search

YunoHost-Apps / pluxml_ynh

PluXml package for YunoHost
http://pluxml.org
GNU General Public License v3.0
9 stars 3 forks source link

Upgrade to 5.7 issue #29

Closed ewilly closed 5 years ago

ewilly commented 5 years ago

ynh_secure_remove ${final_path}/update # modif perso give me an issue. After updating my previous instance i got a message is file update/index.php not found. A manual download the pluxml update directory and applying ownership solve the issue on my installation.

anto1ne commented 5 years ago

Hi @ewilly ,

Thank you for your feedback, I tried to reproduced you bug. 1- Install Pluxml 5.6 from an older commit : https://github.com/YunoHost-Apps/pluxml_ynh/tree/3c9860d5b46aa5027c146cfb15434ea23ff1ae71 2 - Upgrade to Pluxml 5.7 with the latest version https://github.com/YunoHost-Apps/pluxml_ynh/tree/f63bd012e4183fa4825fd65c970df040a5338ca7 3- Browse to https://domain.tld/pluxml/

The browser display "File not found" and the url is https://domain.tld/pluxml/update/index.php

Youl solved this issue by adding the directory "update" from Pluxml's website to "/var/www/pluxml" You are then able to complete the upgrade.

admin@skynet:~$ sudo yunohost -v
yunohost: 
  repo: stable
  version: 3.4.2.4
yunohost-admin: 
  repo: stable
  version: 3.4.2
moulinette: 
  repo: stable
  version: 3.4.2
ssowat: 
  repo: stable
  version: 3.4.2

I think I found the error. Pluxml recommend to remove the update directory and it was done automatically by the script. It should be done later, once the update is done.

anto1ne commented 5 years ago

This pull request will solve the problem https://github.com/YunoHost-Apps/pluxml_ynh/pull/30

Nevertheless, pluxml_ynh had a lot of modifications between the 5.6 and 5.7. In the futur, I think it would be better to 1 - save /var/www/pluxml/data 2- remove the older pluxml 3- install pluxml 5.7 4- restore your /var/www/pluxml/data