Changeset 2385


Ignore:
Timestamp:
Sep 30, 2014, 2:24:45 PM (4 years ago)
Author:
matthijs
Message:

Fix a duplicate variable name

The variable name "filter" was declared twice in the same scope, which
will be a problem in the future (starting with FF35). This was probably
a copy-paste error, since the name doesn't actually make sense for the
second declaration. Renaming it solves both problems.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • desktop/foneradownloader/trunk/chrome/content/js/dlmanager.js

    r1914 r2385  
    446446
    447447        let prefs = Fonera.Preferences.getBranch("extensions.foneradownloader."); // the final . is needed
    448         let filter = prefs.getCharPref('dlview');
    449         let drawFunction = (filter == 'icon-view') ? FoneraDLManager.drawDownloadItem : FoneraDLManager.drawDownloadItemSmall;
     448        let view = prefs.getCharPref('dlview');
     449        let drawFunction = (view == 'icon-view') ? FoneraDLManager.drawDownloadItem : FoneraDLManager.drawDownloadItemSmall;
    450450
    451451        let stringsBundle = document.getElementById("string-bundle");
Note: See TracChangeset for help on using the changeset viewer.