CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST2332] Query aangepast op 2 punten:

1. In drie velden moest een kleine letter vervangen worden door een hoofdletter.

2. Een extra filter toegevoegd op een reeks ID-nummers, om een reeds overzichtelijke reeks producten in de .csv te krijgen. Deze filter wordt later verwijderd opdat de volledige lijst bekomen zou worden.

[UST2319] 1. Nieuwe property: OmschrijvingSamensteller", nodig omdat de method SamenstellingOmschrijving is geschrapt en de functionaliteit ervan nu is overgenomen door de klasse AX.Uitgaand.PM.impl.SamenstellingOmschrijvingProductenInEenString, waarvoor ook een UnitTest bestaat.

2. De method Convert krijgt nu geen ResultSet meer binnen als parameter, maar een Product van het type AX.Uitgaand.PM.Product.impl.Product (waarin pointers zitten naar Prod.Product, Prod.ProductTekst, Prod.ProductKlas en Prod.ProductPrijs). Dit geeft meer en doorzichtigere mogelijkheden voor UnitTesting.

3. Omwille van voorgaande een aantal Set-instructies aangepast.

4. Drie set-instructies gewist, omdat deze velden en bijhorende properties ook gewist zijn wegens niet gebruikt.

5. De in punt vernoemde method SamenstellingOmschrijving is geschrapt en de functionaliteit ervan is nu overgenomen door de klasse AX.Uitgaand.PM.impl.SamenstellingOmschrijvingProductenInEenString, waarvoor ook een UnitTest bestaat.

[UST2319] Query aangepast, wat kan omdat er een nieuwe klasse is gemaakt: AX.Uitgaand.PM.Product.impl.Product, waarin pointers zitten naar Prod.Product, Prod.ProductTekst, Prod.ProductPrijs en Prod.ProductKlas

Object dat doorgegeven wordt aan de convertor is nu een object van het type AX.Uitgaand.PM.Product.impl.Product i.p.v. een ResultSet. Dit geeft meer en doorzichtigere mogelijkheden wat UnitTests betreft.

  • More
  • CR-170
  • finished reviewing
Misschien de moment om de afhankelijk van de testen ten opzichte van zulke wijzigingen te verminderen. Kwestie van een techniek aan te leren hoe zoiets aan te pakken, zodat we die in de toekomst me...

Misschien de moment om de afhankelijk van de testen ten opzichte van zulke wijzigingen te verminderen.
Kwestie van een techniek aan te leren hoe zoiets aan te pakken, zodat we die in de toekomst meer en meer kunnen gebruiken en er meer en onafhankelijker van worden.

[UST2319] Drie ongebruikte velden verwijderd (Properties zijn ook verwijderd in klasse AXdixf.Product.Product
[UST2319] Iterator-test verwijderd, wegens geen nut meer. Testte enkel nog of de query overeenkwam en dat zijn nu exacte kopieën
  • More
  • CR-168
  • finished reviewing
[UST2319] Pointers naar Prod.Product, Prod.ProductKlas, Prod.ProductTekst en Prod.ProductPrijs, zodat ze in de query (met een join) benaderd kunnen worden via één klasse
    • -0
    • +39
    /AX/Uitgaand/PM/Product/impl/Product.cls.xml
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

[UST2319] Schrapping ongebruikte properties
[UST2319] Bijhorende UnitTest
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.

[UST2319] SamenstellingOmschrijvingProductenInEenString: Voegt de 22 lijnen omschrijving van een product, telkens door een spatie gescheiden, samen tot één lange string.

ConvertorLeveranciersnummerVanHoeckeNaarHalux: Een lijstje waarin de leveranciersnummers omgezet worden van wat ze zijn bij Van Hoecke naar wat ze zijn bij Halux.

    • -0
    • +31
    /AX/Uitgaand/PM/impl/SamenstellingOmschrijvingProductenInEenString.cls.xml
    • -0
    • +86
    /AX/Uitgaand/PM/impl/ConvertorLeveranciersnummerVanHoeckeNaarHalux.cls.xml
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

  • More
  • CR-159
  • finished reviewing
  • More
  • CR-160
  • finished reviewing
Aangezien het fase 1 is is het goed genoeg -> fase 2a zal hopelijk wel ietske beter zijn.

Aangezien het fase 1 is is het goed genoeg -> fase 2a zal hopelijk wel ietske beter zijn.

  • More
  • CR-14
  • finished reviewing
Niet dat het zou belangrijk is maar om toch een goed voorbeeld te stellen zou ik de verwerk methods toch een beetje bundelen en niet Verwerk .. nog wat anders en daarna nog wat Verwerk

Niet dat het zou belangrijk is maar om toch een goed voorbeeld te stellen zou ik de verwerk methods toch een beetje bundelen en niet Verwerk .. nog wat anders en daarna nog wat Verwerk

  • More
  • CR-14
  • started review
[DEF570] Kortingen voor Haefele BE werden niet toegepast voor LBX en TAX.
[DEF570] Kortingen voor Haefele BE werden niet toegepast voor LBX en TAX.
  • More
  • CR-166
  • finished reviewing
Code gaat er vanuit dat alles in een groep zit. Best code omvormen naar visitor zodat alle data naar buiten geschreven wordt. Ik zou via het visitor pattern de data verzamelen en deze dan teruggeve...

Code gaat er vanuit dat alles in een groep zit.
Best code omvormen naar visitor zodat alle data naar buiten geschreven wordt. Ik zou via het visitor pattern de data verzamelen en deze dan teruggeven als stream dan kan deze code de stream appenden aan de body.

wlip'ken

wlip'ken

Falende test gefixt, Aanpassing PaulV moet blijven. Test aangepast zodat in requestheader gebruikte klantnummers niet gebruikt worden.