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

Urlify 5.8.4 201113 #505

Closed
wants to merge 2 commits into from
Closed

Conversation

kazimentou
Copy link
Contributor

Voir discussion sur le forum :
https://forum.pluxml.org/discussion/6757/resolu-jai-un-article-fantome-en-admin#latest


Fix plxUtils::urlify

Pas de point dans une URL (voir RFC 3986 ou urlencode() en PHP)

Fix plxMotor::parseArticle():
On controle d'abord que le nom du fichier est valide. On retourne false si echec

Fix plxMotor::getArticles():
On controle que plxMotor::parseArticle retourne bien un article
Si ok, on le pousse à la fin du tableau ( pas de tableau associatif ) pour que les indices numériques se suivent même si l'analyse d'un article à échoué

bazooka07 added 2 commits November 13, 2020 22:46
Pas de point dans une URL (voir RFC 3986 ou urlencode() en PHP)

Fix plxMotor::parseArticle():
On controle d'abord que le nom du fichier est valide. On retourne false
si echec

Fix plxMotor::getArticles():
On controle que plxMotor::parseArticle retourne bien un article
Si ok, on le pousse à la fin du tableau ( pas de tableau associatif )
pour que les indices numériques se suivent même si l'analyse d'un
article à échoué
@haruka-7 haruka-7 closed this Nov 18, 2020
@haruka-7 haruka-7 deleted the branch pluxml:5.8.4 November 18, 2020 14:14
@haruka-7
Copy link
Collaborator

Mergé dans master.

@haruka-7 haruka-7 added this to the PluXml 5.8.5 milestone Nov 18, 2020
@haruka-7 haruka-7 added the bug label Nov 18, 2020
@kazimentou kazimentou deleted the urlify-5.8.4-201113 branch November 19, 2020 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants