Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[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
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
  • 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

[UST2674] Vh4_SVC: IsAsapMogelijk moet met verzendwijze van gekozen adres rekening houden
[UST2674] Vh4_SVC: IsAsapMogelijk moet met verzendwijze van gekozen adres rekening houden
[ICT288] VHisie4 : Bereidvoorbestelling proper maken na snelle implementatie voor adresid
[ICT288] VHisie4 : Bereidvoorbestelling proper maken na snelle implementatie voor adresid