diff --git a/htdocs/public/members/new.php b/htdocs/public/members/new.php
index e6b7a3e7a38..677e1e1b906 100644
--- a/htdocs/public/members/new.php
+++ b/htdocs/public/members/new.php
@@ -201,12 +201,12 @@ if (empty($reshook) && $action == 'add') {
$langs->load("errors");
$errmsg .= $langs->trans("ErrorPasswordsMustMatch")."
\n";
}
- if (!GETPOST("email")) {
+ if (!GETPOST('email')) {
$error++;
$errmsg .= $langs->trans("ErrorFieldRequired", $langs->transnoentitiesnoconv("EMail"))."
\n";
}
}
- if (GETPOST('type') <= 0) {
+ if (GETPOST('typeid') <= 0) {
$error++;
$errmsg .= $langs->trans("ErrorFieldRequired", $langs->transnoentitiesnoconv("Type"))."
\n";
}
@@ -214,21 +214,21 @@ if (empty($reshook) && $action == 'add') {
$error++;
$errmsg .= $langs->trans("ErrorFieldRequired", $langs->transnoentitiesnoconv('Nature'))."
\n";
}
- if (!GETPOST("lastname")) {
+ if (!GETPOST('lastname')) {
$error++;
$errmsg .= $langs->trans("ErrorFieldRequired", $langs->transnoentitiesnoconv("Lastname"))."
\n";
}
- if (!GETPOST("firstname")) {
+ if (!GETPOST('firstname')) {
$error++;
$errmsg .= $langs->trans("ErrorFieldRequired", $langs->transnoentitiesnoconv("Firstname"))."
\n";
}
- if (GETPOST("email") && !isValidEmail(GETPOST("email"))) {
+ if (GETPOST('email') && !isValidEmail(GETPOST('email'))) {
$error++;
$langs->load("errors");
- $errmsg .= $langs->trans("ErrorBadEMail", GETPOST("email"))."
\n";
+ $errmsg .= $langs->trans("ErrorBadEMail", GETPOST('email'))."
\n";
}
- $birthday = dol_mktime(GETPOST("birthhour", 'int'), GETPOST("birthmin", 'int'), GETPOST("birthsec", 'int'), GETPOST("birthmonth", 'int'), GETPOST("birthday", 'int'), GETPOST("birthyear", 'int'));
- if (GETPOSTISSET("birthmonth") && empty($birthday)) {
+ $birthday = dol_mktime(GETPOST('birthhour', 'int'), GETPOST('birthmin', 'int'), GETPOST('birthsec', 'int'), GETPOST('birthmonth', 'int'), GETPOST('birthday', 'int'), GETPOST('birthyear', 'int'));
+ if (GETPOSTISSET('birthmonth') && empty($birthday)) {
$error++;
$langs->load("errors");
$errmsg .= $langs->trans("ErrorBadDateFormat")."
\n";
@@ -240,36 +240,32 @@ if (empty($reshook) && $action == 'add') {
}
}
- if (GETPOSTISSET('public')) {
- $public = 1;
- } else {
- $public = 0;
- }
+ $public = GETPOSTISSET('public') ? 1 : 0;
if (!$error) {
// email a peu pres correct et le login n'existe pas
$adh = new Adherent($db);
$adh->statut = -1;
$adh->public = $public;
- $adh->firstname = GETPOST("firstname");
- $adh->lastname = GETPOST("lastname");
- $adh->gender = GETPOST("gender");
- $adh->civility_id = GETPOST("civility_id");
- $adh->societe = GETPOST("societe");
- $adh->address = GETPOST("address");
- $adh->zip = GETPOST("zipcode");
- $adh->town = GETPOST("town");
- $adh->email = GETPOST("email");
+ $adh->firstname = GETPOST('firstname');
+ $adh->lastname = GETPOST('lastname');
+ $adh->gender = GETPOST('gender');
+ $adh->civility_id = GETPOST('civility_id');
+ $adh->societe = GETPOST('societe');
+ $adh->address = GETPOST('address');
+ $adh->zip = GETPOST('zipcode');
+ $adh->town = GETPOST('town');
+ $adh->email = GETPOST('email');
if (empty($conf->global->ADHERENT_LOGIN_NOT_REQUIRED)) {
- $adh->login = GETPOST("login");
- $adh->pass = GETPOST("pass1");
+ $adh->login = GETPOST('login');
+ $adh->pass = GETPOST('pass1');
}
- $adh->photo = GETPOST("photo");
- $adh->country_id = GETPOST("country_id", 'int');
- $adh->state_id = GETPOST("state_id", 'int');
- $adh->typeid = GETPOST("type", 'int');
- $adh->note_private = GETPOST("note_private");
- $adh->morphy = GETPOST("morphy");
+ $adh->photo = GETPOST('photo');
+ $adh->country_id = $conf->global->MEMBER_NEWFORM_FORCECOUNTRYCODE ? $conf->global->MEMBER_NEWFORM_FORCECOUNTRYCODE : GETPOST('country_id', 'int');
+ $adh->state_id = GETPOST('state_id', 'int');
+ $adh->typeid = $conf->global->MEMBER_NEWFORM_FORCETYPE ? $conf->global->MEMBER_NEWFORM_FORCETYPE : GETPOST('typeid', 'int');
+ $adh->note_private = GETPOST('note_private');
+ $adh->morphy = $conf->global->MEMBER_NEWFORM_FORCEMORPHY ? $conf->global->MEMBER_NEWFORM_FORCEMORPHY : GETPOST('morphy');
$adh->birth = $birthday;
@@ -549,9 +545,9 @@ jQuery(document).ready(function () {
';
-print '
| '.$langs->trans("Type").' * | '; - print $form->selectarray("type", $adht->liste_array(), GETPOST('type') ?GETPOST('type') : $defaulttype, $isempty); + print $form->selectarray("typeid", $adht->liste_array(), GETPOST('typeid') ? GETPOST('typeid') : $defaulttype, $isempty); print ' |
| '.$langs->trans('MemberNature').' * | '."\n"; print $form->selectarray("morphy", $morphys, GETPOST('morphy'), 1); @@ -579,33 +575,33 @@ if (empty($conf->global->MEMBER_NEWFORM_FORCEMORPHY)) { print $morphys[$conf->global->MEMBER_NEWFORM_FORCEMORPHY]; print ''; } -// Civility -print ' |
| '.$langs->trans('UserTitle').' | '; -print $formcompany->select_civility(GETPOST('civility_id'), 'civility_id').' |
| '.$langs->trans("Lastname").' * | |
| '.$langs->trans("Firstname").' * | |
| '.$langs->trans("Gender").' | '; -print ''; -$arraygender = array('man'=>$langs->trans("Genderman"), 'woman'=>$langs->trans("Genderwoman")); -print $form->selectarray('gender', $arraygender, GETPOST('gender') ?GETPOST('gender') : $object->gender, 1); -print ' |
| '.$langs->trans("Company").' | |
| '.$langs->trans("Address").' | '."\n"; -print ' |
| '.$langs->trans('Zip').' / '.$langs->trans('Town').' | '; -print $formcompany->select_ziptown(GETPOST('zipcode'), 'zipcode', array('town', 'selectcountry_id', 'state_id'), 6, 1); -print ' / '; -print $formcompany->select_ziptown(GETPOST('town'), 'town', array('zipcode', 'selectcountry_id', 'state_id'), 0, 1); -print ' |
| '.$langs->trans('Country').' | '; -$country_id = GETPOST('country_id'); + // Civility + print ' |
| '.$langs->trans('UserTitle').' | '; + print $formcompany->select_civility(GETPOST('civility_id'), 'civility_id').' |
| '.$langs->trans("Lastname").' * | |
| '.$langs->trans("Firstname").' * | |
| '.$langs->trans("Gender").' | '; + print ''; + $arraygender = array('man'=>$langs->trans("Genderman"), 'woman'=>$langs->trans("Genderwoman")); + print $form->selectarray('gender', $arraygender, GETPOST('gender') ?GETPOST('gender') : $object->gender, 1); + print ' |
| '.$langs->trans("Company").' | |
| '.$langs->trans("Address").' | '."\n"; + print ' |
| '.$langs->trans('Zip').' / '.$langs->trans('Town').' | '; + print $formcompany->select_ziptown(GETPOST('zipcode'), 'zipcode', array('town', 'selectcountry_id', 'state_id'), 6, 1); + print ' / '; + print $formcompany->select_ziptown(GETPOST('town'), 'town', array('zipcode', 'selectcountry_id', 'state_id'), 0, 1); + print ' |
| '.$langs->trans('Country').' | '; + $country_id = GETPOST('country_id'); if (!$country_id && !empty($conf->global->MEMBER_NEWFORM_FORCECOUNTRYCODE)) { $country_id = getCountry($conf->global->MEMBER_NEWFORM_FORCECOUNTRYCODE, 2, $db, $langs); } @@ -620,45 +616,43 @@ if (!$country_id && !empty($conf->geoipmaxmind->enabled)) { } } } -$country_code = getCountry($country_id, 2, $db, $langs); -print $form->select_country($country_id, 'country_id'); -print ' |
| '.$langs->trans('State').' | '; if ($country_code) { print $formcompany->select_state(GETPOST("state_id"), $country_code); - } else { - print ''; } print ' |
| '.$langs->trans("Email").' * | |
| '.$langs->trans("Email").' * | |
| '.$langs->trans("Login").' * | |
| '.$langs->trans("Password").' * | |
| '.$langs->trans("PasswordAgain").' * | |
| '.$langs->trans("DateOfBirth").' | '; -print $form->selectDate($birthday, 'birth', 0, 0, 1, "newmember", 1, 0); -print ' |
| '.$langs->trans("URLPhoto").' | |
| '.$langs->trans("Public").' | |
| '.$langs->trans("Comments").' | '; -print ''; -print ' |
| '.$langs->trans("DateOfBirth").' | '; + print $form->selectDate($birthday, 'birth', 0, 0, 1, "newmember", 1, 0); + print ' |
| '.$langs->trans("URLPhoto").' | |
| '.$langs->trans("Public").' | |
| '.$langs->trans("Comments").' | '; + print ''; + print ' |
| '.$langs->trans("TurnoverOrBudget").' * | '; @@ -675,11 +669,11 @@ if (!empty($conf->global->MEMBER_NEWFORM_DOLIBARRTURNOVER)) { if (jQuery("#budget").val() > 0) { jQuery(".amount").val(jQuery("#budget").val()); } else { jQuery("#budget").val(\'\'); } }); - /*jQuery("#type").change(function() { - if (jQuery("#type").val()==1) { jQuery("#morphy").val(\'mor\'); } - if (jQuery("#type").val()==2) { jQuery("#morphy").val(\'phy\'); } - if (jQuery("#type").val()==3) { jQuery("#morphy").val(\'mor\'); } - if (jQuery("#type").val()==4) { jQuery("#morphy").val(\'mor\'); } + /*jQuery("#typeid").change(function() { + if (jQuery("#typeid").val()==1) { jQuery("#morphy").val(\'mor\'); } + if (jQuery("#typeid").val()==2) { jQuery("#morphy").val(\'phy\'); } + if (jQuery("#typeid").val()==3) { jQuery("#morphy").val(\'mor\'); } + if (jQuery("#typeid").val()==4) { jQuery("#morphy").val(\'mor\'); } initturnover(); });*/ function initturnover() { @@ -703,13 +697,16 @@ if (!empty($conf->global->MEMBER_NEWFORM_DOLIBARRTURNOVER)) { } if (!empty($conf->global->MEMBER_NEWFORM_AMOUNT) || !empty($conf->global->MEMBER_NEWFORM_PAYONLINE)) { // $conf->global->MEMBER_NEWFORM_SHOWAMOUNT is an amount - $amount = 0; + + // Set amount for the subscription + $amount = isset($amount) ? $amount : 0; + if (!empty($conf->global->MEMBER_NEWFORM_AMOUNT)) { $amount = $conf->global->MEMBER_NEWFORM_AMOUNT; } if (!empty($conf->global->MEMBER_NEWFORM_PAYONLINE)) { - $amount = GETPOST('amount') ?GETPOST('amount') : $conf->global->MEMBER_NEWFORM_AMOUNT; + $amount = $amount ? $amount : (GETPOST('amount') ? GETPOST('amount') : $conf->global->MEMBER_NEWFORM_AMOUNT); } // $conf->global->MEMBER_NEWFORM_PAYONLINE is 'paypal', 'paybox' or 'stripe' print ' |
| '.$langs->trans("Subscription").' | '; @@ -722,24 +719,24 @@ if (!empty($conf->global->MEMBER_NEWFORM_AMOUNT) || !empty($conf->global->MEMBER print ' '.$langs->trans("Currency".$conf->currency); print ' |