Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5609] Parametriseren Images path in cache code

- code met expliciete verwijzing naar \\notes01\images aangepast met configItem, met default fallback op Notes01 indien niet gezet.

  1. … 22 more files in changeset.
[ICT-5051][rvNVT] FOP: DocBase Upgrade - Clean up caché-code na de go-live

- Opkuis klasse APPS.common.Document.impl.DocBase.WebServerUrlBepaler

- De enige gebruiker (in method GenereerProductDetailBlad() ) is ook al aangepast

    • -26
    • +0
    ./DocBase/WebServerUrlBepaler.cls.xml
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
[ICT-5021] FOP: DocBase Upgrade - Set up Web-folder (IIS) op DEV/ACP/PRD + aanpassen caché-code

- bij "PRODUCTIEDETAILBLAD;FileDoc" voor FileToWWW (a.k.a. InetPub) nieuwe klasse WebServerUrlDocBaseV7Bepaler gemaakt, omdat locatie van inetpub en Url verschillend zijn dan op de FOP-server.

    • -0
    • +70
    ./DocBase/WebServerUrlDocBaseV7Bepaler.cls.xml
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Reviewaanpassingen => Copy/paste foutje en kleine refactoring => afzondering naar private method en casing

  1. … 1 more file in changeset.
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
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Fix bij mailen ProductieDetailblad => Naam en extensie van bijlage zijn nu correct

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Toevoeging van en uitbreiding met PDFDocumentMailer

    • -0
    • +52
    ./DocBase/DocumentVerwerker/PDFMailer.cls.xml
    • -0
    • +107
    ./PDFDocumentMailer.cls.xml
  1. … 4 more files in changeset.
[ICT][ICT-2743] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder

- Code cleanup : $$$Implementation() werkt niet op String argument : vervangen door $$$If($$$HasLength(..))

[ICT][ICT-2743] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder

- Code cleanup : ReturnType correct ingesteld : private method GeefDocumentNaarDocBaseDocumentConverter()

[ICT-2743][rvPVR] PM: Maatwerk: ALU Kaderdeur detailblad PDF genereren via FopScheduler+Watchfolder

- WebServerUrlBepaler (voor FOP) methods toegevoegd voor NetwerkLocatieViaSharedMap()

- !!! LET OP: de shared map is alleen aanwezig op Fop !

- Mock en Fake aangepast

- Unittest uitgebreid

    • -0
    • +26
    ./DocBase/WebServerUrlBepaler.cls.xml
  1. … 3 more files in changeset.
[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots

- Reviewopmerkingen behandelen

  1. … 2 more files in changeset.
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