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
    ./HF/impl/HalffabSpecKenmerkenAanpasser.cls.xml
  1. … 2 more files in changeset.
[ICT-2930] INK: BOMBOL opstellen

- Klasse uitbreiden voor INK

    • -0
    • +6
    ./HF/impl/HalffabSpecKenmerkenAanpasser.cls.xml
  1. … 2 more files in changeset.
[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken
[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken
[ICT-2902] [rvJBA] BOMBOL .NET met schaduwprijzen mogelijk maken

- Request uitbreiden met 'IsSchaduwBerekening' parameter

- Kostenplaatsen en productInfo tabellen + sync uitbreiden met schaduwprijzen

    • -0
    • +39
    ./Kost/impl/KostenPlaatsWaardeTabel.cls.xml
    • -0
    • +18
    ./Kost/impl/KostenPlaatsWaardeInfo.cls.xml
  1. … 14 more files in changeset.
[ICT-2359] TBX BOMBOL builders porten naar .NET: Commit voor PVR

- Voor oude tbx producten bestaat geen standaard BerekeningSpecificatie en moet die worden afgehaald van de GAData

- Oude variant 'TBX_Mengvorm' hervormen naar variant 'TBX_Antaro'

  1. … 3 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

    • -7
    • +0
    ./HF/impl/TAOR/KantenbandKleurBepaler.cls.xml
  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.

Goed opgemerkt, Jelle. Tom zal wel meteen zien dat de .ProductNummer binnen de Tech.Equals() moet staan. @ iedereen (vooral de jonkies) : Bovenal is de vraag : is er geen UT die dit aan het licht ...

Goed opgemerkt, Jelle.
Tom zal wel meteen zien dat de .ProductNummer binnen de Tech.Equals() moet staan.

@ iedereen (vooral de jonkies) :
Bovenal is de vraag : is er geen UT die dit aan het licht brengt?
Zal zelf al deels antwoorden :
Deze klasse BL.MB.UP.Receptie.cls bevat stokoude implementatie, en er is nauwelijks sprake van "scheiding van dependencies".
Dit laatste maakt het dan ook zeer moeilijk om UT hiervoor te schrijven.
@Tom : misschien toch een systeemke bedenken om de Dna-complexiteit af te zonderen in een helper-klasse, zodat die alsnog kan ge-UT-ed worden :
bvb. DnaCodeSoortBepaler .IsVoorKad() .IsVoorVhoss(), .IsVoorSpaceStep()
( *) woordkeuze "...Soort..." kan vast beter!

Geen Asserts na een "VerwachteExceptie". Deze worden toch niet uitgevoerd, kunnen daarom zelfs foute waarden tonen en niet falen. Dus sowieso de asserts hier weghalen. @iedereen : deze opmerking a...

Geen Asserts na een "VerwachteExceptie".
Deze worden toch niet uitgevoerd, kunnen daarom zelfs foute waarden tonen en niet falen. Dus sowieso de asserts hier weghalen.

@iedereen : deze opmerking altijd maken als reviewer, wanneer iemand hiertegen zondigt. En als developer hier best zelf proberen op te letten :-P

Vind dit persoonlijk wat moeilijk om allemaal te begrijpen wat er juist gebeurt met de globals

Vind dit persoonlijk wat moeilijk om allemaal te begrijpen wat er juist gebeurt met de globals

Waarvoor wordt DnaCode4 gebruikt?

Waarvoor wordt DnaCode4 gebruikt?

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest" #dim IngegevenKenmerken As DOM.PM.Maatwerk.Calc.Common.impl.LBXKenmerken = ##class(vhTest.Dummy.DOM.PM.Maatwerk.Calc.Common.impl.LBXKenm...

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

#dim IngegevenKenmerken As DOM.PM.Maatwerk.Calc.Common.impl.LBXKenmerken = ##class(vhTest.Dummy.DOM.PM.Maatwerk.Calc.Common.impl.LBXKenmerken).DummyLade()
Heb hier ook over zitten peinzen. Probleem is dat voor dit kaartje heel het systeem uitging van 1 DnaCode per Product. Nu maken we 2 DnaCodes voor 1 Product. Was niet van zin om op dit moment dat p...

Heb hier ook over zitten peinzen. Probleem is dat voor dit kaartje heel het systeem uitging van 1 DnaCode per Product. Nu maken we 2 DnaCodes voor 1 Product. Was niet van zin om op dit moment dat principe overal te gaan overhoop halen en daarom heb ik het hier maar in deze method gestoken. Op zich blijven de oproepers van MaakDnaCode-methods dan in het ongewisse.

Staan al bovenaan gedefinieerd in 'OneBeforeOneTest'         Do ..ProductTypeApiStub.IndienMethodCall("IsLegraboxProduct", 1000).DanReturn($$$False)

Staan al bovenaan gedefinieerd in 'OneBeforeOneTest'

        Do ..ProductTypeApiStub.IndienMethodCall("IsLegraboxProduct", 1000).DanReturn($$$False)
heb dat in volgende kaartje aangepast. dingen liepen tegelijk (yesyes, kleine kaartjes en rap committen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif )

heb dat in volgende kaartje aangepast. dingen liepen tegelijk (yesyes, kleine kaartjes en rap committen )

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

Deze staat ook al bovenaan gedefinieerd in "OnBeforeOneTest"

Productnummer wordt vergeleken met boolean?

Productnummer wordt vergeleken met boolean?

Toevoegen dat de method een 'boolean' teruggeeft

Toevoegen dat de method een 'boolean' teruggeeft