phpmyadmin / phpmyadmin/sql-parser
Re-formating LIMIT destroys DELETE and UPDATE queries
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- PHP
- Estrellas
- 485
- Forks
- 119
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
When building queries, the LIMIT is re-formatted. For example SELECT * FROM tbl LIMIT 1 is changed to SELECT * FROM tbl LIMIT 0, 1. This works just fine for SELECT queries, but not for DELETE or UPDATE queries.
Example:
$query1 = "DELETE FROM a LIMIT 1";
$parser = new PhpMyAdmin\SqlParser\Parser($query1);
$statement = $parser->statements[0];
$table2 = new \PhpMyAdmin\SqlParser\Components\Expression("", "b", "", "");
$statement->from[0] = $table2;
echo $statement->build();
results in
DELETE FROM `b` LIMIT 0, 1
The changed query fails, while the original query is perfect SQL.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza reproduciendo el problema con el ejemplo del constructor de Parser y statement->build() mostrado en el informe; después, rastrea cómo se construye LIMIT para las sentencias DELETE y UPDATE. El issue no menciona ningún archivo ni prueba; se considera terminado cuando las consultas DELETE y UPDATE reconstruidas conservan una sintaxis LIMIT válida, mientras el comportamiento de SELECT sigue siendo correcto.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- mysql, php, sql
- Área
- database
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bien especificado
- Aptitud para principiantes
- 48/100