CacheAdminA_trunk2010/DOM/PM/Product/impl

TAORMaatwerkProductCreator.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 22 Nov 19
Commits this week: 0
Total Lines of Code (LoC): 70
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
Thursday 19 Mar 2020
1:58 pm

Jo Willems

Monday 25 Nov 2019
8:07 am

Tom Vermeulen

8:07 am

Tom Vermeulen

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronde...

Eigenlijk zijn dat hier allemaal commits om dependency injection toe te laten op deze Creator. Zouden er dan ook niet wat testen moeten zijn die wat dependencies injecten en het gedrag van hieronder verifiëren?

8:01 am

Tom Vermeulen

overbodige comment?

overbodige comment?

Friday 22 Nov 2019
2:35 pm

Jo Willems

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten
[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten
2:34 pm

Jo Willems

[ICT-860]PM: Maatwerk: ingekort ORGALUX: "ORGALUX" in bestelreferentie zetten

- Beetje refactoren om de constructor aan de coding conventions te laten voldoen.

Options
    • -1
    • +2
    ./TAORMaatwerkProductCreator.cls.xml
  2. … 7 more files in changeset.
Wednesday 30 Oct 2019
9:42 am

Tommy Hebb

Monday 28 Oct 2019
3:08 pm

Kurt Callebaut

Friday 25 Oct 2019
1:20 pm

Wim Vermeulen

Deze method voor TAX is volledig identiek http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/warning.gif aan die erboven voor LBX, op één woordje na : ..GeefProductU...

Deze method voor TAX is volledig identiek aan die erboven voor LBX, op één woordje na : ..GeefProductUpdater("TAX", ...
En ondertussen mogelijk ook nog eens t zelfde voor TBX (niet in deze review)

Zoals reeds eerder opgemerkt : Maak in de UT duidelijk waar de variatie zit, dus niet door de volledige impl te kopieren en één waarde te veranderen.
Oplossing hier is heel simpel : maak private method AssertProductUpdaterVoorLadeVariant (LadeVariant As ...enu)
en dan heb je 2 (of 3 indien TBX ook) testmethods die 1 lijntje code bevat met "LBX" of "TAX" of "TBX"

Monday 21 Oct 2019
3:43 pm

Tommy Hebb

[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:
[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:
11:08 am

Tommy Hebb

[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:

- Louter refactor

Options
    • -6
    • +8
    ./TAORMaatwerkProductCreator.cls.xml
11:04 am

Tommy Hebb

[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:

- Ongebruikte parameter van constructor verwijderd

Options
    • -1
    • +0
    ./TAORMaatwerkProductCreator.cls.xml
Friday 18 Oct 2019
5:16 pm

Tommy Hebb

[ICT-958] [rvWV] PM: Maatwerk: BOMBOL VHIP481: TAX updaten:

- Ook igv TAX wordt nu ProductData netjes doorgegeven om PRNr en Dossiercode te behouden van een bestaand product om het te verwijderen en met andere kenmerken opnieuw te maken

Options
    • -2
    • +2
    ./TAORMaatwerkProductCreator.cls.xml
  2. … 1 more file in changeset.
Thursday 03 Sep 2015
11:25 am

JoC

[UST1253] TAX LC: producten niet dubbel maken

- TAORMaatwerkProductCreator : werkt nu via Prod.KorttekstZoekIndex

Options
    • -7
    • +24
    ./TAORMaatwerkProductCreator.cls.xml
Tuesday 01 Sep 2015
11:49 am

JoC

[UST1253] TAX LC: producten niet dubbel maken

- TAORMaatwerkProductCreator

Options
    • -4
    • +37
    ./TAORMaatwerkProductCreator.cls.xml
8:38 am

PirminVDB

[UST1253] TAX LC: producten niet dubbel maken

- TAORMaatwerkProductCreator

Options
    • -0
    • +18
    ./TAORMaatwerkProductCreator.cls.xml
  2. … 3 more files in changeset.