Change Summary:
Add caching, adapt to changes in r/5531 and resolve Bugzilla url from repository
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+43 -1) |
Enhance Bugzilla support
Review Request #5769 — Created May 4, 2014 and discarded
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. |
 david
|
|
|
These can be combined into one statement. |
david
|
|
|
Please add a docstring. |
david
|
|
|
These can be combined into one statement. |
david
|
|
|
Can you reformat this to put the dictionary entries on their own lines? (newline after { and before }). |
david
|
|
|
If there's an exception, this will return None. Should it return '' instead? |
david
|
|
|
Same question about None vs ''. |
david
|
TO
-
-
-
-
-
-
reviewboard/hostingsvcs/bugzilla.py (Diff revision 2) Can you reformat this to put the dictionary entries on their own lines? (newline after { and before }).
TO
Change Summary:
Fix issues
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 3 (+43 -1) |
TO
Change Summary:
Add Bugzilla url as part of the cache key
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 4 (+45 -1) |
TO
Change Summary:
Add requested docstrings
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 5 (+47 -1) |
-
-
reviewboard/hostingsvcs/bugzilla.py (Diff revision 5) If there's an exception, this will return None. Should it return '' instead?
-
TO
Change Summary:
Fix issues
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 6 (+49 -1) |