DnaCodeAfmeldenResponse.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
hier kunt ge ook AssertTrue (of AssertFalse) gebruiken.

hier kunt ge ook AssertTrue (of AssertFalse) gebruiken.

[ICT-2157] [rvTVE] PA-UB-217: spacesteppopup bij afmelden dnacode

- Verwerken review opmerkingen: Property 'IsSST' default false gemaakt.

  1. … 1 more file in changeset.
Ge kunt u hier natuurlijk afvragen of deze method op deze klasse nog zin heeft tout court. 't Is in een oude naamgeving / package. Waarschijnlijk beter om de code die deze method oproept, rechtstre...

Ge kunt u hier natuurlijk afvragen of deze method op deze klasse nog zin heeft tout court. 't Is in een oude naamgeving / package. Waarschijnlijk beter om de code die deze method oproept, rechtstreeks de ToeleveringService te laten gebruiken. Dan skippen we dit gewoon.

ge hebt bij de oproepende klasse wel testen voorzien die checken of deze wordt opgeroepen, maar er zijn (nog) geen testen voor deze method zelf

ge hebt bij de oproepende klasse wel testen voorzien die checken of deze wordt opgeroepen, maar er zijn (nog) geen testen voor deze method zelf

Hier valt over te discussiëren: Is het niet beter om een gedefinieerd antwoord terug te geven, ook als het een error was? Onder normale omstandigheden wel, omdat je niet altijd weet wat de client g...

Hier valt over te discussiëren: Is het niet beter om een gedefinieerd antwoord terug te geven, ook als het een error was? Onder normale omstandigheden wel, omdat je niet altijd weet wat de client gaat doen bij een niet-geïnitialiseerde variabele. Aangezien je in dit concreet geval de client ook onder controle hebt, kan je ermee wegkomen. Persoonlijk zou ik in DnaCodeAfmeldenResponse een defaultvalue False voorzien voor de nieuwe property.

Delphi is niet case-sensitive, dus het maakt niet zoveel uit: UiContext is met een kleine i

Delphi is niet case-sensitive, dus het maakt niet zoveel uit: UiContext is met een kleine i

[ICT-2157] [rvTVE] PA-UB-217: spacesteppopup bij afmelden dnacode
[ICT-2157] [rvTVE] PA-UB-217: spacesteppopup bij afmelden dnacode
[ICT-2157] [rvTVE] PA-UB-217: spacesteppopup bij afmelden dnacode

- Extra property invoegen die aanduidt of er een SpaceStep in de toelevering zit.

  1. … 1 more file in changeset.
[ICT-1514] PA-UB-110: PlanScan: Vhoss: Afmeldantwoord invullen
[ICT-1514] PA-UB-110: PlanScan: Vhoss: Afmeldantwoord invullen
[ICT-1514] PA-UB-110: PlanScan: Vhoss: Afmeldantwoord invullen

- BatchId gebruiken ipv GuiVhossVolgnummer

  1. … 6 more files in changeset.
[ICT-1514] PA-UB-110: PlanScan: Vhoss: Afmeldantwoord invullen

- Huidige vhosskast in response/request steken en gebruiken

  1. … 5 more files in changeset.
heb ineens nog een test toegevoegd voor het geval de ValidateHeader zelf een exception zou raisen

heb ineens nog een test toegevoegd voor het geval de ValidateHeader zelf een exception zou raisen

Ik had ook deze twijfel, maar heb toch voor een warning gekozen omdat er geen 'harde fout' optreedt naar de gebruiker toe. Ze zullen daar wel een melding krijgen, maar niet "vast" zitten. In dat ge...

Ik had ook deze twijfel, maar heb toch voor een warning gekozen omdat er geen 'harde fout' optreedt naar de gebruiker toe. Ze zullen daar wel een melding krijgen, maar niet "vast" zitten. In dat geval zullen we wel gecontacteerd worden en in de logs gaan zoeken.

GeneriekeResponse OF SpecifiekeMethodResponse ? Kunnen we hier eens over babbelen? Ik ben persoonlijk evenveel voor als tegen! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/ico...

GeneriekeResponse OF SpecifiekeMethodResponse ?
Kunnen we hier eens over babbelen?
Ik ben persoonlijk evenveel voor als tegen!

Assert OmschrijvingVoorLogging ook nog.

Assert OmschrijvingVoorLogging ook nog.

Ik twijfel tussen "Warning" en "Error", of zelfs ErrorMail. Er is ten slotte iets foutgelopen, dat wellicht moet opgelost worden. Dus ik vrees dat bij Warning de boodschap zal verloren gaan.

Ik twijfel tussen "Warning" en "Error", of zelfs ErrorMail.
Er is ten slotte iets foutgelopen, dat wellicht moet opgelost worden. Dus ik vrees dat bij Warning de boodschap zal verloren gaan.

Mag hier ook weg, want zit in superklasse.

Mag hier ook weg, want zit in superklasse.

[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

"... 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