Missing context in cleanUserShares() could create an error on user deletion. Close #1279

cdujeu · cdujeu · commit 9933d87c65f1 · 2016-11-14T12:54:29.000+01:00
diff --git a/core/src/plugins/action.share/src/ShareCenter.php b/core/src/plugins/action.share/src/ShareCenter.php
@@ -1177,7 +1177,7 @@ public function updateNodeSharedData($oldNode=null, $newNode=null, $copy = false
      * @param String $userId
      */
     public function cleanUserShares($ctx, $userId){
-        $shares = $this->getShareStore()->listShares($userId);
+        $shares = $this->getShareStore($ctx)->listShares($userId);
         foreach($shares as $hash => $data){
             $this->getShareStore()->deleteShare($data['SHARE_TYPE'], $hash, false, true);
         }

Original file line number	Diff line number	Diff line change
`@@ -1177,7 +1177,7 @@ public function updateNodeSharedData($oldNode=null, $newNode=null, $copy = false`
`1177`	`1177`	`* @param String $userId`
`1178`	`1178`	`*/`
`1179`	`1179`	`public function cleanUserShares($ctx, $userId){`
`1180`		`- $shares = $this->getShareStore()->listShares($userId);`
	`1180`	`+ $shares = $this->getShareStore($ctx)->listShares($userId);`
`1181`	`1181`	`foreach($shares as $hash => $data){`
`1182`	`1182`	`$this->getShareStore()->deleteShare($data['SHARE_TYPE'], $hash, false, true);`
`1183`	`1183`	`}`