Skip to content

[4.0] Rename some component helper classes#27528

Merged
wilsonge merged 3 commits intojoomla:4.0-devfrom
SharkyKZ:j4/finder-language-helper
Jan 16, 2020
Merged

[4.0] Rename some component helper classes#27528
wilsonge merged 3 commits intojoomla:4.0-devfrom
SharkyKZ:j4/finder-language-helper

Conversation

@SharkyKZ
Copy link
Copy Markdown
Contributor

Summary of Changes

Renames Joomla\Component\Finder\Administrator\Helper\FinderHelperLanguage and Joomla\Component\Users\Administrator\Helper\UsersHelperDebug to follow the same convention as other helper classes.

Testing Instructions

  1. Run Smart Search indexer, search using Smart Search in frontend.
  2. View Users and User Groups, click the icon in Permissions column.

Expected result

Works like before.

Documentation Changes Required

IDK.

@jwaisner
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 8985f5d


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

1 similar comment
@richard67
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 8985f5d


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

@richard67
Copy link
Copy Markdown
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 15, 2020
@richard67
Copy link
Copy Markdown
Member

Drone mysql system test failing seems not to be related to this PR.

@wilsonge wilsonge merged commit d3f427a into joomla:4.0-dev Jan 16, 2020
@wilsonge
Copy link
Copy Markdown
Contributor

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 16, 2020
@wilsonge wilsonge added this to the Joomla 4.0 milestone Jan 16, 2020
brianteeman pushed a commit to brianteeman/joomla-cms that referenced this pull request Feb 4, 2020
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