CacheAdminA_trunk2010/DOM/PM/Maatwerk/Syncs/DotNet/MaatwerkCalculator/Data/impl

HalffabSpecInfoRepository.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 21 Dec 20
Commits this week: 0
Total Lines of Code (LoC): 0
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 21 Dec 2020
9:08 am

Peter Vercoutter

[ICT-1978] Sync van cache naar DotNet voor HalffabSpec

- Reviewopmerkingen: alles onder repositories brengen

Options
    • -13
    • +0
    ./HalffabSpecInfoRepository.cls.xml
  2. … 17 more files in changeset.
9:06 am

Peter Vercoutter

Tuesday 15 Dec 2020
10:51 am

Peter Vercoutter

[ICT-1978] Sync van cache naar DotNet voor HalffabSpec

- Reviewopmerkingen

Options
    • -6
    • +0
    ./HalffabSpecInfoRepository.cls.xml
  2. … 5 more files in changeset.
Monday 14 Dec 2020
12:01 pm

Tom Vermeulen

Nooit een .Save doen: afzonderen naar een Bewaar op een repository

Nooit een .Save doen: afzonderen naar een Bewaar op een repository

11:44 am

Tom Vermeulen

11:44 am

Tom Vermeulen

deze kan dus ineens de ..ZoekAlle() uitvoeren op de repo

deze kan dus ineens de ..ZoekAlle() uitvoeren op de repo

11:43 am

Tom Vermeulen

Is deze dan nodig? De oproeper kan toch direct ..ZoekAlle() uitvoeren op de HalffabSpecInfoRepo?

Is deze dan nodig? De oproeper kan toch direct ..ZoekAlle() uitvoeren op de HalffabSpecInfoRepo?

11:40 am

Tom Vermeulen

Volgens mij gebeuren hier in deze klasse toch twee volledig verschillende dingen? De sync-logica zelf aan de ene kant en de repo-logica van de HalffabSpec aan de andere kant (de private methods Bes...

Volgens mij gebeuren hier in deze klasse toch twee volledig verschillende dingen? De sync-logica zelf aan de ene kant en de repo-logica van de HalffabSpec aan de andere kant (de private methods Bestaat... en Update...) Die twee laatste horen functioneel gezien in een aparte repository-klasse te zitten, zodat ze eenvoudig uit te mocken zijn in je nog niet-gecommitte testen . Volgend voordeel is dat de Sync* method zelf een stukje korter/leesbaarer kan zijn.
Aangezien dit toch een vrij belangrijk stukje code is, zou ik dat toch nog veranderen (en zeker testen voorzien!)

11:28 am

Tom Vermeulen

Ge kunt hier ook gewoon CurrentTimeStamp van de DateTimeAPI gebruiken http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Ge kunt hier ook gewoon CurrentTimeStamp van de DateTimeAPI gebruiken

Friday 11 Dec 2020
1:45 pm

Peter Vercoutter

[ICT-1978] Sync van cache naar DotNet voor HalffabSpec
[ICT-1978] Sync van cache naar DotNet voor HalffabSpec
Wednesday 04 Nov 2020
10:22 am

Peter Vercoutter

[ICT-1978] Sync van cache naar DotNet voor HalffabSpec
Options
    • -0
    • +19
    ./HalffabSpecInfoRepository.cls.xml
  2. … 4 more files in changeset.