GX-Task #68464
Von Moritz Bunjes vor fast 4 Jahren aktualisiert
The third party developer Holger Schrörs from Xycons is complaining about the missing possibility to overload logic in CheckoutShippingContentControl::proceed() and ot_coupon::calculate_tax_deduction(), because the methods are much to long, so you cannot overload parts of it. The code should be split into several methods. Example: In CheckoutShippingContentControl::proceed() it should be possible to overload the value of Holger's message: " Hier nun die Dinger, bei denen man schwer dazwischen kommt und somit echte Probleme bei Überladungen hat, so dass man wenn überhaupt die gesamte Methode überladen muss, was hinsichtlich der Updatesicherheit ein echtes Risiko ist, denn lange Inhalte werden dann doch mal hier und da punktuell geändert und unterscheiden sich leider sehr oft zwischen den Versionen: CheckoutShippingContentControl proceed() Praktisch unmöglich MODULE_ORDER_TOTAL_SHIPPING_FREE_SHIPPING_OVER situativ zu übersteuern z. B. für bestimmte Kundengruppen ☹ Das sind mal eben rd. 200 Zeilen am Stück ot_coupon calculate_tax_deduction Praktisch unmöglich da etwas Flexibler zu agieren; mal eben 500 Zeilen am Stück"