Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
OnAfterAllTests() toegevoegd.

OnAfterAllTests() toegevoegd.

puur theoretische kommaneukerij, maar stel dat een test crasht, dan komen we niet op die Kill dacht ik. Beter om ook nog eens te killen in een OnAfterAllTests. En ik weet het, ik zie in het gebrui...

puur theoretische kommaneukerij, maar stel dat een test crasht, dan komen we niet op die Kill dacht ik.
Beter om ook nog eens te killen in een OnAfterAllTests.

En ik weet het, ik zie in het gebruik van die %-variabele in de andere klassen geen rariteiten, maar ik wou gewoon iéts schrijven

@Tom : deze klasse is een vereenvoudigde kopie van "APPS.common.Document.impl.DocBase.WebServerUrlBepaler" voor FOP. Geen inheritance of uncle Bob-principes toegepast http://subversion02.vanhoecke....

@Tom : deze klasse is een vereenvoudigde kopie van "APPS.common.Document.impl.DocBase.WebServerUrlBepaler" voor FOP.
Geen inheritance of uncle Bob-principes toegepast

[ICT-5021] FOP: DocBase Upgrade - Set up Web-folder (IIS) op DEV/ACP/PRD + aanpassen caché-code
[ICT-5021] FOP: DocBase Upgrade - Set up Web-folder (IIS) op DEV/ACP/PRD + aanpassen caché-code
FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt http://sub...

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt )
De impl in deze afgeleide klasse is wel correct.

De implementatie voor "Lengte" hieronder zou ik wel in een private method steken, analoog aan GeefFrontPlaatHoogte()
Zo blijft deze method simpel leesbaar en overzichtelijk. En dan is de scope ook duidelijk, nl. dat Kleur W7 enkel impact heeft op de kenmerk "Lengte, en niet op de "Breedte" van het Front.

copy-paste foutje : "... kan niet gemaild worden." (ook al is de conditie altijd false :-P )

copy-paste foutje :
"... kan niet gemaild worden."
(ook al is de conditie altijd false :-P )

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2743][rvPVR] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder
[ICT-2743][rvPVR] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder
Zoals elders gezegd... geen waarden via parameters doorgeven in een constructor, dat moet je dan via een aparte public method doen! De ZetTekstLinks en ZetTekstMidden mogen wel blijven staan, omdat...

Zoals elders gezegd... geen waarden via parameters doorgeven in een constructor, dat moet je dan via een aparte public method doen!
De ZetTekstLinks en ZetTekstMidden mogen wel blijven staan, omdat het louter defaultering is.

Ik zie dat dit een copy/paste is van hierboven, maar dan toch even de casing aanpassen aub http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif For Set $Orde...

Ik zie dat dit een copy/paste is van hierboven, maar dan toch even de casing aanpassen aub
For
Set
$Order
Quit

En ook graag een leeg lijntje tussen de For-loop en de uiteindelijk Quit
Misschien gelijk ook de casing elders in deze klasse aanpassen, is niet zo veel.

Oeps! Dit is een beetje tegen de schenen schoppen van onze code-conventions http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif De constructor gebruiken we ...

Oeps! Dit is een beetje tegen de schenen schoppen van onze code-conventions
De constructor gebruiken we ENKEL voor het intantiëren van een object. De énige parameters die daar dus aan meegegeven mogen worden zijn degene die voor die intantiëring essentieel zijn, zoals dependencies.
Wat we zeker NIET doen in constructors, of toch zeker zoveel als mogelijk proberen te vermijden, is het doorgeven van waarden (als parameters) om properties te zetten. Dat maakt namelijk een knoeiboel van eventuele testen.
Wat je dus eigenlijk moet doen is die 2 parameters er weer uithalen en in de klasse waarvan je hier een object maakt een public method maken om het zetten van die properties te regelen.
Dat maakt dan wel dat je hier 2 coderegels gaat hebben: Eentje om een object te instantiëren en direct daarna een tweede om op die instantie die public method te callen met deze 2 parameters, bv:
Do VoetTekst.ZetPaginaNrEnTotaalAantalBladzijden(HuidigPaginaNr, TotaalAantalBladzijden)

[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots
[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots