-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix call to deprecated circles api #35030
Conversation
/backport to stable25 |
/backport to stable24 |
/backport to stable23 |
Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
8298f50
to
1d09083
Compare
rebased |
} | ||
// checking if user is within said circle | ||
// deprecated API, needs to be edited when ICirclesManager is implemented | ||
\OCA\Circles\Api\v1\Circles::getMember($sharedWith, $userId, 1); |
Check notice
Code scanning / Psalm
UndefinedClass
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a stub for that in build/stubs and psalm.xml ? :)
return false; | ||
} catch (QueryException $e) { | ||
} catch (\Exception $e) { | ||
\OC::$server->getLogger()->logException($e); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\OC::$server->getLogger()->logException($e); | |
\OC::$server->get(LoggerInterface::class)->error($e->getMessage(), ['exception' => $e, 'app' => 'files_sharing']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better even from \OCP\Server
As there is no feedback since a while I will close this ticket. If this is still happening please make sure to upgrade to the latest version. After that, feel free to rebase & reopen. |
the current API to getMember() is deactivated and throws Exception by default.
A patch is in progress in Circles to return something if it exists, and throw exception if not.