CacheAdminA_trunk2010/DOM/PM/Maatwerk/impl

IngegevenKenmerkenTypeAPI

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 8
Last Commit: 22 Jul 15
Commits this week: 0
Total Lines of Code (LoC): 185
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
Wednesday 30 Sep 2020
10:20 am

Peter Vercoutter

[ICT-1828] Ingegeven kenmerken aanmaken voor MVX

- VerwerkMVXKenmerken implementeren in verschillende klasses waar een visitor kan passeren

Options
    • -0
    • +6
    ./IsGemonteerdeLadeBepaler.cls.xml
  2. … 2 more files in changeset.
Friday 25 Sep 2020
11:49 am

Peter Vercoutter

[ICT-1828] Ingegeven kenmerken aanmaken voor MVX

- VerwerkMVXKenmerken implementeren in verschillende klasses waar een visitor kan passeren

Options
    • -0
    • +6
    ./IsEenLosseComponentenVerpakkingBepaler.cls.xml
    • -0
    • +6
    ./IsFlatpackV1VerpakkingBepaler.cls.xml
  3. … 12 more files in changeset.
Thursday 23 Jul 2020
10:53 am

Peter Vercoutter

[ICT-1546] PM: Maatwerk: Space Step: Ingegeven Kenmerken Caché-kant inbouwen

- Method VerwerkSSTKenmerken toevoegen op verschillende relevante klassen

Options
    • -0
    • +8
    ./IsGemonteerdeLadeBepaler.cls.xml
    • -0
    • +6
    ./IsFlatpackV1VerpakkingBepaler.cls.xml
    • -0
    • +6
    ./IsEenLosseComponentenVerpakkingBepaler.cls.xml
  4. … 14 more files in changeset.
Tuesday 26 May 2020
8:55 am

LauraL

[ICT-1370] PM: Maatwerk: TAX: Palletverpakking configurator+calculator+productieaansturing

IsGemonteerdeLade instellen voor pallet verpakking

Options
    • -1
    • +1
    ./IsGemonteerdeLadeBepaler.cls.xml
  2. … 1 more file in changeset.
Friday 02 Aug 2019
1:59 pm

Tommy Hebb

1:57 pm

Tommy Hebb

Is bij de laatste wijzigingen terug F geworden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Is bij de laatste wijzigingen terug F geworden

12:26 pm

Wim Vermeulen

12:24 pm

Wim Vermeulen

Blijft deze code "P" behouden? Of toch de "F" van oorspronkelijke "FlatpackV1" ? Best toch effe checken bij JohanB.

Blijft deze code "P" behouden? Of toch de "F" van oorspronkelijke "FlatpackV1" ?
Best toch effe checken bij JohanB.

Tuesday 30 Jul 2019
5:15 pm

Tommy Hebb

[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:
2:20 pm

Tommy Hebb

[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:

- Move class vanwege naamswijziging => FlatpackV1

Options
    • -0
    • +60
    ./IsFlatpackV1VerpakkingBepaler.cls.xml
    • -60
    • +0
    ./IsFlatpackProboxVerpakkingBepaler.cls.xml
  3. … 1 more file in changeset.
12:51 pm

Tommy Hebb

[ICT742] [Kanban UST4277] TAXFP1: Flatpack Probox => Flatpack V1:

- Verheuis code FlatpackProbox naar FlatpackV1

- Nog wat opkuis hier en daar

Options
    • -7
    • +7
    ./IsFlatpackProboxVerpakkingBepaler.cls.xml
  2. … 87 more files in changeset.
Wednesday 26 Jun 2019
12:21 pm

Tommy Hebb

11:26 am

Wim Vermeulen

Tuesday 25 Jun 2019
10:37 am

Tommy Hebb

[UST4235] TAXFP1: Kostprijsberekening - Tijden:

- Na review => Kleine refactor => Equals van StringUtils gebruiken is wa overkill als er maar 1 vergelijkingswaarde is

Options
    • -1
    • +1
    ./IsFlatpackProboxVerpakkingBepaler.cls.xml
10:17 am

Tommy Hebb

Check http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Check

Friday 21 Jun 2019
1:39 pm

Wim Vermeulen

#dim met .%New( ... ) :-/ Zeker met een lange klassenaam zoals hierboven (en hieronder) staat de relevante code gegarandeerd buiten beeld. Ik zie hiervan echt geen meerwaarde. Als de nuttige info i...

#dim met .%New( ... ) :-/
Zeker met een lange klassenaam zoals hierboven (en hieronder) staat de relevante code gegarandeerd buiten beeld.
Ik zie hiervan echt geen meerwaarde.
Als de nuttige info in beeld kan staan (korte klassenaam) dan laat ik da aan mij voorbijgaan tijdens reviewen. Maar in dit geval vind ik da écht hopeloos zinloos!

N.B.: ik ga ervan uit dat je dit ook maar gekopieerd hebt van daarboven hé.

1:15 pm

Wim Vermeulen

Effe ter info : door het aanpassen van de #dim in dezelfde svn-commit is het reviewen van deze klasse behoorlijk lastig geworden. Best volgende keer hoofdletters en andere cosmetische wijzigingen a...

Effe ter info :
door het aanpassen van de #dim in dezelfde svn-commit is het reviewen van deze klasse behoorlijk lastig geworden.
Best volgende keer hoofdletters en andere cosmetische wijzigingen apart committen.
In deze klasse zou de review verbazend simpel worden. Echt waar

Graag effe feedback aub.

12:59 pm

Wim Vermeulen

Voor één waarde te vergelijken geen .Equals() method gebruiken; dit is overkill. Voor 2 (of meer) zeker wel gerechtvaardigd, en tevens mijn persoonlijke voorkeur, eerder dan (value = "A") || (value...

Voor één waarde te vergelijken geen .Equals() method gebruiken; dit is overkill.
Voor 2 (of meer) zeker wel gerechtvaardigd, en tevens mijn persoonlijke voorkeur, eerder dan 

(value = "A") || (value = "B")
12:56 pm

Wim Vermeulen

IngegevenKenmerkenTypeAPI --> IsFlatpackProboxVerpakkingBepaler --> complexe constructie, veel omwegen om uiteindelijk iets simpel op te vragen, iets dat enkel bij TAX zal voorkomen. Bij nader inz...

IngegevenKenmerkenTypeAPI --> IsFlatpackProboxVerpakkingBepaler --> complexe constructie, veel omwegen om uiteindelijk iets simpel op te vragen, iets dat enkel bij TAX zal voorkomen.

Bij nader inzien geldt dezelfde opmerking voor de "IsEenLosseComponentenVerpakkingBepaler". Wellicht heg je je daarop gebaseerd.
Dan is je actie min of meer gerechtvaardigd.

P.S.: ik had deze review-opmerking gewoon kunnen wissen, want je moet dus niks doen. Maar ik wou mijn bedenking toch effe delen.

Friday 14 Jun 2019
3:38 pm

Tommy Hebb

[UST4235] TAXFP1: Kostprijsberekening - Tijden:
[UST4235] TAXFP1: Kostprijsberekening - Tijden:
Wednesday 22 May 2019
12:06 pm

Tommy Hebb

[UST4235] TAXFP1: Kostprijsberekening - Tijden:

- IsFlatpackProboxVerpakking bijgevoegd

Options
    • -0
    • +60
    ./IsFlatpackProboxVerpakkingBepaler.cls.xml
  2. … 3 more files in changeset.
Friday 18 Jan 2019
8:03 am

Kurt Callebaut

Thursday 17 Jan 2019
5:37 pm

Wim Vermeulen

5:36 pm

Wim Vermeulen

Naam aanpassen : ProboxVerpakkingEvaluator Update: lees eerst opmerking hieronder.

Naam aanpassen : ProboxVerpakkingEvaluator

Update: lees eerst opmerking hieronder.

5:20 pm

Wim Vermeulen

Is dit niet pure duplicate code? (V1- en Probox-EnkelOnderdeel) In dit geval lijkt me toch beter van te de-dupliceren (enige parameter is VerpakkingEvaluator). De rest is hetzelfde, toch?

Is dit niet pure duplicate code? (V1- en Probox-EnkelOnderdeel)

In dit geval lijkt me toch beter van te de-dupliceren (enige parameter is VerpakkingEvaluator).
De rest is hetzelfde, toch?

11:02 am

Kurt Callebaut

[UST3929] TAX onlies: Tijden kosten van V1 en probox aanrekenen, maar enkel van de relevante componenten...
[UST3929] TAX onlies: Tijden kosten van V1 en probox aanrekenen, maar enkel van de relevante componenten...
Tuesday 15 Jan 2019
11:59 am

KurtC

[UST3929] TAX onlies: Tijden kosten van V1 en probox aanrekenen, maar enkel van de relevante componenten

- Montage aanrekenen is overbodig. Isgemonteerde lade wordt nu afhankelijk van productiewijze. Flatpack en lossecomp. blijft steeds ongemonteerd. V1 en probox kunnen gemonteerd of niet gemonteerd worden.

=> niet gemonteerd => geen kosten voor montage.

Options
    • -1
    • +5
    ./IsGemonteerdeLadeBepaler.cls.xml
  2. … 57 more files in changeset.
Tuesday 15 Sep 2015
1:51 pm

PirminVDB

[UST1277] TAX-PAS-LC: Bestelbon en Pickinglijst:zijkanten

- IsGemonteerdeLadeBepaler -> LBX en TBX kijken naar de productiewijze om te zien of de lade gemonteerd is.

Options
    • -2
    • +2
    ./IsGemonteerdeLadeBepaler.cls.xml
Thursday 03 Sep 2015
12:29 pm

Wim Vermeulen

[UST1239] TAX-PAS-LC : IngegevenKenmerkenTypeAPI method IsEenLosseComponentenVerpakking() + UT : impl voor MATKenmerken
Options
    • -1
    • +1
    ./IsEenLosseComponentenVerpakkingBepaler.cls.xml
  2. … 1 more file in changeset.
Tuesday 01 Sep 2015
3:07 pm

MartijnV

[UST1225] TAX LC: 10 stuks als aantal in de prijsberekening

- IsEenLosseComponentVerpakking voor LBXKenmerken geeft false terug ipv exception

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