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

[BOOM-6122] ps_legalcompliance error upgrading from 1.6.1.20 to 1.7.4.2 #9553

Closed
prestonBot opened this issue Aug 22, 2018 · 21 comments
Closed
Labels
1.7.4.1 Affects versions Advancedeucompliance Module: advancedeucompliance Autoupgrade Module: autoupgrade Bug Type: Bug Can't reproduce Resolution: issue closed because cannot be reproduced Fixed Resolution: issue closed because fixed Minor Severity: minor bug > https://build.prestashop.com/news/severity-classification
Milestone

Comments

@prestonBot
Copy link
Collaborator

This issue has been migrated from the Forge. Read the original ticket here.

  • Reporter: maofree
  • Created at: Mon, 30 Jul 2018 07:49:14 +0200

hi
using the module 1-click upgrade I got this error
[INTERNAL] /var/www/html/xxxx-backup/modules/ps_legalcompliance/entities/AeucCMSRoleEmailEntity.php line 99 - Cannot declare class AeucCMSRoleEmailEntity, because the name is already in use

I'll try again removing the module ps_legalcompliance

bye

  • How to reproduce the issue ?

Install Prestashop 1.6.1.20


Install the advancedeucompliance module v1.3.2


Install the 1-click upgrade module v4.0.0-beta-5.


Upgrade to the last version 1.7.4.2

@prestonBot prestonBot added 1.7.4.1 Affects versions Advancedeucompliance Module: advancedeucompliance Autoupgrade Module: autoupgrade Bug Type: Bug Minor Severity: minor bug > https://build.prestashop.com/news/severity-classification Ready Status: Issue is ready to be worked on labels Aug 22, 2018
@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 08:15:10 +0200

for the upgrade I've used 1-click upgrade 4 Beta 3

ps_legalcompliance is not installed on the 1.6 so the problem comes out during the upgrade

Can you fix please ? if not it is not possible to upgrade to the 1.7.x

I'll try again with a clean 1.6.1.20

bye

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 08:30:37 +0200

at the start there are 2 errors

[INTERNAL] /var/www/html/prova/classes/Rijndael.php line 121 - Undefined property: Rijndael::$key
Inizio aggiornamento...
L'archivio scaricato proviene da https://download.prestashop.com/download/releases/prestashop_1.7.4.2.zip
[INTERNAL] /var/www/html/prova/modules/autoupgrade/classes/UpgradeTools/Translator.php line 76 - sprintf(): Too few arguments

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 08:37:26 +0200

Installed a clean 1.6.1.20
installed 1-click upgrade 4 Beta 3
upgrade to 1.7.4.2 was ok without that ps_legalcompliance error

how is it possible? in the other site that module is not installed

can I get some info?

thanks

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 09:02:37 +0200

removed the cookie module of Línea Gráfica, I thought it depended on this module
which module use AeucCMSRoleEmailEntity class in the 1.6?

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: khouloud.belguith
  • Created at: Mon, 30 Jul 2018 09:31:44 +0200

Hi maofree,

A new version beta-5 of the 1-click upgrade module is released.
Thanks to check your issue with the last version v4.0.0-beta-5 and feedback.
You can follow this link: https://github.com/PrestaShop/autoupgrade/releases/tag/v4.0.0-beta.5

Best regards, Khouloud

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 09:34:15 +0200

the problem is due from the module "advancedeucompliance"

there are some errors on blockrss

[INTERNAL] /var/www/html/xxx-backup/modules/blockrss/blockrss.php line 30 - include_once(/var/www/html/xxx-backup/classes/../tools/pear/PEAR.php): failed to open stream: No such file or directory
[INTERNAL] /var/www/html/xxx-backup/modules/blockrss/blockrss.php line 30 - include_once(): Failed opening '/var/www/html/xxx-backup/classes/../tools/pear/PEAR.php' for inclusion (include_path='/var/www/html/xxx-backup/vendor/pear/pear_exception:/var/www/html/xxxx-backup/vendor/pear/console_getopt:/var/www/html/xxx-backup/vendor/pear/pear-core-minimal/src:/var/www/html/xxx-backup/vendor/pear/archive_tar:.:/usr/share/php')
[INTERNAL] /var/www/html/xxx-backup/modules/blockrss/blockrss.php line 31 - include_once(/var/www/html/xxx-backup/tools/pear_xml_parser/Parser.php): failed to open stream: No such file or directory
[INTERNAL] /var/www/html/xxx-backup/modules/blockrss/blockrss.php line 31 - include_once(): Failed opening '/var/www/html/xxx-backup/tools/pear_xml_parser/Parser.php' for inclusion (include_path='/var/www/html/xxx-backup/vendor/pear/pear_exception:/var/www/html/xxx-backup/vendor/pear/console_getopt:/var/www/html/xxx-backup/vendor/pear/pear-core-minimal/src:/var/www/html/xxx-backup/vendor/pear/archive_tar:.:/usr/share/php')

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 09:35:02 +0200

enabling the debug mode I see this error

Unknown column 'h.live_edit' in 'field list'

SELECT h.`name` as hook, m.`id_module`, h.`id_hook`, m.`name` as module, h.`live_edit`
FROM `ps_module` m
INNER JOIN ps_module_shop module_shop
ON (module_shop.id_module = m.id_module AND module_shop.id_shop = 1 AND module_shop.enable_device & 1)
INNER JOIN `ps_module_shop` `ms` ON ms.`id_module` = m.`id_module`
INNER JOIN `ps_hook_module` `hm` ON hm.`id_module` = m.`id_module`
INNER JOIN `ps_hook` `h` ON hm.`id_hook` = h.`id_hook`
WHERE (h.name != "displayPayment") AND (hm.id_shop = 1)
GROUP BY hm.id_hook, hm.id_module
ORDER BY hm.`position`

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 09:35:37 +0200

ok I'll try with Beta 5
15 minutes and I'll report the result

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: khouloud.belguith
  • Created at: Mon, 30 Jul 2018 10:01:02 +0200

Hi maofree,

Thanks waiting for your feedback.

Best regards, Khouloud

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 10:14:17 +0200

Beta 5 tested
these problems are still present

at the start it is necessary to remove advancedeucompliance module
then the installation will be completed
it is necessary also to solve the live_edit error

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: khouloud.belguith
  • Created at: Mon, 30 Jul 2018 11:31:09 +0200

Hi maofree,

So after the upgrade, the advancedeucompliance module & the themeconfigurator are installed & enabled in your shop.
During the upgrade to the last version 1.7.4.2, those modules have issues?

Best regards, Khouloud

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 11:33:35 +0200

themeconfigurator is not a problem
advancedeucompliance should be removed because it has a conflict with ps_legalcompliance

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: khouloud.belguith
  • Created at: Mon, 30 Jul 2018 11:56:40 +0200

Hi maofree !

Thank you for your report.
We'll first try to reproduce it and we'll come back to you if we need more information.

Best regards, Khouloud

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Mon, 30 Jul 2018 13:12:11 +0200

then it is necessary to fix the problem on the hook_module because at each upgrade many front modules lost their position and it are not visible

I've creted a topic to report it some months ago

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: khouloud.belguith
  • Created at: Mon, 30 Jul 2018 13:53:25 +0200

Hi maofree,

Yes we aware of this issue, this ticket BOOM-3195 is added to our bug roadmap.

Best regards, Khouloud

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: khouloud.belguith
  • Created at: Mon, 30 Jul 2018 15:35:34 +0200

Hi maofree,

Install Prestashop 1.6.1.20
Install the advancedeucompliance module v1.3.2
Install the 1-click upgrade module v4.0.0-beta-5.
Upgrade to the last version 1.7.4.2
During the upgrade, I have this error

I manage to reproduce the issue.
We will see how to fix it.

Best regards, Khouloud

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: maofree
  • Created at: Tue, 31 Jul 2018 07:35:32 +0200

Hi
tested again
you were right, before to start the upgrade it is necessary to unistall "themeconfigurator" to avoid the live_edit error

so it is necessary to unistall and remove 2 modules before to start to upgrade
advancedeucompliance
themeconfigurator

bye

@prestonBot
Copy link
Collaborator Author

This comment has been migrated from the Forge. Read the original comment here.

  • Author: khouloud.belguith
  • Created at: Wed, 15 Aug 2018 12:06:39 +0200

Hi,

I manage to reproduce the issue using the 1-click upgrade module v4.0.0 stable.
Upgrade from 1.6.1.20 => 1.7.4.2.

Best regards, Khouloud

@hibatallahAouadni
Copy link
Contributor

Hello @maofree

I didn't manage to reproduce your issue with:

  • Prestashop 1.6.1.20
  • Advancedeucompliance module v2.0.2
  • 1-click upgrade module v4.10.1

And then I upgraded to the last version 1.7.6.8.
During the upgrade, I have these warning:

image

image


No errors encountered.
When the upgrade is done, a success upgrade notice has been displayed:

image


Finally, it log me out and my shop is well upgraded to the latest stable version 1.7.6.8:

image

Waiting for your feed-back.

Thanks!

@maofree
Copy link

maofree commented Oct 30, 2020

hi
normally now I remove that module from an upgrade like 1.6 to 1.7
that test has been done many time ago just to report an error

bye

@hibatallahAouadni
Copy link
Contributor

Hi @maofree

Well then, it seems that problem has been solved with the latest version of modules.

So I close the issue.

Thanks!

@prestashop-issue-bot prestashop-issue-bot bot removed the Ready Status: Issue is ready to be worked on label Oct 30, 2020
@hibatallahAouadni hibatallahAouadni added Fixed Resolution: issue closed because fixed Can't reproduce Resolution: issue closed because cannot be reproduced labels Oct 30, 2020
@hibatallahAouadni hibatallahAouadni added this to the 1.7.6.8 milestone Dec 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.7.4.1 Affects versions Advancedeucompliance Module: advancedeucompliance Autoupgrade Module: autoupgrade Bug Type: Bug Can't reproduce Resolution: issue closed because cannot be reproduced Fixed Resolution: issue closed because fixed Minor Severity: minor bug > https://build.prestashop.com/news/severity-classification
Projects
None yet
Development

No branches or pull requests

3 participants