VerzendWijzeBepaler.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Dit is nu echt het minst van mijn zorgen. Beter de juiste klasse namen , variabele namen dan te moeten focussen om iets wat eigenlijk niet belangrijk qua duidelijkheid

Dit is nu echt het minst van mijn zorgen. Beter de juiste klasse namen , variabele namen dan te moeten focussen om iets wat eigenlijk niet belangrijk qua duidelijkheid

Duuh , hier staat geen logica op die lijn é . Het is het samenstellen van een exceptie test , dat is nu echt niet zo belangrijk om te moeten in lezen..

Duuh , hier staat geen logica op die lijn é . Het is het samenstellen van een exceptie test , dat is nu echt niet zo belangrijk om te moeten in lezen..

Nu niet overdrijven ook in de review opmerkingen é http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif het is begot een method van 10 lijnen..

Nu niet overdrijven ook in de review opmerkingen é het is begot een method van 10 lijnen..

  • More
  • CR-259
  • finished reviewing
boyscout principle : best deze method wat herwerken en korter maken, opsplitsen in kleinere methods.

boyscout principle : best deze method wat herwerken en korter maken, opsplitsen in kleinere methods.

Geen test gevonden voor wijzigingen.

Geen test gevonden voor wijzigingen.

casing -> Do

casing -> Do

Lijn code is 217 chars lang. Best inkorten zodat het volledig leesbaar blijft. Eventueel een variabele voorzien voor exceptieOmschrijving op te bouwen.

Lijn code is 217 chars lang. Best inkorten zodat het volledig leesbaar blijft. Eventueel een variabele voorzien voor exceptieOmschrijving op te bouwen.

[DEF590] - <PROPERTY DOES NOT EXIST>zGeefVerzendWijze+10
[DEF590] - <PROPERTY DOES NOT EXIST>zGeefVerzendWijze+10
[DEF590] - <PROPERTY DOES NOT EXIST>zGeefVerzendWijze+10
  • More
  • CR-161
  • finished reviewing
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.

Foutmelding zal hoogstwaarschijnlijk is niet mogelijk moeten zijn. Foutmelding dan ook beetje uniformiseren en de waarde tussen ' plaatsen

Foutmelding zal hoogstwaarschijnlijk is niet mogelijk moeten zijn. Foutmelding dan ook beetje uniformiseren en de waarde tussen ' plaatsen

[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)
[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)
[UST2197] - GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)

- GLS specifieke code verplaatsen naar submap

  1. … 3 more files in changeset.