Forums

By inserting this php code into a switcher event this is the error that returns me after the submit of the form: Undefined variable: codicefiscale in C:\xampp\htdocs\scuole\libraries\cegcore2\admin\extensions\chronofc\functions\php\php_output.php(6) : eval()'d code on line 65
​
This is the code:
​
if(isset($_POST['codicefiscale']) && !empty($_POST['codicefiscale'])){

if (preg_match("/^[0-9]{11}$/", $this->data('codicefiscale'))) {
$this->validation_errors['codicefiscale'] = "Il codice fiscale inserito è relativo ad una persona giuridica.";
return 'errore_pg';
}
elseif
(!preg_match("/^[a-zA-Z]{6}[0-9]{2}[a-zA-Z]{1}[0-9]{2}[a-zA-Z]{1}[0-9]{3}[a-zA-Z]{1}$/",
$this->data('codicefiscale'))) {
$this->validation_errors['codicefiscale'] = "Errore, il codice fiscale contiene caratteri non validi o non è della lunghezza esatta.";
return 'errore_caratteri';
}
else {
$s=0;
for ($i = 1; $i <= 13; $i += 2 ){
$c = $this->data('codicefiscale')[$i];
if ('0' <= $c && $c <= '9') {
$s += ord($c) - ord('0');
}
else {
$s += ord($c) - ord('A');
}
}
for ($i = 0; $i <= 14; $i += 2 ) {
$c = $this->data('codicefiscale')[$i];
switch( $c ){
case '0': $s += 1; break;
case '1': $s += 0; break;
case '2': $s += 5; break;
case '3': $s += 7; break;
case '4': $s += 9; break;
case '5': $s += 13; break;
case '6': $s += 15; break;
case '7': $s += 17; break;
case '8': $s += 19; break;
case '9': $s += 21; break;
case 'A': $s += 1; break;
case 'B': $s += 0; break;
case 'C': $s += 5; break;
case 'D': $s += 7; break;
case 'E': $s += 9; break;
case 'F': $s += 13; break;
case 'G': $s += 15; break;
case 'H': $s += 17; break;
case 'I': $s += 19; break;
case 'J': $s += 21; break;
case 'K': $s += 2; break;
case 'L': $s += 4; break;
case 'M': $s += 18; break;
case 'N': $s += 20; break;
case 'O': $s += 11; break;
case 'P': $s += 3; break;
case 'Q': $s += 6; break;
case 'R': $s += 8; break;
case 'S': $s += 12; break;
case 'T': $s += 14; break;
case 'U': $s += 16; break;
case 'V': $s += 10; break;
case 'W': $s += 22; break;
case 'X': $s += 25; break;
case 'Y': $s += 24; break;
case 'Z': $s += 23; break;
}
}
if( chr($s%26+ord('A'))!=$codicefiscale[15] ){
return 'success';
} else {
return 'fail';
}
}
}

if( chr($s%26+ord('A'))!=$codicefiscale[15] ){

Like it said, you haven't actually DEFINED $codicefiscale anywhere.

Defining the variable at the beginning of the function in this way no error is returned to me anymore.
​
$codicefiscale = 0;

Great. I would suggest though that you return either true or false (no quotes) UNLESS you are using the return solely as a string you're displaying.