Jump to content

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


Recommended Posts

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 by shiva30 (see edit history)
Link to comment
Share on other sites

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

Link to comment
Share on other sites

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. 

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...