CacheAdminA_trunk2010/WSimpl/Vhisie4/Auth

HandelAuthenticator.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 14 Sep 17
Commits this week: 0
Total Lines of Code (LoC): 46
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
Thursday 05 Oct 2017
9:44 am

Tom Vermeulen

Wednesday 04 Oct 2017
2:36 pm

Jo Claes

  • More
  • CR-535
  • finished reviewing
Thursday 21 Sep 2017
1:02 pm

Tom Vermeulen

8:49 am

Tom Vermeulen

Ja, want dat staat ook in de standaardDecrypteerRequest

Ja, want dat staat ook in de standaardDecrypteerRequest

Wednesday 20 Sep 2017
3:50 pm

Jo Claes

  • More
  • CR-551
  • finished reviewing
9:35 am

Tom Vermeulen

Het zou natuurlijk ook kunnen zijn dat ik, bij het overzetten van al die verschillende methods/properties/..., deze per ongeluk ben overgeschoten...

Het zou natuurlijk ook kunnen zijn dat ik, bij het overzetten van al die verschillende methods/properties/..., deze per ongeluk ben overgeschoten...

Tuesday 19 Sep 2017
4:29 pm

Tom Vermeulen

[UST2802] Vh4: Hein: Decrypteer-call met enkel hostname verwerken
[UST2802] Vh4: Hein: Decrypteer-call met enkel hostname verwerken
3:44 pm

Jo Claes

'k weet niet of deze voorwaarde effectief wel klopt ; lijkt het mij niet beter te checken of het feit dat er moet gecontroleerd worden voor deze hostname ? 'k vermoed dat we naar een systeem moete...

'k weet niet of deze voorwaarde effectief wel klopt ; lijkt het mij niet beter te checken of het feit dat er moet gecontroleerd worden voor deze hostname ?

'k vermoed dat we naar een systeem moeten gaan om hostnames te koppelen en personalities en ervoor zorgen dat die hostnames per systeem eventueel anders kunnen ingesteld worden.

3:42 pm

Jo Claes

'k zou zeggen spel wat met de waarden die ja als dummy doorgeeft en zie welke rare effecten uw testen opeens zullen hebben. Als het dan echt dummies zijn zou ik er effectief ook dummy klassen van m...

'k zou zeggen spel wat met de waarden die ja als dummy doorgeeft en zie welke rare effecten uw testen opeens zullen hebben.
Als het dan echt dummies zijn zou ik er effectief ook dummy klassen van maken zodat ze niet de aandacht trekken in uw code en de nadruk ligt op hetgeen je wil benadrukken.

3:39 pm

Jo Claes

Hier zou ik geen dummy string gebruiken want je controleert er effectief op

Hier zou ik geen dummy string gebruiken want je controleert er effectief op

3:38 pm

Jo Claes

Wel raar dat hier opeens een stub staat ?? Is het zo belangrijk dat er "example.com" wordt teruggegeven

Wel raar dat hier opeens een stub staat ?? Is het zo belangrijk dat er "example.com" wordt teruggegeven

3:37 pm

Jo Claes

Niet echt nodig om response te hebben -> het is perfect mogelijk om gewoon de registreer uit te voeren aangezien je het toch niet kan controleren -> kwestie van het geen aandacht te geven

Niet echt nodig om response te hebben -> het is perfect mogelijk om gewoon de registreer uit te voeren aangezien je het toch niet kan controleren -> kwestie van het geen aandacht te geven

3:28 pm

Jo Claes

Als deze foutmelding naar de client gaat en zo lijkt het er op dan is deze verkeerd want de client weet niet dat wij intern in ons systeem de id prefixen om onderscheid te kunnen maken in de handel...

Als deze foutmelding naar de client gaat en zo lijkt het er op dan is deze verkeerd want de client weet niet dat wij intern in ons systeem de id prefixen om onderscheid te kunnen maken in de handelklantdata.

3:25 pm

Jo Claes

?? GeefID verwijderen maar GeefPrefix laten we staan ?? Hoe kan je dit verklaren ?

?? GeefID verwijderen maar GeefPrefix laten we staan ?? Hoe kan je dit verklaren ?

3:23 pm

Jo Claes

Reponse lijkt mij toch eerder een datastructuur dus al het moet genewed worden in de constructor is er iets mis. Ga ik nu echt moeten verplichten dat alle methods 2x moeten uitgevoerd worden in de...

Reponse lijkt mij toch eerder een datastructuur dus al het moet genewed worden in de constructor is er iets mis.

Ga ik nu echt moeten verplichten dat alle methods 2x moeten uitgevoerd worden in de testen en er gecontroleerd wordt dat het resultaat niet dezelfde instantie is dan die van de vorige call

Friday 15 Sep 2017
3:06 pm

Jo Claes

Dummies injecten

Dummies injecten

3:05 pm

Jo Claes

Dummy Decrypteerder meegeven - dat die test niet opeens begint te falen omdat een bepaalde decryptie niet meer geldig is

Dummy Decrypteerder meegeven - dat die test niet opeens begint te falen omdat een bepaalde decryptie niet meer geldig is

3:04 pm

Tom Vermeulen

3:04 pm

Tom Vermeulen

Is al een beetje minder erg na UST2849

Is al een beetje minder erg na UST2849

3:04 pm

Jo Claes

Dummy Registreerder meegeven

Dummy Registreerder meegeven

3:02 pm

Tom Vermeulen

Aangezien het als sinds 2013 als String op de dto staat, heb ik het zo overgenomen.

Aangezien het als sinds 2013 als String op de dto staat, heb ik het zo overgenomen.

2:45 pm

Jo Claes

  • More
  • CR-529
  • finished reviewing
2:41 pm

Jo Claes

%String niet eerder iets %Decimal of %Numeric achtig

%String niet eerder iets %Decimal of %Numeric achtig

2:40 pm

Jo Claes

Wordt het lijstje hier niet etwa te lang -> Bepalers bundelen is misschien een optie Repsonse lijkt mij ook iets wat niet zou moeten genewed worden in de de constructor maar eerder iets is dat per...

Wordt het lijstje hier niet etwa te lang -> Bepalers bundelen is misschien een optie

Repsonse lijkt mij ook iets wat niet zou moeten genewed worden in de de constructor maar eerder iets is dat per call van toepassing is.

KlantDataBewaarder en HandelKantDataRepository kunnen misschien ook gegroepeerd worden.

2:38 pm

Jo Claes

Dit lijkt mij geen TECH.RegisteredObject te zijn maar eerder een TECH.DataTransfertObject Want nu worden bepaalde enum en aspecten van de basis klasse beschikbaar gemaakt zoals daar is de E proper...

Dit lijkt mij geen TECH.RegisteredObject te zijn maar eerder een TECH.DataTransfertObject

Want nu worden bepaalde enum en aspecten van de basis klasse beschikbaar gemaakt zoals daar is de E property

TECH.RegisteredObject is enkel voor Klassen en niet voor Objecten -> misschien had de naam beter geweest TECH.RegisteredClass

Thursday 14 Sep 2017
11:39 am

Tom Vermeulen

alleen maar wat code verplaatst, voor de rest niets

alleen maar wat code verplaatst, voor de rest niets

11:38 am

Tom Vermeulen

[UST2849] Vh4 Hein: HandelAuthenticator opkuisen
[UST2849] Vh4 Hein: HandelAuthenticator opkuisen
11:17 am

Tom Vermeulen

[UST2849] Vh4 Hein: HandelAuthenticator opkuisen

- ContactGegevensRegistreerder gebruiken

Options
    • -139
    • +7
    ./HandelAuthenticator.cls.xml
  2. … 5 more files in changeset.
8:15 am

Tom Vermeulen

[UST2846] Vh4 Hein: korting via registreercontactgegevens

- Voor de volledigheid

Options
    • -2
    • +8
    ./HandelAuthenticator.cls.xml
  2. … 8 more files in changeset.
7:59 am

Tom Vermeulen

[UST2849] Vh4 Hein: HandelAuthenticator opkuisen

- Decrypteer opgeschoven (volledig)

Options
    • -23
    • +2
    ./HandelAuthenticator.cls.xml
  2. … 1 more file in changeset.