diff --git a/src/app/contentManagement/controllers/JnlpController.php b/src/app/contentManagement/controllers/JnlpController.php index a318ac171def73e8b710aecfdc644c74e3f132f4..e441819d4b970e0ec119fb097dc08edc4285dc55 100755 --- a/src/app/contentManagement/controllers/JnlpController.php +++ b/src/app/contentManagement/controllers/JnlpController.php @@ -251,18 +251,19 @@ class JnlpController } } $mergedDocument = MergeController::mergeDocument([ - 'content' => file_get_contents($pathToCopy), - 'data' => $dataToMerge + 'path' => $pathToCopy, + 'data' => $dataToMerge ]); file_put_contents($tmpPath . $newFileOnTmp, base64_decode($mergedDocument['encodedDocument'])); + $pathToCopy = $tmpPath . $newFileOnTmp; } else { $xmlResponse = JnlpController::generateResponse(['type' => 'ERROR', 'data' => ['ERROR' => 'Wrong objectType']]); $response->write($xmlResponse); return $response->withHeader('Content-Type', 'application/xml'); } - if (!file_exists($pathToCopy) || !copy($pathToCopy, $tmpPath . $newFileOnTmp)) { + if ( $pathToCopy != $tmpPath . $newFileOnTmp && (!file_exists($pathToCopy) || !copy($pathToCopy, $tmpPath . $newFileOnTmp))) { $xmlResponse = JnlpController::generateResponse(['type' => 'ERROR', 'data' => ['ERROR' => "Failed to copy on {$tmpPath} : {$pathToCopy}"]]); $response->write($xmlResponse); return $response->withHeader('Content-Type', 'application/xml'); diff --git a/src/app/contentManagement/controllers/MergeController.php b/src/app/contentManagement/controllers/MergeController.php index ba4917ec64d759b01e94e31366c811e94e4e8462..103e3e133e091e3ed94cdd6c96e11106c19c6437 100644 --- a/src/app/contentManagement/controllers/MergeController.php +++ b/src/app/contentManagement/controllers/MergeController.php @@ -87,7 +87,7 @@ class MergeController //Resource if (!empty($args['resId'])) { - $resource = ResModel::getById(['select' => ['*'], 'resId' => [$args['resId']]]); + $resource = ResModel::getById(['select' => ['*'], 'resId' => $args['resId']]); } else { if (!empty($args['modelId'])) { $indexingModel = IndexingModelModel::getById(['id' => $args['modelId'], 'select' => ['category']]); @@ -198,9 +198,10 @@ class MergeController ]); foreach ($copyWorkflow as $value) { if ($value['item_type'] == 'user_id') { - $labelledUser = UserModel::getLabelledUserById(['id' => $value['item_id']]); + $userInfo = UserModel::getByLogin(['login' => $value['item_id'], 'select' => ['id']]); + $labelledUser = UserModel::getLabelledUserById(['id' => $userInfo['id']]); $primaryentity = UserModel::getPrimaryEntityByUserId(['userId' => $value['item_id']]); - $label = "{$labelledUser} ({$primaryentity})"; + $label = "{$labelledUser} ({$primaryentity})"; } else { $entity = EntityModel::getByEntityId(['entityId' => $value['item_id'], 'select' => ['entity_label']]); $label = $entity['entity_label'];