Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Reverse merged revision(s) 59500 from trunk2010:

[ICT-1159] Mapping leveranciersnummer VH/HLX:

- Leveranciersnummer 9106 = 06815

- CODE-AANPASSING MOET IN ANDERE KLASSE GEBEUREN !!!

    • -1
    • +2
    ./Convertor/ConvertorLeveranciersnummerVanHoeckeNaarHalux.cls.xml
  1. … 1 more file in changeset.
[ICT-1159] Mapping leveranciersnummer VH/HLX:

- Leveranciersnummer 9106 = 06815

    • -2
    • +1
    ./Convertor/ConvertorLeveranciersnummerVanHoeckeNaarHalux.cls.xml
  1. … 1 more file in changeset.
DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

my point exactly (zie comment). Afzonderen naar apart klaske dat getest kan worden

my point exactly (zie comment).
Afzonderen naar apart klaske dat getest kan worden

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:

- Na review => Mockable gemaakt om AxProductValidator onder test te brengen

    • -0
    • +26
    ./SoprIteratorFilter.cls.xml
    • -0
    • +27
    ./IsOngewenstProductInMigratieFilter.cls.xml
  1. … 4 more files in changeset.
ClassMethod!

ClassMethod!

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (ju...

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (juist wat parameters zetten enzo) en dan was het geen probleem dat het niet in een niet-testbare classmethod steekt. Met al de extra logica die er ondertussen is bijgekomen is dat eigenlijk niet goed meer.

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
  • More
  • CR-944
  • finished reviewing
Is nu A en B, met beter geformuleerde asserts (AantalA, BestaatA, ...) en een verbeterde VoegToeProductOrderLijn() en VerwijderData(), die tegelijkertijd nu makkelijker uitbreidbaar zijn.

Is nu A en B, met beter geformuleerde asserts (AantalA, BestaatA, ...) en een verbeterde VoegToeProductOrderLijn() en VerwijderData(), die tegelijkertijd nu makkelijker uitbreidbaar zijn.

Na overleg hebben we dit issue laten vallen wegens het sop dat de kolen niet waard is?

Na overleg hebben we dit issue laten vallen wegens het sop dat de kolen niet waard is?

Zoals besproken LegacyPartijAPI.GeefKlantNummerVoorPartijID() gebruikt, wegens niet voor AX bedoeld, maar voor Caché-intern gebruik.

Zoals besproken LegacyPartijAPI.GeefKlantNummerVoorPartijID() gebruikt, wegens niet voor AX bedoeld, maar voor Caché-intern gebruik.

testje? $Char(0) is nl een tricky geval

testje? $Char(0) is nl een tricky geval

Ik weet dat dit op veel plaatsen gebruikt wordt, maar er bestaat ook de AXimpl.AdminToAxUtils (die gaat wel lpadden). En eigenlijk is dat misschien wel juister, aangezien ge hier overal met AX-klan...

Ik weet dat dit op veel plaatsen gebruikt wordt, maar er bestaat ook de AXimpl.AdminToAxUtils (die gaat wel lpadden). En eigenlijk is dat misschien wel juister, aangezien ge hier overal met AX-klantids aan het werken zijt (dus hoe dan ook 5 karakters lang (ook in repo))

Ik zou deze elvendertig calls ook in de onbeforeonetest steken. Op het eerste zicht zie ik maar 1 verwachtmethodcall hierop en principe overrulet die de reeds bestaande indienmethodcall. Op die man...

Ik zou deze elvendertig calls ook in de onbeforeonetest steken. Op het eerste zicht zie ik maar 1 verwachtmethodcall hierop en principe overrulet die de reeds bestaande indienmethodcall. Op die manier blijven de testen wat meer beperkt tot hun essentie. (of ik heb mij miskeken)

Ik zou al die namen met 8'en en 9's hernoemen. Vind het momenteel nogal moeilijk om direct te zien wat ze betekenen. Wat doet VoegToeProductOrdelijn8(9) bv? Kan je die twee methods al niet dezelfde...

Ik zou al die namen met 8'en en 9's hernoemen. Vind het momenteel nogal moeilijk om direct te zien wat ze betekenen. Wat doet VoegToeProductOrdelijn8(9) bv?
Kan je die twee methods al niet dezelfde naam geven en een extra parameter gebruiken (al dan niet gedefault)? Dat bespaart weeral een (bijna) duplicate private method. En waarom 8 en 9 en niet 1 en 2? precies dat de eerste 7 nog ergens anders staan

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:
[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:
[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- Fix SoprContractId => Eerst omzetting naar productnr van principaal

    • -0
    • +6
    ./Convertor/Sopr/ProductConvertor.cls.xml
  1. … 3 more files in changeset.
VHIP Migratie: Aanpassing langteksten en ExtArtOmschrLev:

- Parameter delimiter nu ook netjes doorgegeven

- Kleur moet ook op nieuwe lijn na langtekst => ook pipepipe ertussen en woordkleur met hoofdletter

    • -2
    • +2
    ./SamenstellingBestelreferentieProductenInEenString.cls.xml
  1. … 2 more files in changeset.
VHIP Migratie: Aanpassing langteksten en ExtArtOmschrLev:

- GeefBestelreferentieVoorProductTekst gebruikt nu een delimiter als 2e parameter die default een lege spatie is

- Voor ExternalItemTXT aan GeefBestelreferentieVoorProductTekst een dubbele pipe als delimiter meegegeven

    • -2
    • +2
    ./SamenstellingBestelreferentieProductenInEenString.cls.xml
  1. … 2 more files in changeset.
Yup http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Yup

  • More
  • CR-703
  • finished reviewing
waarom *A? Komt er nog een *B?

waarom *A? Komt er nog een *B?