Opened 16 years ago

Last modified 10 years ago

#12246 new Cleanup/optimization

contrib.admin's JavaScript functions should be namespaced

Reported by: italomaia Owned by: nobody
Component: contrib.admin Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Some methods clash with some frameworks. addEvent is used in mootools and django-admin scripts.

Possible solution

# from FunFactor, IRC - mootools
var django = {}; django.addEvent = function(...);

Change History (5)

comment:1 by Russell Keith-Magee, 16 years ago

Triage Stage: Unreviewed → Accepted

Agreed, although this should probably be done as part of a wholesale cleanup of the pieces of admin JS that aren't using jQuery.

comment:2 by Matt McClanahan, 15 years ago

Severity: → Normal
Type: → Bug

comment:3 by Aymeric Augustin, 14 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:4 by Aymeric Augustin, 14 years ago

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:5 by Tim Graham, 10 years ago

Summary: Django-admin methods should be wraped in a django object → contrib.admin's JavaScript functions should be namespaced
Type: Bug → Cleanup/optimization
Version: 1.1 → master
Note: See TracTickets for help on using tickets.
Back to Top