Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-179
  • finished reviewing
Het is te hopen dat de andere code dat ook doet !? Gelieve hiervoor een betere omschrijving te bedenken. Een korte samenvatting van het wat gedrag is m.a.w. en niet hoe maar wat er juist gedaan wo...

Het is te hopen dat de andere code dat ook doet !?

Gelieve hiervoor een betere omschrijving te bedenken. Een korte samenvatting van het wat gedrag is m.a.w. en niet hoe maar wat er juist gedaan wordt.

Beter een $$$Not(BevatFout) zodoende 0 en "" hetzelfde terug geven.

Beter een $$$Not(BevatFout) zodoende 0 en "" hetzelfde terug geven.

  • More
  • CR-162
  • finished reviewing
Response uitbouwen op het einde van de code.. Niet we doen iets .. we bouwen den Response op en we doen nog iets en we quiten de response. Beter we doen iets , we doen nog iets , we bouwen de res...

Response uitbouwen op het einde van de code..

Niet we doen iets .. we bouwen den Response op en we doen nog iets en we quiten de response.

Beter we doen iets , we doen nog iets , we bouwen de response op en we quiten die

$$$If structuur gebruiken..

$$$If structuur gebruiken..

[UST2241]VHIP_GLS: RegisterContainer implementeren
[UST2241]VHIP_GLS: RegisterContainer implementeren
De parcel en unit id zijn gebaseerd op zelfde veld uit de verzend data. ze moeten dus de zelfde waarde hebben... idem voor de t&t url

De parcel en unit id zijn gebaseerd op zelfde veld uit de verzend data. ze moeten dus de zelfde waarde hebben...
idem voor de t&t url

  • More
  • CR-163
  • finished reviewing
Assert methods mogen geprivatiseerd worden

Assert methods mogen geprivatiseerd worden

Ik zou toch opteren voor andere data zodoende er een verschil tussen de GLSParcelNumber als de GLSUnitNumber zodat we zeker zijn we het juiste veld overgekopieerd hebben. Idem dito voor de TrackAnd...

Ik zou toch opteren voor andere data zodoende er een verschil tussen de GLSParcelNumber als de GLSUnitNumber zodat we zeker zijn we het juiste veld overgekopieerd hebben.
Idem dito voor de TrackAndTraceUrlVoorKlant en dienen voor VH

macro $$$false gebruiken ipv 0

macro $$$false gebruiken ipv 0

Gelieve de bewuste id in de foutmelding op te nemen en indien mogelijk ook welke type bericht het dan wel was, kwestie om zo duidelijk mogelijke foutmelding te kunnen geven. ps : Enkel eigennamen o...

Gelieve de bewuste id in de foutmelding op te nemen en indien mogelijk ook welke type bericht het dan wel was, kwestie om zo duidelijk mogelijke foutmelding te kunnen geven.
ps : Enkel eigennamen of het begin van een zin begint met een hoofdletter

White space http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

White space

GeefVerzendData zegt weinig over de wat voor verzend data het is ? Als het echt niet uitmaakt gelieve dan een ##class(vhTest.Dummy) te gebruiken als het wel van belang is gelieve dan een iets duide...

GeefVerzendData zegt weinig over de wat voor verzend data het is ? Als het echt niet uitmaakt gelieve dan een ##class(vhTest.Dummy) te gebruiken als het wel van belang is gelieve dan een iets duidelijkere benaming te kiezen.

Utils mogen classmethods zijn

Utils mogen classmethods zijn

Gelieve het gedrag en het gewenste resultaat iets duidelijk in de test te vermelden zodoende er kan gecontroleerd worden dat de test setup wel is volgens het vooropgestelde gedrag. De volgende stap...

Gelieve het gedrag en het gewenste resultaat iets duidelijk in de test te vermelden zodoende er kan gecontroleerd worden dat de test setup wel is volgens het vooropgestelde gedrag. De volgende stap is dat dat de code voldoet aan het vooropgesteld gedrag van de test.

Lijkt mij beter om VerzendData uit te breiden met BevatFout en Foutmelding zodoende er maar één object moet doorgegeven worden. Idem dito voor de CancelContainer..

Lijkt mij beter om VerzendData uit te breiden met BevatFout en Foutmelding zodoende er maar één object moet doorgegeven worden. Idem dito voor de CancelContainer..

?? mag deze method niet weg daar de response controleur kan gebruikt worden ??

?? mag deze method niet weg daar de response controleur kan gebruikt worden ??

Er zal wel ergens vhTest.Utils.APPS.TRANSP.dto.LeveringAdres).EenBelg staan vermoed ik .. en als het nog niet zo is .. maak ze maar http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wik...

Er zal wel ergens vhTest.Utils.APPS.TRANSP.dto.LeveringAdres).EenBelg staan vermoed ik .. en als het nog niet zo is .. maak ze maar

??? waarom deze specifieke en niet de algemene ???

??? waarom deze specifieke en niet de algemene ???

De ene keer is het Result en de andere keer is het Response

De ene keer is het Result en de andere keer is het Response

Kan dit niet ergens veralgemeend worden ?? Lijkt me raar dat dit hier moet staan. DRY

Kan dit niet ergens veralgemeend worden ?? Lijkt me raar dat dit hier moet staan. DRY

onelineren ofwel het zetten van LeveringInfo dichter bij deze lijn plaatsen. Niet iets opzetten , ietske anders doen en dan het opgezette gebruiken

onelineren ofwel het zetten van LeveringInfo dichter bij deze lijn plaatsen.

Niet iets opzetten , ietske anders doen en dan het opgezette gebruiken

#dim mag onliner worden met set die er onderstaat

#dim mag onliner worden met set die er onderstaat

Gelieve de #dim's zoveel mogelijk bij code te plaatsen waar ze effectief gebruikt wordt. Refactored iets makkelijker

Gelieve de #dim's zoveel mogelijk bij code te plaatsen waar ze effectief gebruikt wordt.

Refactored iets makkelijker