Merge pull request #461 from ONLYOFFICE/feature/limit-groups-share

Feature/limit groups share
ONLYOFFICE · Mar 4, 2024 · 4488157 · 4488157
2 parents 4f54b43 + af8cabf
commit 4488157
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,6 +6,7 @@
 - offline viewer for share link
 - updatable list of supported formats
 - filling pdf instead oform
+- fixed guest redirect when limiting the app to groups
 
 ## Added
 - reference data from coediting

diff --git a/lib/appconfig.php b/lib/appconfig.php
@@ -1070,7 +1070,7 @@ public function isUserAllowedToUse($userId = null) {
 
 		$groups = $this->getLimitGroups();
 		// no group set -> all users are allowed
-		if (\count($groups) === 0) {
+		if (empty($groups)) {
 			return true;
 		}
 

diff --git a/lib/hookhandler.php b/lib/hookhandler.php
@@ -40,7 +40,7 @@ public static function publicPage() {
 
 		$appConfig = new AppConfig($appName);
 
-		if (!empty($appConfig->getDocumentServerUrl()) && $appConfig->settingsAreSuccessful()) {
+		if (!empty($appConfig->getDocumentServerUrl()) && $appConfig->settingsAreSuccessful() && empty($appConfig->getLimitGroups())) {
 			Util::addScript("onlyoffice", "main");
 			Util::addScript("onlyoffice", "share");