Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • 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
  • More
  • CR-161
  • finished reviewing
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

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

Beetje opletten op de volgorde van methods. Constructor Publieke Private

Beetje opletten op de volgorde van methods.

Constructor
Publieke
Private

Alvorens een eerste keer in productie te brengen er voor zorgen dat de storage ok is maw storage verwijderen en opnieuw toevoegen daar er anders items in blijven zitten die noot in gevuld raken de ...

Alvorens een eerste keer in productie te brengen er voor zorgen dat de storage ok is maw storage verwijderen en opnieuw toevoegen daar er anders items in blijven zitten die noot in gevuld raken de gevolgen van renames etc.

Het is eerder #dim As ipv #Dim as ( laten we dit omschrijven als een slechte gewoonte )

Het is eerder #dim As ipv #Dim as ( laten we dit omschrijven als een slechte gewoonte )

Verzekeren dat er nergens data zit die niet verwijst naar niet GLSv1 als dat zo is deploy klasse maken om die data te corrigeren.

Verzekeren dat er nergens data zit die niet verwijst naar niet GLSv1 als dat zo is deploy klasse maken om die data te corrigeren.

Lijkt mij beter om deze klasse om te vormen naar een vhTest.Utils klasse .. kwestie van een beetje conformiteit in de code base te krijgen

Lijkt mij beter om deze klasse om te vormen naar een vhTest.Utils klasse .. kwestie van een beetje conformiteit in de code base te krijgen

AX testen zou ik niet samen steken met andere testen.. De zaken duidelijk van elkaar gescheiden houden lijkt mij wel handig in dit concept zodoende we makkelijk de zaken kunnen opkuisen eens de zak...

AX testen zou ik niet samen steken met andere testen.. De zaken duidelijk van elkaar gescheiden houden lijkt mij wel handig in dit concept zodoende we makkelijk de zaken kunnen opkuisen eens de zaken effectief in AX zitten.

auwch

auwch

Factory is niet testbaar daar er een classmethod gebruikt wordt . . -> depedency ook naar boven brengen er nu een andere klasse gebruikt wordt om dto klassen te maken.

Factory is niet testbaar daar er een classmethod gebruikt wordt . . -> depedency ook naar boven brengen er nu een andere klasse gebruikt wordt om dto klassen te maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.