CacheAdminA_trunk2010/APPS/PM/Maatwerk

MaatwerkService.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 6
Last Commit: 07 Jul 23
Commits this week: 0
Total Lines of Code (LoC): 183
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 07 Jul 2023
3:32 pm

Tommy Hebb

[ICT-2278] [rvWV] PM: Maatwerk: TAX: End2End-testen aanmaken, ter validatie van de builder(s):

- IngegevenKenmerken worden nu meegegeven aan GeefHalffabItemsBoom op de MaatwerkDetailsConvertor, om op de juiste plek aan de Familie en Variant te geraken.

- Toevoeging DotNetHalffabContext, die de echte HalffabContext als super heeft en er een afgeslankte versie van is speciaal voor het DotNet-pad, om Familie en Variant ervan te kunnen gebruiken op de juiste plek.

- IHalffabItem => De dirty hack van WV, waarbij er default MVX gebruikt wordt voor ophaling PPS-kenmerken, vervangen door een exceptie, omdat er nu normaal gezien altijd een context aanwezig zal zijn en zo niet... exceptie!

- ClassnameFromJsonNaarDomConverter => KlassenaamHalffabItem wordt nu soms hardcoded doorgegeven vanuit DotNet op HalffabItemParsedFromJson. Zo ja, dat gebruiken!

- ClassnameFromJsonNaarDomConverter => Uitbreiding GeefVolledigeKlasseNaam om compatibel te zijn met TAOR.

- HalffabrikatenFromJsonNaarDomConverter => Ingebruikname van de nieuwe DotNetHalffabContext, die in de constructor van elk HalffabItem dat hier genewd wordt meegegeven wordt, opdat Familie en Variant proper doorgegeven worden.

- HalffabrikatenFromJsonNaarDomConverter => Toevoeging van specifieke TAOR-HalffabItems die eigen properties hebben die gemapt moeten worden: Verpakkingv001LosseComponentenZRFFront, Verpakkingv001LosseComponentenZRFRug,

Verpakkingv001LosseComponentenZRFZijkantLinks, Verpakkingv001LosseComponentenZRFZijkantRechts, RugPlaat, ZijkantPlaat, Verpakkingv001BulkPerOnderdeelGegroepeerdBodem en BodemDeksel.

- HalffabrikatenFromJsonNaarDomConverter => Een property van bovenstaand vermelde properties is AantallenPerVerpakking en die heeft wat extra implementatie nodig gehad (private methodje ZetCleaneVersieVanAantallenPerVerpakking).

Options
    • -1
    • +1
    ./MaatwerkService.cls.xml
  2. … 9 more files in changeset.
Thursday 27 May 2021
9:53 am

Peter Vercoutter

[ICT-1329] Multicalc uitbreiden met matmateriaal en kleur

- Probleemrapport van bombol via dotnet is al in de vorm van een string

Options
    • -6
    • +4
    ./MaatwerkService.cls.xml
9:04 am

Peter Vercoutter

[ICT-1329] Multicalc uitbreiden met matmateriaal en kleur

- Reviewopmerkingen

Options
    • -7
    • +5
    ./MaatwerkService.cls.xml
  2. … 2 more files in changeset.
8:56 am

Peter Vercoutter

Wednesday 26 May 2021
3:39 pm

Wim Vermeulen

3:36 pm

Wim Vermeulen

Ik denk dat deze lijn (en de 7 volgende lijnen) duidelijker en copy-paste-"veiliger" zullen worden, wanneer je van dit lijntje een private methodje maakt. Bvb. Do ZetGekoppeldeIKKenmerkWaarde(Gekop...

Ik denk dat deze lijn (en de 7 volgende lijnen) duidelijker en copy-paste-"veiliger" zullen worden, wanneer je van dit lijntje een private methodje maakt.
Bvb. Do ZetGekoppeldeIKKenmerkWaarde(GekoppeldeIK.GetAt(##class(DOM.PM.Maatwerk.Calc.Common.enu.GekoppeldIKType).MatY()), "Kleur", KenmerkWaarden.GetAt(i ) )

3:25 pm

Wim Vermeulen

Cosmetica = hoofdletters, spaties, accolades, open lijnen, en haakjes (dit laatste is soms een randgeval, wanneer het om complexe condities gaat) Aan de lijnen hierboven is niks anders gewijzigd. P...

Cosmetica = hoofdletters, spaties, accolades, open lijnen, en haakjes (dit laatste is soms een randgeval, wanneer het om complexe condities gaat)
Aan de lijnen hierboven is niks anders gewijzigd.
Probeer de cosmetica in een aparte commit te doen, dan kan de reviewer die commit skippen indien gewenst.

3:17 pm

Wim Vermeulen

Clean code principle : het aspect "Visitor" verbergen/wegwerken en de Accept-method niet rechtstreeks vanuit uw implementatie oproepen Hiervoor zijn volgende stappen nodig : *klassenaam wijzigen ...

Clean code principle : het aspect "Visitor" verbergen/wegwerken en de Accept-method niet rechtstreeks vanuit uw implementatie oproepen
Hiervoor zijn volgende stappen nodig :

  • klassenaam wijzigen van "HalffabrikatenVisitor" naar "HalffabItemNaarDtoConverter", idem voor KostItemNaarDtoConverter
  • de visitor/converter heeft 1 public method, die de .Accept() implementeert en het resultaat rechtstreeks teruggeeft.
    In dit geval wordt dit dan logischerwijs "ConverteerNaar...Dto" (of zoiets) 
    Method ConverteerNaarHalffabItemDto(DomHallfabItem As DOM.PM.Maatwerk.Calc.HF.IHalffabItem) As DOM.PM.Maatwerk.Calc.HF.dto.HalffabItem
{
	Do DomHallfabItem.Accept(this)
	Quit ..HalffabItem.HalffabItems.GetAt(1)
}

en : 
	Set MaatwerkDetails.Halffabrikaten = HalffabItemNaarDtoConverter.ConverteerNaarHalffabItemDto(HalffabItemsBoom.Wortel)
  • de method GeefHalffabItem() mag dan weg


Zowel uw oproepende code als uw visitor zullen hierdoor properder en duidelijker worden, en de bijhorende UT kan zeer eenvoudig mee aangepast worden.

N.B. ik weet dat de visitor reeds bestond en in het gedeelte daaronder reeds werd gebruikt, dus deze review-opmerking was in feite aan uw voorganger bestemd :-P

2:42 pm

Wim Vermeulen

$$$HasLength() is alleen zinvol op een %String. "Problemen" is een object --> $IsObject() en/of List.Count() gebruiken

$$$HasLength() is alleen zinvol op een %String.
"Problemen" is een object --> $IsObject() en/of List.Count() gebruiken

Tuesday 18 May 2021
11:33 am

Peter Vercoutter

[ICT-1329] Multicalc uitbreiden met matmateriaal en kleur
[ICT-1329] Multicalc uitbreiden met matmateriaal en kleur
10:38 am

Peter Vercoutter

[ICT-1329] Multicalc uitbreiden met matmateriaal en kleur

- Matten mogelijk maken in de multicalc

- Er kwam nog een probleempje aan het licht in de GeefMaatwerkdetails, bij mvx was het resultaat nog anders dan de andere lades, staat nu in orde

Options
    • -9
    • +14
    ./MaatwerkService.cls.xml
  2. … 1 more file in changeset.
Wednesday 12 May 2021
8:26 am

Peter Vercoutter

[ICT-2233] MVX - berekenen via Multicalc excel

- Reviewopmerking

Options
    • -1
    • +1
    ./MaatwerkService.cls.xml
Monday 03 May 2021
11:19 am

Peter Vercoutter

[ICT-2233] MVX - berekenen via Multicalc excel

- codeChange die de BerekenPrijs nu mogelijk maken zonder foutmelding

Options
    • -13
    • +15
    ./MaatwerkService.cls.xml
  2. … 1 more file in changeset.
Wednesday 28 Apr 2021
1:30 pm

Peter Vercoutter

[ICT-2233] MVX - berekenen via Multicalc excel

- De berekenPrijs van multicalc moest nog geredirect worden naar .NET indien nodig

Options
    • -30
    • +45
    ./MaatwerkService.cls.xml
Wednesday 19 Jun 2019
10:30 am

Wim Vermeulen

10:03 am

Tommy Hebb

Tuesday 18 Jun 2019
3:06 pm

Wim Vermeulen

[UW775] vHintra: LBX S-geleider probleem BerekenDatum via MaatwerkViewer
[UW775] vHintra: LBX S-geleider probleem BerekenDatum via MaatwerkViewer
2:49 pm

Wim Vermeulen

[UW775] vHintra: LBX S-geleider probleem BerekenDatum via MaatwerkViewer

- globale variabele BerekeningsSpecificatie i.p.v. BerekenDatum doorgeven; method GeefMaatwerkDetailsVoorDomeinKenmerken()

Options
    • -3
    • +3
    ./MaatwerkService.cls.xml
2:00 pm

Wim Vermeulen

[UW775] vHintra: LBX S-geleider probleem BerekenDatum via MaatwerkViewer

- globale variabele voor BerekenDatum opzetten (en Kill achteraf) in method GeefMaatwerkDetailsVoorDomeinKenmerken()

Aangezien er niet meteen een propere manier is om deze waarde door te geven, zal deze Quick&Dirty via een %-variabele worden doorgegeven.

Alternatieve oplossing: (properder, maar meer werk) de BerekenDatum doorgeven via een property op de MaatwerkCalcApi, deze kan opgevraagd worden via de DOM.DomeinContext.

Options
    • -0
    • +3
    ./MaatwerkService.cls.xml
Thursday 28 Sep 2017
7:17 am

Jo Claes

Monday 25 Sep 2017
11:51 am

Wim Vermeulen

  • More
  • CR-548
  • finished reviewing
Wednesday 20 Sep 2017
3:14 pm

Wim Vermeulen

De exacte datum verifieren met PM http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif vrijdag = 29/09 zatedag = 30/09 zondag = 01/10

De exacte datum verifieren met PM
vrijdag = 29/09
zatedag = 30/09
zondag = 01/10

Tuesday 19 Sep 2017
2:39 pm

Jo Claes

[UST2819] - Toekomstige prijzen LBX in schaduw
[UST2819] - Toekomstige prijzen LBX in schaduw
1:20 pm

JoC

[UST2819] - Toekomstige prijzen LBX in schaduw

- MaatwerkService voorzien dat deze datum die via UI kan doorgegeven worden verder gebruikt

Options
    • -6
    • +6
    ./MaatwerkService.cls.xml
  2. … 2 more files in changeset.
Tuesday 24 Nov 2015
4:42 pm

MartijnV

Refactor/cleanup
Options
    • -55
    • +84
    ./MaatwerkService.cls.xml
Wednesday 01 Jul 2015
8:58 am

JoC

[UST1170] TAX BOG: Kartonverbruik

- Aantal doorgeven in de maatwerk service

Options
    • -6
    • +6
    ./MaatwerkService.cls.xml
  2. … 7 more files in changeset.