]> source.dussan.org Git - nextcloud-server.git/commit
Skip null groups in group manager (#26871) (#26956)
authorVincent Petry <pvince81@owncloud.com>
Thu, 19 Jan 2017 14:02:46 +0000 (15:02 +0100)
committerMorris Jobke <hey@morrisjobke.de>
Fri, 17 Mar 2017 06:07:03 +0000 (00:07 -0600)
commit377fdf3860e317b68392c678fa25e081251baecc
tree3ca0e680377018a10df0dbf4999dd27d679e3c43
parent39afcbd49feb4ba333746762fe7c9d4701db9860
Skip null groups in group manager (#26871) (#26956)

* Skip null groups in group manager (#26871)

* Skip null groups in group manager

* Also skip null groups in group manager's search function

* Add more group null checks in sharing code

* Add unit tests for null group safety in group manager

* Add unit tests for sharing code null group checks

* Added tests for null groups handling in sharing code

* Ignore moveShare optional repair in mount provider

In some cases, data is inconsistent in the oc_share table due to legacy
data. The mount provider might attempt to make it consistent but if the
target group does not exist any more it cannot work. In such case we
simply ignore the exception as it is not critical. Keeping the
exception would break user accounts as they would be unable to use
their filesystem.

* Adjust null group handing + tests

* Fix new group manager tests

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
apps/files_sharing/lib/Controller/ShareAPIController.php
apps/files_sharing/lib/MountProvider.php
apps/files_sharing/tests/Controller/ShareAPIControllerTest.php
apps/files_sharing/tests/MountProviderTest.php
lib/private/Group/Manager.php
lib/private/Share20/DefaultShareProvider.php
lib/private/Share20/Manager.php
tests/lib/Group/ManagerTest.php
tests/lib/Share20/DefaultShareProviderTest.php
tests/lib/Share20/ManagerTest.php