CacheAdminA_trunk2010/vhDeploy/Maatwerk

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): 697
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 04 Nov 2024
10:44 am

Peter Vercoutter

[ICT-5196] SST_SEL_StekkerType uitbreiden

- SST_SEL_StekkerType Uitbreiden met datum en klantbep velden

Options
    • -0
    • +47
    ./SEL/SSTxSSTxStekkerType.cls.xml
  2. … 1 more file in changeset.
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
Monday 03 Aug 2020
2:57 pm

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Toevoegen laatste halffabbuilder: Verpakking (type deksel+bodem)

- Dit gaat samen met het toevoegen van de SEL en BEW kenmerken

- In de doosberekenaar een workaround voorzien voor SST omdat deze geen IsBinnenlade kenmerk heeft

Options
    • -0
    • +157
    ./SSTxBEWxLengte.cls.xml
    • -0
    • +113
    ./SSTxSELxVerpakking.cls.xml
  3. … 11 more files in changeset.
Wednesday 29 Jul 2020
11:06 am

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Toevoegen selectiekenmerken voor PlaatmateriaalRug en zijkant voor de sokkel

- Halffabbuilders toevoegen voor Plaatmateriaal Rug, -zijkant, -tablet en alu Lat

- rechtzetting in halffabbuilder van SpaceStepOpvullijst voor aantal

Options
    • -0
    • +47
    ./SEL/SSTxSSTxPlaatmateriaalZijkantSokkel.cls.xml
    • -0
    • +47
    ./SEL/SSTxSSTxPlaatmateriaalRugSokkel.cls.xml
  3. … 10 more files in changeset.
Monday 20 Jul 2020
10:14 am

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Toevoegen halffabitem en selectiekenmerk Dwarsprofiel

- Casing aanpassing in Rolnaam

Options
    • -0
    • +47
    ./SEL/SSTxSSTxDwarsprofiel.cls.xml
  2. … 8 more files in changeset.
Wednesday 15 Jul 2020
2:57 pm

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Aanpassing om aangemaakte Deployklasse ivm selectiekenmerken te wissen + gecorrigeerde versie van deployklasse

- Aanpassing naam van rol zodat deze zou kloppen met de naam van het selectiekenmerk

Options
    • -34
    • +1
    ./SEL/SSTxSSTxSpaceStepStekkerType.cls.xml
    • -0
    • +47
    ./SEL/SSTxSSTxStekkerType.cls.xml
  3. … 12 more files in changeset.
Tuesday 14 Jul 2020
4:42 pm

Peter Vercoutter

[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Deployklasse om niet bewerkbare selectiekenmerken aan te maken

Options
    • -0
    • +63
    ./SEL/SSTxSSTxSpaceStepSet.cls.xml
    • -0
    • +47
    ./SEL/SSTxSSTxSpaceStepOpvullijst.cls.xml
    • -0
    • +36
    ./SEL/SSTxSELxBase.cls.xml
  4. ./SEL
    • -0
    • +47
    ./SEL/SSTxSSTxSpaceStepStekkerType.cls.xml
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
    • +32
    ./KostenPlaats.cls.xml
  2. ./SST
  3. … 11 more files in changeset.