CacheAdminA_trunk2010/DOM/VKP/impl

VerkoopAPI.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 31 May 22
Commits this week: 0
Total Lines of Code (LoC): 31
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
Tuesday 31 May 2022
4:23 pm

Peter Vercoutter

[ICT-2649] Multicalc voor TBX mogelijk als berekening via dotnet

- Reviewopmerkingen

Options
    • -4
    • +21
    ./VerkoopAPI.cls.xml
  2. … 2 more files in changeset.
4:21 pm

Peter Vercoutter

2:52 pm

Wim Vermeulen

Monday 30 May 2022
5:19 pm

Wim Vermeulen

Is het nodig om de rollen uit te schakelen voor bestaande ladetypes in caché? Voor mij speelt het geen rol. Misschien heeft PM dit gevraagd, dan is het voor mij oké.

Is het nodig om de rollen uit te schakelen voor bestaande ladetypes in caché?
Voor mij speelt het geen rol. Misschien heeft PM dit gevraagd, dan is het voor mij oké.

5:06 pm

Wim Vermeulen

Opgelet met de uitdrukking %ClassName(1) = "..." want dit zal falen op afgeleide klassen --> beter object.%Extends(...) alsook gebruik maken van #class(DOM.PM.Maatwerk.Calc.Common.impl.TBXKenmerken...

Opgelet met de uitdrukking %ClassName(1) = "..."
want dit zal falen op afgeleide klassen --> beter object.%Extends(...) alsook gebruik maken van #class(DOM.PM.Maatwerk.Calc.Common.impl.TBXKenmerken).%ClassName($$$True)

De method naam kan wel tot verwarring leiden, want wat met een LBX-kenmerk?
Ik zou het stukje "Tbx" weglaten en de implementatie lichtjes herschijven, met een lokale variabele en een lijn "If ... %Extends (TBXKenmerken) then ... "

4:45 pm

Wim Vermeulen

Domme opmerking, maar om de "issen" te onderscheiden zet ik meestal haakjes rond de conditie. LES 1 in caché : haakjes is altijd veiliger dan geen haakjes http://subversion02.vanhoecke.be/static/og...

Domme opmerking, maar om de "issen" te onderscheiden zet ik meestal haakjes rond de conditie. LES 1 in caché : haakjes is altijd veiliger dan geen haakjes

4:39 pm

Wim Vermeulen

LegacyPartijApi : Door deze implementatie wordt de method GeefLegacyPartijAPI() van de context minstens 5x opgeroepen. Dit valt nu eenmaal op door de aanpassing in deze method. Dus boyscout-gewijs ...

LegacyPartijApi :
Door deze implementatie wordt de method GeefLegacyPartijAPI() van de context minstens 5x opgeroepen. Dit valt nu eenmaal op door de aanpassing in deze method.
Dus boyscout-gewijs : graag de LegacyPartijApi aanmaken in de constructor, op de gekende wijze. Voor mij hoeft er nog geen $$$Inject() te staan, gewoon newen is oké.

4:28 pm

Wim Vermeulen

VHoss : A of B Misschien VerpakkingType eerst in een variabele steken. Dit maakt de "OF"-condities dan leesbaarder.

VHoss : A of B
Misschien VerpakkingType eerst in een variabele steken. Dit maakt de "OF"-condities dan leesbaarder.

4:26 pm

Wim Vermeulen

VerpakkingType Probox : X of Y

VerpakkingType Probox : X of Y

Wednesday 25 May 2022
3:30 pm

Peter Vercoutter

[ICT-2649] Multicalc voor TBX mogelijk als berekening via dotnet
[ICT-2649] Multicalc voor TBX mogelijk als berekening via dotnet
Tuesday 10 May 2022
2:34 pm

Peter Vercoutter

[ICT-2649] Multicalc voor TBX mogelijk als berekening via dotnet

- KlantID moet ook mogelijk zijn zonder prefix van 'K||' voor multicalc

Options
    • -5
    • +9
    ./VerkoopAPI.cls.xml
Wednesday 01 Jun 2016
7:17 pm

JoC

[UST1660] VHIP OS - Klanten

- Uitbreiding van de VerkoopAPI met een GeefKlantIdIterator

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