CacheAdminA_trunk2010/WSimpl/AX/CalculatedProduct

ProductUpdater.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 02 Dec 22
Commits this week: 0
Total Lines of Code (LoC): 166
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
Friday 02 Dec 2022
9:37 am

Tommy Hebb

9:24 am

Tommy Hebb

[ICT-3246] [rvWV] PM: Maatwerk: TAX: BOMBOL: Vaste producten BOM aanpassen maar prijs behouden:

- Verwerking reviewopmerkingen => toch 5 seconden lock pakken

Options
    • -2
    • +2
    ./ProductUpdater.cls.xml
9:06 am

Wim Vermeulen

9:06 am

Wim Vermeulen

Lock timeout toch op 5 sec laten staan, lijkt me veiliger.

Lock timeout toch op 5 sec laten staan, lijkt me veiliger.

Thursday 01 Dec 2022
5:41 pm

Tommy Hebb

[ICT-3246] [rvWV] PM: Maatwerk: TAX: BOMBOL: Vaste producten BOM aanpassen maar prijs behouden:

- Verwerking reviewopmerkingen

Options
    • -5
    • +7
    ./ProductUpdater.cls.xml
  2. … 1 more file in changeset.
12:08 pm

Wim Vermeulen

ProductServiceMock heeft enkel een "IndienMethodCall", dan geen Verifieer() nodig. Of omgekeerd : VerwachtMethodCall gebruiken

ProductServiceMock heeft enkel een "IndienMethodCall", dan geen Verifieer() nodig.
Of omgekeerd : VerwachtMethodCall gebruiken

11:43 am

Wim Vermeulen

Deze manier "Lock ... Else Do" is compacter (en beter) alternatief dan hierboven met de aparte "If $Test ... " Dit gewoon ter info http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wik...

Deze manier "Lock ... Else Do" is compacter (en beter) alternatief dan hierboven met de aparte "If $Test ... "

Dit gewoon ter info

11:39 am

Wim Vermeulen

De "If $Test ... " moet meteen na de Lock op ^KPR staan, anders heeft dat geen zin meer.

De "If $Test ... " moet meteen na de Lock op ^KPR staan, anders heeft dat geen zin meer.

Tuesday 29 Nov 2022
5:03 pm

Tommy Hebb

[ICT-3246] [rvWV] PM: Maatwerk: TAX: BOMBOL: Vaste producten BOM aanpassen maar prijs behouden:
[ICT-3246] [rvWV] PM: Maatwerk: TAX: BOMBOL: Vaste producten BOM aanpassen maar prijs behouden:
12:55 pm

Tommy Hebb

[ICT-3246] [rvWV] PM: Maatwerk: TAX: BOMBOL: Vaste producten BOM aanpassen maar prijs behouden:

- Wat commentjes verwijderd.

- ProductUpdater neemt nu ook een lock op de bouwsteen van het product

- UnitTest werkt nu met OrigineelProductMock en GeupdatetProductMock en met VerwachtMethodCall

Options
    • -1
    • +23
    ./ProductUpdater.cls.xml
  2. … 2 more files in changeset.
Wednesday 24 Aug 2022
8:26 pm

Jelle Baele

[ICT-3041] [rvPVR] Revego: Eerste opzet BomBol generatie:

- Ook checken op Revego

Options
    • -1
    • +1
    ./ProductUpdater.cls.xml
Thursday 14 Jul 2022
8:40 am

Peter Vercoutter

8:39 am

Peter Vercoutter

Je doet een vergelijking van de verschillen, dus je wilt dat er geen verschillen zijn, daarom empty list

Je doet een vergelijking van de verschillen, dus je wilt dat er geen verschillen zijn, daarom empty list

Wednesday 13 Jul 2022
4:42 pm

Jelle Baele

4:41 pm

Jelle Baele

Vanwaar assert empty list?

Vanwaar assert empty list?

4:40 pm

Jelle Baele

wlips mogen weg of het zal trakteren worden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

wlips mogen weg of het zal trakteren worden

Monday 11 Jul 2022
10:57 am

Peter Vercoutter

[ICT-2932] INK: Productcreatie INK
[ICT-2932] INK: Productcreatie INK
Monday 13 Jun 2022
8:42 am

Peter Vercoutter

[ICT-2932] INK: Productcreatie INK

- Uitbreidingen voor INK variant

- Correctie in econ naar apps converter

Options
    • -1
    • +1
    ./ProductUpdater.cls.xml
  2. … 2 more files in changeset.
Thursday 28 Oct 2021
9:44 am

Peter Vercoutter

Tuesday 14 Sep 2021
4:50 pm

Wim Vermeulen

De vhUtils standaard-kenmerken kiezen is soms nogal tricky: *liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras) *liefst zo ondubbelzinnig mogelijk : bvb. X en Y versc...

De vhUtils standaard-kenmerken kiezen is soms nogal tricky:

  • liefst zo standaard mogelijk (behalve bij speciallekes zoals LadeMetVeelExtras)
  • liefst zo ondubbelzinnig mogelijk : bvb. X en Y verschillend bij sifonlade --> check.
  • in de meeste gevallen zo werkelijkheidsgetrouw mogelijk


in deze method vind ik enige verwarring :

  • StandaardMat heeft LadeType=TAX, die heeft (nog) geen spoelbak.
  • Spoelbak voor LBX? of eerder voor TBX/MVX? --> wellicht zijn beide methods zinvol, en dan zou ik één method expliciet MatVoorLbxSpoelbakLade() noemen.
  • bij deze LBX moeten de uitsparingBreedtes X en Y dan best wel de juiste afmeting bevatten, want dat is ook een "vaste maat"


In eerdere gesprekken had ik al wel aangegeven dat een goede keuze van kenmerkwaarden, niet zo vanzelfsprekend is, en dat ik je hierbij zeker wel zou bijstaan. Bij deze, call me

4:42 pm

Wim Vermeulen

Friday 10 Sep 2021
5:22 pm

Wim Vermeulen

Ik heb nog wat uitleg over die "E" "Exceptions" in de Json-export, maar dat is wat veel om hier in de review-opmerking te zetten. Contacteer me maar eens. Eventueel zelfs interessant voor een meeti...

Ik heb nog wat uitleg over die "E" "Exceptions" in de Json-export, maar dat is wat veel om hier in de review-opmerking te zetten.
Contacteer me maar eens. Eventueel zelfs interessant voor een meeting met JBA en THB.

4:56 pm

Wim Vermeulen

copy-paste typo (2x) : #dim ... As MVX-kenmerken ??

copy-paste typo (2x) :
#dim ... As MVX-kenmerken ??

4:48 pm

Wim Vermeulen

copy-paste 2x zelfde

copy-paste 2x zelfde

4:47 pm

Wim Vermeulen

copy-paste 2x zelfde

copy-paste 2x zelfde

4:46 pm

Wim Vermeulen

Mijn eerste review-opmerking ging zijn : "In een AssertEquals-expressie zou geen $Select() mogen staan." Dat topic wil ik eventueel nog met jou bespreken. Door dit op te merken, viel m'n oog wel o...

Mijn eerste review-opmerking ging zijn : "In een AssertEquals-expressie zou geen $Select() mogen staan."
Dat topic wil ik eventueel nog met jou bespreken.

Door dit op te merken, viel m'n oog wel op een typo in bovenstaande lijn: RugBreedteX en in de $Select() staat BreedteM :-O

4:30 pm

Wim Vermeulen

Bij voorkeur Z1 en Z2 verschillend kiezen. idem voor X en Y

Bij voorkeur Z1 en Z2 verschillend kiezen.
idem voor X en Y

3:55 pm

Wim Vermeulen

Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.

Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.

3:55 pm

Wim Vermeulen

Deze IF legt een onnodige dependency, en heeft geen meerwaarde binnen de converter. Integendeel, bij uitbreiding voor LBX sifon of TBX sifon moet je hier iets toevoegen, terwijl de code binnen de I...

Deze IF legt een onnodige dependency, en heeft geen meerwaarde binnen de converter.
Integendeel, bij uitbreiding voor LBX sifon of TBX sifon moet je hier iets toevoegen, terwijl de code binnen de IF toch "universeel" is.

3:44 pm

Wim Vermeulen

Ook de MatKenmerken hebben property "NietMeeleveren", dus ik zou deze method wel laten oproepen, ook al is de impl dan enkel met de eerste en de laatste lijn : %New() en Quit. Oftewel rechtstreeks ...

Ook de MatKenmerken hebben property "NietMeeleveren", dus ik zou deze method wel laten oproepen, ook al is de impl dan enkel met de eerste en de laatste lijn : %New() en Quit.
Oftewel rechtstreeks in de method Converteer() , dan mag de commentaar hier volledig weg.