[SOLUCIONADO] Duda de funcion

[tampon]

Hola tengo que implementar un captcha pero tengo una duda en esta funcion.

Código PHP:

  <?php
 
// The form processor PHP code
function process_si_contact_form()
{
 
    $userName = @$_POS['userName'];
    $password =  @$_POS['password'];
 
    if ($_SERVER['REQUEST_METHOD'] == 'POST' && @$_POST['do'] == 'contact') {
        // if the form has been submitted
 
        foreach($_POST as $key => $value) {
            if (!is_array($key)) {
                // sanitize the input data
                if ($key != 'ct_message') $value = strip_tags($value);
                $_POST[$key] = htmlspecialchars(stripslashes(trim($value)));
            }
        }
 
        $captcha = @$_POST['ct_captcha']; // the user's entry for the captcha code
 
        $errors = array();  // initialize empty error array
 
        // Only try to validate the captcha if the form has no errors
        // This is especially important for ajax calls
        if (sizeof($errors) == 0) {
            require_once dirname(__FILE__) . '/securimage.php';
            $securimage = new Securimage();
 
            if ($securimage->check($captcha) == false) {
                $errors['captcha_error'] = 'Verifica el resultado del captcha.';
 
//echo '<script language="javascript">alert("Verifica el resultado del captcha.");</script>';
 
            }
        }
 
        if (sizeof($errors) == 0) {
            // no errors, send the form
            $return = array('error' => 0, 'message' => $userName);
            die(json_encode($return));
        } else {
            $errmsg = 'mensaje ok?';
            foreach($errors as $key => $error) {
                // set up error messages to display with each field
                $errmsg .= " - {$error}\n";
            }
            $return = array('error' => 1, 'message' => $errmsg);
            die(json_encode($return));
        }
    } // POST
} // function process_si_contact_form()
 
?>   

Veo que se realiza 2 veces la misma condición

Código PHP:

  if (sizeof($errors) == 0) 
   

y que lanza varios mensajes de error (innecesarios a mi punto de vista)

Código PHP:

  $errmsg = 'mensaje ok?';
            foreach($errors as $key => $error) {
                // set up error messages to display with each field
                $errmsg .= " - {$error}\n"; 
   

me gustaria simplificarlo pero al meterle mano se me desconfigura todo, quiero que cuando sea error solo mande un alert con un mensaje de error y cuando la condición se cumpla que envié unas mediante post las variables $userName y $password.. ¿alguna ayuda? Gracias!

[Triby2]

Bueno, no tiene mucha complicacion, solo ha que seguir el orden y la estructura.

Código PHP:

<?php 
// The form processor PHP code 
function process_si_contact_form() 
{ 
    $userName = @$_POST['userName']; 
    $password =  @$_POST['password']; 
 
    if ($_SERVER['REQUEST_METHOD'] == 'POST' && @$_POST['do'] == 'contact') { 
        // if the form has been submitted 
 
        foreach($_POST as $key => $value) { 
            if (!is_array($key)) { 
                // sanitize the input data 
                if ($key != 'ct_message') $value = strip_tags($value); 
                $_POST[$key] = htmlspecialchars(stripslashes(trim($value))); 
            } 
        } 
 
        $captcha = @$_POST['ct_captcha']; // the user's entry for the captcha code 
 
        $errors = array();  // initialize empty error array 
 
        // Only try to validate the captcha if the form has no errors 
        // This is especially important for ajax calls 
        if (sizeof($errors) == 0) { 
            require_once dirname(__FILE__) . '/securimage.php'; 
            $securimage = new Securimage(); 
 
            if ($securimage->check($captcha) == false) { 
                $errors['captcha_error'] = 'Verifica el resultado del captcha.'; 
                //echo '<script language="javascript">alert("Verifica el resultado del captcha.");</script>'; 
            }
 
            // no errors, send the form 
            $return = array('error' => 0, 'message' => $userName);  
        } else { 
            $errmsg = 'mensaje ok?'; 
            foreach($errors as $key => $error) { 
                // set up error messages to display with each field 
                $errmsg .= " - {$error}\n"; 
            } 
            $return = array('error' => 1, 'message' => $errmsg);  
        }
        die(json_encode($return));
    } // POST 
} // function process_si_contact_form() 
 
?>
 

Pero el código que tu compartiste tiene el $_POST de las variables userName y password mal escrito, le falta una "T"

[tampon]

Hola Triby2 ! Gracias por responder.
Creo que ahora fallo, toma el resultado incorrecto como correcto, pero sigo buscando una solución.
Dentro del código intento declarar una acción javascript como variable de php ¿eso se puedo?
Saludos.

[hhs]

La función regresa un json como error así que solo tienes que usar el json desde javascript, no requieres alterar la función.

[alvaro_trewhela]

No entiendo lo que quieres hacer con esa función pero lo del captcha es facil

Asignas un valor aleatorio alfanumerico a una variable, haces la rutina de imagen de la variable, luego encriptas la variable y la asignas a una variable sesion...

Todo eso entorno al formulario ...

[tampon]

Lose, estoy teniendo problemas solo con ie11, todos los demas navegadores no tienen ningun problema