Page MenuHomePhabricator
Differential D11064

Define JavaScript functions before they are used
ClosedPublic
Actions

Authored by joshuaspence on Dec 30 2014, 10:14 AM.
Tags
None
Referenced Files
F18237069: D11064.id.diff
Wed, Aug 20, 9:02 PM
F18209062: D11064.id26557.diff
Mon, Aug 18, 10:16 PM
F18203965: D11064.id.diff
Mon, Aug 18, 1:08 PM
Unknown Object (File)
Jul 6 2025, 9:23 PM
Unknown Object (File)
May 24 2025, 3:04 PM
Unknown Object (File)
May 20 2025, 11:55 AM
Unknown Object (File)
Apr 25 2025, 8:12 PM
Unknown Object (File)
Apr 19 2025, 5:48 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
Restricted Diffusion Commit
rP558e1ee3b8d2: Define JavaScript functions before they are used
Summary

This should be a fairly minor change that silences a bunch of JSHint warnings.

Test Plan

arc lint showed less warnings.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 26557.Dec 30 2014, 10:14 AM
joshuaspence retitled this revision from to Define JavaScript functions before they are used.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptDec 30 2014, 10:14 AM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.Dec 30 2014, 10:53 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Dec 30 2014, 10:53 AM
Closed by commit rP558e1ee3b8d2: Define JavaScript functions before they are used (authored by joshuaspence, committed by epriestley). · Explain WhyDec 30 2014, 10:53 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
156 lines
webroot/
rsrc/
js/
application/
countdown/
4 lines
maniphest/
6 lines
pholio/
6 lines
phortune/
4 lines
3 lines
3 lines
core/
3 lines

Diff 26577

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/js/application/countdown/timer.js

Loading...
View Options

webroot/rsrc/js/application/maniphest/behavior-batch-editor.js

Loading...
View Options

webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js

Loading...
View Options

webroot/rsrc/js/application/phortune/behavior-balanced-payment-form.js

Loading...
View Options

webroot/rsrc/js/application/phortune/behavior-stripe-payment-form.js

Loading...
View Options

webroot/rsrc/js/application/phortune/behavior-test-payment-form.js

Loading...
View Options

webroot/rsrc/js/core/behavior-dark-console.js

Loading...
Phabricator · Built by Phacility