Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2978] Oude kleur TS-M bij LBX uitschakelen
[ICT-2978] Oude kleur TS-M bij LBX uitschakelen
[ICT-3470] Opkuisen code LBX LadeKleur PS-M en BodemKleur LG
[ICT-3470] Opkuisen code LBX LadeKleur PS-M en BodemKleur LG
[ICT-3470] Opkuisen code LBX LadeKleur PS-M en BodemKleur LG

    • -1
    • +0
    ./OCCKenmerkenConverter/BlumNaarEconKenmerkenConverter.cls.xml
  1. … 19 more files in changeset.
[ICT-2978] Oude kleur TS-M bij LBX uitschakelen

    • -1
    • +0
    ./OCCKenmerkenConverter/BlumNaarEconKenmerkenConverter.cls.xml
  1. … 4 more files in changeset.
[ICT-3244] [rvWV] OCC Laatste openstaande issues:

- FrontStabilisatorAantal => CorpusBreedte is irrelevant => Ook Econ houdt er geen rekening mee en laat die bepaling over aan de calculator (builders)

  1. … 1 more file in changeset.
Opdeling betreffende InterneBreedte eruit gehaald, wegens irrelevant (zie uitleg bij UnitTests).

Opdeling betreffende InterneBreedte eruit gehaald, wegens irrelevant (zie uitleg bij UnitTests).

Na overleg en herbekijken issue samen met Wim: *Bovenstaande test mag weg, want InterneBreedte is irrelevant hierbij. *Onderstaande test is ok, mits InterneBreedte uit de vergelijking gehaald wor...

Na overleg en herbekijken issue samen met Wim:

  • Bovenstaande test mag weg, want InterneBreedte is irrelevant hierbij.
  • Onderstaande test is ok, mits InterneBreedte uit de vergelijking gehaald wordt, wegens irrelevant.


Reden: Ook Econ laat de afhandeling omtrent InterneBreedte hierbij over aan de calculator. De keuze voor "Auto" komt dus neer op: "laat het maar aan het onderliggend systeem over".

[ICT-2995] [rvWV] OCC: Default VerpakkingType Probox ipv V1:

- VerpakkingType is nu gedefaulteerd op Probox i.p.v. V1

  1. … 1 more file in changeset.
[ICT-2851] [rvWV] EC: OCC: Aanpassingen vanwege BLUM-decisionlist-veranderingen:

- Aanpassingen ivm pull-out type

    • -5
    • +3
    ./OCCKenmerkenConverter/BlumNaarEconKenmerkenConverter.cls.xml
  1. … 2 more files in changeset.
Deze lijn is overbodig. (idem)

Deze lijn is overbodig. (idem)

Deze lijn is overbodig. t.t.z. het is niet fout, maar het heeft totaal geen meerwaarde. Integendeel: mensen gaan denken "wat is hier speciaal? of wat is er veranderd?" antwoord : "Niks" :-P

Deze lijn is overbodig.
t.t.z. het is niet fout, maar het heeft totaal geen meerwaarde. Integendeel: mensen gaan denken "wat is hier speciaal? of wat is er veranderd?" antwoord : "Niks" :-P

[ICT-1451] [rvWV] OCC JSON met amount implementeren:
[ICT-1451] [rvWV] OCC JSON met amount implementeren:
[ICT-1451] [rvWV] OCC JSON met amount implementeren:

- Indien BlumKenmerk Amount meegestuurd werd => geen vertaling nodig

- TestResponse uitgebreid met Data.Lijn.Aantal en gedefault op 1

- Werkelijke response => default Aantal 1 enkel indien BlumKenmerk Amount niet meegestuurd werd of leeg was => anders aantal overnemen

  1. … 1 more file in changeset.
[ICT-1171] [rvWV] EC: OCC: VOORGEMONTEERDE LADES: Echte implementatie van vertaling Blum-kenmerken naar VH-kenmerken:

- Verwerking review-opmerkingen, nml...

- Enum TBXZijwandverhoginginschuifelementkleur uitgebreid met ZonderVerhoging ZV => is in Econ wel een mogelijke waarde maar enum in Cache was (nog niet) uitgebreid.

- Typefoutjes corrigeren

- In enkele testen input uitgebreid met kenmerk IsBinnenlade voor meer consistentie met andere kenmerken in die UT.

- Verduidelijkingen via comment of in UT-naam waar nodig

    • -1
    • +1
    ./OCCKenmerkenConverter/BlumNaarEconKenmerkenConverter.cls.xml
  1. … 1 more file in changeset.
Is omwille van historisch gefoefel. In Cache bestaat ZV (ZonderVerhoging) niet als waarde op ZijwandVerhogingInschuifelementKleur, maar in Econ kan die waarde wèl. ECON.PM.Maatwerk.enu.TBXZijwandVe...

Is omwille van historisch gefoefel. In Cache bestaat ZV (ZonderVerhoging) niet als waarde op ZijwandVerhogingInschuifelementKleur, maar in Econ kan die waarde wèl.
ECON.PM.Maatwerk.enu.TBXZijwandVerhogingInschuifelementKleur uitgebreid met ZonderVerhoging = ZV

ICT-1171: [rvWV] EC: OCC: VOORGEMONTEERDE LADES: Echte implementatie van vertaling Blum-kenmerken naar VH-kenmerken
ICT-1171: [rvWV] EC: OCC: VOORGEMONTEERDE LADES: Echte implementatie van vertaling Blum-kenmerken naar VH-kenmerken
Nope. Crappy mapping van Blum, waardoor kenmerken mekaar bevuilen. In andere test een vette comment gezet ter verduidelijking.

Nope. Crappy mapping van Blum, waardoor kenmerken mekaar bevuilen. In andere test een vette comment gezet ter verduidelijking.

Mocht inderdaad wat duidelijker, maar heb voor een iets eenvoudigere aanpak gekozen, nml.: Method "Test: Het Request vraagt om een test-Response omdat het test-kenmerk erin zit"() { #dim VoegToeT...

Mocht inderdaad wat duidelijker, maar heb voor een iets eenvoudigere aanpak gekozen, nml.:

Method "Test: Het Request vraagt om een test-Response omdat het test-kenmerk erin zit"()
{
	#dim VoegToeTestRequestKenmerk As %Boolean = $$$True
	#dim OCCKenmerkenRequest As WS.Vhisie4.OCC.GeefNaarEconGeconverteerdeKenmerkenRequest = ..GeefRequestMetKenmerken(4, VoegToeTestRequestKenmerk)
	#dim Resultaat As WS.Vhisie4.OCC.GeefNaarEconGeconverteerdeKenmerkenResponse = ..OCCKenmerkenConverter.ConverteerBlumKenmerkenNaarEconKenmerken(OCCKenmerkenRequest)
Komt door crappy samenspel van kenmerken vanwege Blum. Verklarende comment toegevoegd!

Komt door crappy samenspel van kenmerken vanwege Blum. Verklarende comment toegevoegd!

Komt omdat IsBinnenlade false gedefault wordt toegevoegd indien het niet expliciet meegegeven werd. Omdat de testopzet inderdaad aangeeft dat het over een binnenlade gaat heb ik de expliciete IsBin...

Komt omdat IsBinnenlade false gedefault wordt toegevoegd indien het niet expliciet meegegeven werd.
Omdat de testopzet inderdaad aangeeft dat het over een binnenlade gaat heb ik de expliciete IsBinnenlade nu voorzien.
Idem voor volgende UnitTest.

Het ene is InbouwHoogte, het andere ZijkantHoogte

Het ene is InbouwHoogte, het andere ZijkantHoogte

Logica is voorzien per requirements opgesteld door de analist en daarbij moest in dit geval zijwandverhogingtype niet ingevuld worden. Haar testen zouden dat ook bevestigd hebben. Mogelijk wordt er...

Logica is voorzien per requirements opgesteld door de analist en daarbij moest in dit geval zijwandverhogingtype niet ingevuld worden. Haar testen zouden dat ook bevestigd hebben. Mogelijk wordt er gedefault in Econ hé.

Nope! Ene is pure, andere is free http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Nope! Ene is pure, andere is free

Ga het nu niet speciaal veranderen, want is wel correct en ik vind het zelf best leesbaar, maar zal in 't vervolg dan ook de andere manier gebruiken.

Ga het nu niet speciaal veranderen, want is wel correct en ik vind het zelf best leesbaar, maar zal in 't vervolg dan ook de andere manier gebruiken.