CacheAdminA_trunk2010/DOM/PM/Maatwerk/Calc/Kost/impl

SST

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 14 Jul 20
Commits this week: 0
Total Lines of Code (LoC): 106
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
Wednesday 27 Oct 2021
1:42 pm

Peter Vercoutter

Wednesday 01 Sep 2021
5:05 pm

Wim Vermeulen

Als kritische reviewer ben ik verplicht om te melden dat bovenstaande If - Else {} nogal slordige code is. Wat je zou kunnen doen is iets in de aard van If (...) { Set ProcentueelKostItem = .....

Als kritische reviewer ben ik verplicht om te melden dat bovenstaande If - Else {} nogal slordige code is.
Wat je zou kunnen doen is iets in de aard van 

If (...) {
    Set ProcentueelKostItem = ..GeefProcentueelKostItemVoorLbxMetSpaceStep( Naam, DecimaalPercentage, TotaalKostItem) 
}
Else {
    Set ProcentueelKostItem = ..GeefProcentueelKostItemVoorLbxLade( Naam, DecimaalPercentage, TotaalKostItem) 
}
Do TotaalKostItem.VoegToe(ProcentueelkostItem)
}

+ de 2 private methods, waarin je dan proper het verschil kan zien tussen SSTProcentueelKostItem met de ExcludeFilter versus het gewone ProcentueelKostItem.

5:05 pm

Wim Vermeulen

Deze klasse mag rechstreeks afleiden van MaatwerkProductAlsHFKostItemBuilder (i.p.v. MaatwerkProductAlsHFKostItemBuilderSST) Zie ook opmerking bij klasse MaatwerkProductAlsHFKostItemBuilderSST

Deze klasse mag rechstreeks afleiden van MaatwerkProductAlsHFKostItemBuilder (i.p.v. MaatwerkProductAlsHFKostItemBuilderSST)
Zie ook opmerking bij klasse MaatwerkProductAlsHFKostItemBuilderSST

5:02 pm

Wim Vermeulen

5:00 pm

Wim Vermeulen

Als de 3 Test-methods alle 3 dezelfde SST-kenmerken gebruiken, dan zal de test meteen veel duidelijker worden wanneer je deze in een private method afzondert, en enkel de variatie als parameter doo...

Als de 3 Test-methods alle 3 dezelfde SST-kenmerken gebruiken, dan zal de test meteen veel duidelijker worden wanneer je deze in een private method afzondert,
en enkel de variatie als parameter doorgeeft. In dit geval is dat property Uitvoering en Bewegingstechnologie, denk.

Een andere insteek van Unittest :
Vul enkel die waarden in die noodzakelijk zijn. M.a.w. Hoogte, Breedte, Diepte, Kleur, stekkerType, ... zijn allemaal niet relevant voor deze UT.
Je zou dus kunnen opteren voor een GeefLegeSSTKenmerken() en nadien enkel Uitvoering en technologie in te vullen (or whatever is needed).

4:26 pm

Wim Vermeulen

Volgens mijn wiskundegevoel is dit gewoon gelijk aan   KostLade * ..Percentage of anders heb ik hier net een bug ontdekt? :-P

Volgens mijn wiskundegevoel is dit gewoon gelijk aan 

    KostLade * ..Percentage


of anders heb ik hier net een bug ontdekt?
:-P

4:18 pm

Wim Vermeulen

In .NET moeten we wel de "nieuwe standaarden" van VoegToeBuilders() volgen, kwestie van geen KostItemBuilders in de %OnNew toe te voegen. Hier in caché mag dat wel zo blijven :-P

In .NET moeten we wel de "nieuwe standaarden" van VoegToeBuilders() volgen, kwestie van geen KostItemBuilders in de %OnNew toe te voegen.
Hier in caché mag dat wel zo blijven :-P

3:22 pm

Wim Vermeulen

Deze klasse is een exacte kopie van "DOM.PM.Maatwerk.Calc.Kost.MaatwerkProductAlsHFKostItemBuilder" Ik zie geen enkele reden of specifieke dependency om een specifieke versie voor SST te maken. Dus...

Deze klasse is een exacte kopie van "DOM.PM.Maatwerk.Calc.Kost.MaatwerkProductAlsHFKostItemBuilder"
Ik zie geen enkele reden of specifieke dependency om een specifieke versie voor SST te maken. Dus wat mij betreft mag deze klasse verwijderd worden.
Of, zie ik iets over t hoofd?

Saturday 03 Jul 2021
10:45 am

Peter Vercoutter

[ICT-2417] [rvWV] Te grote afrondingsfouten bij kostenbuilders SpaceStep

- Geen prijswijziging in testfile zichtbaar omdat die prijs al de correcte was zonder afrondingsfouten

Options
    • -2
    • +2
    ./KostVolledigOfSokkelEnTabletv001.cls.xml
    • -2
    • +2
    ./KostEnkelTabletv001.cls.xml
  3. … 1 more file in changeset.
Wednesday 02 Sep 2020
4:05 pm

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep
[ICT-1634] Implementatie kostenbuilders SpaceStep
Thursday 13 Aug 2020
5:38 pm

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep

- De correctie van boekjaar 2018 moet enkel op de lade gebeuren en niet op de spacestep, voorgaand implementatie was niet 100% correct. De toeslag werd berekend op het bedrag waarvan de korting voor spacestep al gebeurd was

Options
    • -25
    • +0
    ./SpecKostv001.cls.xml
  2. … 5 more files in changeset.
Thursday 06 Aug 2020
10:26 am

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep

- De procentuele extra kost mag enkel op de lade en niet op de spacestep toegepast worden, daarom de korting bij spacestep

Options
    • -0
    • +25
    ./SpecKostv001.cls.xml
  2. … 1 more file in changeset.
Tuesday 14 Jul 2020
3:20 pm

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep

- Aanmaken kostenplaats (bestond al op cache01 omdat deze op die omgeving werd toegevoegd en niet via de normale manier) -> nodig om bepaalde andere info mbt de kostenplaats te genereren.

- SST KostenSpecInfo toevoegen in de databank

- SST kostenbuilder toevoegen aan LBX kostenbuilder

Options
    • -0
    • +37
    ./KostEnkelTabletv001.cls.xml
  2. ./SST
    • -0
    • +32
    ./SpecKostv001.cls.xml
    • -0
    • +37
    ./KostVolledigOfSokkelEnTabletv001.cls.xml
  5. … 9 more files in changeset.