#1426 closed defect (fixed)
patch: core.js String.prototype.pad_left sets/overrides global variable 'new_string'
| Reported by: | anonymous | Owned by: | Adrian Holovaty |
|---|---|---|---|
| Component: | contrib.admin | Version: | magic-removal |
| Severity: | normal | Keywords: | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The attached patch fixes that bug and adds some other small improvements.
contrib/admin/media/dateparse.js:
Check for existing methods before emulating
Array.prototype.indexOf
Array.prototype.filter
Upcoming versions of Mozilla might allready have these.
Added local variables for .length props for use in for-loops.
contrib/admin/media/core.js:
Added missing semicolons.
Added missing var statement.
contrib/admin/media/calendar.js:
Moved Calendar methods to prototype object.
Saves memory when multiple calendar instances are created.
contrib/admin/media/admin/RelatedObjectLookups.js:
Added missing semicolons.
Attachments (1)
Change History (2)
by , 18 years ago
| Attachment: | admin-js-fixes.diff added |
|---|
comment:1 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
(In [2448]) Fixed #1426 -- Made several admin JavaScript improvements. Thanks, anonymous