Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.0] Tags: Add rebuild button to toolbar #42586

Merged
merged 5 commits into from Jan 10, 2024

Conversation

Hackwar
Copy link
Member

@Hackwar Hackwar commented Jan 1, 2024

Summary of Changes

On Facebook a user had an issue with invalid lft and rgt values in the tags table. When I told him to click "rebuild", it turns out that the tags component doesn't have that button. Upon adding it back in, it turns out that exactly only this button is missing, but the task in the controller and all other necessary code (even including the language strings) is already there. That is why I'm putting this PR against 5.0-dev as a bugfix and not a new feature against 5.1-dev. The code is there, the only thing missing right now is the button in the toolbar.

CC @brianteeman just to let you know that this issue would be fixed with this PR.

Testing Instructions

  1. Go to backend of tags component, see the missing rebuild button.
  2. Apply PR, see the button appear.
  3. Click rebuild button and see that it reports a successfull rebuild of the data.

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

@toivo
Copy link
Contributor

toivo commented Jan 2, 2024

I have tested this item ✅ successfully on 1c7f4b5

Tested successfully in Joomla 5.0.2-rc2-dev of 2 January in Wampserver using PHP 8.2.13.


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

@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 1c7f4b5


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

@Quy
Copy link
Contributor

Quy commented Jan 2, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 2, 2024
@Quy Quy added the bug label Jan 2, 2024
@bembelimen bembelimen changed the base branch from 5.0-dev to 5.1-dev January 2, 2024 23:31
@bembelimen
Copy link
Contributor

New feature => 5.1

@Quy Quy removed the bug label Jan 3, 2024
@CCMLanger
Copy link

tested succsessvully


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

@CCMLanger
Copy link

I have tested this item ✅ successfully on 3d4f0d6


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

@LadySolveig LadySolveig added this to the Joomla! 5.1.0 milestone Jan 10, 2024
@LadySolveig LadySolveig merged commit f85b8d7 into joomla:5.1-dev Jan 10, 2024
3 checks passed
@LadySolveig
Copy link
Contributor

Thank you for the quick fix @Hackwar and for the tests @toivo @viocassel @CCMLanger

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 10, 2024
@Hackwar Hackwar deleted the 5.0-tags-rebuild branch March 22, 2024 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants