CacheAdminA_trunk2010/DOM/VKP/impl/DataM

DataMProductOrderlijn.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 13 Oct 20
Commits this week: 0
Total Lines of Code (LoC): 2.383
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
Monday 26 Oct 2020
11:01 am

Tommy Hebb

8:27 am

Tom Vermeulen

Thursday 15 Oct 2020
4:41 pm

Tommy Hebb

[ICT-1876] BIDI: Orderingave: Error zGeefIngegevenKenmerken in BLDTOE+29^FLOWORD7 (bij order2admin):...
[ICT-1876] BIDI: Orderingave: Error zGeefIngegevenKenmerken in BLDTOE+29^FLOWORD7 (bij order2admin):...
Tuesday 13 Oct 2020
5:19 pm

Tommy Hebb

[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.

Options
    • -11
    • +23
    ./DataMProductOrderlijn.cls.xml
  2. … 3 more files in changeset.
Thursday 31 Oct 2019
10:14 am

Tommy Hebb

9:58 am

Tommy Hebb

Telegramstijl, omdat ze anders te lang worden en de variaties nu eenmaal soms ingewikkeld zijnhttp://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Telegramstijl, omdat ze anders te lang worden en de variaties nu eenmaal soms ingewikkeld zijn

9:57 am

Tommy Hebb

Idem reply hierboven.

Idem reply hierboven.

9:57 am

Tommy Hebb

Zoals besproken... Dit ging over de property-namen, maar... toch liever niet. In de properties kunnen soms mocks, soms stubs gestoken worden, afhankelijk van wat de UnitTest nodig heeft.

Zoals besproken... Dit ging over de property-namen, maar... toch liever niet. In de properties kunnen soms mocks, soms stubs gestoken worden, afhankelijk van wat de UnitTest nodig heeft.

Wednesday 30 Oct 2019
12:57 pm

Tommy Hebb

Done!

Done!

12:51 pm

Tommy Hebb

Done

Done

12:43 pm

Tommy Hebb

Done! (smartass http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif )

Done! (smartass )

12:39 pm

Tommy Hebb

Done!

Done!

12:15 pm

Tommy Hebb

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

12:09 pm

Tommy Hebb

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

Done, maar staat momenteel al nieuwe openstaande wijziging op die klasse, die nog niet gecommit mag worden, dus komt daarna wel.

12:05 pm

Tommy Hebb

Done en done en done! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Done en done en done!

10:51 am

Tommy Hebb

Normaal binnen de scope blijven, ik weet het, maar ik dacht de while op deze manier zo proper en leesbaar mogelijk te houden. Uiteindelijk wordt elk van die dim'ekes toch in die while gebruikt en o...

Normaal binnen de scope blijven, ik weet het, maar ik dacht de while op deze manier zo proper en leesbaar mogelijk te houden. Uiteindelijk wordt elk van die dim'ekes toch in die while gebruikt en omhelst die while de volledige method.

10:44 am

Tommy Hebb

Denk het niet. Nu krijgt ge alle OrderLijnIterators netjes op een rijtje via code completion. Eentje is dan de volledige, sommige zijn "via" dit, andere "via" dat, ...

Denk het niet. Nu krijgt ge alle OrderLijnIterators netjes op een rijtje via code completion. Eentje is dan de volledige, sommige zijn "via" dit, andere "via" dat, ...

9:44 am

Tommy Hebb

Had ik al aangepast inmiddels, zit al in volgende review http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Had ik al aangepast inmiddels, zit al in volgende review

Tuesday 29 Oct 2019
2:45 pm

Kurt Callebaut

12:20 pm

Kurt Callebaut

test namen zeggen niet duidelijk wat er getest wordt.

test namen zeggen niet duidelijk wat er getest wordt.

12:18 pm

Kurt Callebaut

(Is bestaande code natuurlijk) hoeft geen lokale var te zijn. mag rechtstreeks op ..ProductService(Mock)

(Is bestaande code natuurlijk) hoeft geen lokale var te zijn. mag rechtstreeks op ..ProductService(Mock)

12:16 pm

Kurt Callebaut

Refact nodig zodat namen van bestaande code ook ProductServiceMock worden, etc..

Refact nodig zodat namen van bestaande code ook ProductServiceMock worden, etc..

12:02 pm

Kurt Callebaut

in deze situatie gebruiken we veelal if $$asserttrue(.. hasnext){ assert.. next() } omdat de .next hiervan afhankelijk is. ..

in deze situatie gebruiken we veelal
if $$asserttrue(.. hasnext)

Unknown macro: { assert.. next() }


omdat de .next hiervan afhankelijk is. ..

12:00 pm

Kurt Callebaut

es afstemmen met team of we dim buiten lussen houden als ze enkel binnen lus gebruikt worden.

es afstemmen met team of we dim buiten lussen houden als ze enkel binnen lus gebruikt worden.

11:58 am

Kurt Callebaut

"Order heeft 1 orderlijn met gevraagde InventransId"

"Order heeft 1 orderlijn met gevraagde InventransId"

11:56 am

Kurt Callebaut

"Order heeft geen orderlijnen." voor mij maakt het niet uit of ze zitten of liggen. :o)

"Order heeft geen orderlijnen." voor mij maakt het niet uit of ze zitten of liggen. :o)

11:44 am

Kurt Callebaut

Toelevering heeft normaal gezien ook een productlijniterator, dan hoef je type niet mee te geven.

Toelevering heeft normaal gezien ook een productlijniterator, dan hoef je type niet mee te geven.

11:40 am

Kurt Callebaut

$$$haslength ?

$$$haslength ?

11:37 am

Kurt Callebaut

ik vind deze ook goed, maar kan het zijn dat we declaraties altijd op de plaats zetten waar ze gebruikt worden ? in dit geval binnen de while ?

ik vind deze ook goed, maar kan het zijn dat we declaraties altijd op de plaats zetten waar ze gebruikt worden ? in dit geval binnen de while ?

11:35 am

Kurt Callebaut

Moet "iterator" altijd achteraan in benaming ? (ben ook niet zeker) -> bv GeefOrderlijnenViaAXInventTransIdIterator als naam

Moet "iterator" altijd achteraan in benaming ? (ben ook niet zeker) -> bv GeefOrderlijnenViaAXInventTransIdIterator als naam