Skip to content
Snippets Groups Projects
Verified Commit cf892d29 authored by Florian Azizian's avatar Florian Azizian
Browse files

FIX #12332 TIME 3:00 merge field in indexing form

parent 3d6b481a
No related branches found
No related tags found
No related merge requests found
...@@ -251,18 +251,19 @@ class JnlpController ...@@ -251,18 +251,19 @@ class JnlpController
} }
} }
$mergedDocument = MergeController::mergeDocument([ $mergedDocument = MergeController::mergeDocument([
'content' => file_get_contents($pathToCopy), 'path' => $pathToCopy,
'data' => $dataToMerge 'data' => $dataToMerge
]); ]);
file_put_contents($tmpPath . $newFileOnTmp, base64_decode($mergedDocument['encodedDocument'])); file_put_contents($tmpPath . $newFileOnTmp, base64_decode($mergedDocument['encodedDocument']));
$pathToCopy = $tmpPath . $newFileOnTmp;
} else { } else {
$xmlResponse = JnlpController::generateResponse(['type' => 'ERROR', 'data' => ['ERROR' => 'Wrong objectType']]); $xmlResponse = JnlpController::generateResponse(['type' => 'ERROR', 'data' => ['ERROR' => 'Wrong objectType']]);
$response->write($xmlResponse); $response->write($xmlResponse);
return $response->withHeader('Content-Type', 'application/xml'); 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}"]]); $xmlResponse = JnlpController::generateResponse(['type' => 'ERROR', 'data' => ['ERROR' => "Failed to copy on {$tmpPath} : {$pathToCopy}"]]);
$response->write($xmlResponse); $response->write($xmlResponse);
return $response->withHeader('Content-Type', 'application/xml'); return $response->withHeader('Content-Type', 'application/xml');
......
...@@ -87,7 +87,7 @@ class MergeController ...@@ -87,7 +87,7 @@ class MergeController
//Resource //Resource
if (!empty($args['resId'])) { if (!empty($args['resId'])) {
$resource = ResModel::getById(['select' => ['*'], 'resId' => [$args['resId']]]); $resource = ResModel::getById(['select' => ['*'], 'resId' => $args['resId']]);
} else { } else {
if (!empty($args['modelId'])) { if (!empty($args['modelId'])) {
$indexingModel = IndexingModelModel::getById(['id' => $args['modelId'], 'select' => ['category']]); $indexingModel = IndexingModelModel::getById(['id' => $args['modelId'], 'select' => ['category']]);
...@@ -198,9 +198,10 @@ class MergeController ...@@ -198,9 +198,10 @@ class MergeController
]); ]);
foreach ($copyWorkflow as $value) { foreach ($copyWorkflow as $value) {
if ($value['item_type'] == 'user_id') { 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']]); $primaryentity = UserModel::getPrimaryEntityByUserId(['userId' => $value['item_id']]);
$label = "{$labelledUser} ({$primaryentity})"; $label = "{$labelledUser} ({$primaryentity})";
} else { } else {
$entity = EntityModel::getByEntityId(['entityId' => $value['item_id'], 'select' => ['entity_label']]); $entity = EntityModel::getByEntityId(['entityId' => $value['item_id'], 'select' => ['entity_label']]);
$label = $entity['entity_label']; $label = $entity['entity_label'];
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment