Il 12/03/2015 21:41, Aymeric Augustin ha scritto:
2015-03-12 18:47 GMT+01:00 Riccardo Magliocchetti
<[email protected] <mailto:[email protected]>>:
Finally it looks to me that everything in the app_dict but the permissions
could be cached somehow. Would it make sense to cache the app_dict so it
would be cheaper to add the app_list in the other views?
Yes, it looks like there's room for optimization.
PR please? ;-)
I've taken at stab at it, but test_overriding_has_module_permission test fail.
Looks like the deepcopy of the dict does not work but haven't debugged much.
https://github.com/django/django/compare/master...xrmx:admin_applist?expand=1
I don't like how the code ended up after the second patch though. Need to
measure it is a gain or not.
--
Riccardo Magliocchetti
@rmistaken
http://menodizero.it
--
You received this message because you are subscribed to the Google Groups "Django
developers (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/django-developers.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-developers/55036067.5070300%40gmail.com.
For more options, visit https://groups.google.com/d/optout.
