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

CustomOrganizationFolderDescriptor breaks when multiple branch sources are added

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Critical Critical
    • branch-api-plugin
    • None

      If you install one branch source (such as github-branch-source; plus workflow-multibranch), then later dynamically install a second one (such as cloudbees-bitbucket-branch-source), you get an error:

      ... hudson.ExtensionList fireOnChangeListeners
      SEVERE: Error firing ExtensionListListener.onChange().
      java.lang.UnsupportedOperationException
      	at hudson.util.Iterators$5.remove(Iterators.java:299)
      	at hudson.util.AdaptedIterator.remove(AdaptedIterator.java:60)
      	at java.util.AbstractCollection.removeAll(AbstractCollection.java:376)
      	at jenkins.branch.CustomOrganizationFolderDescriptor.doAddSpecificDescriptors(CustomOrganizationFolderDescriptor.java:153)
      	at jenkins.branch.CustomOrganizationFolderDescriptor.access$100(CustomOrganizationFolderDescriptor.java:53)
      	at jenkins.branch.CustomOrganizationFolderDescriptor$ListenerImpl.onChange(CustomOrganizationFolderDescriptor.java:138)
      	at ...
      

          [JENKINS-39520] CustomOrganizationFolderDescriptor breaks when multiple branch sources are added

          Jesse Glick created issue -
          Jesse Glick made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Jesse Glick made changes -
          Link New: This issue blocks JENKINS-33106 [ JENKINS-33106 ]

          Jesse Glick added a comment -

          Apparently ExtensionList supports remove but not removeAll.

          Jesse Glick added a comment - Apparently ExtensionList supports remove but not removeAll .
          Jesse Glick made changes -
          Remote Link New: This issue links to "branch-api PR 56 (Web Link)" [ 15022 ]
          Jesse Glick made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]

          Code changed in jenkins
          User: Jesse Glick
          Path:
          src/main/java/jenkins/branch/CustomOrganizationFolderDescriptor.java
          src/test/java/jenkins/branch/CustomOrganizationFolderDescriptorTest.java
          http://jenkins-ci.org/commit/branch-api-plugin/b95b07f52d3d4e5191cf6109b770cc2010026b20
          Log:
          [FIXED JENKINS-39520] ExtensionList.removeAll is unimplemented.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/jenkins/branch/CustomOrganizationFolderDescriptor.java src/test/java/jenkins/branch/CustomOrganizationFolderDescriptorTest.java http://jenkins-ci.org/commit/branch-api-plugin/b95b07f52d3d4e5191cf6109b770cc2010026b20 Log: [FIXED JENKINS-39520] ExtensionList.removeAll is unimplemented.
          Jesse Glick made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Resolved [ 5 ]

          Code changed in jenkins
          User: Jesse Glick
          Path:
          src/main/java/jenkins/branch/CustomOrganizationFolderDescriptor.java
          src/test/java/jenkins/branch/CustomOrganizationFolderDescriptorTest.java
          http://jenkins-ci.org/commit/branch-api-plugin/1f75509b9f033532fc852e4937d59cd42b1ab732
          Log:
          Merge pull request #56 from jglick/UnsupportedOperationException-JENKINS-39520

          JENKINS-39520 ExtensionList.removeAll is unimplemented
          (cherry picked from commit f5c8342f2fd73f95e70471d64c2fdeb4f050e035)

          Compare: https://github.com/jenkinsci/branch-api-plugin/compare/0d23abd35af7^...1f75509b9f03

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/jenkins/branch/CustomOrganizationFolderDescriptor.java src/test/java/jenkins/branch/CustomOrganizationFolderDescriptorTest.java http://jenkins-ci.org/commit/branch-api-plugin/1f75509b9f033532fc852e4937d59cd42b1ab732 Log: Merge pull request #56 from jglick/UnsupportedOperationException- JENKINS-39520 JENKINS-39520 ExtensionList.removeAll is unimplemented (cherry picked from commit f5c8342f2fd73f95e70471d64c2fdeb4f050e035) Compare: https://github.com/jenkinsci/branch-api-plugin/compare/0d23abd35af7 ^...1f75509b9f03
          Jesse Glick made changes -
          Remote Link New: This issue links to "core PR 2612 (Web Link)" [ 15024 ]

            jglick Jesse Glick
            jglick Jesse Glick
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: