Ist halt nicht so ganz best practice. Sauberer wäre es ein ordentliches Plug-In zu schreiben.
Ein „sauberes“ Plugin zu schreiben, um in den Einkaufswelten etwas zu machen ist aber seit Shopware 5.3 und dem EmotionBundle wesentlich komplizierter. Da kann man sich eben nicht mehr „mal einfach schnell“ auf ein Event registrieren und schon geht alles.
Und ob man das jetzt so rum macht oder anders herum mit der Smarty Function ist doch letztendlich egal. Beides führt zum Ziel und beides ist performant.