Closed Bug 692373 Opened 13 years ago Closed 13 years ago

Link within brief log to the full log doesn't have the (ignored) tree param

Categories

(Tree Management Graveyard :: TBPL, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philor, Unassigned)

References

Details

Attachments

(1 file)

STR:

1. Load https://tbpl.mozilla.org/php/getParsedLog.php?id=6694839&tree=Mozilla-Inbound
2. Realize that an Android jsrefest timeout calls for the full log, since the test running at the time is miles and miles away from the timeout.
3. Click the "View Full Log" link in the brief log, and discover that you're now at https://tbpl.mozilla.org/php/getParsedLog.php?id=6694839&full=1 without the &tree=Mozilla-Inbound tacked on.
Attached patch v1Splinter Review
Unfortunately, PHP land is oblivious of trees and only thinks in terms of buildbot branches, so the only quick fix here is to append e.g.
&branch=mozilla-central
Attachment #565238 - Flags: review?(arpad.borsos)
Attachment #565238 - Flags: review?(arpad.borsos) → review+
http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/b6996e9b3b5c
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Depends on: 693121
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: