Merge pull request Dolibarr#24907 from mc2contributor/fix_element

Fix undefined and invalid access issues

htdocs/projet/element.php

@@ -1164,8 +1164,10 @@
 			for ($i = 0; $i < $num; $i++) {
 				$tmp = explode('_', $elementarray[$i]);
 				$idofelement = $tmp[0];
+
 				$idofelementuser = isset($tmp[1]) ? $tmp[1] : "";
 
+
 				$element->fetch($idofelement);
 				if ($idofelementuser) {
 					$elementuser->fetch($idofelementuser);
@@ -1191,9 +1193,10 @@
 
 					$total_ht_by_third = 0;
 					$total_ttc_by_third = 0;
-				}
+
 				$saved_third_id = !empty($element->thirdparty->id) ? $element->thirdparty->id : 0;
 
+
 				$qualifiedfortotal = true;
 				if ($key == 'invoice') {
 					if (!empty($element->close_code) && $element->close_code == 'replaced') {
@@ -1239,7 +1242,8 @@
 
 					$element_doc = $element->element;
 					$filename = dol_sanitizeFileName($element->ref);
-					$filedir = $conf->{$element_doc}->multidir_output[$element->entity].'/'.dol_sanitizeFileName($element->ref);
+					$path = $conf?->{$element_doc}?->multidir_output[$element->entity] ?? null;
+					$filedir = $path.'/'.dol_sanitizeFileName($element->ref);
 
 					if ($element_doc === 'order_supplier') {
 						$element_doc = 'commande_fournisseur';
@@ -1489,7 +1493,8 @@
 					$total_ht_by_third += $total_ht_by_line;
 					$total_ttc_by_third += $total_ttc_by_line;
 
-					$total_time = $total_time + $total_time_by_line;
+					if (!isset($total_time)) $total_time = $total_time_by_line;
+					else $total_time += $total_time_by_line;
 				}
 
 				if (canApplySubtotalOn($tablename)) {