D20230607xConfigItemOphalenObjectiveSqlData.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Idd, geen testen. Code van PV moeilijk te unittesten. Nieuw stuk toegevoegde code is enkel om een ConfigItem op te halen, daarvoor is een UT overkill.

Idd, geen testen.
Code van PV moeilijk te unittesten. Nieuw stuk toegevoegde code is enkel om een ConfigItem op te halen, daarvoor is een UT overkill.

Terechte opmerking. Early Quit als eerste lijn van een method, dat is nog wel aanvaardbaar. Wat echter niet aanvaardbaar zou zijn, is wanneer die Quit middenin een blok code staat. Hoe dan ook, de...

Terechte opmerking.
Early Quit als eerste lijn van een method, dat is nog wel aanvaardbaar.
Wat echter niet aanvaardbaar zou zijn, is wanneer die Quit middenin een blok code staat.

Hoe dan ook, deze lijn code bevat een tweede probleem, namelijk dat het teruggeven van "" (leeg) zou de code verderdoor doen crashen, wat enigzins mijn bedoeling was, bij gebrek aan een "Empty ResultSetIterator".
Daarom heb ik het aangepast van early Quit "" naar Throw Exception.
Bovendien heb ik de method private gemaakt, want ik zie zie geen andere gebruikers dan in deze klasse.

Inderdaad, ReturnType moet %Boolean zijn. (is wellicht een copy-paste probleemke geweest)

Inderdaad, ReturnType moet %Boolean zijn.
(is wellicht een copy-paste probleemke geweest)

Zijn unit testen hier relevant? zoja, waar zijn deze dan?

Zijn unit testen hier relevant? zoja, waar zijn deze dan?

Ik heb geleerd dat een 'geef' altijd een object of een error teruggeeft. Hier wordt er in de early quit mogelijks een leeg object teruggegeven. Kan dit voor verwarring zorgen hier?

Ik heb geleerd dat een 'geef' altijd een object of een error teruggeeft.
Hier wordt er in de early quit mogelijks een leeg object teruggegeven.

Kan dit voor verwarring zorgen hier?

Ik zie dat er een Boolean wordt opgehaald, maar een String wordt teruggegeven. Cache is untyped, dat weet ik, maar dit is een beetje vreemd.

Ik zie dat er een Boolean wordt opgehaald, maar een String wordt teruggegeven.
Cache is untyped, dat weet ik, maar dit is een beetje vreemd.

[ICT-3699] Objective gerelateerde wijzigingen backported
[ICT-3699] Objective gerelateerde wijzigingen backported
[ICT] Objective gerelateerde wijzigingen backported

- Probleem in Scherm 7 van Admin oplossen door SqlConnection niet op te roepen indien de caché-server niet aan OBJT kan.

- Deploy-klasse toegevoegd voor ConfigItem in te stellen : "OBJTimpl.WMSOrder.KanOphalenObjectiveSqlData" (alleen op True indien Cache01 of CacheACP2010)

    • -0
    • +33
    ./D20230607xConfigItemOphalenObjectiveSqlData.cls.xml
  1. … 1 more file in changeset.