Fix admin media prefix error in Django 1.3.x.

wardi / django-filebrowser-no-grappelli

django-filebrowser for default Django admin site. Based on https://github.com/sehmaschine/django-filebrowser v3.1 (before it was in git) Consider using https://github.com/smacker/django-filebrowser-no-grappelli which is based on a newer version

Other

150 stars 114 forks source link

Fix admin media prefix error in Django 1.3.x. #23

Closed jasondavies closed 11 years ago

jasondavies commented 12 years ago

Anything I can do to help get this merged?

marltu commented 12 years ago

Admin-media-prefix is deprecated in in django 1.4. Maybe we should consider using another alternative?

https://docs.djangoproject.com/en/dev/ref/settings/#admin-media-prefix

jsma commented 12 years ago

This project still supports Django < 1.4 and this pull request solves the issue (the template tag wasn't being called so we wind up with 'href="<function admin_media_prefix at 0x110af31b8>css/changelists.css"' etc in the templates).

wardi commented 11 years ago

Sorry I've left this for so long. I've made you a contributor, please compare with what @IanLewis has changed here https://github.com/wardi/django-filebrowser-no-grappelli/pull/48 and merge what you think makes sense.

jasondavies commented 11 years ago

Fixed in @b5bc6c8d66a6c9481a411bed15998b1c226955eb.