Skip to content

Return to voku/portable-utf8#39

Merged
HLeithner merged 3 commits into
wamania:5.0-devfrom
D3R:revert-33-voku
Jun 5, 2026
Merged

Return to voku/portable-utf8#39
HLeithner merged 3 commits into
wamania:5.0-devfrom
D3R:revert-33-voku

Conversation

@dmnc
Copy link
Copy Markdown

@dmnc dmnc commented May 26, 2026

Reverts #33 to move from joomla\string back to voku/portable-utf8. The latter is not abandoned as suggested in the previous PR, although there did seem to be some indication in the discussion there that joomla/string may be soon.

The main reason I would like to do this is to address #38.

I also removed some deprecations by deleting the CsvFileVerboseIterator that was only used for one language and adding #[ReturnTypeWillChange] attributes to maintain PHP 7 support.

@dmnc dmnc changed the title Revert "Removing dependency on voku/portable-utf8" Revert too voku/portable-utf8 May 26, 2026
@dmnc dmnc marked this pull request as ready for review May 26, 2026 10:38
@dmnc dmnc changed the title Revert too voku/portable-utf8 Return to voku/portable-utf8 May 26, 2026
Comment thread src/Stemmer/English.php Outdated
Comment thread src/Stemmer/German.php Outdated
Comment thread src/Stemmer/German.php Outdated
Comment thread src/Stemmer/Italian.php Outdated
Comment thread src/Stemmer/Italian.php
Comment thread src/Stemmer/Spanish.php Outdated
Comment thread src/Stemmer/Catalan.php Outdated
Comment thread src/Stemmer/Dutch.php Outdated
Comment thread src/Stemmer/Dutch.php Outdated
Comment thread src/Stemmer/French.php Outdated
@dmnc dmnc requested a review from HLeithner May 26, 2026 16:39
@dmnc
Copy link
Copy Markdown
Author

dmnc commented May 26, 2026

Thank you @HLeithner for your very speedy review. I have now removed the deprecated UTF8::str_replace calls.

@HLeithner
Copy link
Copy Markdown
Collaborator

Thank you @HLeithner for your very speedy review. I have now removed the deprecated UTF8::str_replace calls.

I think not all?

@dmnc
Copy link
Copy Markdown
Author

dmnc commented May 27, 2026

Thank you @HLeithner for your very speedy review. I have now removed the deprecated UTF8::str_replace calls.

I think not all?

I can't find any more with a few different methods of code search. Are you able to link to where you are seeing an issue please? One of your comments may look like I missed it, as it was referring to the line after the call itself so may not have auto-resolved.

Copy link
Copy Markdown
Collaborator

@HLeithner HLeithner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, it seem github ui tricked me...

@dmnc
Copy link
Copy Markdown
Author

dmnc commented Jun 5, 2026

@HLeithner Thank you for approving. Do you have merge and release privileges or do we need to alert another maintainer?

@HLeithner HLeithner changed the base branch from master to 5.0-dev June 5, 2026 09:10
@HLeithner
Copy link
Copy Markdown
Collaborator

Yes I can but I would like to have some other things fixed too and move forward in php support with type hints.
Would be nice if you can help on this topic.

I created a 5.0 branch and will merge this pr into 5.0-dev, so you can reference it even if it's not tagged yet

@HLeithner HLeithner merged commit 81a03ba into wamania:5.0-dev Jun 5, 2026
14 checks passed
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.

2 participants