Projekt

Allgemein

Profil

GX-Task #68464

Von Moritz Bunjes vor fast 3 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"

Zurück