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

stripslashes() missing for title #299

Closed
nicosomb opened this issue Nov 29, 2013 · 9 comments
Closed

stripslashes() missing for title #299

nicosomb opened this issue Nov 29, 2013 · 9 comments
Assignees
Labels
Milestone

Comments

@nicosomb
Copy link

It lacks a stripslashes() call when you display the title of the FreshRSS instance.

For example, I name my instance "nicosomb's feeds".

And it displays " nicosomb's feeds".

marienfressinaud added a commit that referenced this issue Nov 29, 2013
@marienfressinaud
Copy link
Member

Corrected but only for the version 0.7. If I release a version 0.6.2 it should be backported

@nicosomb
Copy link
Author

nicosomb commented Jan 5, 2014

This bug is still present in 0.7 b4.

@Alkarex
Copy link
Member

Alkarex commented Jan 5, 2014

Pas de problème dans mes tests, mais peut-être un bug d'une version précédente qui reste présent dans le config.php. Pourrais-je voir cette partie du config.php ?

@nicosomb
Copy link
Author

nicosomb commented Jan 5, 2014

J'ai corrigé le titre tout à l'heure là dedans donc plus de bug.

c'était OK avec la première beta, j'ai mis à jour et les slashs sont apparus (ça donnait : c\'est mon install de freshrss)

@Alkarex
Copy link
Member

Alkarex commented Jan 5, 2014

Ok merci. Je pense que le bug ne devrait plus y être pour les nouvelles installations.
Je vais peut-être ajouter un stripslashes lors de la mise à jour, mais j'hésite un peu car ça pourrait casser d'autres chaînes valides.

@Alkarex
Copy link
Member

Alkarex commented Jan 5, 2014

P.S. : Est-ce que je comprends bien et que le problème a disparu après avoir corrigé manuellement le config.php qui provenait d'une ancienne version ?

@nicosomb
Copy link
Author

nicosomb commented Jan 5, 2014

Je ne peux pas te confirmer, mon install n'est pas dispo actuellement, petit souci chez mon hébergeur.

@nicosomb
Copy link
Author

nicosomb commented Jan 5, 2014

je confirme. tout semble OK maintenant.

@Alkarex
Copy link
Member

Alkarex commented Jan 5, 2014

Parfait, merci :-)
J'ai ajouté un petit stripslashes, mais uniquement une fois pour la migration depuis une v0.6.

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

No branches or pull requests

3 participants