Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix apostrophes in database #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Holo795
Copy link

@Holo795 Holo795 commented Apr 21, 2021

No description provided.

@Holo795
Copy link
Author

Holo795 commented Apr 21, 2021

voila juste une petite modif qui faisait erreur a certaine personne

@Holo795
Copy link
Author

Holo795 commented Apr 21, 2021

J'ai rien dit le theme noir est niquel, c'est pas sur ce plugin

@PHPierrre
Copy link
Member

Si je comprends bien, les simples quotes posent problème au moteur SQL à plusieurs endroits du code ?
Il existe deux méthodes un peu moins "brutes" pour corriger le problème :

  • la première technique et brouillon qui nécessited e rajouter des quotes manuellement ici ; ['visible' => "'".$visible."'"].
  • la seconde meilleure à mon goût est en sécurisant les champs un par un avec des $value = $db->value($value, 'string');.

Si j'ai bien compris ton problème, il s'agira d'appliquer la seconde technique que j'ai découverte après plusieurs heures bloquantes sur CakePHP il y a plusieurs années, mais je partage l'information ici, car ça reste une bizarrerie de CakePHP qui n'a besoin que d'être mise en évidence pour de futures contributions.

Qu'en penses-tu ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants