search

PAYONE-GmbH / oxid-6

The PAYONE Oxid 6 payment module
https://www.payone.com/
MIT License
5 stars 26 forks source link

ADD COLUMN `OXTIMESTAMP` missing from migrations for fcpocheckedaddresses #237

Open TiiFuchs opened 2 years ago

TiiFuchs commented 2 years ago

Found another issue with the migrations in 1.6.2:

https://github.com/PAYONE-GmbH/oxid-6/blob/c5c7973949ce3194b83195b45fd38facb169f351/core/fcpayone_events.php#L634

In this line the data from fcpo_checkdate gets copied to OXTIMESTAMP for the fcpocheckedaddresses table. But earlier in the migration this OXTIMESTAMP column never gets added. It's only in the CREATE TABLE, but if you migrate from an earlier version where the table already exists without OXTIMESTAMP it gets not created and therefore the copying e.t.c. does not work.

I had to create the column manually and deactivate and activate the module again for this to work.

AlexHupe commented 2 years ago

But there are many other differences between the tables for version 1.3.2 to 1.6.2. Is there a migration script?

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.