Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#7254 closed (fixed)

patch: exclude apps from dumpdir

Reported by: Carl Karsten <carl@…> Owned by: nobody
Component: Core (Management commands) Version: dev
Severity: Keywords:
Cc: "Russell, Keith-Magee", <freakboy3742@…> Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

To help deal with loaddata issues caused by a single app (like http://code.djangoproject.com/ticket/7052 ) this adds support for --exclude to dumpdata. (use multiple --exclude to exclude multiple apps.)

It also adds --verbosity which shows lets you see what apps have been excluded, what will be processed, and what is being processed as the data is being dumped. It sends this to stderr because dumpdata currently sends to stdout.

If stderr is poor form, I was considering adding a --file option so that sdtout could be used for verbosity messages. but that seemed just as wrong as stderr, and stderr was much easier to code.

Attachments (2)

dumpdata.py.diff (3.5 KB ) - added by Carl Karsten <carl@…> 16 years ago.
dumpdata.py.2.diff (3.3 KB ) - added by Carl Karsten <carl@…> 16 years ago.
removed unnecessary changes, s/print stderr/sys.stderr.write

Download all attachments as: .zip

Change History (9)

by Carl Karsten <carl@…>, 16 years ago

Attachment: dumpdata.py.diff added

comment:1 by Ludvig Ericson, 16 years ago

Patch needs improvement: set

There are unnecessary changes in the patch. Please have them removed.

     option_list = BaseCommand.option_list + (
-        make_option('--format', default='json', dest='format',
+        make_option('--format', dest='format', default='json',
             help='Specifies the output serialization format for fixtures.'),
-        make_option('--indent', default=None, dest='indent', type='int',
+        make_option('--indent', dest='indent', type='int', default=None,
             help='Specifies the indent level to use when pretty-printing output'),

Also, the direct usage of stderr is layer violation, AFAICT.

by Carl Karsten <carl@…>, 16 years ago

Attachment: dumpdata.py.2.diff added

removed unnecessary changes, s/print stderr/sys.stderr.write

comment:2 by Jeff Anderson, 16 years ago

Triage Stage: UnreviewedDesign decision needed

comment:3 by Carl Karsten <carl@…>, 16 years ago

Cc: "Russell Keith-Magee" <freakboy3742@…> added

Russ - you kinda mentored me on this - mind chiming in?

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

Triage Stage: Design decision neededAccepted

This is a reasonable idea, especially as a workaround for #7052 issues. I can't review the patch right now, but I should have a chance in the near future.

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

Resolution: fixed
Status: newclosed

(In [7615]) Fixed #7254 -- Added an '--exclude' option to dumpdata, allowing specific applications to be removed from the dump output. Thanks to Carl Karsten for the idea and patch.

comment:6 by Carl Karsten <carl@…>, 16 years ago

What happened to

make_option('-v', '--verbosity', action='store', dest='verbosity', default='0',

?

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

On looking at the patch, I couldn't see the value in the output being provided by verbosity - it was useful output for debugging the command itself, but I didn't see the value for end users.

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