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

Bug : tags avec le caractère & #85

Closed
nymous opened this issue May 13, 2013 · 3 comments · Fixed by shaarli/Shaarli#54
Closed

Bug : tags avec le caractère & #85

nymous opened this issue May 13, 2013 · 3 comments · Fixed by shaarli/Shaarli#54

Comments

@nymous
Copy link

nymous commented May 13, 2013

Lorsqu'on assigne à un lien un tag contenant le symbole "&" (ex : trucs&astuces), la recherche sur ce tag ne donne aucun résultat.

(Je sens que ça va être difficile à corriger, si seulement c'est possible. Empêcher PHP de récupérer ce qu'il croit être des nouveaux paramètres... Si j'en crois mes maigres connaisances)


When a tag containing "&" is given to a link (such as "tips&tricks"), the search of this tag returns no element.

@sijnak
Copy link

sijnak commented May 15, 2013

J'ai fait une petite adaptation de shaarli pour séparer les tags avec des
virgules et sur cette version, ajouter un "&" ne m'a pas posé de problème.
Alors tu peux peut-être jeter un coup d'œil du côté de
https://github.com/sijnak/Shaarli. Seul petit problème, rien n'est encore
fait pour migrer une base de données dans le "nouveau" format. Enfin, ça ne
serait pas de refus d'avoir des retours sur d'éventuels bugs apportés avec
les changements.

Il y aura un "pull request" quand je serai sûr que ça tient le coup.

Sinon, le problème en soi ne doit pas être compliqué à corriger. Le code se
lit plutôt bien.


I made some little changes to Shaarli to allow coma-separated tags. With
this changes, the character "&" makes no problem.
So, if you would like, you can have a look at it at
https://github.com/sijnak/Shaarli. The only known drawback is that nothing
is currently done to migrate from one to another tagging system. I would
appreciate some feedback on some bugs I could have missed.

As soon as these changes are prooftested, I'll make a pull requet.

Besides, your problem should not be hard to fix since the source code is
fairly easy to read.

Le 14 mai 2013 00:31, tikiki [email protected] a écrit :

Lorsqu'on assigne à un lien un tag contenant le symbole "&" (ex :
trucs&astuces), la recherche sur ce tag ne donne aucun résultat.

(Je sens que ça va être difficile à corriger, si seulement c'est possible.
Empêcher PHP de récupérer ce qu'il croit être des nouveaux paramètres... Si

j'en crois mes maigres connaisances)

When a tag containing "&" is given to a link (such as "tips&tricks"), the
search of this tag returns no element.


Reply to this email directly or view it on GitHubhttps://github.com//issues/85
.

@nodiscc
Copy link

nodiscc commented Nov 8, 2014

@tikiki @sijnak merci du rapport de bug, je confirme.

Il y a un petit fork communautaire sur https://github.com/shaarli/Shaarli/ avec quelques fixes et améliorations (en attendant que Seb décide si il merge nos changements, pas de réponse pour l'instant). J'ai copié ce bug sur shaarli#48 pour qu'on pense à travailler là dessus.

J'ai jeté un coup d'oeil au fix dans ta version de Shaarli et ca représente un énorme changement (séparation des tags par des virgules) et donc difficile à intégrer sans casser la compatibilité avec le format actuel.

Pas vraiment de solution pour le moment, mais je pense qu'on peut empêcher ça en interdisant de stocker les tags avec un & (message d'erreur ou simplement le remplacer à la volée). SI tu as d'autres suggestions elles sont bienvenues.

Est il possible de fermer ce bug et de continuer sur shaarli#48 ? Un petit coup de ménage dans le tracker ne ferait pas de mal, et on peut toujours réimporter nos bugs ici plus tard (https://github.com/IQAndreas/github-issues-import)

Merci d'avance

@nodiscc
Copy link

nodiscc commented Nov 9, 2014

@tikiki @sijnak C'était juste un bug dans l'encodage des liens propre à la page tag cloud. Le stockage et la recherche des tags contenant & fonctionne très bien.

Le bug est résolu sur shaarli#54

Merci de penser à fermer ce bug si tout est ok :)

pikzen pushed a commit to pikzen/Shaarli that referenced this issue Jan 21, 2015
@nymous nymous closed this as completed Jun 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants