Comment Search

Waarom is er geen versie voor FlatpackV1 ??

Lijkt mij misschien al eens handig om een helper method aan te maken :

do ..AssertConverter( .. ().LadeMetVerlaagdeRug(), (...).LadeMetVerlaagdeRug())

Standaardbinnenlade wordt getest met ---> CopyListOfDataTypes(..StandaardLadeItems(),..StandaardBinnenladeItems()) maw gebruikt standaardlade
en binnenlade bevat geen setschroeven.

De klasse wordt maar éénmalig gegenereerd en ik vind het niet correct om foutieve code te laten staan, ook al is het een gegenereerde klasse en ook al wordt deze overridden in een andere klasse. Dit kan enkel maar verwarrend werken en kans geven op fouten als anderen de gegenereerde gaan gebruiken.

Wordt gebruikt in afgeleide klassen... mogelijk als indicatie dat je deze zeker moet meegeven. eventueel set verstuurder naar base Class, Maar gezien dat de spy een andere implementatie heeft voor verstuurder in constructor.

ik mis hier een test met alpnach die aangeeft dat er bij alpnach dus geen alpnach etiket meer geprint wordt.

properties worden pas later in de flow gezet. daarom nu expliciet blank.

Dan best parameter laten staan en deze in de afgeleide klassen doorgeven via de ##super()

Leesbaarheid: code voluit schrijven of in een method plaatsen met een duidelijke naam

Gaan we hier geen verkeerd beeld krijgen ?? Door het feit dat er bepaalde zaken al in AX zitten ?

Tests ontbreken (zie: vhUnitTest.APPS.VKP.Offerte.impl.BesteldeOfferteMelder.MailVerzender.Test)

Builder opnieuw genereren, het selectiekenmerk voor LadeKleur ontbreekt nog.

Is er al een kaartje voor de aanpassingen die nodig zijn als de conversies meegenomen worden vanuit econ?