User tests: Successful: Unsuccessful:
Please see #8274 and #8057 for more details.
We need to correct the nested sets after we've removed one entry in the menu.
Thanks to @matrikular and @roland-d for providing valued input on this topic.
How to test?
Do an installation and an update (with the changed files included in the package) and check whether the menu still works properly!
Status | New | ⇒ | Pending |
Labels |
Added:
?
|
Milestone |
Added: |
Something I observed while getting prepared to test this patch. I installed the current staging branch and then went to Manage > Database. There I get this items:
able 'v1j2o_banners' does not have column 'alias' with type 'varchar(191)'. (From file 3.5.0-2015-07-01.sql.)
Table 'v1j2o_categories' does not have column 'alias' with type 'varchar(191)'. (From file 3.5.0-2015-07-01.sql.)
Table 'v1j2o_contact_details' does not have column 'alias' with type 'varchar(191)'. (From file 3.5.0-2015-07-01.sql.)
Table 'v1j2o_content' does not have column 'alias' with type 'varchar(191)'. (From file 3.5.0-2015-07-01.sql.)
Table 'v1j2o_menu' does not have column 'alias' with type 'varchar(191)'. (From file 3.5.0-2015-07-01.sql.)
Table 'v1j2o_newsfeeds' does not have column 'alias' with type 'varchar(191)'. (From file 3.5.0-2015-07-01.sql.)
Table 'v1j2o_tags' does not have column 'alias' with type 'varchar(191)'. (From file 3.5.0-2015-07-01.sql.)
Table 'v1j2o_ucm_content' does not have column 'core_alias' with type 'varchar(191)'. (From file 3.5.0-2015-07-01.sql.)
Can you adapt joomla.sql appropriately?
@waader @infograf768 Please test the update. Do a fresh installation and check whether database errors are displayed in "Extensions" - "Manage" - "Database".
Database Up to date. OK for me
@infograf768 Thanks! And the menus are working properly? If yes, please mark as tested successfully. Thank you!
I have tested this item successfully on 03e6fd1
Thanks Kubik-Rubik!
I have tested mysql, postgres and mssql.
Can we try and split things up please. Let's keep all the stuff for the alias's in @zero-24 's PR #8268 (just PR the non-mysql database changes for that into there) and keep this specific to that menu change.
For the menu changes. The old values 18-20 were for weblinks. So any testers MUST make sure at minimum they test starting at 3.3.6 and upgrading through the versions to this version. To be honest whilst I don't mind what we start new installs on I see upgrading existing installs as very risky and bug prone - it could leave people with two items with the same lft value in a worst case scenario. This is why we have the rebuild button in the menu manager and don't do update queries for menus pretty much ever.
I didnĀ“t see this neither. @Kubik-Rubik. you also have the date changes for postgres.
I will close this PR in favor of #8268 and #8284 (comment).
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-11-05 22:04:00 |
Closed_By | ⇒ | Kubik-Rubik |
Milestone |
Removed: |
@Kubik-Rubik With #8274 I checked the component menus and the worked. So obviously that was not enough. I have to check all menu items, right?