Project

General

Profile

Anomalie #4633

argument 'date' du pipeline optimiser_base_disparus

Added by Maïeul Rouquette about 1 month ago. Updated 10 days ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
01/18/2021
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

Description

Hop,

dans mes logs j'ai ceci

2021-01-18 18:22:16 127.0.0.1 (pid 13677) :Pri:ERREUR: Erreur 1525 de mysql: Incorrect TIMESTAMP value: ''2021-01-17 18:22:16''
in /home/mrouquet/Sites/dev/localhost/plugins-dist/forum/forum_pipelines.php L308 [sql_delete(),forum_optimiser_base_disparus(),minipipe(),execute_pipeline_optimiser_base_disparus(),pipeline(),optimiser_base_disparus(),optimiser_base(),genie_optimiser_dist(),queue_start_job(),queue_schedule(),action_forcer_job_dist(),traiter_appels_actions()]
DELETE FROM `spip_localhost`.spip_forum
WHERE statut='redac' AND maj < '\'2021-01-17 18:22:16\''
2021-01-18 18:22:16 127.0.0.1 (pid 13677) :Pri:ERREUR: Erreur 1525 de mysql: Incorrect TIMESTAMP value: ''2021-01-17 18:22:16''
in /home/mrouquet/Sites/dev/localhost/plugins-dist/mots/mots_pipelines.php L97 [sql_delete(),mots_optimiser_base_disparus(),minipipe(),execute_pipeline_optimiser_base_disparus(),pipeline(),optimiser_base_disparus(),optimiser_base(),genie_optimiser_dist(),queue_start_job(),queue_schedule(),action_forcer_job_dist(),traiter_appels_actions()]
DELETE FROM `spip_localhost`.spip_mots
WHERE length(titre)=0 AND maj < '\'2021-01-17 18:22:16\''
2021-01-18 18:22:16 127.0.0.1 (pid 13677) :Pri:ERREUR: Erreur 1525 de mysql: Incorrect TIMESTAMP value: ''2021-01-17 18:22:16''
in /home/mrouquet/Sites/dev/localhost/plugins-dist/petitions/petitions_pipelines.php L58 [sql_delete(),petitions_optimiser_base_disparus(),minipipe(),execute_pipeline_optimiser_base_disparus(),pipeline(),optimiser_base_disparus(),optimiser_base(),genie_optimiser_dist(),queue_start_job(),queue_schedule(),action_forcer_job_dist(),traiter_appels_actions()]
DELETE FROM `spip_localhost`.spip_petitions
WHERE statut='poubelle' AND maj < '\'2021-01-17 18:22:16\''
2021-01-18 18:22:16 127.0.0.1 (pid 13677) :Pri:ERREUR: Erreur 1525 de mysql: Incorrect TIMESTAMP value: ''2021-01-17 18:22:16''
in /home/mrouquet/Sites/dev/localhost/plugins-dist/sites/sites_pipelines.php L228 [sql_delete(),sites_optimiser_base_disparus(),minipipe(),execute_pipeline_optimiser_base_disparus(),pipeline(),optimiser_base_disparus(),optimiser_base(),genie_optimiser_dist(),queue_start_job(),queue_schedule(),action_forcer_job_dist(),traiter_appels_actions()]
DELETE FROM `spip_localhost`.spip_syndic
WHERE maj<'\'2021-01-17 18:22:16\'' AND statut='refuse'
2021-01-18 18:22:16 127.0.0.1 (pid 13677) :Pri:ERREUR: Erreur 1525 de mysql: Incorrect TIMESTAMP value: ''2021-01-17 18:22:16''
in /home/mrouquet/Sites/dev/localhost/plugins-dist/breves/breves_pipelines.php L298 [sql_select(),breves_optimiser_base_disparus(),minipipe(),execute_pipeline_optimiser_base_disparus(),pipeline(),optimiser_base_disparus(),optimiser_base(),genie_optimiser_dist(),queue_start_job(),queue_schedule(),action_forcer_job_dist(),traiter_appels_actions()]
SELECT B.id_breve AS id
FROM `spip_localhost`.spip_breves AS B
            LEFT JOIN `spip_localhost`.spip_rubriques AS R
                ON B.id_rubrique=R.id_rubrique
WHERE R.id_rubrique IS NULL
            AND B.maj < '\'2021-01-17 18:22:16\''

Cela vient du fait que $flux['args']['date'] du pipeline optimiser_base_disparus est valeur qui est déjà passé par sql_quote(). (cf. fonction `optimiser_base_disparus` du code, qui contient la ligne (`$mydate = sql_quote(date("Y-m-d H:i:s", time() - $attente));`)

Or la plupart des plugin(-dist) qui utilise ce paramètre $flux['args']['date'] pour construire leur requete passe en sql_quote. Du couip double sql_quote.

La question que je me pose : faut-il corriger les plugins(-dist) ou le core ?

History

#1 Updated by b b 27 days ago

Version cible ?

#2 Updated by Maïeul Rouquette 26 days ago

Idéalement 3.2

#3 Updated by Maïeul Rouquette 26 days ago

mais je peux pas le modifier

#4 Updated by b b 24 days ago

  • Target version set to 3.2

#5 Updated by Maïeul Rouquette 15 days ago

Je me dis que corriger le core plutot que les plugins est pertinent, car on pourrait avoir besoin dans le pipeline d'utiliser $flux['args']['date'] pour autre chose que de la requete sql (même si je ne sais pas encore quoi).

Si vous êtes d'accord avec mon analyse, je peux me charger de la PR

#6 Updated by cedric - 15 days ago

oui il faut corriger le core, et faire un tour des plugins qui utilisent mais je crois en effet que majoritairement tout le monde prend ça pour une date et non une date quotée

#7 Updated by cedric - 15 days ago

  • Target version changed from 3.2 to 3.3

#8 Updated by b b 15 days ago

  • Status changed from Nouveau to En cours

#9 Updated by cedric - 10 days ago

  • Status changed from En cours to Fermé
  • Resolution set to fixed

c'est intégré

#10 Updated by cedric - 10 days ago

et reporté en 3.2 (mais après la release secu, ça sera pour la prochaine)

Also available in: Atom PDF