• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • extra-columns-plugin
    • None
    • Windows XP
      Jenkins 1.569

      After upgrading from Jenkins 1.544, the "Test Result" column appears to have an incorrect link for each job, doubling up on the view path where we only expect one, i.e.:
      http://jenkinsSite:8080/ view/ViewName / view/viewName /job/jobName/lastCompletedBuild/testReport

      Looking at the "TestResultColumn/column.jelly" file, I can see the link as:
      <a href="${job.url}lastCompletedBuild/${tr.urlName}/">

      It would appear that a change made as part of JENKINS-19310, changes the value of ${job.url} thus probably causing this problem.

          [JENKINS-23583] Column "Test Result" has incorrect link

          centic added a comment -

          I have reported a general bug-report for this at JENKINS-25113

          centic added a comment - I have reported a general bug-report for this at JENKINS-25113

          Diogo Campos added a comment -

          It seems that this only happens for views using a regex to match jobs.

          Jobs selected individually from checkboxes do not have this problem.

          Diogo Campos added a comment - It seems that this only happens for views using a regex to match jobs. Jobs selected individually from checkboxes do not have this problem.

          Ulli Hafner added a comment -

          Ulli Hafner added a comment - Fred, I think you need to check all your columns. job.getUrl() now returns a url containing the view part. See discussions: https://github.com/jenkinsci/cobertura-plugin/commit/1a899111888f458109b47758a0c667c07c3995b8#commitcomment-8302902 https://github.com/jenkinsci/cobertura-plugin/commit/7fd072ed198217974ef275d5662bf882f92147dd JENKINS-24436

          Fred G added a comment -

          Thanks Ulli for the heads-up.
          I will take a look.

          Fred G added a comment - Thanks Ulli for the heads-up. I will take a look.

          @stephenthornhill made a PR on github to fix this issue PR #11.

          I tested it in my instance and it works like a charm.

          Matthieu Ouellette-Vachon added a comment - @stephenthornhill made a PR on github to fix this issue PR #11 . I tested it in my instance and it works like a charm.

          Igor Kostenko added a comment -

          Matthieu, could you upload plugin jar with PR 11 somewhere?

          Igor Kostenko added a comment - Matthieu, could you upload plugin jar with PR 11 somewhere?

          Code changed in jenkins
          User: stephenthornhill
          Path:
          src/main/resources/jenkins/plugins/extracolumns/TestResultColumn/column.jelly
          http://jenkins-ci.org/commit/extra-columns-plugin/8fee58bbe7f2e6c496f8f64bee3dd23a18a69669
          Log:
          [FIXED JENKINS-23583] Test result url made absolute by prepending rootURL

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: stephenthornhill Path: src/main/resources/jenkins/plugins/extracolumns/TestResultColumn/column.jelly http://jenkins-ci.org/commit/extra-columns-plugin/8fee58bbe7f2e6c496f8f64bee3dd23a18a69669 Log: [FIXED JENKINS-23583] Test result url made absolute by prepending rootURL

          Code changed in jenkins
          User: Fred G
          Path:
          src/main/resources/jenkins/plugins/extracolumns/TestResultColumn/column.jelly
          http://jenkins-ci.org/commit/extra-columns-plugin/3ec36ffb2e303757f9033bba9b8a5df9a6c6dc0b
          Log:
          Merge pull request #11 from stephenthornhill/master

          [FIXED JENKINS-23583] Test result url made absolute by prepending rootUR...

          Compare: https://github.com/jenkinsci/extra-columns-plugin/compare/90db80f86d0f...3ec36ffb2e30

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Fred G Path: src/main/resources/jenkins/plugins/extracolumns/TestResultColumn/column.jelly http://jenkins-ci.org/commit/extra-columns-plugin/3ec36ffb2e303757f9033bba9b8a5df9a6c6dc0b Log: Merge pull request #11 from stephenthornhill/master [FIXED JENKINS-23583] Test result url made absolute by prepending rootUR... Compare: https://github.com/jenkinsci/extra-columns-plugin/compare/90db80f86d0f...3ec36ffb2e30

            fredg Fred G
            amolago Chris Lake
            Votes:
            8 Vote for this issue
            Watchers:
            17 Start watching this issue

              Created:
              Updated:
              Resolved: