Skip to content

Conversation

@joomdonation
Copy link
Contributor

Pull Request for Issue # .

Summary of Changes

This PR makes several clean up to Tours module from Guided Tours extension

  • No need to override getEmptyStateQuery method from parent class (This override contains really wrong code, seems copied from Steps model)
  • Remove getTable method. We do not need this method from a List model

Testing Instructions

  • Install Joomla from download package generated by this PR (to make sure Guided Tours is available on your installation). Access to Guided Tours, see list of tours and make sure nothing is broken from this page

Actual result BEFORE applying this Pull Request

Unnecessary wrong code in Tours model

Expected result AFTER applying this Pull Request

The unnecessary wrong 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

@joomdonation
Copy link
Contributor Author

Question for @obuisard : In this model, we also have code to handle extensions filter (no idea how it should work). However, that filter is not in filter form, so should we remove that code as well ?

@obuisard
Copy link
Contributor

obuisard commented Mar 4, 2023

Question for @obuisard : In this model, we also have code to handle extensions filter (no idea how it should work). However, that filter is not in filter form, so should we remove that code as well ?

No, it is used by the Guided Tours module to show tours according to a specific extension that is showing on the screen.

@joomdonation
Copy link
Contributor Author

No, it is used by the Guided Tours module to show tours according to a specific extension that is showing on the screen

Ah, OK. Thanks for the info.

@obuisard
Copy link
Contributor

obuisard commented Mar 9, 2023

I have tested this item ✅ successfully on 17b1444


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40003.

1 similar comment
@Quy
Copy link
Contributor

Quy commented Mar 9, 2023

I have tested this item ✅ successfully on 17b1444


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40003.

@Quy
Copy link
Contributor

Quy commented Mar 9, 2023

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/40003.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 9, 2023
@obuisard obuisard added this to the Joomla! 4.3.0 milestone Mar 9, 2023
@obuisard obuisard merged commit 310182c into joomla:4.3-dev Mar 9, 2023
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Mar 9, 2023
@obuisard
Copy link
Contributor

obuisard commented Mar 9, 2023

Thank you Tuan @joomdonation!

@joomdonation joomdonation deleted the guidertours_cleanup_tours_model branch March 10, 2023 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants