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.1] Complete com_search deprecation #40727 #42891

Merged
merged 6 commits into from
Apr 10, 2024

Conversation

brianteeman
Copy link
Contributor

@hackware deprecated various search methods in libraries/src/Language/Language.php in #40727

This PR completes that PR by deprecating their use in the localise.php file

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

joomla#40727 deprecated various search methods in `libraries/src/Language/Language.php`

This PR completes that PR by deprecating their use in the localise.php file
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.4-dev labels Feb 26, 2024
@HLeithner
Copy link
Member

it's too late to deprecated anything in 4.4, that's only allowed in x.y.0 releases. In this case I would deprecate it with 5.1 and remove it with 7.0

@brianteeman
Copy link
Contributor Author

its a bug that @Hackwar forgot to deprecate it when he deprecated the code and now removes the actual code with #42892. If this can't be deprecated until 5.1 then the code itself will have to stay al;so until 7. Anything else makes no sense

@ceford
Copy link
Contributor

ceford commented Feb 26, 2024

I have tested this item ✅ successfully on 51255d3

I looked at the code and read the comments so it looks OK to me as a bug fix for a previous commit.


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

@laoneo
Copy link
Member

laoneo commented Feb 26, 2024

@brianteeman then we have to change these depreciations you are talking about to 7.0 as well. Better to follow our deprecation policy and postpone already added deprecations than vice versa.

@brianteeman
Copy link
Contributor Author

fine by me

@laoneo
Copy link
Member

laoneo commented Mar 4, 2024

@brianteeman can you rebase this one to 5.1-dev, so we can merge it.

@brianteeman brianteeman changed the base branch from 4.4-dev to 5.1-dev March 4, 2024 15:47
@brianteeman brianteeman changed the title [4.4] Complete com_search deprecation #40727 [5.1] Complete com_search deprecation #40727 Mar 4, 2024
@Quy Quy removed the PR-4.4-dev label Mar 4, 2024
@laoneo
Copy link
Member

laoneo commented Mar 20, 2024

I have tested this item ✅ successfully on f7ac86b

Doc blocks are correct


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

@Quy Quy added the PR-4.4-dev label Mar 20, 2024
@Quy
Copy link
Contributor

Quy commented Mar 20, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Mar 20, 2024
@Quy Quy removed the PR-4.4-dev label Mar 20, 2024
@brianteeman
Copy link
Contributor Author

@laoneo I did what you asked and rebased it. Any reason why it cant be merged now?

@laoneo
Copy link
Member

laoneo commented Apr 9, 2024

This is up to @bembelimen or @LadySolveig as they are the RM.

@Hackwar
Copy link
Member

Hackwar commented Apr 9, 2024

This is now also to late for 5.1, so it would have to go into 5.2-dev

@brianteeman
Copy link
Contributor Author

I am not remaking it. Someone else can fix your mistake. The pr was made and tested in good time.

@LadySolveig LadySolveig added this to the Joomla! 5.1.0 milestone Apr 10, 2024
@LadySolveig LadySolveig merged commit 0a07442 into joomla:5.1-dev Apr 10, 2024
3 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 10, 2024
@LadySolveig
Copy link
Contributor

Thank you @brianteeman and for review @ceford and @laoneo

@brianteeman
Copy link
Contributor Author

thanks - appreciate it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators PR-5.1-dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants