Upgrade from 18.10.11 to 22.7.1

[Moritz]

The siteid of both tables was NULL.
I set both to an empty string and now all users are displayed.

Are these changes become part of the upgrade wizard or do I still execute them manually?

Code: Select all

UPDATE fe_users_address SET mtime = '0001-01-01 00:00:00' WHERE mtime = 0;
UPDATE fe_users_address SET ctime = '0001-01-01 00:00:00' WHERE ctime = 0;

UPDATE fe_users SET siteid = '' WHERE siteid IS NULL;
UPDATE fe_users_address SET siteid = '' WHERE siteid IS NULL;

And the change of this line https://github.com/aimeos/ai-typo3/blob ... o3.php#L49 to

Code: Select all

if( !$db->hasTable( 'mshop_locale_site' ) || !$db->hasColumn( 'mshop_locale_site', 'siteid' ) )

[aimeos]

Are these changes become part of the upgrade wizard or do I still execute them manually?

Code: Select all

UPDATE fe_users_address SET mtime = '0001-01-01 00:00:00' WHERE mtime = 0;
UPDATE fe_users_address SET ctime = '0001-01-01 00:00:00' WHERE ctime = 0;

Can you investigate where the mtime/ctime values in your fe_users_address come from?
Guess, they are not inserted by Aimeos because otherwise, these columns wouldn't have "0" values.

UPDATE fe_users SET siteid = '' WHERE siteid IS NULL;

This should be already done because of this line:
https://github.com/aimeos/ai-typo3/blob ... o3.php#L60

UPDATE fe_users_address SET siteid = '' WHERE siteid IS NULL;[/code]

These lines have been added now to set the site ID values of the fe_users_address table correctly:
https://github.com/aimeos/ai-typo3/blob ... hp#L63-L65

And the change of this line https://github.com/aimeos/ai-typo3/blob ... o3.php#L49 to

Code: Select all

if( !$db->hasTable( 'mshop_locale_site' ) || !$db->hasColumn( 'mshop_locale_site', 'siteid' ) )

This should not be necessary if you upgrade from scratch.

[Moritz]

Can you investigate where the mtime/ctime values in your fe_users_address come from?
Guess, they are not inserted by Aimeos because otherwise, these columns wouldn't have "0" values.

I think I can't...
The values was already there before the update.

This should be already done because of this line:
https://github.com/aimeos/ai-typo3/blob ... o3.php#L60

Ah perfect.
In my v22.7.3-pre6 the tablename bugfix isn't fixed.

This should not be necessary if you upgrade from scratch.

You can evaluate this better.
But is the logic correct?
You skip the migration task if mshop_locale_site has the column siteid?

[aimeos]

This should not be necessary if you upgrade from scratch.

You can evaluate this better.
But is the logic correct?
You skip the migration task if mshop_locale_site has the column siteid?

There's still a dependency error. Currently, the TYPO3 siteid migration task runs after the general siteid migration task and thus, the column is already created and the TYPO3 task won't do anything. We've changed the dependency so the TYPO3 siteid migration task will be executed before the general siteid migration task. Then, it should be correct.

Here's a new package:
https://aimeos.org/fileadmin/download/a ... 3-pre7.zip

[Moritz]

Thank you for the prerelease!

I found another issue when I want to download a file from order.

download.png (57.93 KiB) Viewed 2245 times

There is a foreign key on fe_users_list.type to fe_users_list_type.id of datatype integer.
The SQL tries to insert a string in this column.

The exception occurs in "aimeos/Resources/Libraries/aimeos/ai-client-html/src/Client/Html/Account/Download/Standard.php(243)".

Maybe there is a missing migration of the foreign key?

[aimeos]

The setup task for migrating the type columns in the fe_users_* tables had been removed by accident in 2022.04. Now, it's available again in the 22.10.1-pre1 pre-release. Please test if the typeid column in fe_users_list and fe_users_property is changed from INT to VARCHAR:

https://aimeos.org/fileadmin/download/a ... 1-pre1.zip

[Moritz]

When installing v22.10.1-pre1 with my old v18.10.11 tables I get the following exception.

ext_install_error.png (17.4 KiB) Viewed 2228 times

With v22.7.3-pre7 the upgrade wizard is successfully.

[aimeos]

MySQL doesn't accept quotes in table and column names by default. Here's a new package:
https://aimeos.org/fileadmin/download/a ... 1-pre2.zip

[Moritz]

The installation of v22.10.1-pre2 works now.

But there are some users missing again in the backend.
Also the entries in fe_users_address are away of these users.
Because of that I can't see any orders of those users in frontend account history.

In v22.7.3-pre7 that was working.

[Moritz]

The catalog tree to filter with by category isn't working too...

Edit: Or do you have changed the strukture of the template again?
I modified the styles to display the filter tree like a sidemenu.
Works also in v22.7.3-pre7 but not with v22.10.1-pre2.