Activity CacheAdminA_trunk2010/vhUnitTest/WSimpl/Vhisie4/QRcodeNaarKenmerken/GeefKenmerken

2:05 pm

Joannes Lemmens

More
73948
changed 2 files
root:

[ICT-4366] Wijziging zodat nieuwe distributeurs BlumPL geen opzet meer nodig hebben

- QrCodeNaarKenmerkenService aangepast zodat de VH Personality en gerelateerde klanten gekozen worden uit de OorsprongKetting

- -0
- +28
./Test.cls.xml
- History
- Source
- Diff
./Test.cls.xml
… 1 more file in changeset.

2:12 pm

Joannes Lemmens

More
70552
changed 1 file
root:

[ICT-3380] QRCodeNaarKenmerkenService aanpassen igv oproepen via AdminProductNummer

- Typfout verbeterd: QRnaarKenmerkenService -> QRcodenaarKenmerkenService

- -46
- +46
./Test.cls.xml
- History
- Source
- Diff
./Test.cls.xml

4:21 pm

Tom Vermeulen

More
Re: CR-2567
commented on review

Technically correct is the best kind of correct. Altijd afwegen of een pre/postconditional beter is dan een if/then/else structuur. Als ge dat aan de groep vraagt, krijgt ge uiteenlopende antwoorden.

Technically correct is the best kind of correct.
Altijd afwegen of een pre/postconditional beter is dan een if/then/else structuur. Als ge dat aan de groep vraagt, krijgt ge uiteenlopende antwoorden.

4:13 pm

misschien QRcodeNaarKenmerkenService ipv QRNaar.... ben je zeker dat de eerste parameter van de constructor leeg mag zijn? In dit geval maakt het niet uit, maar weet dat als ie null is, er een écht...

misschien QRcodeNaarKenmerkenService ipv QRNaar....
ben je zeker dat de eerste parameter van de constructor leeg mag zijn? In dit geval maakt het niet uit, maar weet dat als ie null is, er een échte gemaakt gaat worden. In alle andere testen staat hier een *Stub of *Mock. Nogmaals: het zal niet zoveel uitmaken in dit geval, maar het 'kan' tricky zijn.

Na verder te lezen zie ik dat die opmerking van naamgeving QRNaar... ook in mijn oude testen staat.... mea culpa. Your call

3:27 pm

Joannes Lemmens

More
CR-2567
started review

[ICT-3380] QRCodeNaarKenmerkenService aanpassen igv oproepen via AdminProductNummer

1:47 pm

Joannes Lemmens

More
70519
changed 2 files
root: CR-2567

[ICT-3380] QRCodeNaarKenmerkenService aanpassen igv oproepen via AdminProductNummer

- Als de service werd opgeroepen aan de hand van AdminProductnummer dan zetten we VerpakkingType aan de hand van IngegevenKenmerken ipv standaard V1

- Unit test uitgebreid om dit gedrag te reflecteren

- -0
- +87
./Test.cls.xml
- History
- Source
- Diff
./Test.cls.xml
… 1 more file in changeset.

9:41 am

Joannes Lemmens

More
CR-2499
Closed

10:06 am

Tom Vermeulen

More
CR-2499
finished reviewing

1:06 pm

Tom Vermeulen

More
Re: CR-2499
commented on review

zoals ge misschien al van Jo gehoord hebt: %String heeft max length van 50 -> gaf vrijdag problemen in productie. Best maxlen op 100 zetten (AX gebruikt nu 1000)

4:16 pm

Joannes Lemmens

More
Re: CR-2499
replied to Tom Vermeulen

Dit is aangepast: het steekt nu in een try-catch, waarbij in de catch een warning wordt gelogd en in de OorsprongKetting een lege string wordt gestoken. Verderop in de code, waar de OorsprongKettin...

Dit is aangepast: het steekt nu in een try-catch, waarbij in de catch een warning wordt gelogd en in de OorsprongKetting een lege string wordt gestoken.
Verderop in de code, waar de OorsprongKetting gebruikt wordt, is al een check of deze leeg is of niet dus alles zal gewoon kunnen blijven draaien.

8:59 am

Tom Vermeulen

More
Re: CR-2499
commented on review

wat als nieuwe lijnen falen? niet beter om in try-catch te steken en bij error de ketting leeg te maken? 't is maar omdat dit een vrij cruciaal stukje code is. Als de registratie niet gebeurt, word...

wat als nieuwe lijnen falen? niet beter om in try-catch te steken en bij error de ketting leeg te maken? 't is maar omdat dit een vrij cruciaal stukje code is. Als de registratie niet gebeurt, wordt er ook niet geperst en worden de aantallen niet verhoogd.

1:12 pm