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
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/