FIX Dolibarr#32072 PUT API call does not handle extra fields in shipm…

…ent module Store extra field during updating the shipment object
denes44 · Nov 23, 2024 · a599216 · a599216
1 parent 2b264ae
commit a599216
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/htdocs/expedition/class/api_shipments.class.php b/htdocs/expedition/class/api_shipments.class.php
@@ -483,7 +483,12 @@ public function put($id, $request_data = null)
 				$this->shipment->context['caller'] = sanitizeVal($request_data['caller'], 'aZ09');
 				continue;
 			}
-
+			if ($field == 'array_options' && is_array($value)) {
+				foreach ($value as $index => $val) {
+					$this->shipment->array_options[$index] = $this->_checkValForAPI($field, $val, $this->shipment);
+				}
+				continue;
+			}
 			$this->shipment->$field = $this->_checkValForAPI($field, $value, $this->shipment);
 		}
 

diff --git a/htdocs/expedition/class/expedition.class.php b/htdocs/expedition/class/expedition.class.php
@@ -1252,6 +1252,14 @@ public function update($user = null, $notrigger = 0)
 			$this->errors[] = "Error ".$this->db->lasterror();
 		}
 
+		// Actions on extra fields
+		if (!$error) {
+			$result = $this->insertExtraFields();
+			if ($result < 0) {
+				$error++;
+			}
+		}
+
 		if (!$error && !$notrigger) {
 			// Call trigger
 			$result = $this->call_trigger('SHIPPING_MODIFY', $user);