Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5051][rvNVT] FOP: DocBase Upgrade - Clean up caché-code na de go-live

- Opkuis Mock en Fake van klasse APPS.common.Document.impl.DocBase.WebServerUrlBepaler

- De klasse zelf zal ook verwijderd worden. De enige gebruiker (in method GenereerProductDetailBlad() ) is ook al aangepast

  1. … 1 more file in changeset.
[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-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

  1. … 3 more files in changeset.
yep

yep

om niet-nul mee te geven, en die waarde is ok voor de productiedata

om niet-nul mee te geven, en die waarde is ok voor de productiedata

Geef is voor repo's

Geef is voor repo's

Niet echt, een Geef zou enkel op een repository (desnoods een service) moeten zitten als je zeker bent dat er iets wordt teruggegeven.

Niet echt, een Geef zou enkel op een repository (desnoods een service) moeten zitten als je zeker bent dat er iets wordt teruggegeven.

Enkel voor HAEFELE NEDERLAND?

Enkel voor HAEFELE NEDERLAND?

Zal wel een reden hebben, maar voor alle zekerheid... Waarom 19941?

Zal wel een reden hebben, maar voor alle zekerheid... Waarom 19941?

Is 't nie properder om het te doen zoals je in de code zelf gedaan hebt? Like this: #dim Foldernaam As %String = ##class(TECH.Files).AppendSubPath(##class(TECH.Config.ConfigMgr).Instance().GeefStri...

Is 't nie properder om het te doen zoals je in de code zelf gedaan hebt? Like this:
#dim Foldernaam As %String = ##class(TECH.Files).AppendSubPath(##class(TECH.Config.ConfigMgr).Instance().GeefString("MSMGATEWAY"), "Nacalculatie")
#dim Bestandsnaam As %String = ##class(TECH.Files).AppendFile(Foldernaam, "Nacalculatie.csv")
P.s.: In andere UT hier ook.

BeginDatum en EindDatum maken de signature van 'deze' Fake anders dan in de implementatieklasse.

BeginDatum en EindDatum maken de signature van 'deze' Fake anders dan in de implementatieklasse.

Moet het niet GeefTransportData ipv BepaalTransportData zijn? Geeft tenslotte een object met die data terug. Maar hoe doet ge dat dan met die klassenaam hé? http://subversion02.vanhoecke.be/static/...

Moet het niet GeefTransportData ipv BepaalTransportData zijn? Geeft tenslotte een object met die data terug.
Maar hoe doet ge dat dan met die klassenaam hé?

Mag in 1 lijn, want BepaalTransportData geeft een object van dat type terug. Bovendien (zie ook comment in BepaalTransportData() in APPS.TRANSP.impl.TransportService.Nacalculator.TransportdataVoorK...

Mag in 1 lijn, want BepaalTransportData geeft een object van dat type terug.
Bovendien (zie ook comment in BepaalTransportData() in APPS.TRANSP.impl.TransportService.Nacalculator.TransportdataVoorKlantBepaler): Moet het dan niet GeefTransportData ipv BepaalTransportData zijn?

Na ons overleg hebben we beslist dat het plusje hier weg mag (was inconsistent met HuidigeLaatstVerwerkteRit 3 regels verder en LaatstVerwerkteRit in BepaalTransportData van klasse APPS.TRANSP.impl...

Na ons overleg hebben we beslist dat het plusje hier weg mag (was inconsistent met HuidigeLaatstVerwerkteRit 3 regels verder en LaatstVerwerkteRit in BepaalTransportData van klasse APPS.TRANSP.impl.TransportService.Nacalculator.TransportdataVoorKlantBepaler

BeginDatum en EindDatum maken de signature in de interface (en 'deze' Mock) anders dan in de implementatieklasse, maar komt wel goed als ge de interface rechttrekt hé http://subversion02.vanhoecke....

BeginDatum en EindDatum maken de signature in de interface (en 'deze' Mock) anders dan in de implementatieklasse, maar komt wel goed als ge de interface rechttrekt hé

BeginDatum en EindDatum maken de signature in deze interface anders dan in de implementatieklasse.

BeginDatum en EindDatum maken de signature in deze interface anders dan in de implementatieklasse.

nog iets: 't is nogal uiteenlopende code die aangepast werd, dus het kan misschien geen kwaad om ook in svn de log er bij te nemen om wat meer uitleg te krijgen over wat er juist gewijzigd werd

nog iets: 't is nogal uiteenlopende code die aangepast werd, dus het kan misschien geen kwaad om ook in svn de log er bij te nemen om wat meer uitleg te krijgen over wat er juist gewijzigd werd

[UST3808] HNLT: ana+design: nacalculatie door te rekenen transportkost
[UST3808] HNLT: ana+design: nacalculatie door te rekenen transportkost
[UST3808] HNLT: ana+design: nacalculatie door te rekenen transportkost

- BestandGeneratorTemplate mockable gemaakt

    • -0
    • +19
    ./Rapport/impl/BestandGeneratorTemplate.cls.xml
  1. … 2 more files in changeset.
  • More
  • CR-404
  • finished reviewing
Is het in dit geval niet meer leesbaar als je vergelijkt met "" ipv parameter GeenLeverAdresId? Deze parameter wordt slechts 1x gebruikt.

Is het in dit geval niet meer leesbaar als je vergelijkt met "" ipv parameter GeenLeverAdresId? Deze parameter wordt slechts 1x gebruikt.

  • More
  • CR-395
  • finished reviewing
Waarom werk je hier met een fake Winkelkar, een mock of een nieuwe lege instantie van de Winkelkar zou toch ook moeten werken?

Waarom werk je hier met een fake Winkelkar, een mock of een nieuwe lege instantie van de Winkelkar zou toch ook moeten werken?

Initialiseren met "" lijkt mij hier overbodig

Initialiseren met "" lijkt mij hier overbodig