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

Import de flux OPML cassé dans Leed 1.8.2 #466

Closed
theLOICofFRANCE opened this issue Jun 25, 2017 · 5 comments
Closed

Import de flux OPML cassé dans Leed 1.8.2 #466

theLOICofFRANCE opened this issue Jun 25, 2017 · 5 comments
Assignees
Labels

Comments

@theLOICofFRANCE
Copy link
Contributor

Bonjour,

L'importation d'un fichier opml ne fonctionne plus avec la version de Leed 1.8.2.
Voici l'erreur dans les logs :
PHP Warning: require_once(SimplePie.class.php): failed to open stream: No such file or directory in /var/www/Opml.class.php on line 150, referer: http://10.1.1.2/leed/action.php?action=importForm

L'erreur vient du faite qu'avec cette version un fichier utilisé n'hésite plus :
www/Opml.class.php: require_once("SimplePie.class.php");

La solution consiste alors à remplacer dans le fichier la valeur "SimplePie.class.php" par "SimplePie.compiled-1.4.3-2-geb6dd2d.php".

Ou bien pour les utilisateurs de cette version de créer un lien symbolique :
ln -s SimplePie.compiled-1.4.3-2-geb6dd2d.php SimplePie.class.php

Merci, cordialement

@Simounet
Copy link
Member

Bien vu @HacKurx !
@Sbgodin n'aurait-il pas été plus judicieux de mettre la référence à la version de SP utilisée dans le commit de mise à jour plutôt que de nous obliger à mettre à jour toutes les références à ce fichier ?

@Sbgodin
Copy link
Member

Sbgodin commented Jun 26, 2017

Je confirme le problème. @Simounet, la modification suggérée par @HacKurx est bonne. Mais effectivement, remettre le nom de fichier sans la version réglerait aussi le problème.

J'ai préféré mettre la version dans le nom du fichier afin que même ceux qui récupèrent depuis une archive (ou s'ils suppriment le .git) soient quand même sûrs du numéro de version. Si ça pique trop, je peux retirer le numéro de version du nom de fichier et mettre à jour les références de action.php et Opml.class.php.

@Simounet
Copy link
Member

Je pense qu'il vaut mieux reprendre l'ancien nom. Le numéro de version et le numéro de build se trouvent à l'intérieur du fichier de toutes façons.

Sbgodin added a commit that referenced this issue Jun 26, 2017
FIX #466 - Import de flux OPML cassé
@Sbgodin
Copy link
Member

Sbgodin commented Jun 26, 2017

Merci @HacKurx !

@theLOICofFRANCE
Copy link
Contributor Author

De rien, avec plaisir ;)

@Sbgodin Sbgodin self-assigned this Jun 28, 2017
@Sbgodin Sbgodin added the Bug label Jun 28, 2017
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