Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merged revision(s) 54523 from deploy2010:

AX Statistiek - Aanpassing SOPR convertor en prijsbepalen voor SOPR : Klant DKG converteren naar Bruynzeel

........

  1. … 4 more files in changeset.
[PieterVR] [PaulV] AX.Uitgaand.PM.impl.Convertor.Sopr.ProductConvertor: wijziging op cachedev2010 was recenter dan de laatste commit.
  • 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

  1. … 3 more files in changeset.
Testje:

- Ongedaan maken => test heeft nut bewezen

  1. … 1 more file in changeset.
Testje
  1. … 1 more file in changeset.
[UST3255] VHIP Migr - SoprConverter: Aanpassing PV: BSKey
[UST3255] VHIP Migr - SoprConverter: Aanpassing PV: BSKey
  1. … 1 more file in changeset.
[UST2774] Migratie voorbereiding sprint 11 - issues deel 3 Sopr-gerelateerd:

- Toevoeging extra testcases ter illustratie Sopr-probleem => Sommige principalen staan op non-aktief

  • More
  • CR-450
  • finished reviewing
Je gebruikt principieel en principaal door elkaar, best vervangen door principaal (doe een CTRL-F, het zijn vijf files)

Je gebruikt principieel en principaal door elkaar, best vervangen door principaal (doe een CTRL-F, het zijn vijf files)

kan in 1 lijn (hierboven)

kan in 1 lijn (hierboven)

%New in constructor

%New in constructor

Beter een $IsObject ipv $$$HasLength als het over objecten gaat ipv strings. (jaja, untyped enz, maar toch)

Beter een $IsObject ipv $$$HasLength als het over objecten gaat ipv strings. (jaja, untyped enz, maar toch)

[UST2729] VHIP903: Period. voorraad: SoprContract opnemen voor elke SOPR-variant:
[UST2729] VHIP903: Period. voorraad: SoprContract opnemen voor elke SOPR-variant:
[UST2729] VHIP903: Period. voorraad: SoprContract opnemen voor elke SOPR-variant:

- Aanpassingen ter voorbereiding (methods -> public gemaakt + herschikking public/private methods)

  1. … 2 more files in changeset.
[UST2668] VHIP Migratie: Sync klassen met Caché01 (code PV)
  1. … 3 more files in changeset.
  • More
  • CR-379
  • finished reviewing