Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ik begrijp ni goe waarva, de 26.40 vandaan komt. Er is immers toch maar 1 geldige herberekende lijn. Kan je mij wat uitleg geven?

Ik begrijp ni goe waarva, de 26.40 vandaan komt.
Er is immers toch maar 1 geldige herberekende lijn.
Kan je mij wat uitleg geven?

Mijn dyslectisch brein kan deze "Name" niet aan. graag WasNogGeldig in één woord aub http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif en hieronder ook.

Mijn dyslectisch brein kan deze "Name" niet aan.
graag WasNogGeldig in één woord aub
en hieronder ook.

de twee Mocks in comment worden nergens gebruikt in de tests. Mogen dus geheel wel, volgens mij.

de twee Mocks in comment worden nergens gebruikt in de tests.
Mogen dus geheel wel, volgens mij.

Benaming ...Mock ?

Benaming ...Mock ?

Blokje commentaar ...

Blokje commentaar ...

[ICT-766] Vhisie4 : Synchroniseer / HerberekenMaatwerk rekenig laten houden met Ongeldig
[ICT-766] Vhisie4 : Synchroniseer / HerberekenMaatwerk rekenig laten houden met Ongeldig
[ICT-766] Vhisie4 : Synchroniseer / HerberekenMaatwerk rekenig laten houden met Ongeldig

- bericht uitgebreid bij verwijderen van een winkelkar lijn indien deze al niet meer bestond. het kan zijn dat deze verwijderd werd tijdens een synchroniseer omdat ze ongeldig gemarkeerd was.

  1. … 1 more file in changeset.
Method verwijderd in review.

Method verwijderd in review.

wijziging is om de exceptie aan onze kant niet meer te smijten, maar te loggen. we kunnen er toch niets aan doen. De code aan ITR kant zou voorzien zijn op een Data not found exceptie, maar doet er...

wijziging is om de exceptie aan onze kant niet meer te smijten, maar te loggen.
we kunnen er toch niets aan doen.
De code aan ITR kant zou voorzien zijn op een Data not found exceptie, maar doet er op zoch niets speciaal mee.

Ik denk dat deze melding ook in de controleerexterneid() method kon gezet worden. ITR verwerkte normaal gezien (zoals afgesproken) de datanotfoundexceptie als : "Er klopt iets niet, maar we mogen d...

Ik denk dat deze melding ook in de controleerexterneid() method kon gezet worden.
ITR verwerkte normaal gezien (zoals afgesproken) de datanotfoundexceptie als :
"Er klopt iets niet, maar we mogen de lijn toch verwijderen aan ITR kant zodat de klant kan verderwerken."
Geen idee of deze implementatie nog in orde was aan ITR kant, maar na deze wijziging krijgen ze deze melding/code niet meer van Cache.
Best eens checken met hen of dat nog nodig is ?
(Het zal in ieder geval niet meer als errormelding in de log komen.)

Commit 50446 en 50443 ? UW466

wordt niet meer gebruikt ?

wordt niet meer gebruikt ?

[DEF699] Vh4: Winkelkarsvc: VerwijderLijn: Winkelkarlijn met ExterneId '420162' van Winkelkar met...
[DEF699] Vh4: Winkelkarsvc: VerwijderLijn: Winkelkarlijn met ExterneId '420162' van Winkelkar met...
[DEF699] Vh4: Winkelkarsvc: VerwijderLijn: Winkelkarlijn met ExterneId '420162' van Winkelkar met ExterneId '103264', bestaat niet!

- indien de lijn niet bestaat aan de cahce kant, loggen, en doen alsof de lijn verwijderd is (ttz geen exceptie smijten)

  1. … 1 more file in changeset.

[UW466] Vhisie4 Unieke foutcode indien lijn of winkelkar niet bestaat. Hiermee kan ITR de errorcode...
[UW466] Vhisie4 Unieke foutcode indien lijn of winkelkar niet bestaat. Hiermee kan ITR de errorcode...
[UW466] Vhisie4 Unieke foutcode indien lijn of winkelkar niet bestaat. Hiermee kan ITR de errorcode opvangen.
  1. … 1 more file in changeset.
[UST2065] Vh4_SVC:VerwijderLijn

- VerwachtMethod en VerwachteExceptie vermijden

  • More
  • CR-62
  • finished reviewing
[UST2065] Vh4_SVC:VerwijderLijn

- review opmerkingen

En dan hier EenAndereWinkelkarId||InterneLijnId

En dan hier EenAndereWinkelkarId||InterneLijnId

VanHoeckeId zou ik dus vervangen door InterneWinkelkarId

VanHoeckeId zou ik dus vervangen door InterneWinkelkarId

Om het toch duidelijker te maken zou ik niet DummyId gebruiken maar eerder "InterneWinkelkarId"

Om het toch duidelijker te maken zou ik niet DummyId gebruiken maar eerder "InterneWinkelkarId"

In testen niet dezelfde code gebruiken als in de echte code --> Dus maw gewoon hardcoden in testen

In testen niet dezelfde code gebruiken als in de echte code --> Dus maw gewoon hardcoden in testen