Homec4science

Use better variable names

Authored by Joshua Spence <josh@joshuaspence.com> on Dec 30 2014, 11:56.

Description

Use better variable names

Summary:
Use p and c instead of (reusing) k. This silences the following JSHint warning:

>>> Lint for webroot/rsrc/js/application/projects/behavior-project-boards.js:

   Warning  (W004) JSHintW004
    'k' is already defined.

              57       'phui-tag-shade-blue': (sum > 0 && !over_limit),
              58       'phui-tag-shade-red': (over_limit)
              59     };
    >>>       60     for (var k in color_map) {
              61       JX.DOM.alterClass(data.countTagNode, k, !!color_map[k]);
              62     }
              63   }

Test Plan: arc lint

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D11066

Details

Committed
epriestley <git@epriestley.com>Dec 30 2014, 11:56
Pushed
aubortJan 31 2017, 17:16
Parents
rPH5251f08fb432: Use `window.confirm` instead of `confirm`
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH02b2883555c6: Use better variable names (authored by Joshua Spence <josh@joshuaspence.com>).Dec 30 2014, 11:56