Remove getTable method. It is not used for a List model
Do not need to set filter.search and filter.published states, it is handled by parent class.
Correct getStoreId method, it needs to contains filter.tour_id state value
Rename variable from $tour_id to $tourId to follow camel case naming convension
Testing Instructions
Install Joomla base on download package generated by this PR to make sure you have guided tours available in the installation. Or if you have a site with guide tours already, apply this PR
Access to Guided Tours, then view steps of a tour and make sure nothing is broken on that screen.
Actual result BEFORE applying this Pull Request
Contains unnecessary code
Expected result AFTER applying this Pull Request
Unnecessary code removed
Link to documentations
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
I have tested this item✅ successfully on d930866
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40004.