Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
OEPS !!! Great call!!! Thx!!!

OEPS !!! Great call!!! Thx!!!

Het oude productId komt nog op 2 andere plaatsen voor in caché studio. Deze best ook aanpassen. Searching for 1118152 in *.cls,*.mac,*.inc APPS.PM.Maatwerk.MaatwerkModel.Modellen.TaorboxModel.Rolle...

Het oude productId komt nog op 2 andere plaatsen voor in caché studio. Deze best ook aanpassen.

Searching for 1118152 in *.cls,*.mac,*.inc
APPS.PM.Maatwerk.MaatwerkModel.Modellen.TaorboxModel.Rollen.ZijkantBevestigingVeer.cls(GeefRol+2): ##class(APPS.PM.Maatwerk.MaatwerkModel.dto.SpecifiekProductBuilderType).%New(1118152)
DOM.PM.Maatwerk.Syncs.DotNet.MaatwerkCalculator.impl.ProductInfo.cls(GeefLijstVanProductIdVanNietSelectieKenmerken+102): Do ProductIds.Insert(1118152) 
Found 2 occurrence/s in 2 file/s.
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:
ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs
ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs
Oeps! Moest dus zijn: ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs

Oeps! Moest dus zijn: ICT-2807 [rv WV] TAX - Tijdelijke meerkost voor de hoogteverstelling reeds in schaduwprijs

De subject van deze crucible review verwijst vreemd genoeg naar "TAX nieuwe kleur W7" :-P

De subject van deze crucible review verwijst vreemd genoeg naar "TAX nieuwe kleur W7" :-P

Oplossen in de kostItemBuilder door conditie Tijd > 0

Oplossen in de kostItemBuilder door conditie Tijd > 0

Zoals in je script : If Tijd > 0 then VoegToeVariabele Want nu staat er bij de integratietesten van Lossecomponenten telkens een onnodig "tijdkost = 0 item"

Zoals in je script :
If Tijd > 0 then VoegToeVariabele

Want nu staat er bij de integratietesten van Lossecomponenten telkens een onnodig "tijdkost = 0 item"

Quick-win : performantie verhoogt door IngegevenKenmerken.KoppelingType = ##class(DOM.PM.enu.TAORKoppelingType).Movento() eerst te zetten in de conditie, en pas in tweede instantie te testen of Ro...

Quick-win : performantie verhoogt door

IngegevenKenmerken.KoppelingType = ##class(DOM.PM.enu.TAORKoppelingType).Movento()

eerst te zetten in de conditie, en pas in tweede instantie te testen of RolAanwezig is.

Mag uiteraard wel op 1 lijn blijven staat.

[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
Bouwsteen mapping : je mocht gerust de 8 tekens volledig benutten : bijvoorbeeld : VPFROSPC, VPFROINS, VPFROVDP, ...

Bouwsteen mapping : je mocht gerust de 8 tekens volledig benutten :
bijvoorbeeld : VPFROSPC, VPFROINS, VPFROVDP, ...

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.

[UST4202] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Aanpassingen...
[UST4202] TAX-LC-002: Nieuwe verpakking (betere palletstapeling) - deel 1: calculatie kosten: Aanpassingen...
[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...
Done! UST4187

Done! UST4187

SCA: Momenteel geen aandacht besteden aan pallet-inhoud!

SCA: Momenteel geen aandacht besteden aan pallet-inhoud!

Kaartje UST4201 aangemaakt

Kaartje UST4201 aangemaakt