[1.6.1.0] Champ obligatoire : erreur 500 sur page identité

[shiva30]

Bonjour à tous,

 

j'ai voulu rendre le champ siret obligatoire.

'siret' => array('type' => self::TYPE_STRING, 'validate' => 'isSiret', 'required' => true, 'size' => 14),

Jusque la, pas de soucis, tout fonctionne bien. Sauf que si une personne n'a pas rentré de siret à l'inscription (ancien utilisateur par exemple). Il se retrouve avec une erreur 500 sur la page identité de son compte...

 

J'ai lancé un mode debug, et j'obtiens cela :

Property Customer->siret is empty
at line 872 in file classes/ObjectModel.php

867. 
868. 			$message = $this->validateField($field, $this->$field);
869. 			if ($message !== true)
870. 			{
871. 				if ($die)
872. 					throw new PrestaShopException($message);
873. 				return $error_return ? $message : false;
874. 			}
875. 		}
876. 
877. 		return true;

    ObjectModelCore->validateFields - [line 278 - classes/ObjectModel.php]
    ObjectModelCore->getFields - [line 111 - controllers/front/IdentityController.php]
    IdentityControllerCore->postProcess - [line 171 - classes/controller/Controller.php]
    ControllerCore->run - [line 373 - classes/Dispatcher.php]
    DispatcherCore->dispatch - [line 28 - index.php]

Tout aide ou piste bienvenue

Edited March 15, 2017 by shiva30 (see edit history)

[SmartPlugs]

Rendre le champ required dans le tpl afin que cela n'arrive pas ?

[BeComWeb]

Dans ce cas plutôt que de rendre le SIRET obligatoire au niveau du modèle Customer il vaut mieux que tu ajoutes une structure de contrôle dans la partie du AuthController qui s'occupe de valider l'inscription.

Ca n'impactera pas les anciens clients

[shiva30]

Merci, je vais voir pour les pistes que vous m'avez donné, et je reviens vous dire ce qui à focntionné.

[shiva30]

J'ai une piste pour le bug en cours. Il semblerai que ce soit un soucis avec les données NULL dans la base... Maintenant que j'ai identifer une part du soucis. Je vais voir si je trouve un moyen de contourner grace à vos piste. Sinon, j'affronterai le soucis avec une requete directement sur la base (si je pouvais éviter... une base chez 1&1&...fragile comme du verre).

Encore merci pour vos pistes. 

[shiva30]

Je me rends compte que j'ai voulu (encore une fois) ré-inventé la roue 

https://www.prestashop.com/forums/topic/57590-tutorial-devforever-ajouter-un-champ-obligatoire-au-formulaire-d%E2%80%99inscription/

[SmartPlugs]

Il manquait l'entrée dans fieldsRequired en effet...