Opened 17 years ago

Closed 13 years ago

Last modified 13 years ago

#6011 closed Bug (fixed) flush command not working as advertised

Reported by: Collin Grady <cgrady@…> Owned by: nobody
Component: Core (Management commands) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


If you run help flush, it indicates that it will be the same as if you ran the sql in sqlflush

However, once you try to run it, it starts prompting as if it's running syncdb, like reset does.

Flush should be a simple flush as advertised, while reset does the resetting.

Attachments (2)

management-flush-helptext.diff (695 bytes ) - added by Aaron C. de Bruyn 17 years ago.
Updated help text for flush command
6011.flush-help.diff (882 bytes ) - added by Julien Phalip 13 years ago.

Download all attachments as: .zip

Change History (8)

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

Triage Stage: UnreviewedAccepted

Flush is intended to return the database to a post-sync state. It is used extensively by the testing system for this reason. In this case, the documentation is at fault and the behaviour is correct.

Reset operates on a per-app basis. As a result, cross-app references really mess with the reset command. I have mentioned a few times that I think reset should be deprecated, but we probably won't do so until such time as a schema evolution approach is introduced to trunk.

by Aaron C. de Bruyn, 17 years ago

Updated help text for flush command

comment:2 by Aaron C. de Bruyn, 17 years ago

Has patch: set

Updated help text for flush

comment:3 by Gabriel Hurley, 13 years ago

Severity: Normal
Type: Bug

by Julien Phalip, 13 years ago

Attachment: 6011.flush-help.diff added

comment:4 by Julien Phalip, 13 years ago

Easy pickings: unset

Simply replaced the help text with the one that's in the doc.

comment:5 by Preston Holmes, 13 years ago

Triage Stage: AcceptedReady for checkin
UI/UX: unset

Assuming Julien was not core-dev when this patch was attached.

comment:6 by Paul McMillan, 13 years ago

Resolution: fixed
Status: newclosed

In [16877]:

Fixed #6011 -- Improved help text for flush command. Thanks Julien for the patch.

Note: See TracTickets for help on using tickets.
Back to Top