Add utils service and migrate services to Util namespace

[koertho]

State: Draft

This PR add an utils service containing other utils services. It also marks the beginning of the transition of all utils into the Util namespace in preparation of utils bundle v3.

Todo:

• [x] Create Utils service class
• [x] Migrate some services to demonstrate the new structure
• [x] Update documentation

Changes:

• StringUtil::truncateHtml(): Default ending now "…"

[koertho]

@Defcon0 @AlexejKossmann Bei ein paar StringUtil-Methoden bin ich mir nicht sicher, ob wir die noch brauchen. Was meint ihr?

• StringUtil::removeLeadingAndTrailingSlash()
• StringUtil::removeLeadingString()
• StringUtil::removeTrailingString()
• StringUtil::restoreBasicEntities()
• StringUtil::lowerCase() -> ist ne DCA-Callback-Methode -> Sollte wenn dann in ein anderes Util, aber müsste dann trotzdem immer "the-old-way" genutzt werden

Außerdem: Brauchen wir solche Platzhalter-Methoden noch? Wäre eine direkte Einbindung nicht sinnvoller bei den Erweiterungen, welche das wirklich nutzen?

• StringUitl::convertToInlineCss()
• StringUitl::html2Text() -> hier verwenden wir eine nicht mehr gepflegte Library. Ich würde die Methode deprecaten und eine aktuelle Library empfehlen
• StringUitl::replaceUnicodeEmojisByHtml() -> hier sollten wir eine gepflegte Library nutzen, bspw. https://github.com/elvanto/litemoji
• StringUitl::replaceInsertTags()

[Defcon0]

restoreBasicEntities() kann raus, das ist eine 1:1 Kopie des Cores ;-) lowerCase() kann auch weg, würde ich sagen. Solche direkten DCA-Methoden sind mit 4.9 unflexibel geworden convertToInlineCss() und html2Text() -> stimmt, da gebe ich dir Recht. Dann können wir auch die Abhängigkeiten raustun. replaceInsertTags() hatten wir schon disktutiert. Ist verzichtbar ;-)

Den Rest bitte drin lassen.

replaceUnicodeEmojisByHtml() -> das kann ich im kontext von hyp mal testen, ob deine library funktioniert. Die kannte ich damals noch nicht. Bitte vorerst drin lassen.

[koertho]

@Defcon0 Ok, danke schonmal. Ich schmeiße ja nichts aus der aktuellen Klasse raus, sondern schaue nur, was in die neue mit rein kommt. Der alte Code bleibt (fast) unangetestet. Wirklich was rausfliegen würden die Sachen erst in Version 3.

Über removeLeadingAndTrailingSlash würde ich gerne noch diskutieren, das ist doch mit einem simplen trim($string, "/") getan. Die anderen beiden Methoden wären für mich akzeptabel, auch wenn ich dafür kein Util nutzen würde.

[Defcon0]

Bei removeLeadingAndTrailingSlash hast du Recht ;-) Können wir so machen.

[koertho]

@Defcon0 Hast du bei StringUtil::replaceUnicodeEmojisByHtml() schon Ergebnisse?

[Defcon0]

Leider noch nicht. Steht beim aktuelle Contao 4.9-Update unserer Freunde aus Berlin noch auf der Liste.

[koertho]

@Defcon0 @AlexejKossmann Hi, ich bin durch mir dem PR. Könnt ja gerne nochmal drüber schauen.

[Defcon0]

Sieht gut aus 👍 Schön einfache übersichtliche Struktur!

[AlexejKossmann]

Sieht gut aus 👍