¿Cual es el error de esta actualización en mysql?

[alvarols]

Estoy tratando de actualizar información en una base de datos, pero me tira el siguiente error.

Database query failedYou have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'LIMIT 1' at line 1

Este es el código que tengo:

Código PHP:

  $query = "UPDATE subjects SET
                        menu_name = '{$menu_name}', 
                        position = {$position},
                        visible = {$visible}
                      WHERE id = {$id}";
            $result = mysql_query($query, $connection); 
   

[huesos52]

El error dice que cerca a la clausula limit 1.
Creo que estasd posteando la sentencia equivocada.

PD: Esos {} para que son? esa n es la forma correcta de asignar valores a las variables.

[Jamati]

Hay errores en la actualización.
La manera correcta de hacerlo es:

$query = "UPDATE subjects SET menu_name = '$menu_name', position = '$position', visible = '$visible' WHERE id = '$id'"

Las llaves estaban demás.

Suerte

[alvarols]

Ah muchas gracias

[alvarols]

No, me sigue sin funcionar y tirando el mismo error. Ya le quité las llaves y puse el ejempolo de Jamati y nada.

Ya revisé en el phpmyadmin que concordara con la tabla y todo está bien al parecer.

[huesos52]

Iniciado por huesos52

El error dice que cerca a la clausula limit 1.
Creo que estasd posteando la sentencia equivocada.

PD: Esos {} para que son? esa n es la forma correcta de asignar valores a las variables.

Estas revisando la sentencia equivocada. Que sentencia tienes con la clausula limit?

[alvarols]

Una duda ¿Cual es la clausula limit?

[huesos52]

Dices que tu error es

server version for the right syntax to use near 'LIMIT 1' at line 1

En esa pagina, debes estar utilizando esto. La pregunta te la hago yo a ti, no tu a mi. jajaja

[alvarols]

Tengo estas funciones a las cuales llamo por medio de un include. Aquí veo que se encuentra el famoso "LIMIT", porque lo quito y desaparece la palabra de la sentencia esta. Algo tendrán que ver estas funciones me imagino:

Código PHP:

  function get_subject_by_id($subject_id) {
    global $connection;
    $query = "SELECT * ";
    $query.= "FROM SUBJECTS ";
    $query.= "WHERE ID = " . $subject_id ." ";
    $query.= "LIMIT 1 ";
    $result_set = mysql_query($query, $connection);
    confirm_query($result_set);
    // REMEMBER
    // if no rows are returned, fetch array return false.
    if ($subject = mysql_fetch_array($result_set)) {
        return $subject;
    } else {
        return NULL;
    }
}
 
function get_page_by_id($subject_id) {
    global $connection;
    $query = "SELECT * ";
    $query.= "FROM PAGES ";
    $query.= "WHERE ID = " . $subject_id . " ";
    $query.="LIMIT 1 ";
    $result_set = mysql_query($query, $connection);
    confirm_query($result_set);
    // REMEMBER
    // if no rows are returned, fetch array return false.
    if ($subject = mysql_fetch_array($result_set)) {
        return $subject;
    } else {
        return NULL;
    }
} 
   

[alvarols]

¿Alguien sabe?

[huesos52]

Eso parece ser problema de php.

Haz una impresion de la variable $query y pon el resultado de la impresión directamente en mysql.
Postea dicha impresion y dinos el error que te saca.

saludos

[alvarols]

¿Como puedo imprimir la variable $query si está dentro de una función?

[bNd170]

Poniendo un echo $query en la funcion. :D

[alvarols]

esta es la cadena que me sale

SELECT * FROM SUBJECTS WHERE ID = LIMIT 1

lo probé en el phpmyadmin y me tira el mismo error. El error debe de estar en esta sentencia pero no se donde. Un saludo

[gnzsoloyo]

Entonces tu error es simplísimo: la variable $subject_id está llegando vacía, porque debería haber un valor numérico de comparación entre el "=" y el LIMIT.

Revisa tu PHP porque tienes un error de ejecución.

[alvarols]

Pero donde estaría el problema, porque en realidad no estoy llamando en esa función. Hago un include a la página donde tengo guardadas las funciones pero en específico nunca he llamado a esa función como pueden ver. Es lo que no entiendo.

Les dejo el código de la página para que vean que nunca llamo a get_subject_by_id ni a get_page_by_id

Código PHP:

  <?php require_once ("includes/connection.php");?>
<?php require_once ("includes/functions.php");?>
<?php 
    find_selected_page();    
?>
 
<?php
    if (intval($_GET['subj']) == 0 ) {
        redirect_to("content.php");
        
    }
    if (isset($_POST['submit'])) {
        $errors = array();
        
        $required_fields = array('menu_name', 'position', 'visible');
        foreach($required_fields as $fieldname) {
            if (!isset($_POST[$fieldname]) || (empty($_POST[$fieldname]) && ($_POST[$fieldname]) !=0)) {
                $errors[] = $fieldname;
            }
        }
        $fields_with_lenghts = array('menu_name' => 30);
        foreach($fields_with_lenghts as $fieldname => $maxlength) {
            if (strlen(trim(mysql_prep($_POST[$fieldname]))) > $maxlength) {
                $errors[] = $fieldname;
            }
        }
        
        if (empty($errors)) {
            //perform update
            $id = mysql_prep($_GET['subj']);
            $menu_name = mysql_prep($_POST['menu_name']);
            $position = mysql_prep($_POST['position']);
            $visible = mysql_prep($_POST['visible']);
            
            $query = "UPDATE subjects SET
                        menu_name = '$menu_name', 
                        position = $position,
                        visible = $visible
                      WHERE id = $id";
            $result = mysql_query($query, $connection);
            if (mysql_affected_rows() == 1) {
                //Success
            } else {
                //failed
            }
            
        } else {
            //errors occurred    
        }
        
        
        
        
    } // end if (isset($_POST['submit']))
?>
 
<?php include ("includes/header.php");?>
        <table id="structure">
            <tr>
                <td id="navigation">
                    <?php echo navigation($sel_subject, $sel_page);?>
                </td>
                <td id="page">
                <h2>Edit Subject <?php echo $sel_subject['menu_name'];?></h2>
                <form action="edit_subject.php?subj=<?php urlencode($sel_subject['id']); ?>" method="post">
                    <p>Subject Name:
                        <input type="text" name="menu_name" value="<?php echo $sel_subject['menu_name'];?>" id="menu_name" />
                        
                    </p>
                    <p>Position:
                        <select name="position">
                        <?php
                            //subject_count +1 because we're adding a subject
                            $subject_set = get_all_subjects();
                            $subject_count = mysql_num_rows($subject_set);
                            for ($count=1; $count <= $subject_count+1; $count++){
                                echo "<option value=\"{$count}\"";
                                if ($sel_subject['position'] == $count) {
                                    echo "selected";
                                }
                                echo ">{$count}</option>";
                                
                            }
                        ?>
                            
                        </select>
                    </p>
                    <p>Visible:
                    <input type="radio" name="visible" value="0"<?php
                    if ($sel_subject['visible'] == 0) {echo "checked";}?> /> No
                    &nbsp
                    <input type="radio" name="visible" value="1"<?php
                    if ($sel_subject['visible'] == 1) {echo "checked";}?> /> Yes
                    </p>
                    <input type="submit" name="submit" value="Edit Subject" />
                 </form>
                <br />    
                <a href="content.php">Cancel</a>
                     
                </td>
             </tr>
          </table>
       </div>
<?php require ("includes/footer.php");?>   

[huesos52]

Compa esa ayuda no es tema de este foro.
Eso es un problema neto de php y como invocas archivos con funciones.
Pregunta en el foro de php-

El error de la base de datos, ya te dijo gnzsoloyo a que se debe.

saludos

[gnzsoloyo]

Pero donde estaría el problema, porque en realidad no estoy llamando en esa función. Hago un include a la página donde tengo guardadas las funciones pero en específico nunca he llamado a esa función como pueden ver. Es lo que no entiendo.

La estás llamando en alguna parte, porque de lo contrario no te estaría dando ese error...
Pero, como te dice Huesos52: Eso no es tema de este foro. Los asuntos de PHP se tratan en el Foro de PHP.
Postea tu duda allá, que ellos te ayudarán más rápido y mejor.

[alvarols]

Muchas gracias

[gnzsoloyo]

Un sólo tip: Fíjate que en tu código haces esto:

Código PHP:

<?php
//subject_count +1 because we're adding a subject
$subject_set = get_all_subjects();
$subject_count = mysql_num_rows($subject_set);
for ($count=1; $count <= $subject_count+1; $count++){
echo "<option value=\"{$count}\"";
if ($sel_subject['position'] == $count) {
echo "selected";
}
echo ">{$count}</option>";
}
?>
 

¿Te has fijado en esa llamada a get_all_subjects()? ¿has revisado el código de esa función? ¿Verificaste lo que hace?

[alvarols]

Si, pero esa no tiene que ver con la que está jalando el error. Esta es a la función que llama:

Código PHP:

  function get_all_subjects(){
        global $connection;
        $query = "SELECT * 
            FROM subjects 
            ORDER BY position ASC";
    $subject_set = mysql_query($query, $connection);
    confirm_query($subject_set);
    return $subject_set;
} 
   

Gracias

[gnzsoloyo]

Y confirm_query($subject_set), ¿qué es lo que hace?
Aparece siendo llamada desde todas las lecturas...