CacheAdminA_trunk2010/DOM/EC/Winkelkar/impl

LijnRepository.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 09 Jan 17
Commits this week: 0
Total Lines of Code (LoC): 29
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 16 Jan 2017
2:14 pm

Kurt Callebaut

2:14 pm

Kurt Callebaut

thx for the feedback.

thx for the feedback.

2:09 pm

Kurt Callebaut

Dan compileert de klasse niet.

Dan compileert de klasse niet.

2:08 pm

Kurt Callebaut

Dan compileert de klasse niet.

Dan compileert de klasse niet.

2:08 pm

Kurt Callebaut

Dan compileert de klasse niet.

Dan compileert de klasse niet.

2:01 pm

Kurt Callebaut

Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories

Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories

2:01 pm

Kurt Callebaut

Hierboven staat GeefViaId ? Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories

Hierboven staat GeefViaId ?
Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories

1:22 pm

Kurt Callebaut

Hieronder staat GeefViaID ?

Hieronder staat GeefViaID ?

12:54 pm

Kurt Callebaut

Laat maar staan, dan is het duidelijk dat er op 2 waarden gecontroleerd wordt.

Laat maar staan, dan is het duidelijk dat er op 2 waarden gecontroleerd wordt.

12:48 pm

Kurt Callebaut

OKay.

OKay.

12:48 pm

Kurt Callebaut

Een goeie week geleden heb je gezegd dat we niet offerte id en winkelkar id mogen gebruiken in foutmeldingen omdat het een en dezelfde functionaliteit is. We hebben dus voor externeid gekozen. Dus ...

Een goeie week geleden heb je gezegd dat we niet offerte id en winkelkar id mogen gebruiken in foutmeldingen omdat het een en dezelfde functionaliteit is. We hebben dus voor externeid gekozen. Dus laat externeid maar staan. WinkelkarId is gewoon de testwaarde. Niet blokkerend.

12:44 pm

Kurt Callebaut

Er is gekozen voor deze benaming. Hadden ze op voorhand maar moeten bedenken. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Er is gekozen voor deze benaming.
Hadden ze op voorhand maar moeten bedenken.

Friday 13 Jan 2017
11:09 am

Jo Claes

  • More
  • CR-21
  • finished reviewing
10:59 am

Jo Claes

Overbodig dus

Overbodig dus

10:58 am

Jo Claes

Foutmelding is beter Winkelkar met id 'WinkelkarId' bestaat niet.

Foutmelding is beter Winkelkar met id 'WinkelkarId' bestaat niet.

10:56 am

Jo Claes

Hier zou ik gaan voor de WinkelkarLijnRepositoryMock ipv de LijnRepositoryMock .. als we dan toch voluit gaan...

Hier zou ik gaan voor de WinkelkarLijnRepositoryMock ipv de LijnRepositoryMock .. als we dan toch voluit gaan...

10:55 am

Jo Claes

Ik zou de naam van variabele renamen naar Service ofwel moeten we de methods renamen naar Invoke.. Want het is nu een beetje onduidelijk VerwijderLijn.VerwijderLijn .. klikt een beetje raar ofwel ...

Ik zou de naam van variabele renamen naar Service ofwel moeten we de methods renamen naar Invoke..
Want het is nu een beetje onduidelijk VerwijderLijn.VerwijderLijn .. klikt een beetje raar

ofwel VerwijderLijnService.VerwijderLijn(
ofwel VerwijderLijn.Invoke(

10:52 am

Jo Claes

Dit soort type testen schrijf je ofwel met echte data - data opzetten en kijken of de repo zijn ding doet . ofwel met een spy die op zijn minst de Query prepared - zodat we zeker zijn dat het query...

Dit soort type testen schrijf je ofwel met echte data - data opzetten en kijken of de repo zijn ding doet .
ofwel met een spy die op zijn minst de Query prepared - zodat we zeker zijn dat het query statement kan uitgevoerd worden.

Het gevaar van dit soort testen is dat je uw Query statement in uw code gaat overkopieëren naar uw testen en als er een fout in het statement zit tabel naam dat bijvoorbeeld niet meer bestaat dat merk je het niet op.

10:47 am

Jo Claes

Lijkt mij overbodig

Lijkt mij overbodig

10:47 am

Jo Claes

Lijkt mij overbodig

Lijkt mij overbodig

10:47 am

Jo Claes

Lijkt mij overbodig

Lijkt mij overbodig

10:44 am

Jo Claes

GeefViaID

GeefViaID

10:44 am

Jo Claes

BestaatViaID

BestaatViaID

10:43 am

Jo Claes

Controles zijn niet nodig daar de GeefViaId op de winkelkar repo en de lijn repo de juiste foutmeldingen zullen geven

Controles zijn niet nodig daar de GeefViaId op de winkelkar repo en de lijn repo de juiste foutmeldingen zullen geven

10:40 am

Jo Claes

BestaatViaID WSimpl.Vhisie4.Winkelkar.WinkelkarRepos.. geen notie hebben van externeid in hun interface enkel in de implementatie

BestaatViaID WSimpl.Vhisie4.Winkelkar.WinkelkarRepos.. geen notie hebben van externeid in hun interface enkel in de implementatie

10:39 am

Jo Claes

GeefViaID method gebruiken daar de WSimpl.Vhisie4.Winkelkar.WinkelkarRepos.. geen notie hebben van externeid in hun interface enkel in de implementatie

GeefViaID method gebruiken daar de WSimpl.Vhisie4.Winkelkar.WinkelkarRepos.. geen notie hebben van externeid in hun interface enkel in de implementatie

Thursday 12 Jan 2017
8:09 am

Kurt Callebaut

  • More
  • CR-21
  • started review
[UST2065] Vh4_SVC:VerwijderLijn
[UST2065] Vh4_SVC:VerwijderLijn
Monday 09 Jan 2017
1:36 pm

KurtC

[UST2065] Vh4_SVC:VerwijderLijn

- VerwijderLijn + test. Testen te vereenvoudigen en nog via verkoopservice te laten verlopen (na bespreking.)

Options
    • -1
    • +1
    ./LijnRepository.cls.xml
  2. … 7 more files in changeset.
1:02 pm

KurtC

[UST2065] Vh4_SVC:VerwijderLijn

- Correctie returnvalue.

Options
    • -1
    • +1
    ./LijnRepository.cls.xml
11:32 am

KurtC

[UST2065] Vh4_SVC:VerwijderLijn

- ExterneID aan LijnRepository toegevoegd.

Options
    • -0
    • +16
    ./LijnRepository.cls.xml
  2. … 3 more files in changeset.