#24518: OptionParser verbosity not cast to int (Python3)
-------------------------------------+-------------------------------------
     Reporter:  jthi3rry             |                    Owner:  jthi3rry
         Type:  Bug                  |                   Status:  closed
    Component:  Core (Management     |                  Version:  1.8
  commands)                          |
     Severity:  Normal               |               Resolution:  fixed
     Keywords:                       |             Triage Stage:
                                     |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  1                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by bkonkle):

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


Comment:

 I found the problem:

 
https://github.com/django/django/commit/f17b24e407385eb18651bf023a187347aa9c1f75#commitcomment-10567852

 Verbosity used to be set like this:

 `self.verbosity = int(options.get('verbosity'))`

 Now it's set like this:

 `self.verbosity = options.get('verbosity')`

 There's no int conversion in 1.8 like there was in 1.7, a change made
 during the conversion to argparse. I assume the fix would be to go back to
 converting to int. I'll work up a pull request on Github.

--
Ticket URL: <https://code.djangoproject.com/ticket/24518#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 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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.4864261f260267e02b6439e92621220c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to