variable "private" in launchpad.js is future reserved word and should be renamed

Bug #385817 reported by Anon Sricharoenchai
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Low
Deryck Hodge

Bug Description

According to, "Standard ECMA-262 ECMAScript Language Specification 3rd edition (December 1999)" (http://www.ecma-international.org/publications/files/ECMA-ST/ECMA-262.pdf)
"private" is future reserved word (page 14), and should not be used as variable name.

Then the variable named "private" in https://launchpad.net/+icing/rev8109/build/launchpad.js
should be renamed.

Source File: https://launchpad.net/+icing/rev8109/build/launchpad.js
Line: 4640, Column: 4
Source Code:
var private=data['field.private']!==undefined;var security_related=data['field.security_related']!==undefined;lp_bug_entry.set('private',private);lp_bug_entry.set('security_related',security_related);var config={on:{success:function(updated_entry){privacy

Related branches

Jonathan Lange (jml)
affects: launchpad-code → launchpad
affects: launchpad → malone
Changed in malone:
status: New → Triaged
tags: added: cleanup trivial
Curtis Hovey (sinzui)
Changed in launchpad:
importance: Undecided → Low
Deryck Hodge (deryck)
Changed in launchpad:
assignee: nobody → Deryck Hodge (deryck)
status: Triaged → In Progress
tags: added: bugjam2010
Revision history for this message
Launchpad QA Bot (lpqabot) wrote : Bug fixed by a commit
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Deryck Hodge (deryck)
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui)
Changed in launchpad:
status: Fix Committed → Fix Released
Curtis Hovey (sinzui)
Changed in launchpad:
milestone: none → 11.01
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.