? Composer Dependency Changed PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
27 Feb 2023

Pull Request for Issue #39512.

Summary of Changes

Our wamania/php-stemmer requires voku/portable-utf8, which in the 5.4 version is not compatible with PHP 8.2. Version 6.0 actually is and since the code for the stemmer is compatible, we can thus update voku/portable-utf8 now with this. php-stemmer will most likely not update it for the 2.x version we are using. For Joomla 5 this is probably going to be done through our dependency updates.

Testing Instructions

Apply the change (together with composer install) and then run the indexer of smart search and afterwards search something in the frontend. Make sure that the stemmer is selected in the global options of Smart Search by setting "Default Language" to "Default Site Language". All of this in PHP 8.2

Actual result BEFORE applying this Pull Request

You see a notice.

Expected result AFTER applying this Pull Request

You don't see a notice.

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

avatar joomla-cms-bot joomla-cms-bot - change - 27 Feb 2023
Category External Library Composer Change
avatar Hackwar Hackwar - open - 27 Feb 2023
avatar Hackwar Hackwar - change - 27 Feb 2023
Status New Pending
avatar Hackwar Hackwar - change - 27 Feb 2023
Labels Added: Composer Dependency Changed PR-4.3-dev
avatar laoneo laoneo - test_item - 6 Mar 2023 - Tested successfully
avatar laoneo
laoneo - comment - 6 Mar 2023

I have tested this item successfully on 44f9ff1

When I run joomla in a webserver with xdebug enabled, like I do with DPDocker, then I get always on the first page load a deprecated warning from voku. With this patch not anymore.


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

avatar carlitorweb carlitorweb - test_item - 6 Mar 2023 - Tested successfully
avatar carlitorweb
carlitorweb - comment - 6 Mar 2023

I have tested this item successfully on 44f9ff1


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

avatar Quy Quy - change - 6 Mar 2023
Status Pending Ready to Commit
avatar Quy
Quy - comment - 6 Mar 2023

RTC


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

avatar obuisard obuisard - change - 8 Mar 2023
Labels Added: ?
avatar obuisard obuisard - change - 8 Mar 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-03-08 02:17:15
Closed_By obuisard
avatar obuisard obuisard - close - 8 Mar 2023
avatar obuisard obuisard - merge - 8 Mar 2023
avatar obuisard
obuisard - comment - 8 Mar 2023

Thank you Hannes @Hackwar !

avatar infograf768
infograf768 - comment - 8 Mar 2023

We have the same problem in 4.2.9
Impossible to install a new 4.2.9 and other issues.

avatar Hackwar
Hackwar - comment - 8 Mar 2023

This class is not used during installation of Joomla.

Add a Comment

Login with GitHub to post a comment