| Re: Xoops 2.3 version finale : la voilí !!! |
Titre du sujet : Re: Xoops 2.3 version finale : la voilí !!!
par RC78 sur 12/10/2008 01:19:30
Dans un premier temps merci au travail effectué sur la 2.31 car le sujet n'est pas simple et demande énormément de travaille aux développeurs.
PROFIL DE LA VERSION 2.31 EN GENERAL :
En effet la version 2.31 Finale à régler certains problème, mais reste encore incomplète pour moi à l'heure actuelle, par rapport à un sujet développé ultérieurement.
Pour moi le module "Ext Profil" est un plus pour les personnes qui ne savent pas hacker le core, mais ceci ne règle pas le problème de cette ajout dans "../class/formselect.php".
Pour rappel depuis la RC3 dans "formselect.php" à été ajouter ceci :
/**
* Render custom javascript validation code
*
* @seealso XoopsForm::renderValidationJS
*/
function renderValidationJS()
{
// render custom validation code if any
if ( !empty( $this->customValidationCode ) ) {
return implode( "n", $this->customValidationCode );
// generate validation code if required
} elseif ($this->isRequired()) {
$eltname = $this->getName();
$eltcaption = $this->getCaption();
$eltmsg = empty($eltcaption) ? sprintf( _FORM_ENTER, $eltname ) : sprintf( _FORM_ENTER, $eltcaption );
$eltmsg = str_replace('"', '"', stripslashes( $eltmsg ) );
return "nvar hasSelected = false; var selectBox = myform.{$eltname};" .
"for (i = 0; i < selectBox.options.length; i++ ) { if (selectBox.options[i].selected == true) { hasSelected = true; break; } }" .
"if (!hasSelected) { window.alert("{$eltmsg}"); selectBox.focus(); return false; }";
}
return '';
}
EN GENERAL :
Vous avez travailler avec le fait que si le champ par défaut est vide (Aucun) le champ devient obligatoire.
Par contre pour les webmasters qui ont déjà travailler sur le sujet en ajoutant un hack avec des champs déroulants dans le core et en imposant obligatoirement un champ quelconque du type : choisir, ---, etc... (mais avec une écriture) ceci ne fonctionne plus avec la fonction TRUE
Donc impossible de bloquer un champ.
Il faudrait donc tout abandonner pour la bonne raison que le module "Ext profil" existe ?
Billet d'humeur 
A+ Titi
|
|
