Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam v...

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam verschillend (cq. AantalDwarsverdelingen in APPS).

Aangepast naar "Dwarsverdelingen te picken"

Aangepast naar "Dwarsverdelingen te picken"

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logi...

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logica.
Maar ik geloof best, zonder het na te kijken, dat in deze klasse nog code staat die vertrekt van de ingegevenKenmerken i.p.v. halffabItem.

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties() hiervoor is geen "uitzondering nodig". Tijdens de uitbreidingsfase en review lijkt het me handig om te zien ...

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties()
hiervoor is geen "uitzondering nodig".
Tijdens de uitbreidingsfase en review lijkt het me handig om te zien dat beide properties zijn toegevoegd. Maar eenmaal in productie (en ondersteund door automatische testen) kan dit het standaard pad volgen.
Of zie ik iets over t hoofd?

Tekst correct? Zonder kar --> geen vakken, right?

Tekst correct?
Zonder kar --> geen vakken, right?

Staat al in productie en is voordien al uitgebreid end-to-end getest.

Staat al in productie en is voordien al uitgebreid end-to-end getest.

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:

- Uitbreiding nodige dto-kes en Utils-klassen en convertoren en bijhorende UnitTests met AantalDwarsverdelingen en DwarsverdelingKleur

- Uitsluiting DwarsverdelingKleur in KenmerkDomeinPostProcessor om VHintra-configurator niet kapot te maken

  1. … 15 more files in changeset.
Hier gebruik je ook de dom enumeratie, de waarden zullen echter gelijk zijn

Hier gebruik je ook de dom enumeratie, de waarden zullen echter gelijk zijn

ook terecht.. was met geefproductiewijze te gebruiken ook aangepast.

ook terecht.. was met geefproductiewijze te gebruiken ook aangepast.

Goe gezien.. is inderdaad nog gewijzigd... http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif waarschijnlijk in andere review terug te vinden.

Goe gezien.. is inderdaad nog gewijzigd... waarschijnlijk in andere review terug te vinden.

[UST4140] TAXON : DEEL 1 - LosseComponent property vervangen door productiewijze. (hebben dezelfde functie voor...
[UST4140] TAXON : DEEL 1 - LosseComponent property vervangen door productiewijze. (hebben dezelfde functie voor...
[UST4140] TAXON : LosseComponent property vervangen door productiewijze. (hebben dezelfde functie voor V1, probox en lossecomponenten verpakking. (ook vhitnra.)

- Conversie dom naar apps ook productiewijze volledig. (conversies zijn voorzien om zowel "" als "V" om te zetten. MUST om te testen op accept dat APPS laag nog steeds ok is.

- getest met vhintra. Maatwerkconfigurator en maatwerkviewer = kostberekening. OK

  1. … 220 more files in changeset.
Type TAX

Type TAX

Moet dit niet zijn GeefProductieWijze ?

Moet dit niet zijn GeefProductieWijze ?

Idem: TAOREnum

Idem: TAOREnum

Bovenstaande zal wss werken omdat de values gelijk zijn, voor alle zekerheid toch TAOREnum gebruiken

Bovenstaande zal wss werken omdat de values gelijk zijn, voor alle zekerheid toch TAOREnum gebruiken

[UST4140] TAXON : LosseComponent property vervangen door productiewijze. (hebben dezelfde functie voor V1, probox en lossecomponenten verpakking. (ook vhitnra.)

- Losse component verwijderd van dto, samen met vhintra release. vhintra gebruikt nu ook voor losse component de productiewijze.

- productiewijze afgestemd voor intern. bij losse component geen volledig. bij flatpackv1 enkel volledig. bij v1 en probox keuze.

- manueel getest met vhintra. (eventueel nog testen voorzien voor kenmerkenpostprocessor.)

- lossecomponent property is vervangen door productiewijze had dus ook impact op DOM of APPS in convertoren. Verschill tussen "" leeg en "V" volledig. Dit is rechtgezet voor TAX

  1. … 288 more files in changeset.
[UST3926] TAXON: TAX onlies: Vhintra: productiewijze-keuze open laten bij verpakking V1. Initieel...
[UST3926] TAXON: TAX onlies: Vhintra: productiewijze-keuze open laten bij verpakking V1. Initieel...
[UST3926] TAXON: TAX onlies: Vhintra: productiewijze-keuze open laten bij verpakking V1. Initieel enkel PM.

- Test ook productiewijze

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-? Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 . De review heb ik wel hier toegevoegd, w...

Vreemd dat deze klasse in de review van UST3954 zit, want is gecommit onder UST3942 :-?
Hoe dan ook : nieuwe UT toegevoegd, maar ook gecommit onder UST3942 .
De review heb ik wel hier toegevoegd, want gij hebt er achter gevraagd :-D :-D

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven! *vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: ...

Er is een testcase waarbij verpakking probox moet geconverteerd worden. Dit moet uiteraard probox blijven!

  • vhUnitTest.EXT.Unishop.PurchaseOrder001.ProductConverter.ExtNaarEdi.Tax.Test.cls(Test: TAX LadeMetVeelExtras)
    Maar ik maak een nieuwe testmethod bij, die controleert of verpakking "" wordt geconverteerd naar V1.