You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lenya.apache.org by th...@apache.org on 2006/08/31 14:36:53 UTC

svn commit: r438893 - /lenya/sandbox/ac-restricted-1.4-src/modules-core/ac-impl/java/src/org/apache/lenya/ac/file/FilePolicyManager.java

Author: thorsten
Date: Thu Aug 31 05:36:52 2006
New Revision: 438893

URL: http://svn.apache.org/viewvc?rev=438893&view=rev
Log:
Adding code comment to the reverting loop like suggested by Joern Nettingsmeier.

Modified:
    lenya/sandbox/ac-restricted-1.4-src/modules-core/ac-impl/java/src/org/apache/lenya/ac/file/FilePolicyManager.java

Modified: lenya/sandbox/ac-restricted-1.4-src/modules-core/ac-impl/java/src/org/apache/lenya/ac/file/FilePolicyManager.java
URL: http://svn.apache.org/viewvc/lenya/sandbox/ac-restricted-1.4-src/modules-core/ac-impl/java/src/org/apache/lenya/ac/file/FilePolicyManager.java?rev=438893&r1=438892&r2=438893&view=diff
==============================================================================
--- lenya/sandbox/ac-restricted-1.4-src/modules-core/ac-impl/java/src/org/apache/lenya/ac/file/FilePolicyManager.java (original)
+++ lenya/sandbox/ac-restricted-1.4-src/modules-core/ac-impl/java/src/org/apache/lenya/ac/file/FilePolicyManager.java Thu Aug 31 05:36:52 2006
@@ -609,6 +609,8 @@
             url += directories[i] + "/";
             policy = buildSubtreePolicy(controller, url);
             Credential[] tmp = policy.getCredentials();
+            // we need to revert the order of the credentials
+            // to keep the most important policy on top
             for (int j = tmp.length-1; j >= 0; j--) {
                 Credential credential = tmp[j];
                 orderedCredential.put(String.valueOf(position), credential);



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@lenya.apache.org
For additional commands, e-mail: commits-help@lenya.apache.org


Re: svn commit: r438893 - /lenya/sandbox/ac-restricted-1.4-src/modules-core/ac-impl/java/src/org/apache/lenya/ac/file/FilePolicyManager.java

Posted by Joern Nettingsmeier <po...@uni-due.de>.
thorsten@apache.org wrote:
> Author: thorsten
> Date: Thu Aug 31 05:36:52 2006
> New Revision: 438893
> 
> URL: http://svn.apache.org/viewvc?rev=438893&view=rev
> Log:
> Adding code comment to the reverting loop like suggested by Joern Nettingsmeier.

if only i could code half as well as nit-pick :)

thanks for adding it.





-- 
"I don't need backups. I need restore!" - Trad.

--
Jörn Nettingsmeier, EDV-Administrator
Institut für Politikwissenschaft
Universität Duisburg-Essen, Standort Duisburg
Mail: pol-admin@uni-due.de, Telefon: 0203/379-2736


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lenya.apache.org
For additional commands, e-mail: dev-help@lenya.apache.org