CacheAdminA_trunk2010

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates

Deze tests met (te)véél opzet zijn een reden temeer om de ZetDoorHandelNietMeeleverens af te zonderen

Deze tests met (te)véél opzet zijn een reden temeer om de ZetDoorHandelNietMeeleverens af te zonderen

ProjectSettingsApi newen

ProjectSettingsApi newen

ProjectSettingsApi newen

ProjectSettingsApi newen

om mee te nemen in de refactor http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif : set --> Set #Dim --> #dim if ---> If

om mee te nemen in de refactor :
set --> Set
#Dim --> #dim
if ---> If

Api *het ophalen van een projectsetting hoeft niet van de domeincontext
  • Api
  • het ophalen van een projectsetting hoeft niet van de domeincontext
Rekent deze klasse op de domeincontext?

Rekent deze klasse op de domeincontext?

Is het niet logischer/performanter dat de 2 bovenstaande lijnen gewisseld worden?

Is het niet logischer/performanter dat de 2 bovenstaande lijnen gewisseld worden?

Met een kleine moeite kan dit dynamisch gemaakt worden zodat slechts één setting nodig is voor eventueel toekomstige personalities met hetzelfde gedrag.

Met een kleine moeite kan dit dynamisch gemaakt worden zodat slechts één setting nodig is voor eventueel toekomstige personalities met hetzelfde gedrag.

Naamgeving bij nieuwe code: ...Api

Naamgeving bij nieuwe code: ...Api

voorstel: de ZetDoorHandelNietMeeleverens afzonderen naar een aparte klasse, en voor elke variant een afgeleide maken. Dan kun je gebruik maken van de autocomplete-enums. Do ..ZetDoorHandelNietMeel...

voorstel: de ZetDoorHandelNietMeeleverens afzonderen naar een aparte klasse, en voor elke variant een afgeleide maken.
Dan kun je gebruik maken van de autocomplete-enums.

Do ..ZetDoorHandelNietMeeleverensVoorRol(Lijn, HalffabItemsBoom, ..Enum.Rol.TiponBlumotionSynchronisatieStang)


Of minder werk: in elke method de juiste autocomplete-enum newen

#dim Rol As DOM.PM.Maatwerk.Calc.HF.enu.LBXRol.AutoComplete = ##class(DOM.PM.Maatwerk.Calc.HF.enu.LBXRol.AutoComplete).%New()
[BUS734] EC: MP: Daro: niet meeleveren aanvinken bij doorbestellen, en lijstje componenten mailen...
[BUS734] EC: MP: Daro: niet meeleveren aanvinken bij doorbestellen, en lijstje componenten mailen...
[UST4206] DPT1: Vh4: MP: Du-Pont: rechtstreeks transport: Transportkosten doorrekenen via systematisch...
[UST4206] DPT1: Vh4: MP: Du-Pont: rechtstreeks transport: Transportkosten doorrekenen via systematisch...
idem: ConsolidatieNummer As %Numeric , ... , ...

idem:
ConsolidatieNummer As %Numeric , ... , ...

ConsolidatieNummer As %Numeric , ... , ...

ConsolidatieNummer As %Numeric , ... , ...

[VRB630]RPKKL2 Logging: mail met links
[VRB630]RPKKL2 Logging: mail met links
[UST4204] PA-UB-021: Exceptions-handling backend
[UST4204] PA-UB-021: Exceptions-handling backend
In principe gebruiken we IsValid idd, maar aangezien deze method ook nog ligt te foefelen aan de Response-parameter doet ie iets meer dan enkel een boolean teruggeven. Maar ik zal hem renamen http:...

In principe gebruiken we IsValid idd, maar aangezien deze method ook nog ligt te foefelen aan de Response-parameter doet ie iets meer dan enkel een boolean teruggeven.
Maar ik zal hem renamen

Misschien kan je toch stukken code, die 3x voorkomen, verplaatsen naar private methods. Bijvoorbeeld .GeefProductOrderlijnMock() of .GeefProductOrderlijnMockVoorStandaardOrderMetProductLijn() ofzo....

Misschien kan je toch stukken code, die 3x voorkomen, verplaatsen naar private methods. Bijvoorbeeld .GeefProductOrderlijnMock() of .GeefProductOrderlijnMockVoorStandaardOrderMetProductLijn() ofzo.
idem OrderMock

Dat zou de 3 testmethods mogelijk overzichtelijker maken (lees: duidelijkere focus op de verschillen tussen de 3)

Als het toch veel te lastig blijkt te zijn, dan wellicht toch beter zo laten staan. Kies zelf maar.

"... voor Toelevering ... " i.p.v. "... DNACode ..."

"... voor Toelevering ... " i.p.v. "... DNACode ..."

Ook een testje maken waarbij de DnaCodeRepositoryMock.ZoekViaDnaCode nen Exception returns --> Response header asserten.

Ook een testje maken waarbij de DnaCodeRepositoryMock.ZoekViaDnaCode nen Exception returns --> Response header asserten.

De Fout-case "Geen data gevonden voor DNACode '"ToeleveringId"'" ook nog onder test brengen

De Fout-case "Geen data gevonden voor DNACode '"ToeleveringId"'" ook nog onder test brengen

Beetje verwarrende naamgevingen: Als onwetende denk je: "Als Hafele actief is, komt de naam op "HAEFELE NEDERLAND" om dan, als Dupont ook actief staat, overschreven te worden door "DU PONT BVBA"" ...

Beetje verwarrende naamgevingen:
Als onwetende denk je: "Als Hafele actief is, komt de naam op "HAEFELE NEDERLAND" om dan, als Dupont ook actief staat, overschreven te worden door "DU PONT BVBA""

Beter om "MoetDuPontAdresGebruiken" ofzoiets te nemen. Die van Hafele kan zelfs veranderd worden door Do:(KLNr="dievanhafele"), want die configsettings is toch niet meer relevant

$$$False ook in dat van Haefele. Ge kunt natuurlijk ook argumenteren dat die IsActiefSetting daar ineens verwijderd mag worden.

$$$False
ook in dat van Haefele. Ge kunt natuurlijk ook argumenteren dat die IsActiefSetting daar ineens verwijderd mag worden.