Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1876] BIDI: Orderingave: Error zGeefIngegevenKenmerken in BLDTOE+29^FLOWORD7 (bij order2admin):...
[ICT-1876] BIDI: Orderingave: Error zGeefIngegevenKenmerken in BLDTOE+29^FLOWORD7 (bij order2admin):...
[ICT-1876] BIDI: Orderingave: Error zGeefIngegevenKenmerken in BLDTOE+29^FLOWORD7 (bij order2admin):

- Gerelateerd aan vorige aanpassing, zijnde [ICT-759] [rvWV] PM: Maatwerk: BOMBOL VHIP481: Waarschuwing indien wijzigen variant die reeds voorbij orderklaar is:

- OrderService => Niet het Order moet bewaard worden, maar wel de OrderLijn, want daar zit de wijziging op. Order bewaren op die plek is overhead, want is op PRNr op OrderLijn na ongewijzigd.

- ProductOrderlijn => Property ProductIdIsGewijzigd als pointer om op juiste moment indexen te killen en rebuilden => Gebeurde blijkbaar (nog) niet in hoofde van de ProductUpdater, door vorige implementatie van VerwijderIndexenIndienNodig en BouwIndexenIndienNodig.

- ProductOrderlijn => Indexen rebuilden in ZetProductID weggehaald (was toevoeging in bovengenoemde story) => Was onzinnig, want in SWNODE^FLOWORD2 wordt gepersisteerde data opgehaald en die is op dat moment nog ongewijzigd (nog met oude PRNr) => Zou dus enkel de VerwijderIndexen ongedaan maken ipv de nieuwe te zetten.

- ProductOrderlijn => Voor rebuilden van indexen na zetten van ProductID voor ProductUpdate wordt nu vertrouwd op de Save => Die doet de Save van de Super vooraleer de indexen te rebuilden, dus al met juiste data opgehaald in SWNODE^FLOWORD2.

- ProductOrderlijn => Geen event ingesloten => Order2Admin volgt sowieso nog.

- DataMProductOrderlijn-Mock uitgebreid met Save.

  1. … 3 more files in changeset.
  • More
  • CR-730
  • finished reviewing
[UST3087] VHIP901: voorbereiding op toevoegen pakket functionaliteit. -> refactor van huidige sorteerder + test voor sorteerde
[UST3087] VHIP901: voorbereiding op toevoegen pakket functionaliteit. -> refactor van huidige sorteerder + test voor sorteerde
  • More
  • CR-691
  • finished reviewing
[UST3087] VHIP901: ORDER2ADMIN: pakket (nieuwe lijn)

- Move sorteerder deel 3 - overbodige klassen

    • -19
    • +0
    ./DataMOrder/OrderlijnenSorteerder.cls.xml
  1. … 2 more files in changeset.
[UST3087] VHIP901: ORDER2ADMIN: pakket (nieuwe lijn)

- Refactor benaming

- Voorbereiding voor pakket implementatie.

    • -2
    • +2
    ./DataMOrder/OrderlijnenSorteerder.cls.xml
  1. … 5 more files in changeset.
[UST3054] VHIP901: ORDER2ADMIN: Teksten en referenties
[UST3054] VHIP901: ORDER2ADMIN: Teksten en referenties
[UST3054] VHIP901: ORDER2ADMIN: Teksten en referenties

- Sorteerder mockable voor uitmocken bij testen van converter.

    • -0
    • +19
    ./DataMOrder/OrderlijnenSorteerder.cls.xml
  1. … 2 more files in changeset.
Aangepast. Variabele naam "ReceiptDate" spreekt voor zich ? Als we $$INTDATE gebruiken dan kiezen hier ook voor conversies in onze testen ipv hardcoded values.

Aangepast. Variabele naam "ReceiptDate" spreekt voor zich ?
Als we $$INTDATE gebruiken dan kiezen hier ook voor conversies in onze testen ipv hardcoded values.

idem met multiple inheritance : Class vhTest.Mock.DOM.VKP.impl.DataM.DataMProductOrderlijn Extends (vhTest.Mock.DOM.VKP.ProductOrderlijn, vhTest.Mock.DOM.VKP.PrijsOrderlijn) Welke moet ik dan kiez...

idem met multiple inheritance :
Class vhTest.Mock.DOM.VKP.impl.DataM.DataMProductOrderlijn Extends (vhTest.Mock.DOM.VKP.ProductOrderlijn, vhTest.Mock.DOM.VKP.PrijsOrderlijn)

Welke moet ik dan kiezen ?

er is nog niks voorzien ook niet door Jow. De vraag is gaan we dit in data bij steken en opzoeken met queries ? of gaan we hardcoded omdat het tijdelijk is en er in cache toch geen wijzigingen meer...

er is nog niks voorzien ook niet door Jow. De vraag is gaan we dit in data bij steken en opzoeken met queries ? of gaan we hardcoded omdat het tijdelijk is en er in cache toch geen wijzigingen meer zullen zijn ?

in progress...

in progress...

Is aangepast, maar zie het nut er niet van in bij multiple inheritance omdat code completion dan niet alle beschikbare methods weer geeft. implementatie heeft bv ook Zetleverdag terwijl de interfac...

Is aangepast, maar zie het nut er niet van in bij multiple inheritance omdat code completion dan niet alle beschikbare methods weer geeft.
implementatie heeft bv ook Zetleverdag terwijl de interface Prijsorderlijn dat niet heeft.

  • More
  • CR-362
  • finished reviewing
Opletten met commit waar in je zTryouts zet , altijd gevaarlijk bij het mergen

Opletten met commit waar in je zTryouts zet , altijd gevaarlijk bij het mergen

Mocken naar interface niet naar implementatie

Mocken naar interface niet naar implementatie

$$INTDATE^vhLib. kwestie dat het duidelijker is

$$INTDATE^vhLib. kwestie dat het duidelijker is

Dit soort zou ik niet in de convert steken want ik denk dat best is om deze te controleren op hun bestaan .. anders ééns vragen aan JoW hoe het Lookup mechanisme werkt dat we in AXimpl.Admin hebben...

Dit soort zou ik niet in de convert steken want ik denk dat best is om deze te controleren op hun bestaan .. anders ééns vragen aan JoW hoe het Lookup mechanisme werkt dat we in AXimpl.Admin hebben gemaakt.

Anders omvormen naar de $$$If(OrderLine.ReceiptDateConfirmed,OrderLine.ReceiptDateConfirmed,OrderLine,ReceiptDateRequested) ,leest makkelijker als het één of het ander

Anders omvormen naar de $$$If(OrderLine.ReceiptDateConfirmed,OrderLine.ReceiptDateConfirmed,OrderLine,ReceiptDateRequested) ,leest makkelijker als het één of het ander

Code in commentaar -> YAGNI http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Code in commentaar -> YAGNI

#dimmen naar Interface ipv implementatie

#dimmen naar Interface ipv implementatie

[UST2570] VHIP901: ORDER2ADMIN ProductLijn simpelste case..
[UST2570] VHIP901: ORDER2ADMIN ProductLijn simpelste case..
[UST2570] VHIP901: ORDER2ADMIN ProductLijn simpelste case..

- Totaal Netto zetten op orderlijn.

    • -0
    • +8
    ./DataMProductOrderlijn.cls.xml
  1. … 3 more files in changeset.