Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-30569

JS error triggered by collapsing build history

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: core
    • Labels:
    • Environment:
      OS: Win 8.1
      Browsers: All popular browsers, including Chrome (v45.0), FireFox (v40.0.3), IE (v10,v11), etc.
    • Similar Issues:

      Description

      To reach said error, go to any job page and collapse (/after expanding) the build history box.
      After collapsing, 'Uncaught SyntaxError: Unexpected token )' error should be visible in browser console.

      Apparently, no functionality is compromised by this error.

        Attachments

          Activity

          Hide
          csimons Christopher Simons added a comment - - edited

          Fixed in PR #1856.

          Show
          csimons Christopher Simons added a comment - - edited Fixed in PR #1856 .
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: tfennelly
          Path:
          .gitignore
          changelog.html
          cli/pom.xml
          core/pom.xml
          core/src/main/java/hudson/model/AbstractProject.java
          core/src/main/java/hudson/model/Run.java
          core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly
          core/src/main/resources/lib/form/confirm.js
          core/src/main/resources/lib/form/select/select.js
          core/src/main/resources/lib/layout/layout.jelly
          plugins/pom.xml
          pom.xml
          test/pom.xml
          test/src/test/groovy/hudson/model/AbstractProjectTest.groovy
          war/pom.xml
          http://jenkins-ci.org/commit/jenkins/95ca3da67d217c90d31819ec92e521e2072acd5a
          Log:
          Merge branch 'master' into plugin-manager-dependants

          • master:
            Update the changelog by new merges:
            [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error
            [FIXED JENKINS-29014] render API link conditional on getApi() presence
            JENKINS-21720 JS alert preventig to leave a configuration page even without formulary changes
            add ctags file 'tags' to .gitignore
            [maven-release-plugin] prepare release jenkins-1.632
            [FIXED JENKINS-29888] Handling all exceptions returned by logRotator
            JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()
          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: tfennelly Path: .gitignore changelog.html cli/pom.xml core/pom.xml core/src/main/java/hudson/model/AbstractProject.java core/src/main/java/hudson/model/Run.java core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly core/src/main/resources/lib/form/confirm.js core/src/main/resources/lib/form/select/select.js core/src/main/resources/lib/layout/layout.jelly plugins/pom.xml pom.xml test/pom.xml test/src/test/groovy/hudson/model/AbstractProjectTest.groovy war/pom.xml http://jenkins-ci.org/commit/jenkins/95ca3da67d217c90d31819ec92e521e2072acd5a Log: Merge branch 'master' into plugin-manager-dependants master: Update the changelog by new merges: [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error [FIXED JENKINS-29014] render API link conditional on getApi() presence JENKINS-21720 JS alert preventig to leave a configuration page even without formulary changes add ctags file 'tags' to .gitignore [maven-release-plugin] prepare release jenkins-1.632 [FIXED JENKINS-29888] Handling all exceptions returned by logRotator JENKINS-30742 Fixed possible NPE in AbstractProject.resolveForCLI()
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: tfennelly
          Path:
          core/src/main/resources/lib/layout/pane.jelly
          war/src/main/webapp/scripts/hudson-behavior.js
          http://jenkins-ci.org/commit/jenkins/01b4ac7d902381f65b88a45738b696a0f9ea5cfa
          Log:
          Disable build history refresh if the pane is collapsed

          See JENKINS-30569 and https://github.com/jenkinsci/jenkins/commit/6cadf70d3afb1d1786bf02e549baea99d3ab308a

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: tfennelly Path: core/src/main/resources/lib/layout/pane.jelly war/src/main/webapp/scripts/hudson-behavior.js http://jenkins-ci.org/commit/jenkins/01b4ac7d902381f65b88a45738b696a0f9ea5cfa Log: Disable build history refresh if the pane is collapsed See JENKINS-30569 and https://github.com/jenkinsci/jenkins/commit/6cadf70d3afb1d1786bf02e549baea99d3ab308a
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Christopher Simons
          Path:
          core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly
          http://jenkins-ci.org/commit/jenkins/6cadf70d3afb1d1786bf02e549baea99d3ab308a
          Log:
          [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Christopher Simons Path: core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly http://jenkins-ci.org/commit/jenkins/6cadf70d3afb1d1786bf02e549baea99d3ab308a Log: [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Oleg Nenashev
          Path:
          core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly
          http://jenkins-ci.org/commit/jenkins/a14acf71d0cfc8569268ee0ad53beaba3809d2f7
          Log:
          Merge pull request #1856 from csimons/fix-js-syntax

          [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: core/src/main/resources/hudson/widgets/HistoryWidget/index.jelly http://jenkins-ci.org/commit/jenkins/a14acf71d0cfc8569268ee0ad53beaba3809d2f7 Log: Merge pull request #1856 from csimons/fix-js-syntax [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error
          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oleg Nenashev Path: changelog.html http://jenkins-ci.org/commit/jenkins/a79e2f144a932fe0a97b524f79133d1ea5d0fef0 Log: Update the changelog by new merges: JENKINS-29888 - https://github.com/jenkinsci/jenkins/pull/1790 JENKINS-30742 - https://github.com/jenkinsci/jenkins/pull/1849 JENKINS-29014 - https://github.com/jenkinsci/jenkins/pull/1850 JENKINS-21720 - https://github.com/jenkinsci/jenkins/pull/1854 JENKINS-30569 - https://github.com/jenkinsci/jenkins/pull/1856

            People

            Assignee:
            csimons Christopher Simons
            Reporter:
            radu_ciobanica Radu Ciobanica
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: