CacheAdminA_trunk2010

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 11
Total Committers: 21
Last Commit: 04 Jul 23
Commits this week: 11
Total Lines of Code (LoC): 1.974.943
Net change in LoC this week: 481

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

Subversion Details

3 properties
  • svn:externals = /svn/CacheAdminACommon/trunk2010/TECH TECH /svn/CacheAdminACommon/trunk2010/vhLib vhLib /svn/CacheAdminACommon/trunk2010/Tools Tools /svn/CacheAdminACommon/trunk2010/vhUnitTest/TECH vhUnitTest/TECH /svn/CacheAdminACommon/trunk2010/vhUnitTest/vhLib vhUnitTest/vhLib /svn/CacheAdminACommon/trunk2010/vhUnitTest/Tools vhUnitTest/Tools /svn/CacheAdminASourceControl/trunk2010/svn svn /svn/CacheAdminAStories/trunk2010 vhStory /svn/CacheAdminACommon/trunk2010/vhTest/Mock/TECH vhTest/Mock/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Fake/TECH vhTest/Fake/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Dummy/TECH vhTest/Dummy/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Integratie/TECH vhTest/Integratie/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Spy/TECH vhTest/Spy/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Utils/TECH vhTest/Utils/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Mock/vhLib vhTest/Mock/vhLib /svn/CacheAdminACommon/trunk2010/vhTest/Fake/Tools vhTest/Fake/Tools /svn/CacheAdminACommon/trunk2010/vhTestFiles/UnitTests/TECH vhTestFiles/UnitTests/TECH /svn/CacheAdminACommon/trunk2010/vhTest/Data/TECH vhTest/Data/TECH
  • svn:ignore = .git ClassExplorer Default_KurtC.prj.xml
  • svn:mergeinfo = /deploy:2316,2391,2429,2461,2790,2922,3203,4080,4105,4814,38942 /deploy2010:3001-3010,3420,3443,3457,3491,3525,3620,3642,3646,3650,3677,3680-3683,3686-3687,3689,3703-3707,3709-3711,3716-3718,3726-3728,3733,3735,3743,3747-3749,3751-3752,3762-3764,3766-3767,3770,3772-3773,3775,3782,3784,3794,3839,3847,3861,3910,3945,4008-4009,4030,4145,4151,4217,4231,4268,4437,4448,4477,4507,4519,4612-4613,4622-4624,4723,4749,4870,4922,5121,5177-5178,5183,5210,5762-5763,5980,5997,6077,6084,6086,6287,6299,6336-6338,6506-6507,6637,6706,6916,6928,7057,7059,7067,7070,7137,7220,7250,7330,7346,7720,7758,7792,7914,7952-7953,8018,8024,8027,8102,8135,8176,8214,8250,8360,8362-8363,8406,8409,8484,8491,8561,8659,8716,8733-8734,8736,8799-8800,8802,8859,9035,9207,9216,9236,9423,10263,10286,10392,10409,10559,10571,10714,10927,10961,11543,11629,11872,11927,12065,12106,12170,12391-12393,12713,12792-12794,12803,12925,12952,12962,13047,13066,13192,13202,13654,14017,14027-14028,14071,14344,14439,14557-14565,14569,14600,14903,15035,15047,15087,15188-15190,15192,15265,15411,15902,16164-16165,16287,16384-16385,16408,16691,16721,16853,16935,16959,16983,17141,17494,17662,18100,18108-18109,18278-18286,18580,18679,18746,19120-19121,19578,20111,21545,22164,22436,22482,22832,22838-22839,23286,23416,23418,23421,23483,23562,23616,23636,29622,30300,30320,31795,31797,31799,34956,34962,35175-35176,35751,35965,36123,36181,36462,37123,37148-37149,38067-38072,38498,38583,38927-38930,39142,39203,39205-39206,39367,39464,39466,39482,39603-39604,39773,39902,39960,40332-40334,40390,40515-40516,40557-40558,40653,40849,40851,41111-41116,41548,41875,41879,42461,42556,42971-42973,44286-44288,44375-44376,46402,46479-46483,46490,46521,46864,46891-46896,47159-47160,47228-47229,47234,47505-47506,47508,47545-47551,48541,48604-48609,51918,52329,52348-52350,53211-53212,53261,53462,54030-54035,54521-54525,54962-54966,55339-55342,55524-55525,55596,55607,56283,56731-56732,56826,56981-56982,57290-57291,57294,57298-57299,57445,57542-57546,58227-58230,58805-58806,58966-58969,60346-60347,60367,60423,60434,61619,61621-61622,62122-62124,62527,63268-63273,65629,65638-65642,65650-65653 /trunk:2977,2979,2985,2987,2990-2996,2999,3002-3005,3007-3009,3011-3015,3019-3045,3049-3050,3052-3053,3055,3060-3062,3067-3068,3081-3091,3094-3095,3097-3099,3104-3108,3113,3115,3120,3123-3126,3128,3134,3142,3145-3148,3157,3164,3173-3174,3183,3185,3189,3199,3202,3204,3211,3213-3215,3222-3223,3225,3247,3252,3264,3289-3292,3298,3309,3356-3357,3362,3374,3389,3391,3402,3405,3409,3430-3431,3448,3451-3452,3463,3467,3477,3479,3481-3482,3484,3486,3500,3526,3535,3539,3551,3561,3571,3573,3614,3616,3636,3638,3640,3647,3654,3664,3668,3672,3674,3678,3690-3692,3699,3701,3717,3724,3729,3737,3739,3741-3742,3744,3787,3789-3790,3792,3803,3833,3837,3844,3846,3850-3851,3857,3862,3868,3872-3873,3879,3909,3914,3927,3930,3932-3933,3941,3957,3966,3968,3970,3972-3973,3982,3986,3990,3994,3996,4000,4006,4019,4022,4034,4037-4040,4045,4056,4079,4104,4134,4156,4159,4173,4397,4817,4990,5059-5060,5093-5094,5108-5137,5144,5222-5223,5446,5651,7918
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 10 Nov 2017
3:20 pm

Tom Vermeulen

  • More
  • CR-713
  • finished reviewing
3:15 pm

Tom Vermeulen

  • More
  • CR-708
  • finished reviewing
3:07 pm

Tommy Hebb

  • More
  • CR-714
  • finished reviewing
3:02 pm

Tommy Hebb

3:01 pm

Tommy Hebb

  • More
  • CR-706
  • finished reviewing
3:00 pm

Tom Vermeulen

  • More
  • CR-709
  • finished reviewing
2:57 pm

Tom Vermeulen

[UST3107][UST3108] ConfigIds meegeven naar InventSync
[UST3107][UST3108] ConfigIds meegeven naar InventSync
2:28 pm

Tommy Hebb

[UST3152] 0601 basisproducten voor ompak aventos:
[UST3152] 0601 basisproducten voor ompak aventos:
11:51 am

Kurt Callebaut

Deze code was dus niet correct...http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif is intussen gewijzigd. Later misschien nog te refactoren.

Deze code was dus niet correct... is intussen gewijzigd. Later misschien nog te refactoren.

11:47 am

Kurt Callebaut

11:47 am

Kurt Callebaut

11:44 am

Kurt Callebaut

terechte opmerking. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

terechte opmerking.

11:43 am

Kurt Callebaut

Aangezien de oude code werkt met API neem ik als variabele naam dezelfde benaming over om een beetje consistent te blijven.

Aangezien de oude code werkt met API neem ik als variabele naam dezelfde benaming over om een beetje consistent te blijven.

11:42 am

Kurt Callebaut

Jo, zegt steeds Set te gebruiken als het op dezelfde plaats gaat over een gaat over een New van een klasse. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Jo, zegt steeds Set te gebruiken als het op dezelfde plaats gaat over een gaat over een New van een klasse.

11:32 am

Kurt Callebaut

  • More
  • CR-712
  • finished reviewing
11:25 am

Kurt Callebaut

idem als overige, het veld reservatie wordt gemaakt via de constructor waarschijnlijk met de bedoeling dat deze niet mag gewijzigd worden. De MaakReservatie is dan ook private. Waarom zouden we dez...

idem als overige, het veld reservatie wordt gemaakt via de constructor waarschijnlijk met de bedoeling dat deze niet mag gewijzigd worden. De MaakReservatie is dan ook private. Waarom zouden we deze property vijrgeven zodat deze rechtstreeks aangepast kan worden vanuit de code, met alle mogelijke gevolgen van dien. Ook hier kan je beter voorkomen dat deze aangepast kan worden.

11:20 am

Kurt Callebaut

ProductID wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een proper...

ProductID wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een property weghalen geven we anderen de mogelijkheid ook na creatie van het object, de property te wijzigen. We zouden op zijn minst moeten zorgen dat deze dan readonly komen te staan. of er een foutmelding gegooid wordt bv via een setter, indien er een poging gedaan wordt om het ProductID te wijzigen.

11:20 am

Kurt Callebaut

IDem als ProductID : VerwachteInvoer wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu ...

IDem als ProductID :

VerwachteInvoer wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een property weghalen geven we anderen de mogelijkheid ook na creatie van het object, de property te wijzigen. We zouden op zijn minst moeten zorgen dat deze dan readonly komen te staan. of er een foutmelding gegooid wordt bv via een setter, indien er een poging gedaan wordt om het veld te wijzigen.

10:48 am

Kurt Callebaut

10:48 am

Kurt Callebaut

10:48 am

Tommy Hebb

  • More
  • CR-701
  • resumed reviewing
10:42 am

Tommy Hebb

Hier ook een paar tabkes...

Hier ook een paar tabkes...

10:41 am

Tommy Hebb

Tabkes my dear Watson, tabkes http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Hier en op meerdere plaatsen in de klasse

Tabkes my dear Watson, tabkes Hier en op meerdere plaatsen in de klasse

10:15 am

Tommy Hebb

Done!

Done!

10:15 am

Tommy Hebb

Was niet van mij, maar heb het verwijderd.

Was niet van mij, maar heb het verwijderd.

8:59 am

Tom Vermeulen

Is het niet simpeler om, voor het geval ge enkel productids krijgt, in de rapte zelf een lijst van AXid-objecten te maken (met enkel productids erin) en die dan door te geven aan de volgende method...

Is het niet simpeler om, voor het geval ge enkel productids krijgt, in de rapte zelf een lijst van AXid-objecten te maken (met enkel productids erin) en die dan door te geven aan de volgende method? Dan moet je niet heel de tijd liggen opsplitsen met die IsAxConfigIdOnBoard e.d.

8:54 am

Tom Vermeulen

Eigenlijk moet je die commentaar hier niet zetten, anders kan je de hele codebase beginnen volschrijven.

Eigenlijk moet je die commentaar hier niet zetten, anders kan je de hele codebase beginnen volschrijven.

7:57 am

Tom Vermeulen

  • More
  • CR-710
  • finished reviewing
7:56 am

Tom Vermeulen

Het enige wat ik kan bedenken is dat, als voor god-weet-welke-reden-dan-ook zowel de CommissieReferentie als de OrderReferentie één van die termen bevat, je twee voorvoegingen kan krijgen. Geen ide...

Het enige wat ik kan bedenken is dat, als voor god-weet-welke-reden-dan-ook zowel de CommissieReferentie als de OrderReferentie één van die termen bevat, je twee voorvoegingen kan krijgen. Geen idee of dat dat in de praktijk kan voorkomen.

(testjes?)

Thursday 09 Nov 2017
4:35 pm

Pieter Van Raemdonck

[ICT319] Properties public ipv private, om in sql fatsoenlijk te kunnen gebruiken.
[ICT319] Properties public ipv private, om in sql fatsoenlijk te kunnen gebruiken.