Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
deploy class-ke toegevoegd.

deploy class-ke toegevoegd.

mag sleutel APPS.OV.impl.SFSOrderBevestiging_GebruikSSL al ineens verwijderd worden?

mag sleutel APPS.OV.impl.SFSOrderBevestiging_GebruikSSL al ineens verwijderd worden?

$$$Not(x="") -> $$$HasLengthhttp://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/error.gif

$$$Not(x="") -> $$$HasLength

[UST4163] VH4: MP: Daro: SSL HttpPost (Integratie webshop-vrijgave in handel-ERP)
[UST4163] VH4: MP: Daro: SSL HttpPost (Integratie webshop-vrijgave in handel-ERP)
Ik ben het eigenlijk niet eens met de default-waarde True.

Ik ben het eigenlijk niet eens met de default-waarde True.

[UW732] Refactor HttpPost code voor MP's
[UW732] Refactor HttpPost code voor MP's
Dat klopt, maar door te newen moet er extra geinject worden. De constructor van die poster is niet volledig kosjer. Daarom de logica van de andere tests aangehouden.

Dat klopt, maar door te newen moet er extra geinject worden. De constructor van die poster is niet volledig kosjer. Daarom de logica van de andere tests aangehouden.

Ik zou de mock durven newen (en meegeven in de constructor want die voorziet dat al) ipv te mock-instancen. Hoe minder testen rekenen op een mock op een context, hoe beter

Ik zou de mock durven newen (en meegeven in de constructor want die voorziet dat al) ipv te mock-instancen. Hoe minder testen rekenen op een mock op een context, hoe beter

#dimmen én type definiëren

#dimmen én type definiëren

Ik zou geneigd zijn om een enumeratie aan te maken

Ik zou geneigd zijn om een enumeratie aan te maken

Het aanmaken van de SSL-configuratie zelf die hier gebruikt wordt, moet manueel op elke omgeving gebeuren. Daar ga je niets van terugvinden in de commits/review

Het aanmaken van de SSL-configuratie zelf die hier gebruikt wordt, moet manueel op elke omgeving gebeuren. Daar ga je niets van terugvinden in de commits/review

[BUS720]MP: SFS: andere server: impact op "Integratie webshop-vrijgave in handel-ERP"
[BUS720]MP: SFS: andere server: impact op "Integratie webshop-vrijgave in handel-ERP"
Tja, snap uw bedenking wel. Tis hier nogal een speciaal geval. Ik wil echt een aparte log van deze melding met de juiste "Groep" en "Actie". Eigenlijk is dat omdat ik onze logging niet handig vind,...

Tja, snap uw bedenking wel.
Tis hier nogal een speciaal geval. Ik wil echt een aparte log van deze melding met de juiste "Groep" en "Actie". Eigenlijk is dat omdat ik onze logging niet handig vind, vandaar :-P

Er is consensus dat de "open accolade" meteen achter de If of Else moet staan, dus op dezelfde lijn. Er is volgens mij geen consensus of de Else/ElseIf op dezelfde lijn of op de volgende lijn moet...

Er is consensus dat de "open accolade" meteen achter de If of Else moet staan, dus op dezelfde lijn.

Er is volgens mij geen consensus of de Else/ElseIf op dezelfde lijn of op de volgende lijn moet staan.
Mijn voorkeur is op de volgende lijn, maar ik heb er geen problemen mee dat anderen de Else op dezelfde lijn zetten, i.e. achter de "}".
Idem Try-Catch

Voor mij is dit voldoende, heb geen nood aan geschreven consensus hiervoor

Na kleine rondvraag: er is een zekere consensus binnen vhict over de schrijfwijze van if else en try catch structuren nl.      if () { ... } else { ... } try { ... } catch { ... }     ...

Na kleine rondvraag: er is een zekere consensus binnen vhict over de schrijfwijze van if else en try catch structuren nl.

    
if () {
   ...
} else {
   ...
}

try {
   ...
} catch {
   ...
}
           
manueel ipv mauneel http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

manueel ipv mauneel

Kun je in dit geval de try ... catch niet gewoon weghalen?

Kun je in dit geval de try ... catch niet gewoon weghalen?

[VRB460] EDI: SFS: Integratie vrijgave in handel-ERP: Error met HTTP POST: kunnen hernemen, en verwittigd...
[VRB460] EDI: SFS: Integratie vrijgave in handel-ERP: Error met HTTP POST: kunnen hernemen, en verwittigd...
  • More
  • CR-275
  • finished reviewing
  • More
  • CR-275
  • finished reviewing
Zal waarschijnlijk ruiken naar feature envy -> dus lijkt mij beter om aparte klasse te maken / dus zal het onliner worden dus niet nodig om private method te maken

Zal waarschijnlijk ruiken naar feature envy -> dus lijkt mij beter om aparte klasse te maken / dus zal het onliner worden dus niet nodig om private method te maken