External_SQL.DB_ZijkantKamerFreesData.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- Nieuwe testbestanden - deel 3 van...

=> Alles met MoventoKoppeling en dus HoogteVerstelling nu => in code is nu L => X en R => Y

=> Wel vakken met MoventoKoppeling en geen vakken meer met SchroefKoppeling

    • -80
    • +80
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 31 more files in changeset.
[ICT-2144] [rvTVE,WV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing (frontverstelmogelijkheid):

- Voor testen op accept wordt Van Hoecke als pilootklant ingesteld => IT gebruiken testklant Van Hoecke => barcodes overal veranderd => L = X en R = Y ==>> GEREVERT WEGENS MERGECONFLICTEN => nog teveel changes van anderen die nog niet gemerged kunnen worden

    • -80
    • +80
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 79 more files in changeset.
[ICT-2144] [rvTVE,WV] HX: TAX koppeling: hoogteverstelling pilootklanten productieaansturing (frontverstelmogelijkheid):

- Voor testen op accept wordt Van Hoecke als pilootklant ingesteld => IT gebruiken testklant Van Hoecke => barcodes overal veranderd => L = X en R = Y

    • -80
    • +80
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 79 more files in changeset.
[ICT-2131] [rvPVR] HX: Frezen: TAX: Technologica: enkel linkse of enkel rechtse: omwisseling.
[ICT-2131] [rvPVR] HX: Frezen: TAX: Technologica: enkel linkse of enkel rechtse: omwisseling.
[ICT-2131] [rvPVR] HX: Frezen: TAX: Technologica: enkel linkse of enkel rechtse: omwisseling.

- Omdraaien invulling kamerfreesdata wanneer er meerdere zijkanten zijn

    • -20
    • +20
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 2 more files in changeset.
[ICT-1554] Aanpassing automilling.dbo.production kolom 'InsertTimeStamp'

- Kleine refactoring zodat timestamp mockable wordt

    • -10
    • +10
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 21 more files in changeset.
[UST4200] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Tijdelijke nieuwe producten opkuisen in productie:

- Nieuwe testbestanden met de nieuwe testproducten

    • -20
    • +20
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 15 more files in changeset.
Ter info : 1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls 2) Testklasse bevat meestal ook de naam van de publi...

Ter info :
1) Alles in één testklasse is oké; maar je kan ook meerdere testklasses maken, met naam .TestLosseComponentInBulk.cls en .TestBOG.cls
2) Testklasse bevat meestal ook de naam van de public method die je aan 't testen bent. Maar misschien werd de naam dan te lang.
Je hoeft dit hier niet te veranderen, maar is taak van reviewer om dit op te merken

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk() Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan d...

Deze method doet exact hetzelfde als ..BestelEenheidLosseComponentenInBulkBepaler.GeefBestelEenheidZRFLosseComponentenInBulk()
Uiteraard klopt dat ook qua resultaat, maar waarom staat de code dan dubbel? --> dubbel onderhoud.
Misschien kan je hier (BOG) de implementatie van LosCompInBulkk laten oproepen.

[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket...
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket...
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket ZRF:

- Nieuwe testbestanden => vorige waren verkeerd door nieuwe data te nemen => verkeerde producten die via codegeneratie geselecteerd waren

    • -20
    • +20
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 7 more files in changeset.
[UST4192] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Default geen ZijkantLogo in korttekst:

- Als ZijkantLogoId leeg of 0 is => XX teruggeven => betekent geen ZijkantLogo in Korttekst => ook op hoger niveau Tommy => Testbestanden

    • -20
    • +20
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 6 more files in changeset.
Done!

Done!

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Die worden allebei in de caller van deze method al uitgesloten

Die worden allebei in de caller van deze method al uitgesloten

object LadeOnderdeel doorgeven, i.p.v. Rol As enu dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

object LadeOnderdeel doorgeven, i.p.v. Rol As enu
dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux". Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de kla...

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux".
Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de klant meegeeft.

Laat voorlopig maar staan, maar hou er wel rekening mee, dat dit soort defaultings "gedurfd" is

De bijhorende UT geeft aan dat *ID = "" geeft "" *ID = 0 geeft "" Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten? Want voor deze waarden is het eigenlijk zinloos ...

De bijhorende UT geeft aan dat

  • ID = "" geeft ""
  • ID = 0 geeft ""
    Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten?
    Want voor deze waarden is het eigenlijk zinloos om de bepaler op te roepen.
Deze lijn zou ik binnen de Else zetten. Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

Deze lijn zou ik binnen de Else zetten.
Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten:...
[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten:...
[UST4166] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: MinimumBestelhoeveelheid:

- Losse componenten => BestelEenheid en Korttekst en Langtekst => gewijzigde integratietestbestanden

    • -20
    • +20
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 12 more files in changeset.
[BUS654] - Sycomore enkel voor SFS

- Lossecomponent - andere volgorde

    • -20
    • +20
    ./External_SQL.DB_ZijkantKamerFreesData.xml
  1. … 14 more files in changeset.
1. Blijkbaar had ik niet de meest recente data om het te detecteren 2. Ik was nog van plan om de setting op te kuisen , maar het moest rap rap in productie wegens LDS bij SFS.

1. Blijkbaar had ik niet de meest recente data om het te detecteren
2. Ik was nog van plan om de setting op te kuisen , maar het moest rap rap in productie wegens LDS bij SFS.

  • More
  • CR-464
  • finished reviewing