Let OC\Group\Group handle the fallback and remove default implementat…

…ion from ABackend Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
nextcloud · Dec 22, 2022 · 2cc2e13 · 2cc2e13
1 parent fc4212d
commit 2cc2e13
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 30 deletions.
diff --git a/apps/user_ldap/lib/Group_LDAP.php b/apps/user_ldap/lib/Group_LDAP.php
@@ -48,12 +48,11 @@
 use OC\ServerNotAvailableException;
 use OCP\Cache\CappedMemoryCache;
 use OCP\GroupInterface;
-use OCP\Group\Backend\ABackend;
 use OCP\Group\Backend\IDeleteGroupBackend;
 use OCP\Group\Backend\IGetDisplayNameBackend;
 use Psr\Log\LoggerInterface;
 
-class Group_LDAP extends ABackend implements GroupInterface, IGroupLDAP, IGetDisplayNameBackend, IDeleteGroupBackend {
+class Group_LDAP extends BackendUtility implements GroupInterface, IGroupLDAP, IGetDisplayNameBackend, IDeleteGroupBackend {
 	protected bool $enabled = false;
 
 	/** @var CappedMemoryCache<string[]> $cachedGroupMembers array of users with gid as key */
@@ -64,15 +63,14 @@ class Group_LDAP extends ABackend implements GroupInterface, IGroupLDAP, IGetDis
 	protected CappedMemoryCache $cachedNestedGroups;
 	protected GroupPluginManager $groupPluginManager;
 	protected LoggerInterface $logger;
-	protected Access $access;
 
 	/**
 	 * @var string $ldapGroupMemberAssocAttr contains the LDAP setting (in lower case) with the same name
 	 */
 	protected string $ldapGroupMemberAssocAttr;
 
 	public function __construct(Access $access, GroupPluginManager $groupPluginManager) {
-		$this->access = $access;
+		parent::__construct($access);
 		$filter = $this->access->connection->ldapGroupFilter;
 		$gAssoc = $this->access->connection->ldapGroupMemberAssocAttr;
 		if (!empty($filter) && !empty($gAssoc)) {
@@ -1333,11 +1331,4 @@ public function getDisplayName(string $gid): string {
 		$this->access->connection->writeToCache($cacheKey, $displayName);
 		return $displayName;
 	}
-
-	public function searchInGroup(string $gid, string $search = '', int $limit = -1, int $offset = 0): array {
-		if (!$this->enabled) {
-			return [];
-		}
-		return parent::searchInGroup($gid, $search, $limit, $offset);
-	}
 }
diff --git a/build/psalm-baseline-ocp.xml b/build/psalm-baseline-ocp.xml
@@ -5,11 +5,6 @@
       <code>OC</code>
     </UndefinedClass>
   </file>
-  <file src="lib/public/Group/Backend/ABackend.php">
-    <UndefinedClass occurrences="2">
-      <code>OC</code>
-    </UndefinedClass>
-  </file>
   <file src="lib/public/AppFramework/ApiController.php">
     <NoInterfaceProperties occurrences="1">
       <code>$this-&gt;request-&gt;server</code>

diff --git a/lib/public/Group/Backend/ABackend.php b/lib/public/Group/Backend/ABackend.php
@@ -26,15 +26,11 @@
 namespace OCP\Group\Backend;
 
 use OCP\GroupInterface;
-use OCP\IUserManager;
-use OCP\Server;
-use OC\User\LazyUser;
 
 /**
  * @since 14.0.0
  */
 abstract class ABackend implements GroupInterface, ISearchableGroupBackend {
-
 	/**
 	 * @deprecated 14.0.0
 	 *
@@ -68,14 +64,4 @@ public function implementsActions($actions): bool {
 
 		return (bool)($actions & $implements);
 	}
-
-	public function searchInGroup(string $gid, string $search = '', int $limit = -1, int $offset = 0): array {
-		// Default implementation for compatibility reasons
-		$userManager = Server::get(IUserManager::class);
-		$users = [];
-		foreach ($this->usersInGroup($gid, $search, $limit, $offset) as $userId) {
-			$users[$userId] = new LazyUser($userId, $userManager);
-		}
-		return $users;
-	}
 }