User tests: Successful: Unsuccessful:
Pull Request for Issue # .
This PR makes some cleanup and code improvement to Tour model base on my code review:
$this->getCurrentUser() to replace deprecated Factory::getUser()if (!$canEditState || !empty($item->default)) { seems to be copied from somewhere, very likely from com_workflow code.$user->authorise('core.create', '__guidedtour_steps') permission check. There is no asset __guidedtour_steps, so that check is uselessContains some bad/not optimized code
Cleaner code.
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
[z] No documentation changes for manual.joomla.org needed
| Category | ⇒ | Administration |
| Status | New | ⇒ | Pending |
| 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-09 23:56:54 |
| Closed_By | ⇒ | obuisard |
Thank you so much Tuan @joomdonation
I have tested this item✅ successfully on 816db2c
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40014.