Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3531] Proboxinhoudetiket - Interne breedte vermelden

- InterneBreedte wordt nu wel opgevuld door uitbreiding convertor

    • -0
    • +1
    ./BulkPerOnderdeelGegroepeerdVandaag.txt
  1. … 96 more files in changeset.
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte
[ICT-3198] TAX - Dubbele dwarsverdeling - Caché gedeelte

- DwarsverdelingType is toegevoegd aan een hoop vhTestFiles, part 2

    • -0
    • +1
    ./BulkPerOnderdeelGegroepeerdVandaag.txt
  1. … 96 more files in changeset.
[ICT-3120] [rvWV] PM: TAX: Front HoogteVerstelling MoventoKoppeling: release alle klanten:

- StringUtils Equals gebruikt want haakskes haakskes haakskes

- HeeftFrontHoogteVerstelling op een product geeft nu voor ALLE KoppelingTypes true terug, waardoor alles in de productieaansturing automatisch met HoogteVerstelling gaat zijn.

=> Eerdere poging tot (her)bereken via DOM.PM.Maatwerk.Calc.HF.impl.HalffabSpecKenmerkenAanpasser lukt niet, omdat de bereken voor productie het product niet wijzigt.

=> Op de nieuwe HalffabItemsboom staat dan wel de MoventoKoppeling, maar uiteindelijk wordt in de productieaansturing toch gekeken naar HeeftFrontHoogteVerstelling op het product zelf => dus dan geen witte rook

=> Nieuwe testbestanden

    • -35
    • +59
    ./BulkPerOnderdeelGegroepeerdVandaag.txt
  1. … 73 more files in changeset.
Ook een klein stukje in Econ! (zie svn)

Ook een klein stukje in Econ! (zie svn)

[ICT-1693] TAXEB1 - PM: Maatwerk: BevatGeleiderBevestigingSchroeven = True indien Alpnach:
[ICT-1693] TAXEB1 - PM: Maatwerk: BevatGeleiderBevestigingSchroeven = True indien Alpnach:
Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam v...

Ge ziet niets over 't hoofd :-D Ge hebt helemaal gelijk! Is een 1-op-1-property, dus mag automatisch geconverteerd worden. In tegenstelling tot DwarsverdelingAantal, want daar is de property-naam verschillend (cq. AantalDwarsverdelingen in APPS).

Aangepast naar "Dwarsverdelingen te picken"

Aangepast naar "Dwarsverdelingen te picken"

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logi...

Als opmerking in de kantlijn (dus je moet niks aanpassen) : hier moeten we eigenlijk gebruik maken van het berekende HalffabItem en daarvan de (bewerkte) lengte opvragen. Nu staat er duplicate logica.
Maar ik geloof best, zonder het na te kijken, dat in deze klasse nog code staat die vertrekt van de ingegevenKenmerken i.p.v. halffabItem.

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties() hiervoor is geen "uitzondering nodig". Tijdens de uitbreidingsfase en review lijkt het me handig om te zien ...

DwarsverdelingKleur zit volgens mij "impliciet" in de ConverteerGegenereerdeProperties()
hiervoor is geen "uitzondering nodig".
Tijdens de uitbreidingsfase en review lijkt het me handig om te zien dat beide properties zijn toegevoegd. Maar eenmaal in productie (en ondersteund door automatische testen) kan dit het standaard pad volgen.
Of zie ik iets over t hoofd?

Tekst correct? Zonder kar --> geen vakken, right?

Tekst correct?
Zonder kar --> geen vakken, right?

[ICT-1693][rvWV] TAXEB1 - PM: Maatwerk: BevatGeleiderBevestigingSchroeven = True indien Alpnach:

- Nieuwe testbestanden => BevatGeleiderBevestigingSchroeven staat nu altijd op false via de kenmerken, tenzij in Econ bepaald is dat het voor Alpnach via SFS is, waarbij het op true gezet wordt

    • -0
    • +1
    ./BulkPerOnderdeelGegroepeerdVandaag.txt
  1. … 87 more files in changeset.
Staat al in productie en is voordien al uitgebreid end-to-end getest.

Staat al in productie en is voordien al uitgebreid end-to-end getest.

[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:
OEPS !!! Great call!!! Thx!!!

OEPS !!! Great call!!! Thx!!!

Het oude productId komt nog op 2 andere plaatsen voor in caché studio. Deze best ook aanpassen. Searching for 1118152 in *.cls,*.mac,*.inc APPS.PM.Maatwerk.MaatwerkModel.Modellen.TaorboxModel.Rolle...

Het oude productId komt nog op 2 andere plaatsen voor in caché studio. Deze best ook aanpassen.

Searching for 1118152 in *.cls,*.mac,*.inc
APPS.PM.Maatwerk.MaatwerkModel.Modellen.TaorboxModel.Rollen.ZijkantBevestigingVeer.cls(GeefRol+2): ##class(APPS.PM.Maatwerk.MaatwerkModel.dto.SpecifiekProductBuilderType).%New(1118152)
DOM.PM.Maatwerk.Syncs.DotNet.MaatwerkCalculator.impl.ProductInfo.cls(GeefLijstVanProductIdVanNietSelectieKenmerken+102): Do ProductIds.Insert(1118152) 
Found 2 occurrence/s in 2 file/s.
[ICT-2728] [rvWV] TAX configurator : uitbreiden met een dwarsverdeling:

- Uitbreiding nodige dto-kes en Utils-klassen en convertoren en bijhorende UnitTests met AantalDwarsverdelingen en DwarsverdelingKleur

- Uitsluiting DwarsverdelingKleur in KenmerkDomeinPostProcessor om VHintra-configurator niet kapot te maken

=> Nieuwe testbestanden

    • -0
    • +2
    ./BulkPerOnderdeelGegroepeerdVandaag.txt
  1. … 123 more files in changeset.
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:
[ICT-3034] [rvWV] Andere veer Vällinge koppeling TA'OR:

- Aanpassing van het productnummer van de veer (nieuwe is 3433118, oude was 1118152).

- Aanpassing van het AantalVeren, zowel voor bevestiging van de rug (volgens LadeHoogte) als van de bodem (volgens LadeDiepte).

=> Nieuwe testbestanden met lichte volgens Kevin aanvaardbare prijsverschillen

    • -10
    • +10
    ./BulkPerOnderdeelGegroepeerdVandaag.txt
  1. … 131 more files in changeset.
FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt http://sub...

FYI: Ik vind het zeker oké dat je hier de ##super hebt weggelaten. (de gegenereerde code die daar stond was eigenlijk zelfs niet correct : LadeHoogte is enum en dus niet in MM uitgedrukt )
De impl in deze afgeleide klasse is wel correct.

De implementatie voor "Lengte" hieronder zou ik wel in een private method steken, analoog aan GeefFrontPlaatHoogte()
Zo blijft deze method simpel leesbaar en overzichtelijk. En dan is de scope ook duidelijk, nl. dat Kleur W7 enkel impact heeft op de kenmerk "Lengte, en niet op de "Breedte" van het Front.