User tests: Successful: Unsuccessful:
Fix duplicated DB queries for non-existing asset.
Enable debug and debug Query in the debug plugin.
Login in administrator section as manager.
Open Articles list table.
Check amount of DB queries in debug panel.
Or run folowing code, in templates/atum/index.php.
for ($i = 0; $i < 100; $i++) {
$app->getIdentity()->authorise('core.edit', 'com_content.category.1');
}
Around 55.
With testing code it is around 155 or more;
Around 45
With testing code it is around 45;
Please select:
Status | New | ⇒ | Pending |
Category | ⇒ | Libraries |
Labels |
Added:
bug
Small
PR-4.4-dev
|
done
I have tested this item ✅ successfully on 892c892
Without Pull Request: 74 Queries
With Pull Request: 47 Queries
Test installed Multilang (de, en, fa)- and Blog Sample Date (14 Articles).
I have tested this item ✅ successfully on 892c892
Status | Pending | ⇒ | Ready to Commit |
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-09-10 14:37:07 |
Closed_By | ⇒ | laoneo |
Thanks!
You can also fix the strange comment "// Else we have to do an extra db query to fetch it from the table fetch it from table." with this PR.