Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3246] [rvWV] PM: Maatwerk: TAX: BOMBOL: Vaste producten BOM aanpassen maar prijs behouden:

- Verduidelijkingske bij iets wat niets met deze story te maken heeft => Wordt nooit gebruikt ivm LadeOnderdeel Bodem, dus dat mag ook blijken uit de naamgeving

  1. … 3 more files in changeset.
Paul had dat indertijd ook al voorgesteld, maar ik heb hem gezegd dat ik daar niet aan ga beginnen. Stille waters, diepe gronden enzo http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/w...

Paul had dat indertijd ook al voorgesteld, maar ik heb hem gezegd dat ik daar niet aan ga beginnen. Stille waters, diepe gronden enzo
En aangezien we in de nieuwe vorm zelfs vijfletterige combinaties gaan hebben, zou de lijst vrij lang kunnen worden.

Laten afleiden van TECH.VhChangeInfo Zie Class DOM.VKP.impl.Korting.Korting Extends TECH.VhChangeInfo Dan kan de .Horolog property (+storage) weggelaten worden. OPGELET : na deze wijziging heb j...

Laten afleiden van TECH.VhChangeInfo

Zie Class DOM.VKP.impl.Korting.Korting Extends TECH.VhChangeInfo

Dan kan de .Horolog property (+storage) weggelaten worden.

OPGELET : na deze wijziging heb je misschien ne factory nodig, om uwe UT werkend te houden :-P

Else do $Increment(AantalMetOngewensteWoorden) en dit aantal kan je op het einde rapporteren als output van deze "powertool" bvb. "CodeGenerator heeft "_(Aantal - AantalMetOngewensteWoorden)_" co...

Else do $Increment(AantalMetOngewensteWoorden)

en dit aantal kan je op het einde rapporteren als output van deze "powertool"

 bvb. "CodeGenerator heeft "_(Aantal - AantalMetOngewensteWoorden)_" codes gegenereerd en "_AantalMetOngewensteWoorden_" codes geskipped."  
Kan je deze lijst vervolledigen aub? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Kan je deze lijst vervolledigen aub?

C:\Temp\ --> Opgepast met hard-coded file path. Beter gebruik maken van onze Config-Items --> E:\CacheExport\...

C:\Temp\ --> Opgepast met hard-coded file path.
Beter gebruik maken van onze Config-Items --> E:\CacheExport\...

om consistent te zijn met de andere parameters, hier ook ..Calculator.CodeNr toekennen.

om consistent te zijn met de andere parameters, hier ook ..Calculator.CodeNr toekennen.

Bij Inject() geen context.Instance ... , maar rechtstreeks newen van de implementatie

Bij Inject() geen context.Instance ... , maar rechtstreeks newen van de implementatie

[ICT-848] Nieuwe Orgalux barcode
[ICT-848] Nieuwe Orgalux barcode
[ICT-848] Nieuwe Orgalux barcode

- OrgaluxLadecodeRepository uitgemockt.

    • -0
    • +71
    ./LadeRegistratie/impl/OrgaluxLadecodeRepository.cls.xml
  1. … 4 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:

- De DoosAfmetingenBepalerFactory geeft de bepaler

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

- TestCase voor BestelEenheidBepaler

  1. … 3 more files in changeset.
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket ZRF:

- BestelEenheidLosseComponentenInBulkBepaler => Move class naar 1 niveau dieper

    • -34
    • +0
    ./impl/BestelService/BestelEenheidLosseComponentenInBulkBepaler.cls.xml
  1. … 6 more files in changeset.
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket ZRF:

- BestelEenheidBulkPerOnderdeelGegroepeerdBepaler => Move class naar 1 niveau dieper

  1. ./impl/BestelService/BestelEenheidBepaler
    • -20
    • +0
    ./impl/BestelService/BestelEenheidBulkPerOnderdeelGegroepeerdBepaler.cls.xml
  2. … 7 more files in changeset.
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket ZRF:

- Move class naar 1 niveau dieper

    • -34
    • +0
    ./impl/BestelEenheidBepaler.cls.xml
  1. … 6 more files in changeset.
[UST4187] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: KlantEtiket ZRF:

- Grondige refactor BestelService => Ts BestelService en haar dependencies die BestelEenheid teruggaven zit nu een BestelEenheidBepaler die BestelEenheid teruggeeft volgens LadeOnderdeel.

Dit was nodig opdat de builders die tussenlaag direct kunnen aanspreken en het van daaruit belangrijk is volgens LadeOnderdeel de BestelEenheid te krijgen en niet volgens Kenmerken,

want igv BulkPerOnderdeelGegroepeerd zijn de Kenmerken niet specifiek genoeg om te weten over welk LadeOnderdeel het gaat.

    • -4
    • +11
    ./impl/BestelService/BestelEenheidLosseComponentenInBulkBepaler.cls.xml
    • -0
    • +34
    ./impl/BestelEenheidBepaler.cls.xml
  1. … 7 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.