Merge pull request #48014 from nextcloud/fix/tags/boolean-user-has-tags

provokateurin · web-flow · commit 77de180faa98 · 2024-09-15T17:48:27.000+02:00
diff --git a/lib/private/Tags.php b/lib/private/Tags.php
@@ -248,8 +248,7 @@ public function getIdsForTag($tag) {
 	 * @param string $user The user whose tags are to be checked.
 	 */
 	public function userHasTag(string $name, string $user): bool {
-		$key = $this->array_searchi($name, $this->getTagsForUser($user));
-		return ($key !== false) ? $this->tags[$key]->getId() : false;
+		return $this->array_searchi($name, $this->getTagsForUser($user)) !== false;
 	}
 
 	/**

Original file line number	Diff line number	Diff line change
`@@ -248,8 +248,7 @@ public function getIdsForTag($tag) {`
`248`	`248`	`* @param string $user The user whose tags are to be checked.`
`249`	`249`	`*/`
`250`	`250`	`public function userHasTag(string $name, string $user): bool {`
`251`		`- $key = $this->array_searchi($name, $this->getTagsForUser($user));`
`252`		`- return ($key !== false) ? $this->tags[$key]->getId() : false;`
	`251`	`+ return $this->array_searchi($name, $this->getTagsForUser($user)) !== false;`
`253`	`252`	`}`
`254`	`253`
`255`	`254`	`/**`