Skip to content

Commit

Permalink
Merge pull request #34563 from nextcloud/debt/noid/carddav-too-many-a…
Browse files Browse the repository at this point in the history
…rguments

Fix TooManyArguments warning for carddav.php
  • Loading branch information
ChristophWurst authored Oct 17, 2022
2 parents e12df79 + 1818be7 commit 44d2eb8
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
2 changes: 1 addition & 1 deletion apps/dav/appinfo/v1/carddav.php
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@
$server->setBaseUri($baseuri);
// Add plugins
$server->addPlugin(new MaintenancePlugin(\OC::$server->getConfig(), \OC::$server->getL10N('dav')));
$server->addPlugin(new \Sabre\DAV\Auth\Plugin($authBackend, 'ownCloud'));
$server->addPlugin(new \Sabre\DAV\Auth\Plugin($authBackend));
$server->addPlugin(new Plugin());

$server->addPlugin(new LegacyDAVACL());
Expand Down
3 changes: 0 additions & 3 deletions build/psalm-baseline.xml
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,6 @@
</UndefinedGlobalVariable>
</file>
<file src="apps/dav/appinfo/v1/carddav.php">
<TooManyArguments occurrences="1">
<code>new \Sabre\DAV\Auth\Plugin($authBackend, 'ownCloud')</code>
</TooManyArguments>
<UndefinedGlobalVariable occurrences="1">
<code>$baseuri</code>
</UndefinedGlobalVariable>
Expand Down

0 comments on commit 44d2eb8

Please sign in to comment.