CacheAdminA_trunk2010/WSimpl/PM/Maatwerk/MaatwerkService/DotNet

KostItemFromJsonNaarDomConverter.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 30 Jun 21
Commits this week: 0
Total Lines of Code (LoC): 115
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 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.

1:19 pm

Wim Vermeulen

VH code conventions : VulAanMatKenmerken()

VH code conventions : VulAanMatKenmerken()

1:17 pm

Wim Vermeulen

Als het object MatKenmerken één en dezelfde oref blijft (dus geen %Clone ofzo) dan is het niet nodig om de SetAt() te doen, want de pointer in de GekoppeldeKenmerken verwjist nog steeds naar hetzel...

Als het object MatKenmerken één en dezelfde oref blijft (dus geen %Clone ofzo) dan is het niet nodig om de SetAt() te doen, want de pointer in de GekoppeldeKenmerken verwjist nog steeds naar hetzelfde object in memory.

1:06 pm

Wim Vermeulen

Een kleine bedenking: ik ben niet helemaal zeker of de method wel thuishoort in deze klasse, omdat die enkel geschikt is voor MatKenmerken. Voorlopig is dit oké, en functioneel zeker geen probleem...

Een kleine bedenking:
ik ben niet helemaal zeker of de method wel thuishoort in deze klasse, omdat die enkel geschikt is voor MatKenmerken.

Voorlopig is dit oké, en functioneel zeker geen probleem.
Ik wil niet verder uitweiden nu, over deze bedenking, dus enkel als review-opmerking, en om te vermijden dat we vanalles in deze klasse gaan steken dat niet voor alle IngegevenKenmerken van toepassing is.

12:48 pm

Wim Vermeulen

Er is geen IF nodig, want : if "true" or "false" --> altijd dus :-P

Er is geen IF nodig, want :
if "true" or "false" --> altijd dus
:-P

Friday 03 Sep 2021
8:58 am

Wim Vermeulen

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

8:57 am

Wim Vermeulen

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

Thursday 02 Sep 2021
1:04 pm

Peter Vercoutter

1:04 pm

Peter Vercoutter

[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator
[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator
Wednesday 30 Jun 2021
10:38 am

Peter Vercoutter

[ICT-2348] BOMBOL matten uitbreiden

- MaatwerkProductHalffabItem + kostItem implementeren

- Properties toevoegen voor matkenmerken zodat mapping goed verloopt, bewust niet gekozen om een ingegevenKenmerk object ervan te maken, dit zou de conversie complexer maken, zeker nu dat MaatwerkhalffabItem enkel nog maar bestaat bij matjes

Options
    • -0
    • +9
    ./KostItemFromJsonNaarDomConverter.cls.xml
  2. … 4 more files in changeset.
Thursday 24 Jun 2021
11:39 am

Peter Vercoutter

[ICT-2129] PM: Maatwerk: dev MERIVOBOX voor De Decker (MVX2/DEV)

- Exceptie gooien als een BOL builder in het respons zit van .NET met aantal 0. Dit zorgt voor fouten langs de kant van cache

Options
    • -2
    • +6
    ./KostItemFromJsonNaarDomConverter.cls.xml
Tuesday 15 Jun 2021
2:01 pm

Peter Vercoutter

[ICT-2296] Uitbreidingen in cache voor nieuwe mattenconfigurator

- Uitbreiding nodig om procentueelKostitem vanuit .NET te kunnen converten naar cache procentueelKostItem. .NET is op dat moment de master van het kostitem dus cache moet niet opnieuw iets gaan berekenen

Options
    • -7
    • +4
    ./KostItemFromJsonNaarDomConverter.cls.xml
  2. … 6 more files in changeset.
Wednesday 03 Mar 2021
2:28 pm

Wim Vermeulen

1:49 pm

Peter Vercoutter

Monday 01 Mar 2021
3:32 pm

Wim Vermeulen

ICT-1782: [rvPVR] MFW: Errorhandling en LogExceptie in Caché
ICT-1782: [rvPVR] MFW: Errorhandling en LogExceptie in Caché
Monday 18 Jan 2021
2:23 pm

Wim Vermeulen

[ICT-1782][rv?] MFW: Errorhandling en LogExceptie in Caché

- ValideerHalffabrikaten en ValideerKostItems : meteen een leesbare exceptie smijten als x geen object is. Anders crasht een "willekeurige" eerstvolgende lijn met een invalid oref. Dat is meestal een obscure melding.

Options
    • -1
    • +13
    ./KostItemFromJsonNaarDomConverter.cls.xml
  2. … 1 more file in changeset.
Thursday 17 Dec 2020
8:23 am

Peter Vercoutter

[ICT-1975] BOM BOL omleiden naar .net voor MVX

- In de convertor moet de kost gedeeld worden door aantal, want .net heeft de kost al * het aantal gedaan, op deze manier komen PPL en kost weer overeen

Options
    • -3
    • +4
    ./KostItemFromJsonNaarDomConverter.cls.xml
Monday 07 Dec 2020
1:20 pm

Peter Vercoutter

[ICT-1975] BOM BOL omleiden naar .net voor MVX

- halffabitemsboomonherwerkt voor mvx voorlopig invullen met de echte halffabitemsboom, hiervoor is nog een vergadering nodig voor een definitieve oplossing

- HalffabRepository uitbreiden om 1 halffab op te kunnen vragen

- de properties van ieder halffabitem moeten expliciet worden ingevuld na het converteren van JSON naar DOM

- casten van string naar numerieke datatype voor bij kostitems en halffabs omdat cache dit wel zo verwacht op sommige plaatsen

- logging toevoegen aan MaatwerkService.DotNet.GeefMaatwerkDetails + klantnummer correct doorsturen (niet rechtstreeks het AX klantnummer) + VersieNr momenteel defaulteren op 0

Options
    • -9
    • +10
    ./KostItemFromJsonNaarDomConverter.cls.xml
  2. … 4 more files in changeset.
Monday 30 Nov 2020
4:05 pm

Peter Vercoutter

[ICT-1975] BOM BOL omleiden naar .net voor MVX

- KostenplaatsKostItem toevoegen aan KostItemFromJsonNaarDomConverter

Options
    • -0
    • +3
    ./KostItemFromJsonNaarDomConverter.cls.xml