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

f:select should show a spinner while AJAX requests are in-flight

    XMLWordPrintable

    Details

    • Type: New Feature
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: core
    • Labels:
    • Similar Issues:

      Description

      Port the fix from JENKINS-32007 to core

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Stephen Connolly
            Path:
            core/src/main/java/hudson/util/FormFillFailure.java
            core/src/main/resources/lib/form/select/select.js
            http://jenkins-ci.org/commit/jenkins/ed329d1088c164df560b9299cd9096ae76519418
            Log:
            [FIXED JENKINS-42443] Make f:select display spinner during AJAX requests

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/util/FormFillFailure.java core/src/main/resources/lib/form/select/select.js http://jenkins-ci.org/commit/jenkins/ed329d1088c164df560b9299cd9096ae76519418 Log: [FIXED JENKINS-42443] Make f:select display spinner during AJAX requests
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Stephen Connolly
            Path:
            core/src/main/java/hudson/util/FormFillFailure.java
            http://jenkins-ci.org/commit/jenkins/79c761d51a244db006b655160017989039c8b44b
            Log:
            Fixing @since tags from merge of JENKINS-42443

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Stephen Connolly Path: core/src/main/java/hudson/util/FormFillFailure.java http://jenkins-ci.org/commit/jenkins/79c761d51a244db006b655160017989039c8b44b Log: Fixing @since tags from merge of JENKINS-42443
            Hide
            jordangs Goran Sarenkapa added a comment -

            Users reported a HTTP Error 404 in my plugin, after investigating and running git bisect, i've managed to narrow the culprit down to this Jenkins Issue / commit. For more information please see the google group discussion.

             

            Thank you very much.

            Show
            jordangs Goran Sarenkapa added a comment - Users reported a HTTP Error 404 in my plugin, after investigating and running git bisect, i've managed to narrow the culprit down to this Jenkins Issue / commit. For more information please see the google group discussion.   Thank you very much.
            Hide
            stephenconnolly Stephen Connolly added a comment -

            Goran Sarenkapa I believe the issue here is how you are configuring things.

            The select CSS style is used to trigger the AJAX field population and the intent is that the style should only be present on select tags generated by f:select.

            Your plugin is using a very non-Jenkins way to build its configuration, in fact you are not even using the AJAX select population that is triggered by adding the select CSS style: https://github.com/jenkinsci/zap-plugin/blob/development/src/main/resources/org/jenkinsci/plugins/zap/ZAPDriver/config.jelly#L246

            I suspect that your plugin has always been making 404 requests and getting 404 responses but this change is now just showing the error, have a look at the network request inspector when loading the config page to see if my theory is correct.

            There are two ways to your issue:

            1. Just remove the select CSS class from your <select> tags; or
            2. Switch to the Jenkins idiomatic AJAX select population by adding a doFillSelectedExportFormats() method to your descriptor and either provide the required fillUrl attribute in your <select> tag or switch to using <f:select>

            Show
            stephenconnolly Stephen Connolly added a comment - Goran Sarenkapa I believe the issue here is how you are configuring things. The select CSS style is used to trigger the AJAX field population and the intent is that the style should only be present on select tags generated by f:select. Your plugin is using a very non-Jenkins way to build its configuration, in fact you are not even using the AJAX select population that is triggered by adding the select CSS style: https://github.com/jenkinsci/zap-plugin/blob/development/src/main/resources/org/jenkinsci/plugins/zap/ZAPDriver/config.jelly#L246 I suspect that your plugin has always been making 404 requests and getting 404 responses but this change is now just showing the error, have a look at the network request inspector when loading the config page to see if my theory is correct. There are two ways to your issue: 1. Just remove the select CSS class from your <select> tags; or 2. Switch to the Jenkins idiomatic AJAX select population by adding a doFillSelectedExportFormats() method to your descriptor and either provide the required fillUrl attribute in your <select> tag or switch to using <f:select>
            Hide
            stephenconnolly Stephen Connolly added a comment -

            Closing after re-open, because the re-open was for the change functioning as designed and reporting a 404 response from the AJAX fill request

            Show
            stephenconnolly Stephen Connolly added a comment - Closing after re-open, because the re-open was for the change functioning as designed and reporting a 404 response from the AJAX fill request

              People

              • Assignee:
                stephenconnolly Stephen Connolly
                Reporter:
                stephenconnolly Stephen Connolly
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: