CacheAdminA_trunk2010/vhTestFiles/UnitTests/APPS.PM.Maatwerk.End2End.LBX

TestLBXGewoneLadeMetSSTPrijzen

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 4
Last Commit: 13 Aug 20
Commits this week: 0
Total Lines of Code (LoC): 315
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
Monday 08 Aug 2022
1:08 pm

Peter Vercoutter

9:31 am

Jelle Baele

9:11 am

Jelle Baele

Hier staat nog een regel commentaar dat waarschijnlijk weg mag

Hier staat nog een regel commentaar dat waarschijnlijk weg mag

9:09 am

Jelle Baele

Geen idee of dit van u is, maar hier staat een schrijffout http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Geen idee of dit van u is, maar hier staat een schrijffout

Friday 05 Aug 2022
12:47 pm

Peter Vercoutter

[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
[ICT-2896] MyLegrabox: Legrabox design zijkanten in configurator
Wednesday 27 Oct 2021
1:42 pm

Peter Vercoutter

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?

3:07 pm

Wim Vermeulen

Tuesday 31 Aug 2021
5:45 pm

Wim Vermeulen

Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() ) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builde...

Deze voorwaarde komt vele malen voor in de Builders (idem voor ..IsNietUitvoeringTablet() )
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

5:44 pm

Wim Vermeulen

Deze method komt vele malen voor in de Builders (idem voor =Servodrive) Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd ...

Deze method komt vele malen voor in de Builders (idem voor =Servodrive)
Kunnen we een propere en efficiente oplossing vinden waarbij die method niet telkens opnieuw in de builder moet gedefinieerd worden. Laten we dat samen eens bekijken.

5:21 pm

Wim Vermeulen

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt. Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en B...

Het "tussen-niveau" ...HalffabItemBuilderBasis is bij SST eigenlijk niet nodig. Dit was wel zo bij de "Matten" omdat je MatX,MatY, ... hebt.
Hier, bij SST, heb je maar 1 klasse, t.t.z. Builder en BuilderBasis vallen samen.
Maar wat mij betreft hoeft dit niet weggewerkt te worden in caché. Misschien wel bij het porten naar .NET

FYI: het was uiteindelijk de dubbele definitie van de method GeefGekoppeldIKType() die voor enige verwarring zorgde. Vandaar de review opm.

Wednesday 02 Sep 2020
4:05 pm

Peter Vercoutter

[ICT-1634] Implementatie kostenbuilders SpaceStep
[ICT-1634] Implementatie kostenbuilders SpaceStep
4:03 pm

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep