diff --git a/dev/setup/codesniffer/ruleset.xml b/dev/setup/codesniffer/ruleset.xml
index 015733ed95f..8c01eea6d99 100644
--- a/dev/setup/codesniffer/ruleset.xml
+++ b/dev/setup/codesniffer/ruleset.xml
@@ -186,7 +186,7 @@
-
+
diff --git a/htdocs/contrat/agenda.php b/htdocs/contrat/agenda.php
index e96425a0dae..df1629b46a6 100644
--- a/htdocs/contrat/agenda.php
+++ b/htdocs/contrat/agenda.php
@@ -164,19 +164,20 @@ if ($id > 0)
$morehtmlref.='
'.$langs->trans('Project') . ' ';
if ($user->rights->contrat->creer)
{
- if ($action != 'classify')
+ if ($action != 'classify') {
$morehtmlref.='' . img_edit($langs->transnoentitiesnoconv('SetProject')) . ' : ';
- if ($action == 'classify') {
- //$morehtmlref.=$form->form_project($_SERVER['PHP_SELF'] . '?id=' . $object->id, $object->socid, $object->fk_project, 'projectid', 0, 0, 1, 1);
- $morehtmlref.='
';
- } else {
- $morehtmlref.=$form->form_project($_SERVER['PHP_SELF'] . '?id=' . $object->id, $object->thirdparty->id, $object->fk_project, 'none', 0, 0, 0, 1);
- }
+ }
+ if ($action == 'classify') {
+ //$morehtmlref.=$form->form_project($_SERVER['PHP_SELF'] . '?id=' . $object->id, $object->socid, $object->fk_project, 'projectid', 0, 0, 1, 1);
+ $morehtmlref.='';
+ } else {
+ $morehtmlref.=$form->form_project($_SERVER['PHP_SELF'] . '?id=' . $object->id, $object->thirdparty->id, $object->fk_project, 'none', 0, 0, 0, 1);
+ }
} else {
if (! empty($object->fk_project)) {
$proj = new Project($db);
diff --git a/htdocs/contrat/card.php b/htdocs/contrat/card.php
index ef57c84443b..2ec6f63d84c 100644
--- a/htdocs/contrat/card.php
+++ b/htdocs/contrat/card.php
@@ -331,9 +331,8 @@ if (empty($reshook))
// Extrafields
$array_options = array();
- if (empty($conf->global->MAIN_EXTRAFIELDS_DISABLED) && method_exists($lines[$i], 'fetch_optionals')) // For avoid conflicts if
- // trigger used
- {
+ // For avoid conflicts if trigger used
+ if (empty($conf->global->MAIN_EXTRAFIELDS_DISABLED) && method_exists($lines[$i], 'fetch_optionals')) {
$lines[$i]->fetch_optionals($lines[$i]->rowid);
$array_options = $lines[$i]->array_options;
}
diff --git a/htdocs/contrat/services_list.php b/htdocs/contrat/services_list.php
index fac71516e99..81c0b13449b 100644
--- a/htdocs/contrat/services_list.php
+++ b/htdocs/contrat/services_list.php
@@ -705,17 +705,18 @@ while ($i < min($num, $limit))
// Status
if (! empty($arrayfields['status']['checked']))
{
- print '';
- if ($obj->cstatut == 0) // If contract is draft, we say line is also draft
- {
- print $contractstatic->LibStatut(0, 5);
- }
- else
- {
- print $staticcontratligne->LibStatut($obj->statut, 5, ($obj->date_fin_validite && $db->jdate($obj->date_fin_validite) < $now)?1:0);
- }
- print ' | ';
- if (! $i) $totalarray['nbfield']++;
+ print '';
+ if ($obj->cstatut == 0)
+ {
+ // If contract is draft, we say line is also draft
+ print $contractstatic->LibStatut(0, 5);
+ }
+ else
+ {
+ print $staticcontratligne->LibStatut($obj->statut, 5, ($obj->date_fin_validite && $db->jdate($obj->date_fin_validite) < $now)?1:0);
+ }
+ print ' | ';
+ if (! $i) $totalarray['nbfield']++;
}
// Action column
print '';
diff --git a/htdocs/contrat/tpl/linkedobjectblock.tpl.php b/htdocs/contrat/tpl/linkedobjectblock.tpl.php
index bda12ea87cb..b52a6939882 100644
--- a/htdocs/contrat/tpl/linkedobjectblock.tpl.php
+++ b/htdocs/contrat/tpl/linkedobjectblock.tpl.php
@@ -23,11 +23,9 @@ if (empty($conf) || ! is_object($conf))
exit;
}
-?>
-
+print "\n";
- $objectlink)
$trclass='oddeven';
if ($ilink == count($linkedObjectBlock) && empty($noMoreLinkedObjectBlockAfter) && count($linkedObjectBlock) <= 1) $trclass.=' liste_sub_total';
-?>
+ ?>
|
| trans("Contract"); ?> |
getNomUrl(1); ?> |
|
date_contrat, 'day'); ?> |
rights->contrat->lire && empty($conf->global->CONTRACT_SHOW_TOTAL_OF_PRODUCT_AS_PRICE))
- {
- $totalcontrat = 0;
- foreach ($objectlink->lines as $linecontrat) {
- $totalcontrat = $totalcontrat + $linecontrat->total_ht;
- $total = $total + $linecontrat->total_ht;
- }
- echo price($totalcontrat);
- } ?> |
+ // Price of contract is not shown by default because a contract is a list of service with
+ // start and end date that change with time andd that may be different that the period of reference for price.
+ // So price of a contract does often means nothing. Prices is on the different invoices done on same contract.
+ if ($user->rights->contrat->lire && empty($conf->global->CONTRACT_SHOW_TOTAL_OF_PRODUCT_AS_PRICE))
+ {
+ $totalcontrat = 0;
+ foreach ($objectlink->lines as $linecontrat) {
+ $totalcontrat = $totalcontrat + $linecontrat->total_ht;
+ $total = $total + $linecontrat->total_ht;
+ }
+ echo price($totalcontrat);
+ } ?>
getLibStatut(7); ?> |
- id.'&action=dellink&dellinkid='.$key; ?>">transnoentitiesnoconv("RemoveLink"), 'unlink'); ?> |
+ id.'&action=dellink&dellinkid='.$key.'">'.img_picto($langs->transnoentitiesnoconv("RemoveLink"), 'unlink'); ?> |
-
+
+print "\n";