Changeset 24078

Timestamp:

2011-05-21T19:55:39+12:00 (13 years ago)

Author:

ak19

Message:

There was a bug in Admin pages when listing users where the list users page could only display status and groups info for the admin user (and this was wrong to: the last user's status and groups would be displayed under the admin user instead). This was because the ids assigned to the status and group cells were identical, and so these got successively overwritten on each pass through the user list for displaying their details. Not sure how this ever worked before, so wasn't too sure of the solution and therefore don't know that the current fix is the most optimal kind: there may be better ways of expressing the same in xslt and javascript.

File:

• main/trunk/greenstone3/web/interfaces/default/transform/authen.xsl (modified) (2 diffs)

main/trunk/greenstone3/web/interfaces/default/transform/authen.xsl

@@ -382 +382 @@
       </tr>
       <xsl:for-each select="/page/pageResponse/authenticationNode/service/userNodeList/userNode">
+    <script type="text/javascript">
+    <xsl:text disable-output-escaping="yes">
+    var username="</xsl:text><xsl:value-of select="@umun"/><xsl:text disable-output-escaping="yes">";
+    </xsl:text>
+    </script>
     <tr>
       <td bgcolor="#eeeeee"><xsl:value-of select="@umun"/></td>
-      <td id="status" bgcolor="#eeeeee">
+      <!--<td id="status" bgcolor="#eeeeee">-->
+      <td bgcolor="#eeeeee"><xsl:attribute name="id">status<xsl:value-of select="@umun"/></xsl:attribute>
         <script type="text/javascript">
           <xsl:text disable-output-escaping="yes">
         var status="</xsl:text><xsl:value-of select="@umas"/><xsl:text disable-output-escaping="yes">";
         if (status=="true"){
-          document.getElementById("status").innerHTML="enabled";
+          document.getElementById("status"+username).innerHTML="enabled";
         }
         if (status=="false"){
-          document.getElementById("status").innerHTML="disabled";
+          document.getElementById("status"+username).innerHTML="disabled";
         }
           </xsl:text>        
         </script>
       </td>
-      <td id="group" bgcolor="#eeeeee">
+      <!--<td id="group" bgcolor="#eeeeee">-->
+      <td bgcolor="#eeeeee"><xsl:attribute name="id">group<xsl:value-of select="@umun"/></xsl:attribute>
         <script type="text/javascript">
           <xsl:text disable-output-escaping="yes">
@@ -406 +413 @@
           new_groups+=split_groups[j]+" &lt;br /&gt; ";
         }
-        document.getElementById('group').innerHTML=new_groups;
+        document.getElementById('group'+username).innerHTML=new_groups;
           </xsl:text>        
         </script>