From d848e5922a02983c19d5c05e1dd90d76086d1f3c Mon Sep 17 00:00:00 2001
From: jygaulier <gaulier@alchemy.fr>
Date: Thu, 7 Jan 2021 10:34:44 +0100
Subject: [PATCH] cs

---
 .../Worker/WriteMetadatasWorker.php           | 29 ++++++++++---------
 1 file changed, 16 insertions(+), 13 deletions(-)

diff --git a/lib/Alchemy/Phrasea/WorkerManager/Worker/WriteMetadatasWorker.php b/lib/Alchemy/Phrasea/WorkerManager/Worker/WriteMetadatasWorker.php
index f651630d60..153fc16bd0 100644
--- a/lib/Alchemy/Phrasea/WorkerManager/Worker/WriteMetadatasWorker.php
+++ b/lib/Alchemy/Phrasea/WorkerManager/Worker/WriteMetadatasWorker.php
@@ -12,6 +12,8 @@ use Alchemy\Phrasea\Model\Repositories\WorkerRunningJobRepository;
 use Alchemy\Phrasea\WorkerManager\Event\SubdefinitionWritemetaEvent;
 use Alchemy\Phrasea\WorkerManager\Event\WorkerEvents;
 use Alchemy\Phrasea\WorkerManager\Queue\MessagePublisher;
+use DateTime;
+use Exception;
 use Monolog\Logger;
 use PHPExiftool\Driver\Metadata\Metadata;
 use PHPExiftool\Driver\Metadata\MetadataBag;
@@ -21,6 +23,7 @@ use PHPExiftool\Driver\Value\Multi;
 use PHPExiftool\Exception\TagUnknown;
 use PHPExiftool\Writer;
 use Psr\Log\LoggerInterface;
+use record_adapter;
 
 class WriteMetadatasWorker implements WorkerInterface
 {
@@ -113,7 +116,7 @@ class WriteMetadatasWorker implements WorkerInterface
                 $em->beginTransaction();
 
                 try {
-                    $date = new \DateTime();
+                    $date = new DateTime();
                     $workerRunningJob = new WorkerRunningJob();
                     $workerRunningJob
                         ->setDataboxId($databoxId)
@@ -129,14 +132,14 @@ class WriteMetadatasWorker implements WorkerInterface
                     $em->flush();
 
                     $em->commit();
-                } catch (\Exception $e) {
+                } catch (Exception $e) {
                     $em->rollback();
                 }
             }
 
             try {
                 $subdef = $record->get_subdef($payload['subdefName']);
-            } catch (\Exception $e) {
+            } catch (Exception $e) {
                 $workerMessage = "Exception catched when try to get subdef " .$payload['subdefName']. " from DB for the recordID: " .$recordId;
                 $this->logger->error($workerMessage);
 
@@ -221,7 +224,7 @@ class WriteMetadatasWorker implements WorkerInterface
                                 try {
                                     $value = self::fixDate($value); // will return NULL if the date is not valid
                                 }
-                                catch (\Exception $e) {
+                                catch (Exception $e) {
                                     $value = null;    // do NOT write back to iptc
                                 }
                             }
@@ -230,7 +233,7 @@ class WriteMetadatasWorker implements WorkerInterface
                                 $value = new Mono($value);
                             }
                         }
-                    } catch(\Exception $e) {
+                    } catch(Exception $e) {
                         // the field is not set in the record, erase it
                         if ($fieldStructure->is_multi()) {
                             $value = new Multi(array(''));
@@ -260,8 +263,8 @@ class WriteMetadatasWorker implements WorkerInterface
                     $this->writer->write($subdef->getRealPath(), $metadata);
 
                     $this->messagePublisher->pushLog(sprintf('meta written for sbasid=%1$d - recordid=%2$d (%3$s)', $databox->get_sbas_id(), $recordId, $subdef->get_name() ));
-                } catch (\Exception $e) {
-                    $workerMessage = sprintf('meta NOT written for sbasid=%1$d - recordid=%2$d (%3$s) because "%s"', $databox->get_sbas_id(), $recordId, $subdef->get_name() , $e->getMessage());
+                } catch (Exception $e) {
+                    $workerMessage = sprintf('meta NOT written for sbasid=%1$d - recordid=%2$d (%3$s) because "%4$s"', $databox->get_sbas_id(), $recordId, $subdef->get_name() , $e->getMessage());
                     $this->logger->error($workerMessage);
 
                     $count = isset($payload['count']) ? $payload['count'] + 1 : 2 ;
@@ -297,11 +300,11 @@ class WriteMetadatasWorker implements WorkerInterface
             $em->beginTransaction();
             try {
                 $workerRunningJob->setStatus(WorkerRunningJob::FINISHED);
-                $workerRunningJob->setFinished(new \DateTime('now'));
+                $workerRunningJob->setFinished(new DateTime('now'));
                 $em->persist($workerRunningJob);
                 $em->flush();
                 $em->commit();
-            } catch (\Exception $e) {
+            } catch (Exception $e) {
                 $em->rollback();
             }
 
@@ -314,7 +317,7 @@ class WriteMetadatasWorker implements WorkerInterface
         return str_replace("\0", "", $value);
     }
 
-    private function updateJeton(\record_adapter $record)
+    private function updateJeton(record_adapter $record)
     {
         $connection = $record->getDatabox()->get_connection();
 
@@ -344,16 +347,16 @@ class WriteMetadatasWorker implements WorkerInterface
             $a = explode(';', preg_replace('/\D+/', ';', trim($value)));
             switch (count($a)) {
                 case 3:     // yyyy;mm;dd
-                    $date = new \DateTime($a[0] . '-' . $a[1] . '-' . $a[2]);
+                    $date = new DateTime($a[0] . '-' . $a[1] . '-' . $a[2]);
                     $date = $date->format('Y-m-d H:i:s');
                     break;
                 case 6:     // yyyy;mm;dd;hh;mm;ss
-                    $date = new \DateTime($a[0] . '-' . $a[1] . '-' . $a[2] . ' ' . $a[3] . ':' . $a[4] . ':' . $a[5]);
+                    $date = new DateTime($a[0] . '-' . $a[1] . '-' . $a[2] . ' ' . $a[3] . ':' . $a[4] . ':' . $a[5]);
                     $date = $date->format('Y-m-d H:i:s');
                     break;
             }
         }
-        catch (\Exception $e) {
+        catch (Exception $e) {
             $date = null;
         }