[JENKINS-30569] JS error triggered by collapsing build history

[jenkins-infra-bot]

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.

---

Originally reported by radu_ciobanica, imported from: JS error triggered by collapsing build history

• assignee: csimons
• status: Resolved
• priority: Minor
• component(s): core
• label(s): build-history
• resolution: Fixed
• resolved: 2015-10-06T02:38:41+00:00
• votes: 0
• watchers: 3
• imported: 2025-11-24

Raw content of original issue

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.

environment

OS: Win 8.1<br/>
Browsers: All popular browsers, including Chrome (v45.0), FireFox (v40.0.3), IE (v10,v11), etc.

[jenkins-infra-bot]

csimons:

• Original comment link
• Raw content of original comment:

  Fixed in PR #1856.
  

Fixed in PR #1856.

[jenkins-infra-bot]

scm_issue_link:

• Original comment link
• Raw content of original 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()
  
  
  
  
  

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()

[jenkins-infra-bot]

scm_issue_link:

• Original comment link
• Raw content of original 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 6cadf70
  
  
  

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 6cadf70

[jenkins-infra-bot]

scm_issue_link:

• Original comment link
• Raw content of original 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
  
  
  

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

[jenkins-infra-bot]

scm_issue_link:

• Original comment link
• Raw content of original 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
  
  
  

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

[jenkins-infra-bot]

scm_issue_link:

• Original comment link
• Raw content of original 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 - [FIXED JENKINS-29888] - Handling all LogRotator exceptions #1790
  	
  JENKINS-30742 - [JENKINS-30742] Fixed possible NPE in AbstractProject.resolveForCLI() #1849
  	
  JENKINS-29014 - [FIXED JENKINS-29014] fix broken 'REST API' link #1850
  	
  JENKINS-21720 - [JENKINS-21720] JS alert preventing to leave a configuration page without changes #1854
  	
  JENKINS-30569 - [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error #1856
  
  
  
  
  

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 - [FIXED JENKINS-29888] - Handling all LogRotator exceptions #1790
• JENKINS-30742 - [JENKINS-30742] Fixed possible NPE in AbstractProject.resolveForCLI() #1849
• JENKINS-29014 - [FIXED JENKINS-29014] fix broken 'REST API' link #1850
• JENKINS-21720 - [JENKINS-21720] JS alert preventing to leave a configuration page without changes #1854
• JENKINS-30569 - [FIXED JENKINS-30569] HistoryWidget: fix JS syntax error #1856