IngegevenKenmerkenInfoDiverseBepaler

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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)

- Test IsSmalleLadebepaler

    • -1
    • +1
    ./BepaalInfoDiverseItems/LBX/Test.cls.xml
  1. … 2 more files in changeset.
[ICT-816] Vhintra Configurator - gebruiker attent maken op ingave van een smalle lade. (met mogelijkse meerkost)

- Smalle lade vermelding in info diverse. Test idem als andere LBX testen. Misschien eens tijd maken om deze aan te passen en kenmerkentypeapi uit te mocken etc...

    • -0
    • +7
    ./BepaalInfoDiverseItems/LBX/Test.cls.xml
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:
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:

- Enkel noodzakelijke afwijkingen t.o.v. standaard laden om het te testen issue aan te kaarten nu

    • -20
    • +8
    ./BepaalInfoDiverseItems/TAOR/SifonladeTest.cls.xml
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen:

- Indien X en/of Y < 204 => dan voegen we in de kolom Diverse volgende vermelding toe => SPECIALE LADE VOOR HALUX => uitbreiding TestCase

    • -0
    • +56
    ./BepaalInfoDiverseItems/TAOR/SifonladeTest.cls.xml
[ICT362] [UST4255] TAXSY - extra mogelijkheden om Talsee binnen te halen: (i.s.m. Tommy)

- Indien X en/of Y < 204 => dan voegen we in de kolom Diverse volgende vermelding toe => SPECIALE LADE VOOR HALUX

    • -2
    • +2
    ./BepaalInfoDiverseItems/TAOR/SifonladeTest.cls.xml
  1. … 6 more files in changeset.
[UST4167r] TAX-LL-001 : Diverse-kolom in interne configurator (voor BIDI en PM)

- Geen fake nodig.

    • -1
    • +1
    ./BepaalInfoDiverseItems/TAOR/Test.cls.xml
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)

- Verwachtmethod voor repo, verifieer en onbestaande logonaam. (geen echte data.).

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

- Testje logo met uitgemockte repo.

    • -2
    • +9
    ./BepaalInfoDiverseItems/TestBase.cls.xml
    • -1
    • +7
    ./BepaalInfoDiverseItems/TAOR/Test.cls.xml
[UST4167] TAX-LL-001 : Diverse-kolom in interne configurator (voor BIDI en PM)

- Testje. Bekijken om repo uit te mocken.

    • -0
    • +7
    ./BepaalInfoDiverseItems/TAOR/Test.cls.xml
[UST3911] TAX-SY: InfoDiverse kolom bepaling (Vhintra/Unishop):

- correcte vertalingskey gebruiken voor Sifonlade - UT aangepast

    • -2
    • +2
    ./BepaalInfoDiverseItems/TAOR/SifonladeTest.cls.xml
Deze mag gerust ook "IsTaxLadeMetVerlaagdeRug" heten, en dan met function-argument (TaxKenmerken As ...Calc.Common.TAORKenmerken) .

Deze mag gerust ook "IsTaxLadeMetVerlaagdeRug" heten, en dan met function-argument (TaxKenmerken As ...Calc.Common.TAORKenmerken) .

Lijkt me niet nodig om ..IngegevenKenmerken als property te assign'en. Geef rechtstreeks door : If (..IsSifonlade(TAORKenmerken)) Idem met IsLadeMetVerlaagdeRug(TAORKenmerken)

Lijkt me niet nodig om ..IngegevenKenmerken als property te assign'en.
Geef rechtstreeks door : If (..IsSifonlade(TAORKenmerken))
Idem met IsLadeMetVerlaagdeRug(TAORKenmerken)