CacheAdminA_trunk2010/DOM/PM/Maatwerk/Calc/Kost

SamengesteldKostItem.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 5
Last Commit: 13 Aug 20
Commits this week: 0
Total Lines of Code (LoC): 253
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?

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
    • -0
    • +23
    ./SamengesteldKostItem.cls.xml
  2. … 5 more files in changeset.
Wednesday 31 Jul 2019
10:05 am

Tommy Hebb

Wednesday 24 Jul 2019
6:45 pm

Wim Vermeulen

12:07 pm

Wim Vermeulen

Is eigenlijk niet nodig om 4x hetzelfde te testen, namelijk dat bij DubbeleSifonlade de kosten "x 3" i.p.v. "x 2" is. Graag effe samen bespreken.

Is eigenlijk niet nodig om 4x hetzelfde te testen, namelijk dat bij DubbeleSifonlade de kosten "x 3" i.p.v. "x 2" is.
Graag effe samen bespreken.

11:26 am

Wim Vermeulen

Beter :  Quit ##super(FormatterInput,IngegevenKenmerken) _ ..GeefSpecialeLadeVoorHaluxTekst(IngegevenKenmerken) en de method dan laten staan op de basisklasse, en dan moet ge de method in de LBX-...

Beter : 

 Quit ##super(FormatterInput,IngegevenKenmerken) _ ..GeefSpecialeLadeVoorHaluxTekst(IngegevenKenmerken)


en de method dan laten staan op de basisklasse,
en dan moet ge de method in de LBX-klasse niet overriden.

9:59 am

Wim Vermeulen

Beter : Do ##super(IngegevenKenmerkenTypeAPI)

Beter : Do ##super(IngegevenKenmerkenTypeAPI)

Monday 22 Jul 2019
4:58 pm

Tommy Hebb

[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:
Friday 19 Jul 2019
4:02 pm

Tommy Hebb

[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:

- Verdubbeling ExtraKostVoorTaxSifonlade igv Dubbele Sifonlade

Options
    • -0
    • +3
    ./SamengesteldKostItem.cls.xml
  2. … 2 more files in changeset.
2:53 pm

Tommy Hebb

[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:

- KostItem => Bepalen en doorgeven of het over Dubbele Sifonlade gaat

- Indien Dubbele Sifonlade => FactorVoorIederKostItem = 3

Options
    • -1
    • +1
    ./SamengesteldKostItem.cls.xml
  2. … 2 more files in changeset.
Thursday 13 Sep 2018
10:18 am

Tommy Hebb

10:12 am

Wim Vermeulen

9:41 am

Tommy Hebb

Wednesday 12 Sep 2018
6:19 pm

Wim Vermeulen

Naam van het kostItem aanpassen : Set Naam = "TAX Sifonlade Extra kost + " _ Naam

Naam van het kostItem aanpassen :
Set Naam = "TAX Sifonlade Extra kost + " _ Naam

6:11 pm

Wim Vermeulen

4:43 pm

Tommy Hebb

[UST4007] TAX-SY: Kostprijs - Filter toepassen op TAX-rollen:
[UST4007] TAX-SY: Kostprijs - Filter toepassen op TAX-rollen:
4:15 pm

Tommy Hebb

[UST4007] TAX-SY: Kostprijs - Filter toepassen op TAX-rollen:

- Bijsturing implementatie GeefExtraKostVoorTaxSifonlade => SamengesteldKostItem volledig negeren

Options
    • -4
    • +6
    ./SamengesteldKostItem.cls.xml
  2. … 1 more file in changeset.
12:22 pm

Tommy Hebb

[UST4006] TAX-SY: Kostprijs - KostItemsFilter toepassen op basis van de Rolnamen:
[UST4006] TAX-SY: Kostprijs - KostItemsFilter toepassen op basis van de Rolnamen:
12:20 pm

Tommy Hebb

[UST4005] TAX-SY: Kostprijs TaxVerdubbelingEnProcentueelKostItem toevoegen:

- Naamgeving => KostItemsToExcludeFilter => KostItemNamenToExcludeFilter

Options
    • -3
    • +3
    ./SamengesteldKostItem.cls.xml
  2. … 2 more files in changeset.
11:22 am

Tommy Hebb

[UST4005] TAX-SY: Kostprijs TaxVerdubbelingEnProcentueelKostItem toevoegen:

- SamengesteldKostItem => Toepassing van filter => KostItems te negeren

- TaxVerdubbelingEnProcentueelKostItem => GeefKost => geeft nu volledige kost => normale + verdubbeling met aftrek van te negeren KostItems + percentage

- Bijhorende testjes

Options
    • -2
    • +4
    ./SamengesteldKostItem.cls.xml
  2. … 2 more files in changeset.
Tuesday 11 Sep 2018
3:21 pm

Tommy Hebb

2:33 pm

Wim Vermeulen