diff --git a/htdocs/margin/admin/margin.php b/htdocs/margin/admin/margin.php index 1035c45cbcc..399dc03cc14 100644 --- a/htdocs/margin/admin/margin.php +++ b/htdocs/margin/admin/margin.php @@ -23,13 +23,7 @@ * \brief Page to setup margin module */ -include '../../main.inc.php'; - -require_once DOL_DOCUMENT_ROOT.'/margin/lib/margins.lib.php'; -require_once DOL_DOCUMENT_ROOT.'/core/lib/admin.lib.php'; -require_once DOL_DOCUMENT_ROOT.'/core/class/html.formcompany.class.php'; -require_once DOL_DOCUMENT_ROOT."/compta/facture/class/facture.class.php"; - +require_once '../../main.inc.php'; /** * @var Conf $conf * @var DoliDB $db @@ -37,6 +31,10 @@ require_once DOL_DOCUMENT_ROOT."/compta/facture/class/facture.class.php"; * @var Translate $langs * @var User $user */ +require_once DOL_DOCUMENT_ROOT.'/margin/lib/margins.lib.php'; +require_once DOL_DOCUMENT_ROOT.'/core/lib/admin.lib.php'; +require_once DOL_DOCUMENT_ROOT.'/core/class/html.formcompany.class.php'; +require_once DOL_DOCUMENT_ROOT."/compta/facture/class/facture.class.php"; $langs->loadLangs(array("admin", "bills", "margins", "stocks")); @@ -72,29 +70,25 @@ if (preg_match('/del_([a-z0-9_\-]+)/i', $action, $reg)) { } } -if ($action == 'remises') { - if (dolibarr_set_const($db, 'MARGIN_METHODE_FOR_DISCOUNT', GETPOST('MARGIN_METHODE_FOR_DISCOUNT'), 'chaine', 0, '', $conf->entity) > 0) { - setEventMessages($langs->trans("RecordModifiedSuccessfully"), null, 'mesgs'); - } else { +if ($action == 'update') { + $error = 0; + if (dolibarr_set_const($db, 'MARGIN_METHODE_FOR_DISCOUNT', GETPOST('MARGIN_METHODE_FOR_DISCOUNT'), 'chaine', 0, '', $conf->entity) <= 0) { dol_print_error($db); } + + if (dolibarr_set_const($db, 'MARGIN_TYPE', GETPOST('MARGIN_TYPE'), 'chaine', 0, '', $conf->entity) <= 0) { + dol_print_error($db); + } + + if (dolibarr_set_const($db, 'AGENT_CONTACT_TYPE', GETPOST('AGENT_CONTACT_TYPE'), 'chaine', 0, '', $conf->entity) <= 0) { + dol_print_error($db); + } + + if (!$error) { + setEventMessages($langs->trans("RecordModifiedSuccessfully"), null, 'mesgs'); + } } -if ($action == 'typemarges') { - if (dolibarr_set_const($db, 'MARGIN_TYPE', GETPOST('MARGIN_TYPE'), 'chaine', 0, '', $conf->entity) > 0) { - setEventMessages($langs->trans("RecordModifiedSuccessfully"), null, 'mesgs'); - } else { - dol_print_error($db); - } -} - -if ($action == 'contact') { - if (dolibarr_set_const($db, 'AGENT_CONTACT_TYPE', GETPOST('AGENT_CONTACT_TYPE'), 'chaine', 0, '', $conf->entity) > 0) { - setEventMessages($langs->trans("RecordModifiedSuccessfully"), null, 'mesgs'); - } else { - dol_print_error($db); - } -} /* * View @@ -113,53 +107,49 @@ $head = marges_admin_prepare_head(); print dol_get_fiche_head($head, 'parameters', $langs->trans("Margins"), -1, 'margin'); +print '
'; print dol_get_fiche_end();