• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Component/s: htmlpublisher-plugin
• Labels:

  None

[JENKINS-29679] Back link doesnt work after job renaming

Collapse comment: mcrooney added a comment - 2015-07-28 22:49

mcrooney added a comment - 2015-07-28 22:49

So this was originally changed because using relative paths didn't work reliably as the report can be accessed from both project or job levels. Probably it would be better to have the report rendered through some kind of templating that populates the job path dynamically.

Expand comment: mcrooney added a comment - 2015-07-28 22:49

mcrooney added a comment - 2015-07-28 22:49 So this was originally changed because using relative paths didn't work reliably as the report can be accessed from both project or job levels. Probably it would be better to have the report rendered through some kind of templating that populates the job path dynamically.

Collapse comment: mcrooney added a comment - 2015-08-03 15:14

mcrooney added a comment - 2015-08-03 15:14

We can't use history.back() because those links still need to work if someone sent you a link of the report and you want to visit the job from there, but you didn't just come from the job. If you end up finding an approach that works, please do feel free to send a pull request however!

Expand comment: mcrooney added a comment - 2015-08-03 15:14

mcrooney added a comment - 2015-08-03 15:14 We can't use history.back() because those links still need to work if someone sent you a link of the report and you want to visit the job from there, but you didn't just come from the job. If you end up finding an approach that works, please do feel free to send a pull request however!

Collapse comment: Daniel Beck added a comment - 2016-02-04 22:56

Daniel Beck added a comment - 2016-02-04 22:56

mcrooney This should be taken care of by serveWrapper from PR 22

Expand comment: Daniel Beck added a comment - 2016-02-04 22:56

Daniel Beck added a comment - 2016-02-04 22:56 mcrooney This should be taken care of by serveWrapper from PR 22

Collapse comment: mcrooney added a comment - 2016-02-05 18:32

mcrooney added a comment - 2016-02-05 18:32

Thanks for the hint Daniel and thanks so much for all your bug triaging, you are an HTML Publisher hero Tested and looks good to me. I'll update the wiki as well.

Expand comment: mcrooney added a comment - 2016-02-05 18:32

mcrooney added a comment - 2016-02-05 18:32 Thanks for the hint Daniel and thanks so much for all your bug triaging, you are an HTML Publisher hero Tested and looks good to me. I'll update the wiki as well.