• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • None
    • Tomcat 7, Linux

      Links in FindBugs/PMD/ Duplicate Code/Checkstyle/Coverage column in all views are bad.
      For example link to cobertura is http://server/view/ViewName/view/ViewName/job/Jobname/cobertura , which leads to 404 page.
      There's duplicated "view/ViewName" fragment in URL. Link should be: http://server/view/ViewName/job/Jobname/cobertura

      I have jenkins 1.577, with plugins:
      analysis-collector 1.41
      analysis-core 1.58
      checkstyle 3.39
      cobertura 1.9.5
      crap4j 0.8
      dry 2.40
      findbugs 4.57
      pmd 3.38
      tasks 4.40
      token-macro 1.10
      warnings 4.41

          [JENKINS-24436] Bad URLs in view

          Adam S created issue -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/test/java/plugins/CheckstylePluginTest.java
          http://jenkins-ci.org/commit/acceptance-test-harness/b8a7a0ef9a477789a60019be13cc2b866fe376da
          Log:
          JENKINS-24436 Added test case that exposes the bug.

          Seems that in Jenkins core the base URLs now already contain the view part.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/test/java/plugins/CheckstylePluginTest.java http://jenkins-ci.org/commit/acceptance-test-harness/b8a7a0ef9a477789a60019be13cc2b866fe376da Log: JENKINS-24436 Added test case that exposes the bug. Seems that in Jenkins core the base URLs now already contain the view part.
          Ulli Hafner made changes -
          Link New: This issue is related to JENKINS-19310 [ JENKINS-19310 ]

          Ulli Hafner added a comment -

          Since core 1.568.

          Ulli Hafner added a comment - Since core 1.568.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          clean.sh
          http://jenkins-ci.org/commit/analysis-suite-plugin/e6e4d739a0552b21579f50835a84bb2441ad3963
          Log:
          [FIXED JENKINS-24436] Don't use AbstractItem.getUrl anymore when creating job links.

          Compare: https://github.com/jenkinsci/analysis-suite-plugin/compare/84ef2b4c31a4...e6e4d739a055

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: clean.sh http://jenkins-ci.org/commit/analysis-suite-plugin/e6e4d739a0552b21579f50835a84bb2441ad3963 Log: [FIXED JENKINS-24436] Don't use AbstractItem.getUrl anymore when creating job links. Compare: https://github.com/jenkinsci/analysis-suite-plugin/compare/84ef2b4c31a4...e6e4d739a055
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/analysis/views/WarningsCountColumn.java
          src/main/resources/hudson/plugins/analysis/views/WarningsCountColumn/column.jelly
          http://jenkins-ci.org/commit/analysis-core-plugin/b5986361cfecb3220811fd63286f81d4dd83176c
          Log:
          [FIXED JENKINS-24436] Don't use AbstractItem.getUrl anymore when creating job links.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/analysis/views/WarningsCountColumn.java src/main/resources/hudson/plugins/analysis/views/WarningsCountColumn/column.jelly http://jenkins-ci.org/commit/analysis-core-plugin/b5986361cfecb3220811fd63286f81d4dd83176c Log: [FIXED JENKINS-24436] Don't use AbstractItem.getUrl anymore when creating job links.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          analysis-core
          http://jenkins-ci.org/commit/analysis-suite-plugin/129182c8dbc042c638475b9829e819249c982fa0
          Log:
          [FIXED JENKINS-24436] Don't use AbstractItem.getUrl anymore when creating job links.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: analysis-core http://jenkins-ci.org/commit/analysis-suite-plugin/129182c8dbc042c638475b9829e819249c982fa0 Log: [FIXED JENKINS-24436] Don't use AbstractItem.getUrl anymore when creating job links.

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/cobertura/CoverageColumn.java
          src/main/resources/hudson/plugins/cobertura/CoverageColumn/column.jelly
          http://jenkins-ci.org/commit/cobertura-plugin/7fd072ed198217974ef275d5662bf882f92147dd
          Log:
          JENKINS-24436 Don't use AbstractItem.getUrl when creating job links.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/cobertura/CoverageColumn.java src/main/resources/hudson/plugins/cobertura/CoverageColumn/column.jelly http://jenkins-ci.org/commit/cobertura-plugin/7fd072ed198217974ef275d5662bf882f92147dd Log: JENKINS-24436 Don't use AbstractItem.getUrl when creating job links.

          Jesse Glick added a comment -

          Commented on the cobertura-plugin change; these fixes look misguided to me.

          Jesse Glick added a comment - Commented on the cobertura-plugin change; these fixes look misguided to me.

            drulli Ulli Hafner
            asw Adam S
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: