[JENKINS-64341] Code review fixes

jenkinsci · Feb 19, 2021 · f14e0a7 · f14e0a7
1 parent 6f98105
commit f14e0a7
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/...ns/authorizeproject/strategy/SpecificUsersAuthorizationStrategy/checkPasswordRequested.js b/...ns/authorizeproject/strategy/SpecificUsersAuthorizationStrategy/checkPasswordRequested.js
@@ -46,6 +46,13 @@ Behaviour.specify(".specific-user-authorization", "checkPasswordRequired", 0, fu
   var useridField = findFormItem(e, "userid", findChild(e));
   var passwordField = findFormItem(e, "password", findChild(e));
   var passwordFieldBlock = findAncestor(passwordField, "TR");
+  /*
+    [JENKINS-64341] - In Jenkins version greater than 2.263.4
+    the table tags for fields (in forms) were replaced by div tags.
+    Since the changes were applied the 'findAncestor(passwordField, "TR")'
+    method returns null. It is necessary to use the method
+    'findAncestorClass(passwordField, "tr")' instead of.
+  */
   if (passwordFieldBlock == null) {
       passwordFieldBlock = findAncestorClass(passwordField, "tr");
     }