Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Nooit een .Save doen: afzonderen naar een Bewaar op een repository

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

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

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

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?

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!)

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

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

- routine toevoegen aan vhCARETAKER

- Klasse en dto toevoegen die sync mogelijk maken

    • -0
    • +108
    ./HalffabSpec.cls.xml
  1. … 2 more files in changeset.