[Zope3-checkins] CVS: Zope3/src/zope/app/browser/security/grants - manage_access.pt:1.5 manage_permissionform.pt:1.3 manage_roleform.pt:1.3 principal_permission_edit.pt:1.3 principal_role_association.pt:1.4

Philipp von Weitershausen philikon@philikon.de
Wed, 25 Jun 2003 14:43:38 -0400


Update of /cvs-repository/Zope3/src/zope/app/browser/security/grants
In directory cvs.zope.org:/tmp/cvs-serv32346/security/grants

Modified Files:
	manage_access.pt manage_permissionform.pt manage_roleform.pt 
	principal_permission_edit.pt principal_role_association.pt 
Log Message:
More XHTML compliance ("xhtml gheddon"?). Files were validated running
the following command (using the xmllint command line tool from
http://xmlsoft.org/)::

  find . -name "*pt" -exec xmllint --noout --dtdvalid \
  http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd {} \;

Emacs's grep-find mode can parse this output. Instead of specifying
the URL, one can also download the DTD to speed up the process.


=== Zope3/src/zope/app/browser/security/grants/manage_access.pt 1.4 => 1.5 ===
--- Zope3/src/zope/app/browser/security/grants/manage_access.pt:1.4	Mon Dec 30 11:47:51 2002
+++ Zope3/src/zope/app/browser/security/grants/manage_access.pt	Wed Jun 25 14:43:07 2003
@@ -16,7 +16,7 @@
 
 <form action="ChangeAllRolePermissions.html" method="post">
 
-<table width="100%" cellspacing="0" cellpadding="2" border="0" nowrap>
+<table width="100%" cellspacing="0" cellpadding="2" border="0" nowrap="nowrap">
 <tr class="list-header">
     <td align="left" valign="top">
       <div class="form-label">
@@ -41,7 +41,7 @@
       <input type="hidden" name="r0" value=""
         tal:attributes="
         name string:r${repeat/role/index};
-        value  string:${role/getId}">
+        value  string:${role/getId}" />
 
     </div>
   </td>
@@ -51,7 +51,7 @@
 <tr class="row-normal"
     tal:attributes="class python:path('repeat/perm/even') and 'row-normal' or 'row-hilite'"
     >
-  <td align="left" nowrap>
+  <td align="left" nowrap="nowrap">
     <div class="list-item">
        <a href="RolesWithPermission.html"
        tal:attributes="
@@ -61,7 +61,7 @@
        <input type="hidden" name="r0" value=""
          tal:attributes="
          name string:p${repeat/perm/index};
-         value  string:${perm/getId}">
+         value  string:${perm/getId}" />
     </div>
   </td>
   <td align="center" tal:repeat="setting perm/roleSettings">


=== Zope3/src/zope/app/browser/security/grants/manage_permissionform.pt 1.2 => 1.3 ===
--- Zope3/src/zope/app/browser/security/grants/manage_permissionform.pt:1.2	Wed Dec 25 09:12:34 2002
+++ Zope3/src/zope/app/browser/security/grants/manage_permissionform.pt	Wed Jun 25 14:43:07 2003
@@ -33,7 +33,7 @@
     tal:attributes="value perm/getId" />
 <div class="form-element">
 
-<table width="100%" cellspacing="0" cellpadding="2" border="0" nowrap>
+<table width="100%" cellspacing="0" cellpadding="2" border="0" nowrap="nowrap">
 <tr class="list-header">
   <td align="left" valign="top">
     <div class="form-label">


=== Zope3/src/zope/app/browser/security/grants/manage_roleform.pt 1.2 => 1.3 ===
--- Zope3/src/zope/app/browser/security/grants/manage_roleform.pt:1.2	Wed Dec 25 09:12:34 2002
+++ Zope3/src/zope/app/browser/security/grants/manage_roleform.pt	Wed Jun 25 14:43:07 2003
@@ -20,7 +20,7 @@
 <input type="hidden" name="role_id" value="Role ID"
     tal:attributes="value role/getId" />
 
-<table width="100%" cellspacing="0" cellpadding="2" border="0" nowrap
+<table width="100%" cellspacing="0" cellpadding="2" border="0" nowrap="nowrap"
        tal:define="availableSettings python:view.availableSettings(noacquire=1)">
 <tr class="list-header">
   <td align="left" valign="top"
@@ -35,7 +35,7 @@
   <td align="left" valign="top"
       tal:repeat="settinginfo availableSettings">
     <div class="form-element">
-      <select name="Unset:list" multiple size="20"
+      <select name="Unset:list" multiple="multiple" size="20"
               tal:define="setting settinginfo/id"
               tal:attributes="name string:${setting}:list">
       <option tal:repeat="permissioninfo role/permissionsInfo"


=== Zope3/src/zope/app/browser/security/grants/principal_permission_edit.pt 1.2 => 1.3 ===
--- Zope3/src/zope/app/browser/security/grants/principal_permission_edit.pt:1.2	Wed Dec 25 09:12:34 2002
+++ Zope3/src/zope/app/browser/security/grants/principal_permission_edit.pt	Wed Jun 25 14:43:07 2003
@@ -1,6 +1,8 @@
 <html metal:use-macro="views/standard_macros/page">
 <div metal:fill-slot="body" tal:define="rprincipal_id python:request['principal_id']">
-<h1>Permission settings for <span tal:replace="python:view.get_principal(rprincipal_id).getTitle()">PrincipalName</span></h1>
+<h1>Permission settings for
+  <span tal:replace="python:view.get_principal(rprincipal_id).getTitle()" />
+</h1>
 
 <form action="unsetPermissions.html" method="post">
 <h2>Permission Settings</h2>
@@ -12,16 +14,17 @@
 	  <th colspan="2" align="center">Allowed Permissions</th>
 	</tr>
 	<tr tal:repeat="perm python:view.get_set_permissions_for_principal(rprincipal_id, 'Allow')">
-	  <td><input type="checkbox" tal:attributes="name perm/getId"></td><td tal:content="perm/getTitle">Permission1</td>
+	  <td><input type="checkbox" tal:attributes="name perm/getId"/></td>
+          <td tal:content="perm/getTitle">Permission1</td>
 	</tr>
 	<tr tal:replace="nothing">
-	  <td><input type="checkbox" name="permission_ids"></td><td>Permission2</td>
+	  <td><input type="checkbox" name="permission_ids" /></td><td>Permission2</td>
 	</tr>
 	<tr tal:replace="nothing">
-	  <td><input type="checkbox" name="permission_ids"></td><td>Permission3</td>
+	  <td><input type="checkbox" name="permission_ids" /></td><td>Permission3</td>
 	</tr>
 	<tr tal:replace="nothing">
-	  <td><input type="checkbox" name="permission_ids"></td><td>Permission5</td>
+	  <td><input type="checkbox" name="permission_ids" /></td><td>Permission5</td>
 	</tr>
       </table>
     </td>
@@ -31,16 +34,21 @@
 	  <th colspan="2" align="center">Denied Permissions</th>
 	</tr>
 	<tr tal:repeat="perm python:view.get_set_permissions_for_principal(rprincipal_id, 'Deny')">
-	  <td><input type="checkbox" tal:attributes="name perm/getId"></td><td tal:content="perm/getTitle">Permission1</td>
+	  <td><input type="checkbox" tal:attributes="name perm/getId" /></td>
+          <td tal:content="perm/getTitle">Permission1</td>
 	</tr>
 	<tr tal:replace="nothing">
-	  <td><input type="checkbox" name="permission_ids"></td><td>Permission2</td>
+	  <td><input type="checkbox" name="permission_ids" /></td>
+          <td>Permission2</td>
 	</tr>
       </table>
     </td>
   </tr>
   <tr>
-    <td colspan="2" align="center"><input type="submit" name="unset" value="Remove selected permission settings"></td></tr>
+    <td colspan="2" align="center">
+      <input type="submit" name="unset" value="Remove selected permission settings" />
+    </td>
+  </tr>
 </table>
 </form>
 <p>&nbsp;</p>
@@ -49,7 +57,7 @@
   <table>
     <tr>
       <td>
-	<select name="permissions" multiple>
+	<select name="permissions" multiple="multiple">
 	  <option 
 	    tal:repeat="perm python:view.get_unset_permissions_for_principal(rprincipal_id)" 
 	    tal:attributes="value perm/getId"
@@ -62,10 +70,10 @@
       </td>	
       <td valign="center">
 	<p>
-	  <input type="submit" name="grantPermissions.html:method" value="Grant">
+	  <input type="submit" name="grantPermissions.html:method" value="Grant" />
 	  </p>
 	<p>
-	  <input type="submit" name="denyPermissions.html:method" value="Deny">
+	  <input type="submit" name="denyPermissions.html:method" value="Deny" />
 	  </p>
       </td>
     </tr>


=== Zope3/src/zope/app/browser/security/grants/principal_role_association.pt 1.3 => 1.4 ===
--- Zope3/src/zope/app/browser/security/grants/principal_role_association.pt:1.3	Mon Dec 30 11:47:51 2002
+++ Zope3/src/zope/app/browser/security/grants/principal_role_association.pt	Wed Jun 25 14:43:07 2003
@@ -11,7 +11,7 @@
         <span class="message"> Apply filter </span>
         <form method="POST" action="@@PrincipalRoles.html">
            Principal(s):
-            <select name="principals:list" multiple>
+            <select name="principals:list" multiple="multiple">
              <option tal:repeat="principal view/getAllPrincipals"
                      tal:attributes="value principal/getId"
                      tal:content="principal/getTitle">my title</option>