#18978: Move cleanup management command to contrib.sessions
-------------------------------------+-------------------------------------
Reporter: rasca | Owner: rasca
Type: | Status: new
Cleanup/optimization | Version: master
Component: Core (Management | Resolution:
commands) | Triage Stage: Accepted
Severity: Normal | Needs documentation: 1
Keywords: | Patch needs improvement: 1
Has patch: 1 | UI/UX: 0
Needs tests: 1 |
Easy pickings: 1 |
-------------------------------------+-------------------------------------
Comment (by Elvard):
Ok, it was easier for me to pull request rather than submit patch here…
I just don't agree with any deprecation warning. The '''cleanup''' command
do the same job as it always did -- it cleans expired sessions and it
requires Sessions installed. I don't think it will be removed in future.
Each app will probably provide it's own '''clean*''' method and
'''cleanup''' will collect them.
Possible feature in this scenario would be new setting, where one could
define list of all '''clean*''' commands triggered by '''cleanup'''
command. This would be handy, eg: django-registration provides
'''cleanupregistration'''. This would be quiet easily implemented. Should
I post a patch for it or open new ticket for this feature?
Last thing, I just can't find any tests for old '''cleanup''' command. I
have to probably write them…
--
Ticket URL: <https://code.djangoproject.com/ticket/18978#comment:4>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
