Welcome to mirror list, hosted at ThFree Co, Russian Federation.

github.com/nextcloud/server.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorChristoph Wurst <christoph@winzerhof-wurst.at>2020-10-14 14:00:20 +0300
committerChristoph Wurst <christoph@winzerhof-wurst.at>2020-10-14 16:40:12 +0300
commitf464ef050bd329f8cac3bb20c932e2021ab49d37 (patch)
treedcdb02e6b8aed1f13d7549e0a82b33c12a787a4c /lib
parentf9f20b63f4d98bd316f03b35ca3c3222aa333d49 (diff)
Fix type errors detected by Psalm
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Diffstat (limited to 'lib')
-rw-r--r--lib/base.php1
-rw-r--r--lib/private/AppFramework/App.php9
-rw-r--r--lib/private/Collaboration/Collaborators/Search.php4
3 files changed, 10 insertions, 4 deletions
diff --git a/lib/base.php b/lib/base.php
index bb9b8963419..a27e8ffc920 100644
--- a/lib/base.php
+++ b/lib/base.php
@@ -859,6 +859,7 @@ class OC {
}
private static function registerAppRestrictionsHooks() {
+ /** @var \OC\Group\Manager $groupManager */
$groupManager = self::$server->query(\OCP\IGroupManager::class);
$groupManager->listen('\OC\Group', 'postDelete', function (\OCP\IGroup $group) {
$appManager = self::$server->getAppManager();
diff --git a/lib/private/AppFramework/App.php b/lib/private/AppFramework/App.php
index d93849a6db6..d6bf7bc81c3 100644
--- a/lib/private/AppFramework/App.php
+++ b/lib/private/AppFramework/App.php
@@ -34,6 +34,7 @@ namespace OC\AppFramework;
use OC\AppFramework\DependencyInjection\DIContainer;
use OC\AppFramework\Http\Dispatcher;
+use OC\AppFramework\Http\Request;
use OC\HintException;
use OCP\AppFramework\Http;
use OCP\AppFramework\Http\ICallbackResponse;
@@ -114,9 +115,13 @@ class App {
*/
public static function main(string $controllerName, string $methodName, DIContainer $container, array $urlParams = null) {
if (!is_null($urlParams)) {
- $container->query(IRequest::class)->setUrlParameters($urlParams);
+ /** @var Request $request */
+ $request = $container->query(IRequest::class);
+ $request->setUrlParameters($urlParams);
} elseif (isset($container['urlParams']) && !is_null($container['urlParams'])) {
- $container->query(IRequest::class)->setUrlParameters($container['urlParams']);
+ /** @var Request $request */
+ $request = $container->query(IRequest::class);
+ $request->setUrlParameters($container['urlParams']);
}
$appName = $container['AppName'];
diff --git a/lib/private/Collaboration/Collaborators/Search.php b/lib/private/Collaboration/Collaborators/Search.php
index e7773858e5c..1e0fd739793 100644
--- a/lib/private/Collaboration/Collaborators/Search.php
+++ b/lib/private/Collaboration/Collaborators/Search.php
@@ -68,14 +68,14 @@ class Search implements ISearch {
foreach ($this->pluginList[$type] as $plugin) {
/** @var ISearchPlugin $searchPlugin */
$searchPlugin = $this->c->resolve($plugin);
- $hasMoreResults |= $searchPlugin->search($search, $limit, $offset, $searchResult);
+ $hasMoreResults = $searchPlugin->search($search, $limit, $offset, $searchResult) || $hasMoreResults;
}
}
// Get from lookup server, not a separate share type
if ($lookup) {
$searchPlugin = $this->c->resolve(LookupPlugin::class);
- $hasMoreResults |= $searchPlugin->search($search, $limit, $offset, $searchResult);
+ $hasMoreResults = $searchPlugin->search($search, $limit, $offset, $searchResult) || $hasMoreResults;
}
// sanitizing, could go into the plugins as well