From 2b0da0f21888ecda6690cc99f290705320054bbc Mon Sep 17 00:00:00 2001 From: Robin Appelman Date: Fri, 5 May 2017 14:44:51 +0200 Subject: [PATCH] handle permissions errors when copying the skeleton for a read only user Signed-off-by: Robin Appelman --- lib/private/User/Session.php | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/lib/private/User/Session.php b/lib/private/User/Session.php index efa11348ef..ac0150ff61 100644 --- a/lib/private/User/Session.php +++ b/lib/private/User/Session.php @@ -46,6 +46,7 @@ use OC_User; use OC_Util; use OCA\DAV\Connector\Sabre\Auth; use OCP\AppFramework\Utility\ITimeFactory; +use OCP\Files\NotPermittedException; use OCP\IConfig; use OCP\IRequest; use OCP\ISession; @@ -480,8 +481,12 @@ class Session implements IUserSession, Emitter { //trigger creation of user home and /files folder $userFolder = \OC::$server->getUserFolder($user); - // copy skeleton - \OC_Util::copySkeleton($user, $userFolder); + try { + // copy skeleton + \OC_Util::copySkeleton($user, $userFolder); + } catch (NotPermittedException $ex) { + // read only uses + } // trigger any other initialization \OC::$server->getEventDispatcher()->dispatch(IUser::class . '::firstLogin', new GenericEvent($this->getUser()));