Properly check for legacy methods

The new shares are completely without parents eventually. But for some
current legacy reasons the methods are still around. But we should
properly check for them.

Fixes #22309
This commit is contained in:
Roeland Jago Douma 2016-02-11 11:30:52 +01:00
parent 06af9b817e
commit dd630e33ea
2 changed files with 10 additions and 3 deletions

View File

@ -720,7 +720,7 @@ class Manager implements IManager {
'itemSource' => $share->getNodeId(),
'shareType' => $shareType,
'shareWith' => $sharedWith,
'itemparent' => $share->getParent(),
'itemparent' => method_exists($share, 'getParent') ? $share->getParent() : '',
'uidOwner' => $share->getSharedBy(),
'fileSource' => $share->getNodeId(),
'fileTarget' => $share->getTarget()

View File

@ -321,7 +321,11 @@ class Share implements \OCP\Share\IShare {
}
/**
* @inheritdoc
* Set the parent of this share
*
* @param int parent
* @return \OCP\Share\IShare
* @deprecated The new shares do not have parents. This is just here for legacy reasons.
*/
public function setParent($parent) {
$this->parent = $parent;
@ -329,7 +333,10 @@ class Share implements \OCP\Share\IShare {
}
/**
* @inheritdoc
* Get the parent of this share.
*
* @return int
* @deprecated The new shares do not have parents. This is just here for legacy reasons.
*/
public function getParent() {
return $this->parent;