HFDiepteVerstellingv001.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken. Alvast bedankt om het aantal testfiles te filteren. Het geheel lijkt me we...

TestFiles reviewen, is in dit soort stories bijna onbegonnen werk. Ik heb zo goed en zo kwaad mogelijk de changes bekeken.
Alvast bedankt om het aantal testfiles te filteren.
Het geheel lijkt me wel oké te zijn.

Ook in eCon enkele commits gereviewed.

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten) In combinatie met svn rev. 69413 is dit wel een correcte overgang. Hierbij is deze cha...

De dossiercode "LI7K" was blijkbaar een onbedoelde tussenstap (Tricky situatie met de TAOR Losse componenten)
In combinatie met svn rev. 69413 is dit wel een correcte overgang.

Hierbij is deze change nagekeken en goedgekeurd!

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om ui...

De meeste testfiles heb ik maar buiten de review gehouden. Een aantal zijn toch toegevoegd, bij wijze van check van het resultaat is dat, denk ik, wel ok en het is bovendien soms ook moeilijk om uit te maken a.d.h.v. de changesets of het al dan niet over code gaat of testfiles.
P.s.: Ook kleine change in Econ (zie svn).

[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Nog wat UnitTest die irrelevant geworden zijn (Schroef- en DrevelKoppeling) uitgeschakeld.

- DiepteVerstelling wordt voortaan altijd los meegeleverd => geen boringen meer omdat nieuwe MoventoKoppeling => frezen ipv boren.

- DiepteVerstellingSchroef => BuilderMoetWerken => voortaan altijd false vanwege bovenstaande + losse schroefjes geregeld door andere builder (SetSchroeven).

- HFSetSchroeven => ConditieDiepteVerstellingSchroefBepaler => Logica i.g.v. DrevelKoppeling gespecifieerd, om afwijking met MoventoKoppeling te hebben. Bij DrevelKoppeling verandert er niets (maar gaat er nooit meer bijzijn).

=> I.g.v. MoventoKoppeling => Altijd schroefjes los meegeleverd, want ook DiepteVerstelling wordt dan los meegeleverd.

  1. … 5 more files in changeset.
Kleine tip : wanneer ik $$$ElkeWaarde zie staan, dan probeer ik die (als reviewer of als developer) steeds toch eens te vervangen door iets specifieker : in dit geval poging met ..IngegevenKenmerke...

Kleine tip :
wanneer ik $$$ElkeWaarde zie staan, dan probeer ik die (als reviewer of als developer) steeds toch eens te vervangen door iets specifieker : in dit geval poging met ..IngegevenKenmerken
maar dat lukte helaas niet.
Dus mag het blijven staan zoals het nu is

[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:

- Comment toegevoegd ter verduidelijking: De combinatie SchroefKoppeling met Binnenlade gaat nooit voorkomen, want i.g.v. een Binnenlade wordt er gedefault op DrevelKoppeling.

[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:
[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:
[ICT-2623] [rvWV] HX: TAX: Movento-koppeling: DiepteVerstelling-builder:

- De ProductieRoute voor de DiepteVerstelling wordt bepaald a.d.h.v. het KoppelingType. Voor de MoventoKoppeling geldt dezelfde logica als voor de DrevelKoppeling.

  1. … 1 more file in changeset.
[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:

- Na review => Eerst Super uitvoeren in constructors

- Rechtzetting in naamgeving UnitTest

  1. … 4 more files in changeset.
Aangezien daar geen automatische testen voor zijn, doen we dat af en toe maar eens manueel hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif Stom copy-p...

Aangezien daar geen automatische testen voor zijn, doen we dat af en toe maar eens manueel hé
Stom copy-paste-foutje hé

Tekst in methodnaam klopt niet meer. Is nog steeds "Onderdelenpicking. om de reviewer te testen, zekers? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Tekst in methodnaam klopt niet meer. Is nog steeds "Onderdelenpicking.

om de reviewer te testen, zekers?

Graa deze aanpassen naar de nieuwere afspraak. Dus eerste lijn Do ##super() en op t einde Quit $$$OK

Graa deze aanpassen naar de nieuwere afspraak.
Dus eerste lijn Do ##super() en op t einde Quit $$$OK

Zie opmerking bij AfdekkapBovenV000: Quit $$$OK

Zie opmerking bij AfdekkapBovenV000:
Quit $$$OK

Deze stond nog volgens de "oude standaard" http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/sad.gif Graa deze aanpassen naar de nieuwere afspraak. Dus eerste lijn ...

Deze stond nog volgens de "oude standaard"
Graa deze aanpassen naar de nieuwere afspraak.
Dus eerste lijn Do ##super() en op t einde Quit $$$OK

Bij %OnNew() doen we ALTIJD eerst do ##super() en op t einde ne Quit $$$OK --> de ##super moet hier sowieso als eerste lijn van deze method gebeuren. Dit is al verschillende keren ter sprake gekom...

Bij %OnNew() doen we ALTIJD eerst do ##super() en op t einde ne Quit $$$OK
--> de ##super moet hier sowieso als eerste lijn van deze method gebeuren.

Dit is al verschillende keren ter sprake gekomen (bij andere collega's), maar het moet zo blijven.
Kan je dit dus terugzetten zoals het er stond?
Uiteraard mag uw lijn met $$$Inject wel blijven hé.

[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:
[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:
[ICT-717] [ICT502] [rvWV] UT VerpakkingTypes: IsgemonteerdeLade() uitmocken:

- IngegevenKenmerkenTypeApi uitgemockt i.g.v. HFDiepteVerstellingv000 => Volledige refactor TestCase

  1. … 1 more file in changeset.
[UST1368] TAX-PAS-FPV1: productiedetailblad wijzigingen:

- groepnaam voor DiepteVerstelling op basis van productieroute: wijzigingen na review

  1. … 10 more files in changeset.
[UST1368] TAX-PAS-FPV1: productiedetailblad wijzigingen:

- groepnaam voor DiepteVerstelling op basis van productieroute

  1. … 11 more files in changeset.