Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3090] RVG: Verdere uitbreidingen BOMBOL

- Klasse uitbreiden voor RVG

    • -0
    • +6
    ./HalffabSpecKenmerkenAanpasser.cls.xml
  1. … 2 more files in changeset.
[ICT-2930] INK: BOMBOL opstellen

- Klasse uitbreiden voor INK

    • -0
    • +6
    ./HalffabSpecKenmerkenAanpasser.cls.xml
  1. … 2 more files in changeset.
[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:...
[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:...
[ICT-2765] [rvWV] PM: Maatwerk: TAX: Builders: KantenbandKleurVolgtLadeKleur ProjectSetting wegwerken:

- Implementatie onder ProjectSetting IsActiefKantenbandKleurVolgtLadeKleur staat nu per default actief en ProjectSetting alsook oude codepad werd weggewerkt

  1. … 6 more files in changeset.
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.

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.

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.

[ICT-1978] Sync van cache naar DotNet voor HalffabSpec

- Reviewopmerkingen: alles onder repositories brengen

    • -0
    • +71
    ./HalffabSpecInfoRepository.cls.xml
  1. … 17 more files in changeset.
[ICT-1828] Ingegeven kenmerken aanmaken voor MVX

- VerwerkMVXKenmerken implementeren in verschillende klasses waar een visitor kan passeren

- Test binnenladefilter uitbreiden voor MVX

- bijhorende fakes en mocks committen

    • -0
    • +6
    ./HalffabSpecKenmerkenAanpasser.cls.xml
  1. … 10 more files in changeset.
[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep
[ICT-1633] Implementatie Halffabbuilders SpaceStep

- Fakes en Mocks committen

    • -2
    • +7
    ./HalffabSpecKenmerkenAanpasser.cls.xml
  1. … 3 more files in changeset.
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug)
[UST4241] Nieuwe programma's afkanter: fix probleem in de aansturing van de programma’s
  1. … 5 more files in changeset.
[BUS708] TAX: kantenband onderaan moet buitenkleur volgen (zijkant en rug): KleurBepaler voor builders
    • -0
    • +27
    ./TAOR/KantenbandKleurBepaler.cls.xml
  1. … 4 more files in changeset.
[VRB639] HLX: (7) LBX probox pickinglijst - afdekkappen

- gegenereerde mocks and fakes

    • -2
    • +68
    ./LBX/Builder/HFBoxCoverVoorv000.cls.xml
    • -5
    • +63
    ./LBX/Builder/HFAfdekkapBinnenv000.cls.xml
    • -2
    • +68
    ./LBX/Builder/HFBoxCoverAchterv000.cls.xml
    • -2
    • +67
    ./LBX/Builder/HFAfdekkapBuitenv000.cls.xml
  1. … 4 more files in changeset.
Done!

Done!

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Is overgenomen van de defaulting van de (énige) implementatieklasse. De mock wordt van de interface gemaakt. Dat maakte dat die ander gedrag vertoonde dan de echte.

Die worden allebei in de caller van deze method al uitgesloten

Die worden allebei in de caller van deze method al uitgesloten

object LadeOnderdeel doorgeven, i.p.v. Rol As enu dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

object LadeOnderdeel doorgeven, i.p.v. Rol As enu
dan moet je voor Bodem niet opnieuw de "..MaakLosseComponent(Kenmerken) " opnieuw oproepen --> vermijd dubbel werk.

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux". Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de kla...

Dit is de algemene MaatwerkCalcAPI --> straf dat je hier de parameter gaat defaulten naar klant "Halux".
Ik zou verwachten dat je ergens een specifieke oproep hebt, waar je dan gecontroleerd de klant meegeeft.

Laat voorlopig maar staan, maar hou er wel rekening mee, dat dit soort defaultings "gedurfd" is

De bijhorende UT geeft aan dat *ID = "" geeft "" *ID = 0 geeft "" Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten? Want voor deze waarden is het eigenlijk zinloos ...

De bijhorende UT geeft aan dat

  • ID = "" geeft ""
  • ID = 0 geeft ""
    Mijn vraag : is dit wenselijk, of zou het beter zijn om een exceptie te smijten?
    Want voor deze waarden is het eigenlijk zinloos om de bepaler op te roepen.
Deze lijn zou ik binnen de Else zetten. Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.

Deze lijn zou ik binnen de Else zetten.
Nu wordt ze uitgevoerd, zelfs bij ID > 1332 en nadien wordt dan toch exception gesmeten. M.a.w. de AflfaKey() uitvoeren was niet nodig.