IngegevenKenmerkenInfoDiverseBepaler.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1546] PM: Maatwerk: Space Step: Ingegeven Kenmerken Caché-kant inbouwen

- Method VerwerkSSTKenmerken toevoegen op verschillende relevante klassen

    • -0
    • +4
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml
  1. … 16 more files in changeset.
zal inderdaad duidelijker zijn.

zal inderdaad duidelijker zijn.

Wat denk je van een Private method AssertSmalleLade met params (Breedte As %Numeric, VerwachteWaarde As %Boolean) ? Alle code kan daarheen verplaatst worden

Wat denk je van een Private method AssertSmalleLade met params (Breedte As %Numeric, VerwachteWaarde As %Boolean) ?
Alle code kan daarheen verplaatst worden

APPS.PM.Maatwerk - was toegevoegd, maar onnodig, enkel DOM laag heeft deze nodig. Dus wijzigingen in APPS laag zijn ongedaan gemaakt. Staan dus wel in deze review.

APPS.PM.Maatwerk - was toegevoegd, maar onnodig, enkel DOM laag heeft deze nodig.
Dus wijzigingen in APPS laag zijn ongedaan gemaakt. Staan dus wel in deze review.

ICT-816: Vhintra Configurator - gebruiker attent maken op ingave van een smalle lade. (met mogelijkse meerkost)
ICT-816: Vhintra Configurator - gebruiker attent maken op ingave van een smalle lade. (met mogelijkse meerkost)
[ICT-816] Vhintra Configurator - gebruiker attent maken op ingave van een smalle lade. (met mogelijkse meerkost)

- Smalle lade vermelding in info diverse

    • -0
    • +11
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml
  1. … 1 more file in changeset.
Is eigenlijk niet nodig om 4x hetzelfde te testen, namelijk dat bij DubbeleSifonlade de kosten "x 3" i.p.v. "x 2" is. Graag effe samen bespreken.

Is eigenlijk niet nodig om 4x hetzelfde te testen, namelijk dat bij DubbeleSifonlade de kosten "x 3" i.p.v. "x 2" is.
Graag effe samen bespreken.

Beter :  Quit ##super(FormatterInput,IngegevenKenmerken) _ ..GeefSpecialeLadeVoorHaluxTekst(IngegevenKenmerken) en de method dan laten staan op de basisklasse, en dan moet ge de method in de LBX-...

Beter :

 Quit ##super(FormatterInput,IngegevenKenmerken) _ ..GeefSpecialeLadeVoorHaluxTekst(IngegevenKenmerken)


en de method dan laten staan op de basisklasse,
en dan moet ge de method in de LBX-klasse niet overriden.

Beter : Do ##super(IngegevenKenmerkenTypeAPI)

Beter : Do ##super(IngegevenKenmerkenTypeAPI)

[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:

- Fix tekst InfoDiverse specifiek voor TAX

- Paar extra testjes

    • -3
    • +3
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml
  1. … 1 more file in changeset.
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:

- UitsparingTekstFormatter => afzonderen verschillende implementaties LBX vs TAX

- Test => mini refactor => is TAX-eigen

    • -1
    • +1
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml
  1. … 4 more files in changeset.
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:
Hier is geen fake voor nodig, dit kan evengoed met de echte implementatieklasse.

Hier is geen fake voor nodig, dit kan evengoed met de echte implementatieklasse.

[UST4167] TAX-LL-001 : Diverse-kolom in interne configurator (voor BIDI en PM)
[UST4167] TAX-LL-001 : Diverse-kolom in interne configurator (voor BIDI en PM)
[UST4167] TAX-LL-001 : Diverse-kolom in interne configurator (voor BIDI en PM)

- Logo naam toevoegen aan diverse.

(te checken op unittesten, gebeurd niet bij comile.)

    • -0
    • +5
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml
[UST4167] TAX-LL-001 : Diverse-kolom in interne configurator (voor BIDI en PM)

- inject repository

    • -1
    • +8
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml
[UST4069] LBX: MeenemerSet voor binnenlade: InfoDiverse:
[UST4069] LBX: MeenemerSet voor binnenlade: InfoDiverse:
[UST4069] LBX: MeenemerSet voor binnenlade: InfoDiverse:

- MeenemerSet in kolom Diverse indien VolledigeSet

    • -0
    • +4
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml
[UST3911] TAX-SY: InfoDiverse kolom bepaling (Vhintra/Unishop):

- correcte vertalingskey gebruiken voor Sifonlade

    • -1
    • +1
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml
[UST3911] TAX-SY: InfoDiverse kolom bepaling (Vhintra/Unishop):

- Na review => TAORKenmerken doorgeven als parameter ipv te assignen als property

    • -10
    • +6
    ./IngegevenKenmerkenInfoDiverseBepaler.cls.xml