CacheAdminA_trunk2010

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 7
Total Committers: 21
Last Commit: 04 Jul 23
Commits this week: 7
Total Lines of Code (LoC): 1.974.914
Net change in LoC this week: 463

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
Thursday 20 Jun 2019
2:26 pm

Tommy Hebb

11:16 am

Tom Vermeulen

10:36 am

Kurt Callebaut

[BUS834] TTC - onderzoek. dubbele code wegwerken.
[BUS834] TTC - onderzoek. dubbele code wegwerken.
Wednesday 19 Jun 2019
2:00 pm

Kurt Callebaut

2:00 pm

Kurt Callebaut

Done

Done

1:04 pm

Wim Vermeulen

1:04 pm

Wim Vermeulen

Zoals besproken : De meest propere, zuivere en correcte oplossing is door de IngegevenKenmerkenTypeApi uit te mocken (1x Returns True en 1x Returns False) en dan is het niet meer nodig om over alle...

Zoals besproken :
De meest propere, zuivere en correcte oplossing is door de IngegevenKenmerkenTypeApi uit te mocken (1x Returns True en 1x Returns False) en dan is het niet meer nodig om over alle verpakkingstypes te lopen (uitz. FlatpackProbox).

Deze "refactor" zal in een aparte story opgenomen worden.
Voorlopig dus geen aanpassingen nodig

12:07 pm

Wim Vermeulen

Complexe If-structuur --> moeilijk begrijpbaar, en zeer lastig voor verdere (latere) uitbreidingen. Beter om hier af te stappen van de compacte versie en iets meer uiteen te halen. Mijn voorstel : ...

Complexe If-structuur --> moeilijk begrijpbaar, en zeer lastig voor verdere (latere) uitbreidingen. Beter om hier af te stappen van de compacte versie en iets meer uiteen te halen.
Mijn voorstel :

	If (IngegevenKenmerken.VerpakkingType = ..Enum.VerpakkingType.FlatpackProbox) {
		Set ProductieRoute = ..Enum.ProductieRoute.OnderdelenPicking
	} ElseIf (IngegevenKenmerken.IsBinnenlade || ..API.IngegevenKenmerkenTypeAPI.IsGemonteerdeLade(IngegevenKenmerken)) {
		Set ProductieRoute = ..Enum.ProductieRoute.TeMonteren
	} Else {
		Set ProductieRoute = ..Enum.ProductieRoute.OnderdelenPicking
	}
11:29 am

Wim Vermeulen

11:26 am

Wim Vermeulen

Dit soort fouten voelt heel erg "awkward" aan. TOP dat je dit gevonden hebt !!!

Dit soort fouten voelt heel erg "awkward" aan.
TOP dat je dit gevonden hebt !!!

11:22 am

Tom Vermeulen

11:21 am

Wim Vermeulen

Setup DnaCode : Mag ik veronderstellen dat er geen CleanUp nodig is, omdat die automatisch gebeurt?

Setup DnaCode :
Mag ik veronderstellen dat er geen CleanUp nodig is, omdat die automatisch gebeurt?

11:18 am

Wim Vermeulen

Test met meerdere toeleveringen --> zal falen want geen While() in de DnaCodeService impl.

Test met meerdere toeleveringen --> zal falen want geen While() in de DnaCodeService impl.

11:10 am

Wim Vermeulen

Methodnaam IsWegwerpPallet() is beter dan GeefIsWegwerpPallet()

Methodnaam IsWegwerpPallet() is beter dan GeefIsWegwerpPallet()

10:50 am

Wim Vermeulen

Geen while ? Er zitten toch meerdere toeleveringen in een Batch? or am I missing the point?

Geen while ?
Er zitten toch meerdere toeleveringen in een Batch? or am I missing the point?

10:31 am

Tommy Hebb

10:30 am

Wim Vermeulen

10:26 am

Tom Vermeulen

10:25 am

Tom Vermeulen

Omdat deze testklasse voorral het Service/Repo gedrag test. De DnaCodeRegistratie is een noodzakelijk kwaad dat buiten de scope van de test valt

Omdat deze testklasse voorral het Service/Repo gedrag test. De DnaCodeRegistratie is een noodzakelijk kwaad dat buiten de scope van de test valt

10:21 am

Tom Vermeulen

Heb code met try-catch uitgebreid zodat ie zeker op de lock - terechtkomt. In principe is dit de enige plek waar met die lock gewerkt wordt, dus zou er geen probleem mogen zijn

Heb code met try-catch uitgebreid zodat ie zeker op de lock - terechtkomt. In principe is dit de enige plek waar met die lock gewerkt wordt, dus zou er geen probleem mogen zijn

10:03 am

Tommy Hebb

Tuesday 18 Jun 2019
4:57 pm

Wim Vermeulen

4:54 pm

Wim Vermeulen

Waarom niet "VerwachtMethodCall" en Verifieer() ? mogelijk wel met Verwacht... ExactAantalKeer(0) en dan Idem bij de andere methods.

Waarom niet "VerwachtMethodCall" en Verifieer() ? mogelijk wel met Verwacht... ExactAantalKeer(0)

en dan Idem bij de andere methods.

4:21 pm

Wim Vermeulen

Ik zou deze blok (lijn 19-23 of zoiets) in een private method zetten, met een naam à la "GeefVolgendeGeldige/Bruikbare/ZonderRareTekens/.../DnaDeelOfNummerOfzoiets" #BestNameEver http://subversion0...

Ik zou deze blok (lijn 19-23 of zoiets) in een private method zetten, met een naam à la "GeefVolgendeGeldige/Bruikbare/ZonderRareTekens/.../DnaDeelOfNummerOfzoiets"
#BestNameEver
Als deze methodnaam dan voldoende sprekend is, dan wordt deze code ineens een pak leesbaarder en begrijpbaarder.

3:53 pm

Wim Vermeulen

Geen timeout gedefinieerd? --> oneindig lang wachten op lock dus?

Geen timeout gedefinieerd? --> oneindig lang wachten op lock dus?

3:06 pm

Wim Vermeulen

[UW775] vHintra: LBX S-geleider probleem BerekenDatum via MaatwerkViewer
[UW775] vHintra: LBX S-geleider probleem BerekenDatum via MaatwerkViewer
2:43 pm

Tom Vermeulen

2:43 pm

Tom Vermeulen

in constructor steken (wordt nergens gedaan in deze klasse, ik weet het...) als ge u dan niet kunt inhouden, kunt ge hem zelfs injecteren en valideren dat ie wordt opgeroepen indien nodig

in constructor steken (wordt nergens gedaan in deze klasse, ik weet het...)
als ge u dan niet kunt inhouden, kunt ge hem zelfs injecteren en valideren dat ie wordt opgeroepen indien nodig

2:42 pm

Kurt Callebaut

2:32 pm

Tom Vermeulen