CacheAdminA_trunk2010/APPS/Halux/Machine

Kantenband

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 7
Last Commit: 03 Jul 15
Commits this week: 0
Total Lines of Code (LoC): 4.234
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
10:27 am

Peter Vercoutter

[ICT-2968] TAX - ander kantenbandprogramma voor sifonlade

- alle onderkanten van de kantenbanden van taor ruggen voor sifon moeten Pass2Program 'PVC_13_G' krijgen en Pass2Edgeband 'Z8D08B18'. Ik heb de kleur van de kantenband onderaan in de uitvoeringinfo niet aangepast naar zwart omdat dit anders een hoop andere zaken in de war stuurt. Enkel de reeds vermelde 2 parameters zijn aangepast

Options
    • -1
    • +8
    ./impl/PassageBepaler.cls.xml
    • -9
    • +9
    ./PassageDefinitieDefinieerder/RugSifonlade.cls.xml
  3. … 2 more files in changeset.
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 )

Monday 23 May 2022
11:29 am

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Vanwege Giel: Nieuwe kantenbandprogramma's specifiek voor W7

Options
    • -0
    • +2
    ./ProgrammaDefinitieDefinieerder.cls.xml
    • -1
    • +15
    ./enu/ProgrammaNaam.cls.xml
    • -0
    • +14
    ./enu/ProgrammaNaam/AutoComplete.cls.xml
    • -1
    • +1
    ./PassageDefinitieDefinieerder/Front.cls.xml
    • -7
    • +7
    ./PassageDefinitieDefinieerder/Rug.cls.xml
    • -7
    • +7
    ./PassageDefinitieDefinieerder/Zijkant.cls.xml
    • -0
    • +16
    ./enu/ProgrammaNaam/Verwerker.cls.xml
  8. … 1 more file in changeset.
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:
Monday 21 Mar 2022
1:45 pm

Tommy Hebb

[ICT-2588] [rvWV] TAX - nieuwe kleur W7 - Walnut medium Brown:

- Uitbreiding voor Kantenband-aansturing

Options
    • -0
    • +1
    ./PassageDefinitieDefinieerder/Front.cls.xml
    • -0
    • +5
    ./ProgrammaDefinitieDefinieerder.cls.xml
    • -1
    • +36
    ./enu/ProgrammaNaam.cls.xml
    • -0
    • +7
    ./PassageDefinitieDefinieerder/Zijkant.cls.xml
    • -0
    • +35
    ./enu/ProgrammaNaam/AutoComplete.cls.xml
    • -0
    • +40
    ./enu/ProgrammaNaam/Verwerker.cls.xml
    • -0
    • +7
    ./PassageDefinitieDefinieerder/Rug.cls.xml
Thursday 02 Dec 2021
1:52 pm

Peter Vercoutter

[ICT-2600] HX: Afkanten: TAX: fineer ruggen 1 pas minder - vervolg

- Enkel voor fineerruggen is een resultaat te vinden met parameter sifonlade = 1

Options
    • -1
    • +1
    ./impl/PassageDefinitieRepository.cls.xml
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
11:05 am

Peter Vercoutter

[ICT-2600] HX: Afkanten: TAX: fineer ruggen 1 pas minder - vervolg

- Nieuwe PassageDefinities voorzien voor ruggen van sifonlades

- Update testbestand met correcte aanpassing

Options
    • -1
    • +10
    ./data/PassageDefinitie.cls.xml
    • -1
    • +1
    ./impl/PassageBepaler.cls.xml
    • -0
    • +1
    ./PassageDefinitieDefinieerder.cls.xml
    • -1
    • +6
    ./impl/PassageDefinitieRepository.cls.xml
    • -1
    • +2
    ./PassageDefinitieDefinieerder/Base.cls.xml
    • -0
    • +95
    ./PassageDefinitieDefinieerder/RugSifonlade.cls.xml
  7. … 10 more files in changeset.
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):
11:06 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Kantenband.PassageDefinitie uitgebreid met OudZwart

Options
    • -0
    • +1
    ./PassageDefinitieDefinieerder/Front.cls.xml
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 21 Sep 2021
11:18 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- KantenbandBepaler => conversie OudZwart dan toch volgens KantenbandKleur wegens uitbreiding van ook die enum => naamgeving property Kleur aangepast naar KantenbandKleur om verwarring te vermijden.

- LadeKleur-doorgave via dto teruggedraaid wegens niet meer nodig

Options
    • -11
    • +6
    ./impl/KantenbandBepaler.cls.xml
  2. … 4 more files in changeset.
Friday 13 Aug 2021
10:27 am

Tommy Hebb

[ICT-2203] [rvTVE] PM - TAX front voor LBX - oplossing voor "OudZwart" (Z8/Z9):

- Indien LadeKleur = OudZwart => KantenbandKleur = Zwart

Options
    • -2
    • +21
    ./impl/KantenbandBepaler.cls.xml
  2. … 3 more files in changeset.
Tuesday 16 Mar 2021
4:47 pm

Tommy Hebb

4:46 pm

Tom Vermeulen

Thursday 11 Mar 2021
4:43 pm

Tommy Hebb

Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt. Ik dacht dat een OnBeforeOne...

Heum... waarom? Die verandert toch nooit en hoeft tussen UTs toch ook niet eventueel te veranderen? Toch zeker niet sinds er in de constructor ervan niets meer gebeurt.
Ik dacht dat een OnBeforeOneTest bedoeld is voor als er een gevaar is dat hij door eerdere UTs zou kunnen bevuild worden, wat hier niet het geval is.

3:13 pm

Tommy Hebb

Bwa, is een beetje zo gegroeid door refactoren... Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests? Daarna de afweging: Waarom dan nog de call on...

Bwa, is een beetje zo gegroeid door refactoren...
Waarom een Bepaler new'en per UnitTest, als je hem gewoon 1 keer kunt new'en in een OnBeforeAllTests?
Daarna de afweging: Waarom dan nog de call onder test speciaal gaan afzonderen? En bovendien: Waarom beschikken we anders over een $$$AssertTrue en $$$AssertFalse?
Het was bovendien daardoor ook niet meer nodig om een betekenisvolle, extra lokale variabele te voorzien in de UnitTesten (VerwachtInOpstartfase), dus ook al een regel minder per UT, zonder aan leesbaarheid in te boeten (eerder het tegendeel).
Doe de nieuwe versie eens open in studio en zie eens hoe leesbaar die is
Misschien moeten we dan eerder (mettertijd, als die klassen eens onder change komen) die van LBX en TBX aanpassen naar dit model?

8:57 am

Wim Vermeulen

Wednesday 10 Mar 2021
7:51 pm

Wim Vermeulen

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval. Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (pro...

Ik had verwacht dat "Product" of "ProductId" reeds werd doorgeven via de LadeInfo, maar dat blijkt toch niet het geval.
Als dit wel zo was, dan had het niet nodig geweest om de extra parameter (productieSequentie) toe te voegen.
Na al je inspanningen, keur ik deze oplossing goed