Skip to content

[RTM 2] Use Utf8::toAscii instead of \utf8_romanize (fixes plenty deprecation hints)#9

Merged
stefanheimes merged 2 commits intoMetaModels:masterfrom
bezin:utf8-to-ascii
Feb 16, 2020
Merged

[RTM 2] Use Utf8::toAscii instead of \utf8_romanize (fixes plenty deprecation hints)#9
stefanheimes merged 2 commits intoMetaModels:masterfrom
bezin:utf8-to-ascii

Conversation

@bezin
Copy link
Copy Markdown
Contributor

@bezin bezin commented Oct 1, 2019

The global helper functions are all marked deprecated. This PR fixes plenty deprecation hints by using the advised alternative of Patchwork\Utf8.

Copy link
Copy Markdown
Member

@richardhj richardhj left a comment

Choose a reason for hiding this comment

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

Travis will fail as of malicious Code Style (space) but good besides this.

@zonky2 zonky2 modified the milestones: 2.2.0, 2.1.1 Dec 19, 2019
@zonky2 zonky2 changed the title Use Utf8::toAscii instead of \utf8_romanize (fixes plenty deprecation hints) [RTM] Use Utf8::toAscii instead of \utf8_romanize (fixes plenty deprecation hints) Jan 1, 2020
@zonky2
Copy link
Copy Markdown
Contributor

zonky2 commented Feb 5, 2020

@stefanheimes merge this #11 as first

@zonky2 zonky2 changed the title [RTM] Use Utf8::toAscii instead of \utf8_romanize (fixes plenty deprecation hints) [RTM 2] Use Utf8::toAscii instead of \utf8_romanize (fixes plenty deprecation hints) Feb 5, 2020
discordier added a commit that referenced this pull request Feb 10, 2020
This changes `\utf8_romanize` calls to `Utf8::toAscii` and
therefore fixes plenty of deprecations.
@discordier
Copy link
Copy Markdown
Member

Merged in c0dc54b

@stefanheimes stefanheimes merged commit 65bfdc1 into MetaModels:master Feb 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants