search

heimrichhannot / contao-utils-bundle

This bundle offers various utility functionality for the Contao CMS.
GNU Lesser General Public License v3.0
8 stars 4 forks source link

Silbentrennung bei Sprache "cs" auch in Scripts #36

Closed FlowinBeatz closed 3 years ago

FlowinBeatz commented 3 years ago

Context Contao version: 4.9.21 Bundle version: 2.203.1 PHP version: 7.4.24

Description Wenn ich einen Seitenbaum mit Sprache "cs" verwende greift die Silbentrennung aus src/Util/Locale/LocaleUtil.php. Diese sorgt allerdings auch dafür, dass die in der Seite verwendeten Scripts & nbsp;-Leerzeichen eingefügt bekommen und dadurch unbrauchbar werden.

fritzmg commented 3 years ago

Since this package is installed as a dependency of many other huh extensions, this functionality should imho be removed entirely from this package and instead should be put into a separate extension, since it is a hidden feature that will also always be active.

koertho commented 3 years ago

Wow, that's a really bad one.

@Defcon0 I would suggest to delete this listener completely, as it makes in my opinion absolutely no sense to use it on the complete rendered page: https://github.com/heimrichhannot/contao-utils-bundle/blob/fbc8a05e79dff89fad3d2630ed0c8a9e10a0e066/src/EventListener/FrontendPageListener.php#L38 WDYT?

Defcon0 commented 3 years ago

Totally agreed. This should not have been implemented so hard. We should remove the listener.

Defcon0 commented 3 years ago

Commit was https://github.com/heimrichhannot/contao-utils-bundle/commit/fa55f00e2f70d879a530c4f31000209ac1bf2d04

FlowinBeatz commented 3 years ago

Thanks for the fast response and your commitment :-)

FlowinBeatz commented 3 years ago

Was muss ich tun um das in mein Produktivsystem zu bekommen? Das util-Bundle ist keine direkte Abhängigkeit in meinem System, sondern kam durch die slick-erweiterung dazu.

Defcon0 commented 3 years ago

Nutzt du composer oder den contao manager?

FlowinBeatz commented 3 years ago

Manager

Defcon0 commented 3 years ago

Mhm, da kenne ich mich nicht soo gut aus. @koertho Kann man dort auch einzelne Extensions updaten ohne alles upzudaten?

koertho commented 3 years ago

Du musst nur deine Pakete aktualisieren über Composer oder Contao Manager. Es ist allerdings noch nicht freigegeben, da ich noch eine offene Baustelle habe. Wird heute noch.

koertho commented 3 years ago

Habe soeben 2.206.0 getaggt, da ist der Fix drin. Einfach das Update machen :)

FlowinBeatz commented 3 years ago

Jetzt war das Update drin. Danke für euren Support!