Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
testjes toevoegen aan vhUnitTest.APPS.Halux.PPS.*.TestDoos ? of hier niet relevant (heb niet in detail gekeken naar die files, maar dat zijn wel de files die regelmatig meegecommit werden in het ve...

testjes toevoegen aan vhUnitTest.APPS.Halux.PPS.*.TestDoos ? of hier niet relevant (heb niet in detail gekeken naar die files, maar dat zijn wel de files die regelmatig meegecommit werden in het verleden als er aan de DoosBerekenaar iets wijzigde)

[ICT-3061] [rvTVE] TAX - te grote V1-doos voor smalle F-lade
[ICT-3061] [rvTVE] TAX - te grote V1-doos voor smalle F-lade
Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() ) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builde...

Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() )
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

Deze method komt vele malen voor in de Builders (idem voor =Servodrive) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd ...

Deze method komt vele malen voor in de Builders (idem voor =Servodrive)
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt. Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en B...

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt.
Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en BuilderBasis vallen samen.
Maar wat mij betreft hoeft dit niet weggewerkt te worden in caché. Misschien wel bij het porten naar .NET

FYI: het was uiteindelijk de dubbele definitie van de method GeefGekoppeldIKType() die voor enige verwarring zorgde. Vandaar de review opm.

[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep
Ik heb ze er zelf onlangs opgezet en ze zijn nooit in gebruik genomen, dus er is geen risico hier http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Ik heb ze er zelf onlangs opgezet en ze zijn nooit in gebruik genomen, dus er is geen risico hier

Oppassen met opkuisen van Storage !!! In bestaande data zullen items 4, 5 en 6 nog steeds ingevuld zijn. Wanneer je later een nieuwe property toevoegd, zal die onterecht verwijzen naar item 4. In ...

Oppassen met opkuisen van Storage !!!
In bestaande data zullen items 4, 5 en 6 nog steeds ingevuld zijn.
Wanneer je later een nieuwe property toevoegd, zal die onterecht verwijzen naar item 4.

In deze klasse zal het mogelijk geen gevaar vormen, maar let er wel op, in de toekomst. Bij twijfel, kom maar vragen.

[ICT831] TAXFP1: Afwerking:
[ICT831] TAXFP1: Afwerking:
Zelfde opmerkingen vorige method http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Zelfde opmerkingen vorige method

GeefDoosLade() beter in een variabele steken, want nu wordt die 7x herhaald/berekend in één testmethod.

GeefDoosLade() beter in een variabele steken, want nu wordt die 7x herhaald/berekend in één testmethod.

F-lade --> dat mag je gerust vermelden in de methodnaam; bvb. "Test: Hoge ondiepe niet-binnenlade verpakt in ... " of "Test: niet-binnenlade F-hoogte verpakt in ... "

F-lade --> dat mag je gerust vermelden in de methodnaam;
bvb. "Test: Hoge ondiepe niet-binnenlade verpakt in ... " of "Test: niet-binnenlade F-hoogte verpakt in ... "

Hier in de UT zou ik de eerder "VerwachteAfmetingK.." noemen i.p.v. DefaultAfm... Je gaat deze immers asserten. Ik denk zelfs dat "K1=100" geen default waarde is, maar effectief een berekende waar...

Hier in de UT zou ik de eerder "VerwachteAfmetingK.." noemen i.p.v. DefaultAfm...
Je gaat deze immers asserten.

Ik denk zelfs dat "K1=100" geen default waarde is, maar effectief een berekende waarden.

Beter: GeefIngegevenKenmerken() kan je direct aan MaatwerkProduct vragen.

Beter: GeefIngegevenKenmerken() kan je direct aan MaatwerkProduct vragen.

[UST4239] TAXFP1: Productie-aansturing - Kartonmachine:
[UST4239] TAXFP1: Productie-aansturing - Kartonmachine:
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...