From a32a2e7237842e1844b285519b5653f08f90b886 Mon Sep 17 00:00:00 2001
From: steinm <steinm@577cc39b-cfd1-4aa2-8bc3-eda205cea05a>
Date: Fri, 3 Dec 2010 07:22:56 +0000
Subject: [PATCH] - do not use global variable settings->_guestID anymore. Use 
  LetoDMS_User::isGuest() instead

---
 op/op.AddEvent.php              |  2 +-
 op/op.EditEvent.php             |  2 +-
 op/op.EditUserData.php          |  2 +-
 op/op.ManageNotify.php          |  2 +-
 op/op.UserDefaultKeywords.php   |  2 +-
 op/op.UsrMgr.php                | 10 +++++++---
 out/out.AddEvent.php            |  2 +-
 out/out.ApprovalSummary.php     |  2 +-
 out/out.DocumentAccess.php      |  4 ++--
 out/out.DocumentNotify.php      |  4 ++--
 out/out.EditEvent.php           |  2 +-
 out/out.EditUserData.php        |  2 +-
 out/out.FolderAccess.php        |  4 ++--
 out/out.FolderNotify.php        |  4 ++--
 out/out.GroupView.php           |  2 +-
 out/out.ManageNotify.php        |  2 +-
 out/out.MyAccount.php           |  2 +-
 out/out.MyDocuments.php         |  2 +-
 out/out.RemoveUser.php          |  8 +++-----
 out/out.ReviewSummary.php       |  2 +-
 out/out.SearchForm.php          |  2 +-
 out/out.UserDefaultKeywords.php |  2 +-
 out/out.UserList.php            |  2 +-
 out/out.UsrMgr.php              | 16 ++++++++++------
 out/out.UsrView.php             |  4 ++--
 out/out.ViewDocument.php        |  2 +-
 26 files changed, 48 insertions(+), 42 deletions(-)

diff --git a/op/op.AddEvent.php b/op/op.AddEvent.php
index 8bc9444b1..1d2311656 100644
--- a/op/op.AddEvent.php
+++ b/op/op.AddEvent.php
@@ -29,7 +29,7 @@ include("../inc/inc.ClassUI.php");
 include("../inc/inc.Calendar.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("edit_event"),getMLText("access_denied"));
 }
 
diff --git a/op/op.EditEvent.php b/op/op.EditEvent.php
index 7e1ff207f..217091571 100644
--- a/op/op.EditEvent.php
+++ b/op/op.EditEvent.php
@@ -29,7 +29,7 @@ include("../inc/inc.ClassUI.php");
 include("../inc/inc.Calendar.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("edit_event"),getMLText("access_denied"));
 }
 
diff --git a/op/op.EditUserData.php b/op/op.EditUserData.php
index 94ba9cbb0..804b5b647 100644
--- a/op/op.EditUserData.php
+++ b/op/op.EditUserData.php
@@ -25,7 +25,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("edit_user_details"),getMLText("access_denied"));
 }
 
diff --git a/op/op.ManageNotify.php b/op/op.ManageNotify.php
index 7c956f927..e1aa92ab5 100644
--- a/op/op.ManageNotify.php
+++ b/op/op.ManageNotify.php
@@ -25,7 +25,7 @@ include("../inc/inc.ClassUI.php");
 include("../inc/inc.ClassEmail.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("my_account"),getMLText("access_denied"));
 }
 
diff --git a/op/op.UserDefaultKeywords.php b/op/op.UserDefaultKeywords.php
index 88198dbac..02eeb7799 100644
--- a/op/op.UserDefaultKeywords.php
+++ b/op/op.UserDefaultKeywords.php
@@ -25,7 +25,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("edit_default_keywords"),getMLText("access_denied"));
 }
 
diff --git a/op/op.UsrMgr.php b/op/op.UsrMgr.php
index 6732a5a39..bb1517f24 100644
--- a/op/op.UsrMgr.php
+++ b/op/op.UsrMgr.php
@@ -43,13 +43,14 @@ if ($action == "adduser") {
 	$email   = sanitizeString($_POST["email"]);
 	$comment = sanitizeString($_POST["comment"]);
 	$isAdmin = (isset($_POST["isadmin"]) && $_POST["isadmin"]==1 ? 1 : 0);
+	$isGuest = (isset($_POST["isguest"]) && $_POST["isguest"]==1 ? 1 : 0);
 	$isHidden = (isset($_POST["ishidden"]) && $_POST["ishidden"]==1 ? 1 : 0);
 
 	if (is_object($dms->getUserByLogin($login))) {
 		UI::exitError(getMLText("admin_tools"),getMLText("user_exists"));
 	}
 
-	$newUser = $dms->addUser($login, md5($_POST["pwd"]), $name, $email, $settings->_language, $settings->_theme, $comment, $isAdmin, $isHidden);
+	$newUser = $dms->addUser($login, md5($_POST["pwd"]), $name, $email, $settings->_language, $settings->_theme, $comment, $isAdmin, $isGuest, $isHidden);
 	if ($newUser) {
 
 		if (isset($_FILES["userfile"]) && is_uploaded_file($_FILES["userfile"]["tmp_name"]) && $_FILES["userfile"]["size"] > 0 && $_FILES['userfile']['error']==0)
@@ -106,7 +107,7 @@ else if ($action == "removeuser") {
 		$userid = $_GET["userid"];
 	}
 
-	if (($userid==$settings->_adminID)||($userid==$settings->_guestID)) {
+	if ($userid==$settings->_adminID) {
 		UI::exitError(getMLText("admin_tools"),getMLText("cannot_delete_admin"));
 	}
 	if (!isset($userid) || !is_numeric($userid) || intval($userid)<1) {
@@ -114,7 +115,7 @@ else if ($action == "removeuser") {
 	}
 
 	$userToRemove = $dms->getUser($userid);
-	if (!is_object($userToRemove) || ($userToRemove->getID() == $settings->_adminID) || ($userToRemove->getID() == $settings->_guestID)) {
+	if (!is_object($userToRemove) || ($userToRemove->getID() == $settings->_adminID)) {
 		UI::exitError(getMLText("admin_tools"),getMLText("invalid_user_id"));
 	}
 
@@ -148,6 +149,7 @@ else if ($action == "edituser") {
 	$email   = sanitizeString($_POST["email"]);
 	$comment = sanitizeString($_POST["comment"]);
 	$isAdmin = (isset($_POST["isadmin"]) && $_POST["isadmin"]==1 ? 1 : 0);
+	$isGuest = (isset($_POST["isguest"]) && $_POST["isguest"]==1 ? 1 : 0);
 	$isHidden = (isset($_POST["ishidden"]) && $_POST["ishidden"]==1 ? 1 : 0);
 	
 	if ($editedUser->getLogin() != $login)
@@ -162,6 +164,8 @@ else if ($action == "edituser") {
 		$editedUser->setComment($comment);
 	if ($editedUser->isAdmin() != $isAdmin)
 		$editedUser->setAdmin($isAdmin);
+	if ($editedUser->isGuest() != $isGuest)
+		$editedUser->setGuest($isGuest);
 	if ($editedUser->isHidden() != $isHidden)
 		$editedUser->setHidden($isHidden);
 
diff --git a/out/out.AddEvent.php b/out/out.AddEvent.php
index 259fa149c..a7b67a581 100644
--- a/out/out.AddEvent.php
+++ b/out/out.AddEvent.php
@@ -24,7 +24,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("edit_event"),getMLText("access_denied"));
 }
 
diff --git a/out/out.ApprovalSummary.php b/out/out.ApprovalSummary.php
index 08ed4900c..c2da3e371 100644
--- a/out/out.ApprovalSummary.php
+++ b/out/out.ApprovalSummary.php
@@ -27,7 +27,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("my_documents"),getMLText("access_denied"));
 }
 
diff --git a/out/out.DocumentAccess.php b/out/out.DocumentAccess.php
index c3b0e50dd..73ec10153 100644
--- a/out/out.DocumentAccess.php
+++ b/out/out.DocumentAccess.php
@@ -93,7 +93,7 @@ if ($user->isAdmin()) {
 	<?php
 	$owner = $document->getOwner();
 	foreach ($allUsers as $currUser) {
-		if ($currUser->getID() == $settings->_guestID)
+		if ($currUser->isGuest())
 			continue;
 		print "<option value=\"".$currUser->getID()."\"";
 		if ($currUser->getID() == $owner->getID())
@@ -192,7 +192,7 @@ if (count($accessList["users"]) != 0 || count($accessList["groups"]) != 0) {
 <option value="-1"><?php printMLText("select_one");?></option>
 <?php
 foreach ($allUsers as $userObj) {
-	if ($userObj->getID() == $settings->_guestID) {
+	if ($userObj->isGuest()) {
 		continue;
 	}
 	print "<option value=\"".$userObj->getID()."\">" . $userObj->getFullName() . "</option>\n";
diff --git a/out/out.DocumentNotify.php b/out/out.DocumentNotify.php
index d49de6bc7..149d2f640 100644
--- a/out/out.DocumentNotify.php
+++ b/out/out.DocumentNotify.php
@@ -119,11 +119,11 @@ print "</table>\n";
 					if ($user->isAdmin()) {
 						$allUsers = $dms->getAllUsers();
 						foreach ($allUsers as $userObj) {
-							if ($userObj->getID() != $settings->_guestID && !in_array($userObj->getID(), $userNotifyIDs))
+							if (!$userObj->isGuest() && !in_array($userObj->getID(), $userNotifyIDs))
 								print "<option value=\"".$userObj->getID()."\">" . $userObj->getFullName() . "\n";
 						}
 					}
-					else if ($user->getID() != $settings->_guestID && !in_array($user->getID(), $userNotifyIDs)) {
+					elseif (!$user->isGuest() && !in_array($user->getID(), $userNotifyIDs)) {
 						print "<option value=\"".$user->getID()."\">" . $user->getFullName() . "\n";
 					}
 				?>
diff --git a/out/out.EditEvent.php b/out/out.EditEvent.php
index 61d2ee104..69a46cf6a 100644
--- a/out/out.EditEvent.php
+++ b/out/out.EditEvent.php
@@ -25,7 +25,7 @@ include("../inc/inc.ClassUI.php");
 include("../inc/inc.Calendar.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("edit_event"),getMLText("access_denied"));
 }
 
diff --git a/out/out.EditUserData.php b/out/out.EditUserData.php
index f6d75a857..28d6e0338 100644
--- a/out/out.EditUserData.php
+++ b/out/out.EditUserData.php
@@ -26,7 +26,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("edit_user_details"),getMLText("access_denied"));
 }
 
diff --git a/out/out.FolderAccess.php b/out/out.FolderAccess.php
index 4e7f45cef..7ff4dda24 100644
--- a/out/out.FolderAccess.php
+++ b/out/out.FolderAccess.php
@@ -91,7 +91,7 @@ if ($user->isAdmin()) {
 	<?php
 	$owner = $folder->getOwner();
 	foreach ($allUsers as $currUser) {
-		if ($currUser->getID() == $settings->_guestID)
+		if ($currUser->isGuest())
 			continue;
 		print "<option value=\"".$currUser->getID()."\"";
 		if ($currUser->getID() == $owner->getID())
@@ -192,7 +192,7 @@ if ((count($accessList["users"]) != 0) || (count($accessList["groups"]) != 0)) {
 <option value="-1"><?php printMLText("select_one");?>
 <?php
 foreach ($allUsers as $userObj) {
-	if ($userObj->getID() == $settings->_guestID) {
+	if ($userObj->isGuest()) {
 		continue;
 	}
 	print "<option value=\"".$userObj->getID()."\">" . $userObj->getFullName() . "\n";
diff --git a/out/out.FolderNotify.php b/out/out.FolderNotify.php
index 8f5d826c3..2ce22babf 100644
--- a/out/out.FolderNotify.php
+++ b/out/out.FolderNotify.php
@@ -118,11 +118,11 @@ print "</table>\n";
 					if ($user->isAdmin()) {
 						$allUsers = $dms->getAllUsers();
 						foreach ($allUsers as $userObj) {
-							if ($userObj->getID() != $settings->_guestID && !in_array($userObj->getID(), $userNotifyIDs))
+							if (!$userObj->isGuest() && !in_array($userObj->getID(), $userNotifyIDs))
 								print "<option value=\"".$userObj->getID()."\">" . $userObj->getFullName() . "\n";
 						}
 					}
-					else if ($user->getID() != $settings->_guestID && !in_array($user->getID(), $userNotifyIDs)) {
+					elseif (!$user->isGuest() && !in_array($user->getID(), $userNotifyIDs)) {
 						print "<option value=\"".$user->getID()."\">" . $user->getFullName() . "\n";
 					}
 				?>
diff --git a/out/out.GroupView.php b/out/out.GroupView.php
index 34d1d7e1e..352976266 100644
--- a/out/out.GroupView.php
+++ b/out/out.GroupView.php
@@ -24,7 +24,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("my_account"),getMLText("access_denied"));
 }
 
diff --git a/out/out.ManageNotify.php b/out/out.ManageNotify.php
index 7d37e2a5f..cd7685845 100644
--- a/out/out.ManageNotify.php
+++ b/out/out.ManageNotify.php
@@ -24,7 +24,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("my_account"),getMLText("access_denied"));
 }
 
diff --git a/out/out.MyAccount.php b/out/out.MyAccount.php
index 7803ea2b3..b9c17158f 100644
--- a/out/out.MyAccount.php
+++ b/out/out.MyAccount.php
@@ -26,7 +26,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("my_account"),getMLText("access_denied"));
 }
 
diff --git a/out/out.MyDocuments.php b/out/out.MyDocuments.php
index 6efbb1507..336ed5892 100644
--- a/out/out.MyDocuments.php
+++ b/out/out.MyDocuments.php
@@ -26,7 +26,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("my_documents"),getMLText("access_denied"));
 }
 
diff --git a/out/out.RemoveUser.php b/out/out.RemoveUser.php
index 4fe611a6a..e16e82b5a 100644
--- a/out/out.RemoveUser.php
+++ b/out/out.RemoveUser.php
@@ -35,14 +35,12 @@ if (!isset($_GET["userid"]) || !is_numeric($_GET["userid"]) || intval($_GET["use
 }
 
 $userid = $_GET["userid"];
+$currUser = $dms->getUser($userid);
 
-
-if (($userid==$settings->_adminID)||($userid==$settings->_guestID)) {
+if (($userid==$settings->_adminID) || $currUser->isGuest()) {
 	UI::exitError(getMLText("rm_user"),getMLText("access_denied"));
 }
 
-$currUser = $dms->getUser($userid);
-
 if (!is_object($currUser)) {
 	UI::exitError(getMLText("rm_user"),getMLText("invalid_user_id"));
 }
@@ -69,7 +67,7 @@ UI::contentContainerStart();
 <?php
 	$users = $dms->getAllUsers();
 	foreach ($users as $currUser) {
-		if (($currUser->getID() == $settings->_adminID) || ($currUser->getID() == $settings->_guestID) || ($currUser->getID() == $userid) )
+		if (($currUser->getID() == $settings->_adminID) || $currUser->isGuest() || ($currUser->getID() == $userid) )
 			continue;
 			
 		if (isset($_GET["userid"]) && $currUser->getID()==$_GET["userid"]) $selected=$count;
diff --git a/out/out.ReviewSummary.php b/out/out.ReviewSummary.php
index 56fbdd19d..5b06dc7b5 100644
--- a/out/out.ReviewSummary.php
+++ b/out/out.ReviewSummary.php
@@ -26,7 +26,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("my_documents"),getMLText("access_denied"));
 }
 
diff --git a/out/out.SearchForm.php b/out/out.SearchForm.php
index 3bb9d6a80..2c4c3e70f 100644
--- a/out/out.SearchForm.php
+++ b/out/out.SearchForm.php
@@ -112,7 +112,7 @@ UI::contentContainerStart();
 $allUsers = getAllUsers();
 foreach ($allUsers as $userObj)
 {
-	if ($userObj->getID() == $settings->_guestID)
+	if ($userObj->isGuest())
 		continue;
 	print "<option value=\"".$userObj->getID()."\">" . $userObj->getFullName() . "\n";
 }
diff --git a/out/out.UserDefaultKeywords.php b/out/out.UserDefaultKeywords.php
index 4e1ee147d..33db5079a 100644
--- a/out/out.UserDefaultKeywords.php
+++ b/out/out.UserDefaultKeywords.php
@@ -25,7 +25,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("edit_default_keywords"),getMLText("access_denied"));
 }
 
diff --git a/out/out.UserList.php b/out/out.UserList.php
index 714a0de2e..7af984302 100644
--- a/out/out.UserList.php
+++ b/out/out.UserList.php
@@ -37,7 +37,7 @@ UI::contentContainerStart();
 $users = getAllUsers();
 for ($i = 0; $i < count($users); $i++) {
 	$currUser = $users[$i];
-	if ($currUser->getID() == $settings->_guestID)
+	if ($currUser->isGuest())
 		continue;
 
 	UI::contentSubHeading(getMLText("user") . ": \"" . $currUser->getFullName() . "\"");
diff --git a/out/out.UsrMgr.php b/out/out.UsrMgr.php
index c9beedcfc..2cdd3b181 100644
--- a/out/out.UsrMgr.php
+++ b/out/out.UsrMgr.php
@@ -101,7 +101,7 @@ UI::contentContainerStart();
 	$selected=0;
 	$count=2;
 	foreach ($users as $currUser) {
-		if (($currUser->getID() == $settings->_adminID) || ($currUser->getID() == $settings->_guestID))
+		if (($currUser->getID() == $settings->_adminID) || $currUser->isGuest())
 			continue;
 			
 		if (isset($_GET["userid"]) && $currUser->getID()==$_GET["userid"]) $selected=$count;
@@ -168,7 +168,7 @@ UI::contentContainerStart();
 				<ul class="cbSelectList"><?php
 				foreach ($users as $usr) {
 				
-					if ($usr->getID() == $settings->_guestID) continue;
+					if ($usr->isGuest()) continue;
 				
 					print "<li class=\"cbSelectItem\"><input id='revUsr".$usr->getID()."' type='checkbox' name='usrReviewers[]' value='". $usr->getID() ."'>".$usr->getLogin();
 				}
@@ -198,7 +198,7 @@ UI::contentContainerStart();
 <?php
 				foreach ($users as $usr) {
 				
-					if ($usr->getID() == $settings->_guestID) continue;
+					if ($usr->isGuest()) continue;
 				
 					print "<li class=\"cbSelectItem\"><input id='appUsr".$usr->getID()."' type='checkbox' name='usrApprovers[]' value='". $usr->getID() ."'>".$usr->getLogin();
 				}
@@ -230,7 +230,7 @@ UI::contentContainerStart();
 	<?php
 	foreach ($users as $currUser) {
 	
-		if (($currUser->getID() == $settings->_adminID) || ($currUser->getID() == $settings->_guestID))
+		if (($currUser->getID() == $settings->_adminID) || $currUser->isGuest())
 			continue;
 		
 		print "<td id=\"keywords".$currUser->getID()."\" style=\"display : none;\">";
@@ -274,6 +274,10 @@ UI::contentContainerStart();
 			<td><?php printMLText("is_admin");?>:</td>
 			<td><input type="checkbox" name="isadmin" value="1"<?php print ($currUser->isAdmin() ? " checked='checked'" : "");?>></td>
 		</tr>
+		<tr>
+			<td><?php printMLText("is_guest");?>:</td>
+			<td><input type="checkbox" name="isguest" value="1"<?php print ($currUser->isGuest() ? " checked='checked'" : "");?>></td>
+		</tr>
 		<tr>
 			<td><?php printMLText("is_hidden");?>:</td>
 			<td><input type="checkbox" name="ishidden" value="1"<?php print ($currUser->isHidden() ? " checked='checked'" : "");?>></td>
@@ -312,7 +316,7 @@ UI::contentContainerStart();
 
 				foreach ($users as $usr) {
 				
-					if (($usr->getID() == $settings->_guestID) || ($usr->getID() == $currUser->getID()))
+					if ($usr->isGuest() || ($usr->getID() == $currUser->getID()))
 						continue;
 						
 					$checked=false;
@@ -352,7 +356,7 @@ UI::contentContainerStart();
 				
 				foreach ($users as $usr) {
 				
-					if (($usr->getID() == $settings->_guestID) || ($usr->getID() == $currUser->getID()))
+					if ($usr->isGuest() || ($usr->getID() == $currUser->getID()))
 						continue;
 						
 					$checked=false;
diff --git a/out/out.UsrView.php b/out/out.UsrView.php
index 4277d5f83..57c411832 100644
--- a/out/out.UsrView.php
+++ b/out/out.UsrView.php
@@ -24,7 +24,7 @@ include("../inc/inc.Language.php");
 include("../inc/inc.ClassUI.php");
 include("../inc/inc.Authentication.php");
 
-if ($user->getID() == $settings->_guestID) {
+if ($user->isGuest()) {
 	UI::exitError(getMLText("my_account"),getMLText("access_denied"));
 }
 
@@ -55,7 +55,7 @@ echo "</tr>\n</thead>\n";
 
 foreach ($users as $currUser) {
 
-	if (($currUser->getID() == $settings->_adminID) || ($currUser->getID() == $settings->_guestID))
+	if (($currUser->getID() == $settings->_adminID) || $currUser->isGuest())
 		continue;
 		
 	if ($currUser->isHidden()=="1") continue;
diff --git a/out/out.ViewDocument.php b/out/out.ViewDocument.php
index 08d748138..106c467a1 100644
--- a/out/out.ViewDocument.php
+++ b/out/out.ViewDocument.php
@@ -491,7 +491,7 @@ if (count($links) > 0) {
 }
 else printMLText("empty_notify_list");
 
-if ($user->getID() != $settings->_guestID){
+if (!$user->isGuest()){
 ?>
 	<br>
 	<form action="../op/op.AddDocumentLink.php" name="form1">