add missing inflectors about legacy-twig-view HOT 9 CLOSED

No problem working on it now.

These are interesting, been mainly using utility classes for things like this but these are actually inside a helper. The Helper element I'm working on should help with these two:

• autoParagraph => TextHelper::autoParagraph()
• autoLinkUrls => TextHelper::autoLinkUrls()

from legacy-twig-view.

slightly unrelated
why is number_format not numberFormat
I seems the convention is camelBacked instead of under_scored for 99% of the methods.

from legacy-twig-view.

the newer ones seem missing as filter:

• dasherize maps to Cake\Utility\Inflector::dasherize()
• slug should be Text:slug() as Inflector::slug() is deprecated

also:

• autoParagraph => TextHelper::autoParagraph()
• autoLinkUrls => TextHelper::autoLinkUrls()

Will add those through a PR, or are you already working on it?

slightly unrelated
why is number_format not numberFormat
I seems the convention is camelBacked instead of under_scored for 99% of the methods.

Sounds like a good target to fix in the release for cake4

from legacy-twig-view.

Nah, I am busy otherwise. Thanks for looking into it.

from legacy-twig-view.

• slug should be Text:slug() as Inflector::slug() is deprecated

Methods look rather different, is the outcome always the same regardless of which I use?

from legacy-twig-view.

Methods look rather different, is the outcome always the same regardless of which I use?

For amost all practical uses cases it's the same.

from legacy-twig-view.

I tested it and it looked quite similar in my cases, but if we want to make sure for all cases, maybe we should add one and deprecate the other.

from legacy-twig-view.

Was expecting Inflector::slug() to use Text::slug() internally due to the deprecation

from legacy-twig-view.

Ok 4.3.6 has been released with the dasherize + @ADmad deprication notice bug fix.

from legacy-twig-view.