Paths

Table of Contentst

Differential D11067

Fix `next_src` being defined as a global variable
ClosedPublic
Actions

Authored by joshuaspence on Dec 30 2014, 10:26 AM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rPfdd1357ce24f: Fix `next_src` being defined as a global variable

Summary

This variable should be local, not global. This silences a bunch of JSHint warnings.

Test Plan

arc lint

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3362
Build 3369: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 26560.Dec 30 2014, 10:26 AM

joshuaspence retitled this revision from to Fix `next_src` being defined as a global variable.

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:26 AM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.Dec 30 2014, 10:56 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 30 2014, 10:56 AM

Closed by commit rPfdd1357ce24f: Fix `next_src` being defined as a global variable (authored by joshuaspence, committed by epriestley). · Explain WhyDec 30 2014, 10:56 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

32 lines

webroot/

rsrc/

js/

application/

pholio/

behavior-pholio-mock-view.js

2 lines

Diff 26560

View Options

resources/celerity/map.php