Paths

Table of Contentst

Differential D11062

Minor JavaScript fix
ClosedPublic
Actions

Authored by joshuaspence on Dec 30 2014, 9:51 AM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rP3f0bc79e0ab1: Minor JavaScript fix

Summary

Currently, editdata is implicitly defined as a global variable (and JSHint complains about this). Instead, change it to be a local variable.

Test Plan

arc lint showed one less warning.

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

No Test Coverage

Build Status

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

Event Timeline

joshuaspence updated this revision to Diff 26553.Dec 30 2014, 9:51 AM

joshuaspence retitled this revision from to Minor JavaScript fix.

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, 9:51 AM

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

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

epriestley edited edge metadata.

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

Closed by commit rP3f0bc79e0ab1: Minor JavaScript fix (authored by joshuaspence, committed by epriestley). · Explain WhyDec 30 2014, 10:50 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

webroot/

rsrc/

js/

core/

behavior-line-linker.js

2 lines

Diff 26553

View Options

webroot/rsrc/js/core/behavior-line-linker.js

Minor JavaScript fixClosedPublicActions