Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NEW select proposal date with date now by default on form confirm #73

Conversation

lvessiller-opendsi
Copy link
Contributor

NEW select proposal date with date now by default on form confirm

@lvessiller-opendsi
Copy link
Contributor Author

@atm-arnaud @john-botella

Could you check and merge it ?

@lvessiller-opendsi
Copy link
Contributor Author

@atm-arnaud @john-botella @atm-maxime
Please, it's important for me to have a check for it or at least one answer.

@john-botella
Copy link
Member

@lvessiller-opendsi sorry but I don't work at ATM anymore.

@atm-maxime
Copy link
Member

Hi @lvessiller-opendsi
I'll submit this to our dev team

@atm-thibaultf
Copy link

@lvessiller-opendsi je viens de commenter votre issue

@lvessiller-opendsi
Copy link
Contributor Author

@lvessiller-opendsi je viens de commenter votre issue

Vous pouvez vérifier le bon fonctionnement.
Je vous laisse remplir traductions pour les langues :" it_IT" et "de_DE"
Pouvez-vous faire le nécessaire pour merger si tout est ok ?

@atm-thibaultf
Copy link

Fonctionnellement tout me parait OK, je transmets à l'équipe technique pour relecture du code.

@@ -149,8 +149,9 @@ function formConfirm($parameters, &$object, &$action, $hookmanager)
if ( array_key_exists('action', $_REQUEST) && $_REQUEST['action'] == 'modif') {
$formquestion = array(
array('type' => 'checkbox', 'name' => 'archive_proposal', 'label' => $langs->trans("ArchiveProposalCheckboxLabel"), 'value' => 1),
array('type' => 'date', 'name' => 'archive_proposal_date_', 'label' => $langs->trans("DatePropal"), 'value' => (!empty($conf->global->PROPALEHISTORY_ARCHIVE_WITH_DATE_NOW) ? dol_now() : $object->date), 'datenow' => 1),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utilise getdolglobalint

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Etes-vous sûr ?
Je n'avais pas utilisé "getDolGlobalInt" pour garder la compatibilité avec les différentes versions de Dolibarr.
La version minimale supportée est 3.2 dans le fichier de classe du module "modPropalehistory.class.php.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En effet, je suis sur oui on modifiera la version dolibarr minimum, c'est étonnant qu'il soit encore déclaré compatible dolibarr 3.2

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est fait et prêt à fusionner.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merci :)

class/actions_propalehistory.class.php Outdated Show resolved Hide resolved
@atm-quentin atm-quentin merged commit 6a0d428 into ATM-Consulting:main Oct 21, 2024
1 check failed
@lvessiller-opendsi lvessiller-opendsi deleted the new-archive-on-modify-with-date-now branch November 15, 2024 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants