diff --git a/htdocs/install/mysql/tables/llx_intracommreport.sql b/htdocs/install/mysql/tables/llx_intracommreport.sql index 9d89320c1fa..d3d2956e4ab 100644 --- a/htdocs/install/mysql/tables/llx_intracommreport.sql +++ b/htdocs/install/mysql/tables/llx_intracommreport.sql @@ -20,7 +20,6 @@ create table llx_intracommreport ( rowid integer AUTO_INCREMENT PRIMARY KEY, - ref varchar(30) NOT NULL, -- report reference number entity integer DEFAULT 1 NOT NULL, -- multi company id type_declaration varchar(32), diff --git a/htdocs/intracommreport/card.php b/htdocs/intracommreport/card.php index 668eca12674..f0a4bd5f070 100644 --- a/htdocs/intracommreport/card.php +++ b/htdocs/intracommreport/card.php @@ -30,54 +30,57 @@ $langs->loadLangs(array("intracommreport")); $action = GETPOST('action'); $exporttype = GETPOST('exporttype'); // DEB ou DES -if (empty($exporttype)) $exporttype = 'deb'; +if (empty($exporttype)) { + $exporttype = 'deb'; +} $form = new Form($db); $formother = new FormOther($db); $year = GETPOST('year'); $month = GETPOST('month'); -$type_declaration = GETPOST('type'); +$label = (string) GETPOST('label', 'alphanohtml'); +$type_declaration = GETPOSTINT('type'); $backtopage = GETPOST('backtopage', 'alpha'); +$declaration = array( + "deb" => $langs->trans("DEB"), + "des" => $langs->trans("DES"), +); /* * Actions */ -if ($user->rights->intracommreport->delete && $action == 'confirm_delete' && $confirm == 'yes') -{ +if ($user->rights->intracommreport->delete && $action == 'confirm_delete' && $confirm == 'yes') { $result = $object->delete($id, $user); - if ($result > 0) - { - if (!empty($backtopage)) - { + if ($result > 0) { + if (!empty($backtopage)) { header("Location: ".$backtopage); exit; - } - else { + } else { header("Location: list.php"); exit; } - } - else { + } else { $errmesg = $object->error; } } if ($action == 'add' && $user->rights->intracommreport->write) { $object->label = trim($label); - $object->type = trim($type); - $object->type_declaration = (int) $statut; - $object->subscription = (int) $subscription; + $object->type = trim($exporttype); + $object->type_declaration = $type_declaration; + $object->subscription = (int) $subscription; // Fill array 'array_options' with data from add form $ret = $extrafields->setOptionalsFromPost($extralabels, $object); - if ($ret < 0) $error++; + if ($ret < 0) { + $error++; + } if (empty($object->label)) { $error++; setEventMessages($langs->trans("ErrorFieldRequired", $langs->transnoentities("Label")), null, 'errors'); - } - else { + } else { $sql = "SELECT libelle FROM ".MAIN_DB_PREFIX."adherent_type WHERE libelle='".$db->escape($object->label)."'"; $result = $db->query($sql); if ($result) { @@ -90,20 +93,16 @@ if ($action == 'add' && $user->rights->intracommreport->write) { } } - if (!$error) - { + if (!$error) { $id = $object->create($user); - if ($id > 0) - { + if ($id > 0) { header("Location: ".$_SERVER["PHP_SELF"]); exit; - } - else { + } else { setEventMessages($object->error, $object->errors, 'errors'); $action = 'create'; } - } - else { + } else { $action = 'create'; } } @@ -113,15 +112,14 @@ if ($action == 'add' && $user->rights->intracommreport->write) { */ // Creation mode -if ($action == 'create') -{ +if ($action == 'create') { $title = $langs->trans("IntracommReportTitle"); llxHeader("", $title); print load_fiche_titre($langs->trans("IntracommReportTitle")); print '