merge all possible conflicts into on array

This commit is contained in:
Björn Schießle 2012-10-02 13:09:34 +02:00
parent 3fdebaa5dc
commit 3e3038cafc
1 changed files with 9 additions and 10 deletions

View File

@ -1002,22 +1002,21 @@ class Share {
} }
} }
// Check if target already exists // Check if target already exists
$targetConflict = false; $checkTarget = array();
if( $itemType == "file" or $itemType == "folder") { if( $itemType == "file" or $itemType == "folder") {
$itemList1 = self::getItems("file", $target, $shareType, $shareWith); $itemList1 = self::getItems("file", $target, $shareType, $shareWith);
$itemList2 = self::getItems("folder", $target, $shareType, $shareWith); $itemList2 = self::getItems("folder", $target, $shareType, $shareWith);
if ( !empty($itemList1) or !empty($itemList2)) { if ( !empty($itemList1) ) {
$targetConflict = true; $checkTarget = array_merge($checkTarget, $itemList1);
}
if ( !empty($itemList2) ) {
$checkTarget = array_merge($checkTarget, $itemList2);
} }
} else { } else {
$itemList = self::getItems($itemType, $target, $shareType, $shareWith); $checkTarget = self::getItems($itemType, $target, $shareType, $shareWith);
if ( !empty($itemList) ) {
$targetConflict = true;
}
} }
if ($targetConflict) { if ( !empty($checkTarget) ) {
foreach ($checkTarget as $item) { foreach ($checkTarget as $item) {
// Skip item if it is the group parent row // Skip item if it is the group parent row
if (isset($groupParent) && $item['id'] == $groupParent) { if (isset($groupParent) && $item['id'] == $groupParent) {