Changeset 5179

Timestamp:

2003-08-19T10:34:04+12:00 (21 years ago)

Author:

sjboddie

Message:

Fixed a bug in the logic of the public_documents and private_documents
directives.

File:

• trunk/gsdl/src/recpt/documentaction.cpp (modified) (1 diff)

trunk/gsdl/src/recpt/documentaction.cpp

@@ -206 +206 @@
             }
              
-             // only public_documents is set, so ask to authenticate for that article/doc
-             else if (cinfo->public_documents.find(docTop) != cinfo->public_documents.end() )
-            {
-               
-               args["uan"] = "1";
-               args["ug"] = cinfo->auth_group;
-            }
+             // if public_documents is set but this document isn't
+             // listed in it we need to authenticate
+             else if ((!cinfo->public_documents.empty()) && 
+                  (cinfo->public_documents.find(docTop) == cinfo->public_documents.end()))
+               {
+             args["uan"] = "1";
+             args["ug"] = cinfo->auth_group;
+               }
              
-             // only private_documents is set, so ask to authenticate for that article/doc
-             else if (cinfo->private_documents.find(docTop) == cinfo->private_documents.end() )
-            {
-               args["uan"] = "1";
-               args["ug"] = cinfo->auth_group;
-            }
+             // if private_documents is set and this document is
+             // listed in it we need to authenticate
+             else if ((!cinfo->private_documents.empty()) &&
+                  (cinfo->private_documents.find(docTop) != cinfo->private_documents.end()))
+               {
+             args["uan"] = "1";
+             args["ug"] = cinfo->auth_group;
+               }
              
           }