Merge with master

lib/classes/caption/record.class.php

@@ -60,8 +60,9 @@ class caption_record implements caption_interface, cache_cacheableInterface
   protected function retrieve_fields()
   {
     if (is_array($this->fields))
-
+    {
       return $this->fields;
+    }
 
     $fields = array();
     try
@@ -125,6 +126,23 @@ class caption_record implements caption_interface, cache_cacheableInterface
     return $fields;
   }
 
+  /**
+   *
+   * @param type $fieldname
+   * @return \caption_field
+   * @throws \Exception
+   */
+  public function get_field($fieldname)
+  {
+    foreach ($this->retrieve_fields() as $meta_struct_id => $field)
+    {
+      if ($field->get_name() == $fieldname)
+        return $field;
+    }
+
+    throw new \Exception('Field not found');
+  }
+
   /**
    *
    * @param type $label
@@ -134,8 +152,10 @@ class caption_record implements caption_interface, cache_cacheableInterface
   {
     $fields = $this->retrieve_fields();
     if (isset($this->dces_elements[$label]))
-
+    {
       return $fields[$this->dces_elements[$label]];
+    }
+
     return null;
   }
 

lib/classes/record/adapter.class.php

@@ -749,12 +749,14 @@ class record_adapter implements record_Interface, cache_cacheableInterface
     if ($data)
     {
       if (isset($this->technical_datas[$data]))
-
+      {
         return $this->technical_datas[$data];
+      }
       else
-
+      {
         return false;
       }
+    }
 
     return $this->technical_datas;
   }
@@ -839,34 +841,39 @@ class record_adapter implements record_Interface, cache_cacheableInterface
   {
     $this->original_name = $original_name;
 
-    foreach ($this->get_caption()->get_fields() as $field)
+    foreach ($this->get_databox()->get_meta_structure()->get_elements() as $data_field)
     {
-      if ($field->get_databox_field()->get_source() != metadata_description_PHRASEANET_tffilename::get_source())
+      if ($data_field->get_metadata_source() != metadata_description_PHRASEANET_tffilename::get_source())
       {
         continue;
       }
 
-      /* @var $field caption_field */
       /**
        * Replacing original name in multi values is non sense
        */
-      if (!$field->is_multi())
+      if (!$data_field->is_multi())
       {
         continue;
       }
-      else
-      {
-        $value = array_pop($field->get_values());
 
-        $this->set_metadatas(
+      try
-          array(
+      {
-            'meta_struct_id' => $field->get_meta_struct_id()
+        $field   = $this->get_caption()->get_field($data_field->get_name())->get_meta_id();
-            , 'meta_id'        => $value->getId()
+        $value   = array_pop($field->get_values());
-            , 'value'          => $original_name
+        $meta_id = $value->getId();
-          )
-          , true
-        );
       }
+      catch (\Exception $e)
+      {
+        $meta_id = null;
+      }
+
+      $metas = array(
+        'meta_struct_id' => $field->get_meta_struct_id()
+        , 'meta_id'        => $meta_id
+        , 'value'          => $original_name
+      );
+
+      $this->set_metadatas($metas, true);
     }
 
     $sql = 'UPDATE record
@@ -1357,7 +1364,9 @@ class record_adapter implements record_Interface, cache_cacheableInterface
     foreach ($metadatas as $param)
     {
       if (!is_array($param))
-        throw new Exception_InvalidArgument();
+      {
+        throw new Exception_InvalidArgument('Invalid metadatas argument');
+      }
 
       $db_field = \databox_field::get_instance($this->get_databox(), $param['meta_struct_id']);
 
@@ -1489,8 +1498,9 @@ class record_adapter implements record_Interface, cache_cacheableInterface
   public function get_reg_name()
   {
     if (!$this->is_grouping())
-
+    {
       return false;
+    }
 
     $balisename = '';
 
@@ -1633,8 +1643,48 @@ class record_adapter implements record_Interface, cache_cacheableInterface
         , ':value'     => $value
       ));
     }
+
     $stmt->closeCursor();
 
+    foreach ($record->get_databox()->get_meta_structure()->get_elements() as $data_field)
+    {
+      if ($data_field->get_metadata_source() != metadata_description_PHRASEANET_tfrecordid::get_source())
+      {
+
+        continue;
+      }
+
+      /**
+       * Replacing record_id in multi values is non sense
+       */
+      if (!$data_field->is_multi())
+      {
+        continue;
+      }
+
+      try
+      {
+        $field   = $record->get_caption()->get_field($data_field->get_name())->get_meta_id();
+        $value   = array_pop($field->get_values());
+        $meta_id = $value->getId();
+      }
+      catch (\Exception $e)
+      {
+        $meta_id = null;
+      }
+
+      $metas = array(
+        array(
+          'meta_struct_id' => $data_field->get_id()
+          , 'meta_id'        => $meta_id
+          , 'value'          => array($record->get_record_id())
+        )
+      );
+
+      $record->set_metadatas($metas, true);
+    }
+
+
     return $record;
   }
 
@@ -1686,7 +1736,6 @@ class record_adapter implements record_Interface, cache_cacheableInterface
   {
     $hd = $this->get_subdef('document');
     if ($hd->is_physically_present())
-
       return new system_file(p4string::addEndSlash($hd->get_path()) . $hd->get_file());
     return null;
   }