Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte

- Uitbreiding van al de dto's, convertoren en utils-klassen met DwarsverdelingType

- Unit tests aangepast voor bovenstaande wijziging

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

  1. … 11 more files in changeset.
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte

- Per ongeluk 1 file teveel meegenomen, changes gerevert en volgende opnieuw gecommit:

- Uitbreiding van al de dto's, convertoren en utils-klassen met DwarsverdelingType

- Unit tests aangepast voor bovenstaande wijziging

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

  1. … 11 more files in changeset.
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte

- Uitbreiding van al de dto's, convertoren en utils-klassen met DwarsverdelingType

- Unit tests aangepast voor bovenstaande wijziging

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

  1. … 12 more files in changeset.
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.
[ICT465] EDI TAX tag EdgeBandColour voorzien
[ICT465] EDI TAX tag EdgeBandColour voorzien
[ICT465] EDI TAX tag EdgeBandColour voorzien

- KantenbandKleur op EDI laag toegevoegd en hier en daar wat conversies/standaardlades aangepast

  1. … 6 more files in changeset.
Ik snap de redenering maar ik vind dit niet correct. Mss eens in de groep gooien en beslissen wat we hiermee doen?

Ik snap de redenering maar ik vind dit niet correct. Mss eens in de groep gooien en beslissen wat we hiermee doen?

De testmethod voor de standaardlade staat nu inderdaad dubbel. Dus ik heb ze in deze klasse verwijderd. Wat de ZijkantLogo-testen van Kurt betreft, daar ga ik wel afblijven, hé http://subversion02...

De testmethod voor de standaardlade staat nu inderdaad dubbel. Dus ik heb ze in deze klasse verwijderd.

Wat de ZijkantLogo-testen van Kurt betreft, daar ga ik wel afblijven, hé

Vond het eerst zelf ook een beetje vreemd, waarom da aangepast is :-P Verklaring vrij eenvoudig : Naam moet consistent zijn met de andere vhTest.Utils ... Kenmerken() klassen, omdat de UT's voor de...

Vond het eerst zelf ook een beetje vreemd, waarom da aangepast is :-P
Verklaring vrij eenvoudig :
Naam moet consistent zijn met de andere vhTest.Utils ... Kenmerken() klassen, omdat de UT's voor de convertoren dit oproepen:

Method "Test: TAX Standaardlade"()
{
Do ..AssertConverter("StandaardLade")
}

Method "Test: TAX BinnenLade"()
{
Do ..AssertConverter("BinnenLade")
}

Method "Test: TAX LadeMetVeelExtras"()
{
Do ..AssertConverter("LadeMetVeelExtras")
}

Anders falen de UT's ... spijtig genoeg. (normaal gezien moet dan de UT-impl aangepast worden en niet de code, maar deze wijziging heeft geen invloed op de werking van de nietmeeleverens, denk.) De...

Anders falen de UT's ... spijtig genoeg.
(normaal gezien moet dan de UT-impl aangepast worden en niet de code, maar deze wijziging heeft geen invloed op de werking van de nietmeeleverens, denk.)
Deze tactiek was ook al toegepast bij LBX door één van m'n voorgangers.

De "ByRef" is hier om aan te geven dat het EconKenmerken-object zal aangepast worden. Technisch is het niet fout om da als .local door te geven. Objecten worden trouwens altijd als "pointer naar ge...

De "ByRef" is hier om aan te geven dat het EconKenmerken-object zal aangepast worden.
Technisch is het niet fout om da als .local door te geven. Objecten worden trouwens altijd als "pointer naar geheugenplaats" doorgegeven.

Principe is bovendien overgenomen van de 2 lijnen eronder, en dat leek mij niet verkeerd om het zo aan te duiden.

Laat zeker weten als je hier toch aan twijfelt.

Zal bij volgende opkuis verwijderd worden.

Zal bij volgende opkuis verwijderd worden.