Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  aaugustin
 Type:  New feature  |   Status:  closed
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:  fixed
 Severity:  Normal   | Triage Stage:  Ready for
 Keywords:  printsettings|  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Aymeric Augustin ):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"20a91cce04c72bc8c64a1c43b7398edac7b709cc"]:
 {{{
 #!CommitTicketReference repository=""
 revision="20a91cce04c72bc8c64a1c43b7398edac7b709cc"
 Fixed #17037 -- Added a --all option to diffsettings.
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  aaugustin
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Ready for
 Keywords:  printsettings|  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by claudep):

 I think this might be useful and does not add so much code, after all. +1

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  aaugustin
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Ready for
 Keywords:  printsettings|  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by aaugustin):

 * needs_tests:  1 => 0
 * stage:  Accepted => Ready for checkin


Comment:

 I added tests and docs.

 Final patch: https://github.com/django/django/pull/914

 ptone: if you're strongly against adding this to Django, just close this
 ticket as wontfix -- I don't care much one way or another.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  aaugustin
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  1|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * status:  assigned => new
 * needs_tests:  0 => 1


-- 
Ticket URL: 
Django 
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.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  aaugustin
 Type:  New feature  |   Status:  assigned
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * status:  new => assigned
 * owner:  msabramo => aaugustin


-- 
Ticket URL: 
Django 
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.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  msabramo
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-

Comment (by msabramo):

 Note that since this stalled, I created a separate Django app for this:

 http://pypi.python.org/pypi/django-print-settings/

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  msabramo
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-

Comment (by anonymous):

 @poirier: Did you get a chance to look at the latest patch in
 attachment:django_manage_py_diffsettings_all_v2.diff

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  msabramo
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-

Comment (by msabramo):

 Replying to [comment:11 poirier]:
 > Instead of using "===" to mark default settings, how about instead
 prefixing those lines with "# "?  Then the output will be a valid settings
 file, with the settings not changed from the defaults commented out, which
 might be useful.

 That's a great idea! OK, see
 attachment:django_manage_py_diffsettings_all_v2.diff

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  msabramo
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by msabramo):

 * owner:  nobody => msabramo


Comment:

 Assigning to myself so I can work on this.

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-

Comment (by poirier):

 Instead of using "===" to mark default settings, how about instead
 prefixing those lines with "# "?  Then the output will be a valid settings
 file, with the settings not changed from the defaults commented out, which
 might be useful.

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-

Comment (by aaugustin):

 By convention, people assign to themselves tickets they intend to work on.

 I triaged this ticket — it's better than leaving it unreviewed — but it
 isn't very high on my priority list, so I didn't assign it to myself.

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-

Comment (by anonymous):

 Replying to [comment:8 aaugustin]:
 > Please don't assign tickets to people without their consent.

 Sorry. Was just trying to make sure you were in the loop on the patch that
 addressed your comment. I'll leave the assignments to you guys, though,
 since you know the protocol.

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * owner:  aaugustin => nobody


Comment:

 Please don't assign tickets to people without their consent.

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  aaugustin
 Type:  New feature  |   Status:  new
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by msabramo):

 * owner:  nobody => aaugustin
 * status:  reopened => new


-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  reopened
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-

Comment (by ptone):

 seems that this would be more suited as a contribution to django-
 extensions

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  reopened
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by msabramo):

 * needs_better_patch:  1 => 0


Comment:

 See new patch
 
[https://code.djangoproject.com/attachment/ticket/17037/django_manage_py_diffsettings_all.diff
 django_manage_py_diffsettings_all.diff].

 Some samples of use:

 {{{
 $ django diffsettings --all | grep '===' | head
 ABSOLUTE_URL_OVERRIDES = {}  ===
 ADMIN_FOR = ()  ===
 ALLOWED_INCLUDE_ROOTS = ()  ===
 APPEND_SLASH = True  ===
 AUTHENTICATION_BACKENDS = ('django.contrib.auth.backends.ModelBackend',)
 ===
 CACHES = {'default': {'LOCATION': '', 'BACKEND':
 'django.core.cache.backends.locmem.LocMemCache'}}  ===
 CACHE_MIDDLEWARE_ALIAS = 'default'  ===
 CACHE_MIDDLEWARE_KEY_PREFIX = ''  ===
 CACHE_MIDDLEWARE_SECONDS = 600  ===
 COMMENTS_ALLOW_PROFANITIES = False  ===
 }}}

 {{{
 $ django diffsettings --help | grep --after=2 -- '--all'
   --all Display all settings; not just ones that differ
 from
 Django defaults. Settings that equal the defaults
 are
 followed by "===".
 }}}

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  reopened
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:  printsettings|  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  1
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * needs_better_patch:  0 => 1
 * stage:  Unreviewed => Accepted


Comment:

 We could add an option to "diffsettings" to show the settings that are set
 to their default value too. They would be highlighted with a special
 marker, for instance `===`.

 I don't expect this option to be widely used — Django has a huge number of
 settings and in most cases the default is OK — but it can't hurt.

 I think we should keep the name "diffsettings". The backwards
 compatibility issues outweigh the semantic improvement of changing it to
 "settings".

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  reopened
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   | Triage Stage:
 Keywords:  printsettings|  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by msabramo):

 * status:  closed => reopened
 * resolution:  worksforme =>


Comment:

 Sorry to reopen.

 I didn't know about diffsettings. Thanks for pointing it out. Looks like
 it prints the settings that differ from Django core's default settings.

 What my command does is slightly different. It prints out all the settings
 regardless of whether they are the same or different as the Django
 default.

 So printsettings is simpler than diffsettings. I suppose both functions
 could be done by a single command (settings?) in order to minimize the #
 of commands, but I'll leave that question open. If folks want that, I can
 rework the patch.

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  closed
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   |  worksforme
 Keywords:  printsettings| Triage Stage:
Has patch:  1|  Unreviewed
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  0|  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by msabramo):

 * cc: msabramo (added)


-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.

Re: [Django] #17037: Add management command to print active project settings

[Django]

#17037: Add management command to print active project settings
-+-
 Reporter:  msabramo |Owner:  nobody
 Type:  New feature  |   Status:  closed
Component:  Core (Management |  Version:  1.3
  commands)  |   Resolution:
 Severity:  Normal   |  worksforme
 Keywords:  printsettings| Triage Stage:
Has patch:  1|  Unreviewed
  Needs tests:  0|  Needs documentation:  0
Easy pickings:  0|  Patch needs improvement:  0
 |UI/UX:  0
-+-
Changes (by aaugustin):

 * status:  new => closed
 * needs_docs:   => 0
 * resolution:   => worksforme
 * needs_tests:   => 0
 * needs_better_patch:   => 0


Comment:

 It seems to me that this already exists:
 [https://docs.djangoproject.com/en/dev/ref/django-admin/#diffsettings
 manage.py diffsettings].

 Did I miss something in your proposal?

-- 
Ticket URL: 
Django 
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 this group at 
http://groups.google.com/group/django-updates?hl=en.