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

[bazaar] Extract inline script block and event handler in WEB-INF/classes/hudson/plugins/bazaar/BazaarTagAction/tagForm.jelly

    • Icon: Task Task
    • Resolution: Unresolved
    • Icon: Minor Minor
    • bazaar-plugin

      Problems

      == Inline Script Block
      Line: 67
      ----
      <script>
      	            <!-- update the visual feedback depending on the checkbox state -->
      	            function updateRow(e,i) {
      	              e.parentNode.parentNode.style.color = e.checked ? "inherit" : "grey";
      	              $("name"+i).disabled = !e.checked;
      	            }
      	          </script>
      ----
      
      == Inline Event Handler
      Line: 45
      ----
      <input type="checkbox" name="tag${loop.index}"
      	                         id="tag${loop.index}" onchange="updateRow(this,${loop.index})" />
      ----
      

      Solutions

      https://www.jenkins.io/doc/developer/security/csp/#inline-javascript-blocks
      https://www.jenkins.io/doc/developer/security/csp/#inline-event-handlers

          [JENKINS-74538] [bazaar] Extract inline script block and event handler in WEB-INF/classes/hudson/plugins/bazaar/BazaarTagAction/tagForm.jelly

          Basil Crow created issue -
          Basil Crow made changes -
          Assignee Original: Monty Taylor [ sdirector ]
          Basil Crow made changes -
          Description Original: h4. Problems

          {noformat}
          == Inline Event Handler
          Line: 45
          ----
          <input type="checkbox" name="tag${loop.index}"
          id="tag${loop.index}" onchange="updateRow(this,${loop.index})" />
          ----

          == Inline Script Block
          Line: 67
          ----
          <script>
          <!-- update the visual feedback depending on the checkbox state -->
          function updateRow(e,i) {
          e.parentNode.parentNode.style.color = e.checked ? "inherit" : "grey";
          $("name"+i).disabled = !e.checked;
          }
          </script>
          ----

          == Inline Event Handler
          Line: 45
          ----
          <input type="checkbox" name="tag${loop.index}"
          id="tag${loop.index}" onchange="updateRow(this,${loop.index})" />
          ----

          == Inline Script Block
          Line: 67
          ----
          <script>
          <!-- update the visual feedback depending on the checkbox state -->
          function updateRow(e,i) {
          e.parentNode.parentNode.style.color = e.checked ? "inherit" : "grey";
          $("name"+i).disabled = !e.checked;
          }
          </script>
          ----
          {noformat}

          h4. Solutions

          [https://www.jenkins.io/doc/developer/security/csp/#inline-javascript-blocks]
          [https://www.jenkins.io/doc/developer/security/csp/#inline-event-handlers]
          New: h4. Problems

          {noformat}
          == Inline Script Block
          Line: 67
          ----
          <script>
          <!-- update the visual feedback depending on the checkbox state -->
          function updateRow(e,i) {
          e.parentNode.parentNode.style.color = e.checked ? "inherit" : "grey";
          $("name"+i).disabled = !e.checked;
          }
          </script>
          ----

          == Inline Event Handler
          Line: 45
          ----
          <input type="checkbox" name="tag${loop.index}"
          id="tag${loop.index}" onchange="updateRow(this,${loop.index})" />
          ----
          {noformat}

          h4. Solutions

          [https://www.jenkins.io/doc/developer/security/csp/#inline-javascript-blocks]
          [https://www.jenkins.io/doc/developer/security/csp/#inline-event-handlers]
          Summary Original: [bazaar] Extract inline script blocks and event handlers in WEB-INF/classes/hudson/plugins/bazaar/BazaarTagAction/tagForm.jelly New: [bazaar] Extract inline script block and event handler in WEB-INF/classes/hudson/plugins/bazaar/BazaarTagAction/tagForm.jelly

            Unassigned Unassigned
            basil Basil Crow
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: