diff --git a/htdocs/comm/propal.php b/htdocs/comm/propal.php index 5cef6fac21a..4b5b50e1f15 100644 --- a/htdocs/comm/propal.php +++ b/htdocs/comm/propal.php @@ -69,7 +69,7 @@ if ($HTTP_POST_VARS["action"] == 'confirm_delete' && $HTTP_POST_VARS["confirm"] if ($HTTP_POST_VARS["action"] == 'add') { - $propal = new Propal($db, $socidp); + $propal = new Propal($db, $_GET["socidp"]); $propal->datep = mktime(12, 1 , 1, $HTTP_POST_VARS["remonth"], @@ -84,7 +84,7 @@ if ($HTTP_POST_VARS["action"] == 'add') $propal->author = $user->id; $propal->note = $HTTP_POST_VARS["note"]; - $propal->ref = $ref; + $propal->ref = $HTTP_POST_VARS["ref"]; for ($i = 1 ; $i <= PROPALE_NEW_FORM_NB_PRODUCT ; $i++) { @@ -92,7 +92,7 @@ if ($HTTP_POST_VARS["action"] == 'add') $xqty = "qty".$i; $xremise = "remise".$i; - $propal->add_product($$xid,$$xqty,$$xremise); + $propal->add_product($HTTP_POST_VARS[$xid],$HTTP_POST_VARS[$xqty],$HTTP_POST_VARS[$xremise]); } $id = $propal->create(); @@ -120,8 +120,8 @@ if ($HTTP_POST_VARS["action"] == 'setstatut' && $user->rights->propale->cloturer * Cloture de la propale */ $propal = new Propal($db); - $propal->fetch($propalid); - $propal->cloture($user, $statut, $note); + $propal->fetch($_GET["propalid"]); + $propal->cloture($user, $HTTP_POST_VARS["statut"], $HTTP_POST_VARS["note"]); } if ($_GET["action"] == 'commande') @@ -156,6 +156,7 @@ if ($HTTP_POST_VARS["addligne"] == 'Ajouter' && $user->rights->propale->creer) $propal = new Propal($db); $propal->fetch($propalid); $propal->insert_product($HTTP_POST_VARS["idprod"], $HTTP_POST_VARS["qty"], $HTTP_POST_VARS["remise"]); + propale_pdf_create($db, $_GET["propalid"], $propal->modelpdf); } } @@ -184,6 +185,7 @@ if ($HTTP_POST_VARS["action"] == 'setremise' && $user->rights->propale->creer) $propal = new Propal($db); $propal->fetch($propalid); $propal->set_remise($user, $HTTP_POST_VARS["remise"]); + propale_pdf_create($db, $_GET["propalid"], $propal->modelpdf); } if ($HTTP_POST_VARS["action"] == 'setpdfmodel' && $user->rights->propale->creer) @@ -202,15 +204,15 @@ if ($action == 'del_ligne' && $user->rights->propale->creer) $propal = new Propal($db); $propal->fetch($propalid); $propal->delete_product($ligne); - + propale_pdf_create($db, $_GET["propalid"], $propal->modelpdf); } -if ($valid == 1 && $user->rights->propale->valider) +if ($_GET["valid"] == 1 && $user->rights->propale->valider) { $propal = new Propal($db); - $propal->fetch($propalid); - $propal->update_price($propalid); - propale_pdf_create($db, $propalid, $propal->modelpdf); + $propal->fetch($_GET["propalid"]); + $propal->update_price($_GET["propalid"]); + propale_pdf_create($db, $_GET["propalid"], $propal->modelpdf); $propal->valid($user); } @@ -226,12 +228,12 @@ llxHeader(); * * */ -if ($propalid) +if ($_GET["propalid"]) { $html = new Form($db); $propal = new Propal($db); - $propal->fetch($propalid); + $propal->fetch($_GET["propalid"]); /* * @@ -246,14 +248,14 @@ if ($propalid) */ if ($action == 'delete') { - $html->form_confirm("$PHP_SELF?propalid=$propalid","Supprimer la proposition","Etes-vous sûr de vouloir supprimer cette proposition ?","confirm_delete"); + $html->form_confirm("$PHP_SELF?propalid=$propal->id","Supprimer la proposition","Etes-vous sûr de vouloir supprimer cette proposition ?","confirm_delete"); } /* * */ $sql = "SELECT s.nom, s.idp, p.price, p.fk_projet,p.remise, p.tva, p.total, p.ref,".$db->pdate("p.datep")." as dp, c.id as statut, c.label as lst, p.note, x.firstname, x.name, x.fax, x.phone, x.email, p.fk_user_author, p.fk_user_valid, p.fk_user_cloture, p.datec, p.date_valid, p.date_cloture"; $sql .= " FROM llx_societe as s, llx_propal as p, c_propalst as c, llx_socpeople as x"; - $sql .= " WHERE p.fk_soc = s.idp AND p.fk_statut = c.id AND x.idp = p.fk_soc_contact AND p.rowid = $propalid"; + $sql .= " WHERE p.fk_soc = s.idp AND p.fk_statut = c.id AND x.idp = p.fk_soc_contact AND p.rowid = $propal->id"; if ($socidp) { @@ -323,9 +325,9 @@ if ($propalid) print ""; - if ($action == 'statut') + if ($_GET["action"] == 'statut') { - print "