User tests: Successful: Unsuccessful:
Pull Request for Issue #40208 (comment) .
This pull request (PR) corrects the following two mistakes with lft and rgt values for the nested #__assets
table for new installations:
Testers need to know and have understood the principles of hierarchical tables using the nested set model as described here:
https://en.wikipedia.org/wiki/Nested_set_model
The #__assets
table uses an extended model which saves as redundant information also parent_id
and level
in addition to lft
and rgt
for speeding up certain queries.
As the structure of the assets table is only a few levels deep, the nested table can be easily verified in tools like phpMyAdmin or phpPgAdmin.
For this you can either make a new installation with the branch of this PR, or you simply use an empty database and execute the statements to create the assets table and insert the records from the base.sql file of this PR.
Then you select all records from the table and order by the lft
column in ascending order. Make sure that really all records are shown.
Then go through the lft and rgt values from top to bottom and lft to rgt, and when at the end of a child level from the last rgt value on that level back to the rgt value of the parent further above. So just follow the lft and rgt values as described on this Wikipedia page about the nested model. Check that the sequence of lft and rgt values is according to that model and that the parent ID and the level are correct, too.
Check that the changes of this PR are correct.
For issue 2. you can also add a new asset e.g. by adding a new category or by reproducing issue #40199 . Make sure that the rgt value of the root asset is updated, too, when the new asset is added.
Please select:
No documentation changes for docs.joomla.org needed
No documentation changes for manual.joomla.org needed
Category | ⇒ | SQL Installation Postgresql |
Status | New | ⇒ | Pending |
Title |
|
Labels |
Added:
PR-4.3-dev
|
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
?
|
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-03-30 15:51:54 |
Closed_By | ⇒ | obuisard |
Thanks for the PR Richard @richard67. Great instructions as well, worth keeping.
I have tested this item✅ successfully on cd26415
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40237.