CacheAdminA_trunk2010/AXimpl/Controle/Order

Admin

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 11 Jul 17
Commits this week: 0
Total Lines of Code (LoC): 327
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 30 Jan 2018
11:00 am

Tommy Hebb

[UST3613] VHIP906 Nachtelijke ordercontrole: Fix lege identnummers igv Orgalux
Options
    • -1
    • +1
    ./AdminOrderLijnConverter.cls.xml
Wednesday 24 Jan 2018
11:36 am

Tommy Hebb

VHIP906 Nachtelijke controle verkooporders: Code THB en code PV krijgen elk hun eigen AxOrderLijnIterator:

- Nog een rechtzetting => check op verkeerd object-type => DOM.VKP.ProductOrderlijn ipv DTO-tje AXimpl.Controle.Order.OrderLijn

Options
    • -1
    • +1
    ./AdminUniekeOrderLijnIterator.cls.xml
11:21 am

Tommy Hebb

10:33 am

Tommy Hebb

[UST3599] VHIP906 Nachtelijke controle verkooporders: Syncen openstaande wijzigingen PV:

- Query-aanpassingen en return-object-aanpassing van PV in iterator => returned nu een ResultSet ipv een DTO.

- Quickfix voor Orders die niet geopend kunnen worden.

Options
    • -1
    • +5
    ./AdminUniekeOrderLijnIterator.cls.xml
  2. … 1 more file in changeset.
8:57 am

Tom Vermeulen

  • More
  • CR-949
  • finished reviewing
Tuesday 23 Jan 2018
3:03 pm

Tommy Hebb

[UST3596] VHIP906 Nachtelijke controle verkooporders: "GEEN ORDER GEVONDEN":
[UST3596] VHIP906 Nachtelijke controle verkooporders: "GEEN ORDER GEVONDEN":
3:03 pm

Tommy Hebb

[UST3596] VHIP906 Nachtelijke controle verkooporders: "GEEN ORDER GEVONDEN":

- LegacyPartijAPI wordt nu gebruikt om LeverancierNummer te verkrijgen van de LeverancierId

- Wegwerken van de quickfix try catch => probleem is nu ten gronde opgelost

- ProductId en het Product zijn nu op meerdere plekken nodig, dus het zetten van die variabelen is verhuisd naar de Convert en ze worden nu meegegeven als parameters

- De logica is aangepast van wanneer en of de principaal moet opgehaald worden (dit is de ten gronde oplossing hierboven vermeld) => LevNr = BLUM en IsSoprInCache

- Aanpassen testen naargelang

Options
    • -31
    • +33
    ./AdminOrderLijnConverter.cls.xml
  2. … 1 more file in changeset.
2:42 pm

Tommy Hebb

2:02 pm

Tom Vermeulen

  • More
  • CR-944
  • finished reviewing
11:07 am

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- Na review => LegacyPartijAPI nu gebruikt voor GeefKlantNummerVoorPartijID

- Na review => GetDataByName ipv GetData in AxOrderLijnConverter

- Na review => Refactoring UnitTests voor leesbaarheid, performantie en uitbreidbaarheid

Options
    • -1
    • +7
    ./AdminUniekeOrderLijnIterator/DataBouwer.cls.xml
  2. … 3 more files in changeset.
10:50 am

Tommy Hebb

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.

10:07 am

Tommy Hebb

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?

9:42 am

Tommy Hebb

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.

8:19 am

Tom Vermeulen

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

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

8:15 am

Tom Vermeulen

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))

8:07 am

Tom Vermeulen

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)

8:04 am

Tom Vermeulen

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

Monday 22 Jan 2018
3:21 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- LeverAdres wordt nu van de VerkoopOrderLijn gehaald

Options
    • -1
    • +1
    ./AdminOrderLijnConverter.cls.xml
  2. … 1 more file in changeset.
2:29 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:
[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:
12:56 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- Voor de AxConfigId aan Admin-kant mag de principaal niet opgehaald worden => de Blumkunde van de Sopr-variant moet geconverteerd worden

- Refactor naar oneliner

Options
    • -8
    • +1
    ./AdminOrderLijnConverter.cls.xml
Friday 19 Jan 2018
4:13 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- Fix SoprContractId => Eerst omzetting naar productnr van principaal

Options
    • -2
    • +9
    ./AdminOrderLijnConverter.cls.xml
  2. … 3 more files in changeset.
12:29 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- Introductie AxInventTransId als extra property die deel uitmaakt van de unieke index

- Verfijning Repository => AxInventTransId en Early Quit als record niet bestaat in de VerwijderVolgensKlantNrOrderIdProductNrAxInventTransId()

- Verfijning en uitbreiding UnitTests van Repository

- Rechtzetting en uitbreiding Fake Repository

Options
    • -3
    • +5
    ./AdminUniekeOrderLijnIterator/DataBouwer.cls.xml
  2. … 5 more files in changeset.
Wednesday 17 Jan 2018
2:56 pm

Tommy Hebb

2:56 pm

Tommy Hebb

Is herleid tot een simpele ResultSet-iterator. Niet veel meer te testen dus. StelSamenProductOrderLijnGlobal() zit nu in de klasse AXimpl.Controle.Order.Admin.AdminUniekeOrderLijnIterator.DataBouwe...

Is herleid tot een simpele ResultSet-iterator. Niet veel meer te testen dus.
StelSamenProductOrderLijnGlobal() zit nu in de klasse AXimpl.Controle.Order.Admin.AdminUniekeOrderLijnIterator.DataBouwer
Zie [UST3554]

2:45 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- ProductOrderLijnRepository wordt nu geinjecteerd

Options
    • -2
    • +2
    ./AdminUniekeOrderLijnIterator/DataBouwer.cls.xml
2:28 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- Aanpassing na review => Afzonderen opbouwen data => niet meer in constructor van iterator, maar in method in Generator

Options
    • -0
    • +61
    ./AdminUniekeOrderLijnIterator/DataBouwer.cls.xml
    • -42
    • +1
    ./AdminUniekeOrderLijnIterator.cls.xml
  3. ./AdminUniekeOrderLijnIterator
  4. … 1 more file in changeset.
12:48 pm

Tommy Hebb

Zie [UST3554]

Zie [UST3554]

Tuesday 16 Jan 2018
4:14 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- Maak en Bewaar van super van repository gebruiken => uitmockbaar

Options
    • -2
    • +2
    ./AdminUniekeOrderLijnIterator.cls.xml
3:08 pm

Tommy Hebb

[UST3554] VHIP906 Nachtelijke controle verkooporders: Tuning:

- Query moet gesorteerd zijn volgens OrderLijnId/inventtransid, zodat de compare wat handiger is

- Lichte aanpassing AX-query (aangereikt door KPR)

- Converter AX => ConfigId en Sopr => zijn soms NUL => geeft CRLF => testen op $Char(0) => dan lege string

- Converter AX => Hoeveelheid => OpenstaandeHoeveelheid min GepickteHoeveelheid ipv som

- Converter Admin => Kleine opkuis => Ongebruikte dependency => OrderWMSLijnRepository => verwijderd

- Converter Admin => Product en Sopr => KlantNummer <> LeverancierNummer => tsssss

- Converter Admin => Product => AxMaatwerkGenerischItemId van OrderLijn halen indien ConfigId aanwezig

- Converter Admin => Product => GeefVervangProductDoorSOPRInAX mag/moet altijd

- Converter Admin =>Sopr => LeverancierNummer ophalen => Indien dan nodig (5005) => SoprContractId ophalen

Options
    • -19
    • +16
    ./AdminOrderLijnConverter.cls.xml
    • -1
    • +1
    ./AdminUniekeOrderLijnIterator.cls.xml
  3. … 3 more files in changeset.
2:21 pm

Tom Vermeulen

  • More
  • CR-907
  • finished reviewing