Activity CacheAdminA_trunk2010/FLOW.mac.rou

8:40 am

Jo Willems

More
67493
changed 22 files
root:

[ICT-2609] Interface EWMS - OBJT Filebased versie

- Backport code van de laatste week.

- -1
- +2
/FLOW.mac.rou
- History
- Source
- Diff
/FLOW.mac.rou
… 21 more files in changeset.

12:08 pm

Tom Vermeulen

More
Re: CR-2410
commented on review

Deze while op het eerste zicht ook. Waarom zit er eigenlijk een aparte if .HasNext() en niet direct een while HasNext()?

Deze while op het eerste zicht ook.

Waarom zit er eigenlijk een aparte if .HasNext() en niet direct een while HasNext()?

12:05 pm

Tom Vermeulen

More
Re: CR-2410
commented on review

deze if mag mee in de bovenstaande if-block

12:04 pm

ik weet het, de opmerkingen zijn rijkelijk laat http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif en ik weet ook dat het in oude Legacy-stuff is http://su...

ik weet het, de opmerkingen zijn rijkelijk laat
en ik weet ook dat het in oude Legacy-stuff is

Hoe dan ook: in-line sql is altijd icky.
Misschien was het beter geweest om het ophalen van deze resultset (die verderop gebruikt wordt om een iterator te maken), op te zetten zoals in AXimpl.Admin.MAG.NietLeverenOorzaak.AxOrderLijnIterator. Dan zit de sql wel nog altijd in die klasse, maar is het beter afgezonderd en gemakkelijker toe te voegen aan de (nog) niet bestaande testen.

10:47 am

Peter Vercoutter

More
CR-2410
started review

[ICT-2678] FSC/PEFC papieren afdrukken voor een toelevering

10:27 am

Peter Vercoutter

More
67110
changed 3 files
root: CR-2410

[ICT-2678] FSC/PEFC papieren afdrukken voor een toelevering

- Enkel toelevering verwijderen indien we weten dat we hem zelf hebben aangemaakt

- journalId zal beginnen met HX, 00 is nu enkel voor test

- -1
- +1
/FLOW.mac.rou
- History
- Source
- Diff
/FLOW.mac.rou
… 2 more files in changeset.

8:57 am

Peter Vercoutter

More
67109
changed 3 files
root: CR-2410

[ICT-2678] FSC/PEFC papieren afdrukken voor een toelevering

- -1
- +7
/FLOW.mac.rou
- History
- Source
- Diff
/FLOW.mac.rou
… 2 more files in changeset.

4:21 pm

Jo Willems

More
CR-1549
Closed

10:37 am

Kurt Callebaut

More
CR-1549
finished reviewing

12:39 pm

Jo Willems

More
CR-1549
started review

[UST4119] MAG: Reden van niet-meenemen in levering kunnen ingeven

12:32 pm

Jo Willems

More
54783
changed 1 file
root: CR-1549

[UST4119] MAG: Reden van niet-meenemen in levering kunnen ingeven

- opvragen van lijst bons die in aanmerking komen gaf een fout. index KU2 is niet meer up to date wegens niet meer facureren in Admin. eerst een check of data wel bestaat in KUL alvorens op te nemen in de gesorteerde lijst.