Activity CacheAdminA_trunk2010/BL/Legacy/FLOW.cls.xml

Tree
Accordion

Stats

Commits this week: 0

Total Committers:	4
Last Commit:	23 May 22
Commits this week:	0
Total Lines of Code (LoC):	720
Net change in LoC this week:	0

Commit Activity

last updated a few seconds ago

Monday 23 May 2022

10:54 am

Jo Willems

More
67867
changed 1 file
root:

[ICT] backport.

- -0
- +1
./FLOW.cls.xml
- History
- Source
- Diff
./FLOW.cls.xml

Monday 09 May 2022

8:37 am

Peter Vercoutter

More
67681
changed 2 files
root:

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

- Logging toevoegen zodat het aanmaken en verwijderen van die tijdelijke toeleveringen traceable is

- -1
- +4
./FLOW.cls.xml
- History
- Source
- Diff
./FLOW.cls.xml
… 1 more file in changeset.

Friday 06 May 2022

8:40 am

Peter Vercoutter

More
CR-2410
Closed

Thursday 05 May 2022

3:04 pm

Peter Vercoutter

More
67656
changed 2 files
root:

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

- Het is correcter om DELOBJ te gebruiken ipv verwijderToelevering. Deze gaat sommige indexen wel beter opkuisen, VerwijderenToelevering wordt eigenlijk nergens concreet gebruikt (het is een beetje de vraag waarom deze dan wel nog steeds bestaat in de code en niet 100% correct werkt)

- Reviewopmerking verwerken

- -4
- +4
./FLOW.cls.xml
- History
- Source
- Diff
./FLOW.cls.xml
… 1 more file in changeset.

Tuesday 12 Apr 2022

12:08 pm

Tom Vermeulen

More
CR-2410
finished reviewing

12:07 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

Tom Vermeulen

More
Re: CR-2410
commented on review

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.