Enhance Bugzilla support

Review Request #5769 — Created May 4, 2014 and discarded

Information

Review Board
master
53dde7f...

Reviewers

Add support for bug infobox on Bugzilla

Tested against a local Bugzilla instance and Bugzilla demo at https://bugzilla.mozilla.org/

Description From Last Updated

Please add a docstring.

daviddavid

These can be combined into one statement.

daviddavid

Please add a docstring.

daviddavid

These can be combined into one statement.

daviddavid

Can you reformat this to put the dictionary entries on their own lines? (newline after { and before }).

daviddavid

If there's an exception, this will return None. Should it return '' instead?

daviddavid

Same question about None vs ''.

daviddavid
There are no open issues
TO
TO
TO
TO
david
  1. 
      
  2. reviewboard/hostingsvcs/bugzilla.py (Diff revision 2)
     
     
    Show all issues

    Please add a docstring.

  3. reviewboard/hostingsvcs/bugzilla.py (Diff revision 2)
     
     
     
    Show all issues

    These can be combined into one statement.

  4. reviewboard/hostingsvcs/bugzilla.py (Diff revision 2)
     
     
    Show all issues

    Please add a docstring.

  5. reviewboard/hostingsvcs/bugzilla.py (Diff revision 2)
     
     
     
    Show all issues

    These can be combined into one statement.

  6. reviewboard/hostingsvcs/bugzilla.py (Diff revision 2)
     
     
     
    Show all issues

    Can you reformat this to put the dictionary entries on their own lines? (newline after { and before }).

  7. 
      
TO
TO
TO
david
  1. 
      
  2. reviewboard/hostingsvcs/bugzilla.py (Diff revision 5)
     
     
     
     
     
     
    Show all issues

    If there's an exception, this will return None. Should it return '' instead?

  3. reviewboard/hostingsvcs/bugzilla.py (Diff revision 5)
     
     
     
     
     
     
    Show all issues

    Same question about None vs ''.

  4. 
      
TO
TO
Review request changed
Status:
Discarded
Change Summary:
Obsoleted by https://reviews.reviewboard.org/r/6047/
Loading...