Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
klopt, maar ondertussen bestaat deze klasse al niet meer http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

klopt, maar ondertussen bestaat deze klasse al niet meer

Testje toevoegen die aantoont dat een DnaCode ouder dan 30 dagen, niks oplevert. Het is immers belangrijk dat er geen items van vroeger worden opgepikt, zo heb ik t toch begrepen.

Testje toevoegen die aantoont dat een DnaCode ouder dan 30 dagen, niks oplevert.
Het is immers belangrijk dat er geen items van vroeger worden opgepikt, zo heb ik t toch begrepen.

[ICT-1250] PA-UB-086: weerkerende nrs voor pallet/toelevering/...
[ICT-1250] PA-UB-086: weerkerende nrs voor pallet/toelevering/...
[ICT-1250] PA-UB-086: weerkerende nrs voor pallet/toelevering/...

- Nieuwe methods die enkel data van laatste 30 dagen teruggeven oproepen

  1. … 2 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

[UST4183] PA-UB-013 DnaCodeAfmelden method toevoegen op DnaCodeService

- juist foutomschrijving

  1. … 1 more file in changeset.
"... 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

(reeds mondeling besproken) DnaCodeIterator : zelfde variabele wordt opnieuw gebruikt en dat maakt het verwarrend i.v.m. HasNext() en Next() Anderzijds zou deze method misschien beter in 2 (private...

(reeds mondeling besproken)
DnaCodeIterator : zelfde variabele wordt opnieuw gebruikt en dat maakt het verwarrend i.v.m. HasNext() en Next()
Anderzijds zou deze method misschien beter in 2 (private) methods opgesplitst worden.

Hebben wij conventies omtrent "Valideer" en "IsValid" ? Deze method geeft een Boolean terug, dus zou ik eerder kiezen voor method naam "IsValid... "

Hebben wij conventies omtrent "Valideer" en "IsValid" ?
Deze method geeft een Boolean terug, dus zou ik eerder kiezen voor method naam "IsValid... "

typo : Iteraot

typo : Iteraot

[UST4201] PA-UB-021: Exceptions-handling backend

- Quotes toegevoegd (nav review opmerking ander kaartje)

  1. … 1 more file in changeset.
[UST4201] PA-UB-021: Exceptions-handling backend

- Headervalidatie gecentraliseerd

  1. … 6 more files in changeset.
[UST4182] PA-UB-012: ZoekViaToelevering method toevoegen op DnaCodeService

- review

  1. … 1 more file in changeset.
Nee, maf hé. Hoera voor de untyped languages http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Nee, maf hé. Hoera voor de untyped languages