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.