CacheAdminA_trunk2010/APPS/Halux

Machine

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 10
Last Commit: 03 Jul 15
Commits this week: 0
Total Lines of Code (LoC): 7.325
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
Tuesday 13 Dec 2022
10:24 am

Peter Vercoutter

9:59 am

Tom Vermeulen

Thursday 25 Aug 2022
10:33 am

Peter Vercoutter

[ICT-2968] TAX - ander kantenbandprogramma voor sifonlade
[ICT-2968] TAX - ander kantenbandprogramma voor sifonlade
Friday 12 Aug 2022
2:34 pm

Joannes Lemmens

Tuesday 09 Aug 2022
11:25 am

Tommy Hebb

Tuesday 02 Aug 2022
12:18 pm

Joannes Lemmens

Extra integratietest is hiervoor aangemaakt: "TestUitsluitendSpecialWorkshop". Momenteel bevat deze enkel een W7 Front, mochten er later meer SpecialWorkshop zaken bijkomen mogen deze hier uiteraar...

Extra integratietest is hiervoor aangemaakt: "TestUitsluitendSpecialWorkshop".
Momenteel bevat deze enkel een W7 Front, mochten er later meer SpecialWorkshop zaken bijkomen mogen deze hier uiteraard ook bij.

10:26 am

Joannes Lemmens

Het probleem is dat je hier echt aan het randje zit van wat Windows aankan met de volledige bestandsnaam (247 karakters). Als dit langer is wordt er helemaal geen bestand aangemaakt. Voor deze IT ...

Het probleem is dat je hier echt aan het randje zit van wat Windows aankan met de volledige bestandsnaam (247 karakters). Als dit langer is wordt er helemaal geen bestand aangemaakt.

Voor deze IT (BinnenladeLadeKleurW7) heb ik alvast de 006 terug toegevoegd. Dan zijn er in totaal 246 karakters voor de special workshop ImportFile met 100-999 aantal stuks.
Voor de LosseComponenten: volgens mij had deze oorspronkelijk ook geen nummering? Hoe dan ook zou dit in totaal dan 247 karakters zijn met een nummering, toch een beetje gevaarlijk lijkt mij (hoewel nipt mogelijk).

Monday 01 Aug 2022
3:58 pm

Tommy Hebb

Ik heb de indruk dat er nu soms ook een leeg bestand wordt gegenereerd (als alles W7 -front is, dan wordt "het originele" bestand ook nog gegenereerd. Klopt dat? Toch precies wel volgens de testfil...

Ik heb de indruk dat er nu soms ook een leeg bestand wordt gegenereerd (als alles W7 -front is, dan wordt "het originele" bestand ook nog gegenereerd. Klopt dat? Toch precies wel volgens de testfiles). Dat lijkt mij onwenselijk.

3:55 pm

Tommy Hebb

ProductTypeApi en ToeleveringService mogen terug verwijderd worden wegens uiteindelijk niet gebruikt (zie ook verderop). P.s.: Ook als parameter en als property te verwijderen! En eveneens niet ve...

ProductTypeApi en ToeleveringService mogen terug verwijderd worden wegens uiteindelijk niet gebruikt (zie ook verderop).
P.s.: Ook als parameter en als property te verwijderen!

En eveneens niet vergeten de nodige aanpassingen door te voeren in bijhorende testcase: vhUnitTest.APPS.Halux.PPS.Activiteit.impl.TAOR.LijstVerwerkers.Opdeelzaag.OptimalisatieBestandenGenerator.VoerUit.TestProductGroep

3:51 pm

Tommy Hebb

Ik mis nog een test met als rol "Front".

Ik mis nog een test met als rol "Front".

3:48 pm

Tommy Hebb

Ai, dat is spijtig, dat je hier die "006" hebt weggelaten. Hierdoor staat de W7-specifieke test niet meer proper net onder de niet-W7-specifieke test. Dat zie je overzichtelijk als je deze klasse o...

Ai, dat is spijtig, dat je hier die "006" hebt weggelaten. Hierdoor staat de W7-specifieke test niet meer proper net onder de niet-W7-specifieke test. Dat zie je overzichtelijk als je deze klasse opzoekt via SVN. Die test-nummering heeft een doel
Hetzelfde geldt tevens ook voor de LosseComponenten-test!

3:42 pm

Tommy Hebb

Ik zou "ProductieGroep" niet afkorten in de variabelenaam, hij wordt door deze ingreep al meer dan voldoende ingekort http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emotic...

Ik zou "ProductieGroep" niet afkorten in de variabelenaam, hij wordt door deze ingreep al meer dan voldoende ingekort
Dus: ProductieGroepEnum.

3:39 pm

Tommy Hebb

Bovenstaande private method BevatBatchProductMetBewerkingInSpecialWorkshop mag verwijderd worden, wegens uiteindelijk toch niet gebruikt (zie ook hogerop)

Bovenstaande private method BevatBatchProductMetBewerkingInSpecialWorkshop mag verwijderd worden, wegens uiteindelijk toch niet gebruikt (zie ook hogerop)

3:37 pm

Tommy Hebb

Property IsSpecialWorkshop mag terug verwijderd worden, want wordt uiteindelijk toch niet gebruikt (zie ook verderop).

Property IsSpecialWorkshop mag terug verwijderd worden, want wordt uiteindelijk toch niet gebruikt (zie ook verderop).

3:33 pm

Tommy Hebb

Opkuis: Lijn in comment doet niet veel http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Opkuis: Lijn in comment doet niet veel

Monday 25 Jul 2022
8:40 am

Joannes Lemmens

[ICT-2966] [rvTHB] Aanpassingen Lodder sturing
[ICT-2966] [rvTHB] Aanpassingen Lodder sturing
Monday 30 May 2022
4:41 pm

Tommy Hebb

Tuesday 24 May 2022
1:09 pm

Wim Vermeulen

1:09 pm

Wim Vermeulen

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt http://sub...

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt )
De impl in deze afgeleide klasse is wel correct.

De implementatie voor "Lengte" hieronder zou ik wel in een private method steken, analoog aan GeefFrontPlaatHoogte()
Zo blijft deze method simpel leesbaar en overzichtelijk. En dan is de scope ook duidelijk, nl. dat Kleur W7 enkel impact heeft op de kenmerk "Lengte, en niet op de "Breedte" van het Front.

10:08 am

Wim Vermeulen

copy-paste foutje : "... kan niet gemaild worden." (ook al is de conditie altijd false :-P )

copy-paste foutje :
"... kan niet gemaild worden."
(ook al is de conditie altijd false :-P )

Wednesday 20 Apr 2022
3:37 pm

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:
Thursday 25 Nov 2021
10:34 am

Peter Vercoutter

10:32 am

Tom Vermeulen

Thursday 18 Nov 2021
11:06 am

Peter Vercoutter

[ICT-2600] HX: Afkanten: TAX: fineer ruggen 1 pas minder - vervolg
[ICT-2600] HX: Afkanten: TAX: fineer ruggen 1 pas minder - vervolg
Tuesday 05 Oct 2021
12:49 pm

Tommy Hebb

12:21 pm

Tom Vermeulen

11:09 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):
Thursday 23 Sep 2021
4:17 pm

Tommy Hebb

Zoals besproken blijkbaar ongewild in deze review terechtgekomen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Zoals besproken blijkbaar ongewild in deze review terechtgekomen

4:06 pm

Tom Vermeulen

is deze wel nodig?

is deze wel nodig?

Tuesday 16 Mar 2021
4:47 pm

Tommy Hebb