Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UW767] AX OrderUpdate: controleer ToeleveringenReedsInBatch

- ControleerToeleveringenInBatch voorlopig uitgeschakeld. Opnieuw te onderzoeken onverwacht gedrag, onterecht opkuisen van toeleveringen.

[UW767] AX OrderUpdate: controleer ToeleveringenReedsInBatch

- Bugfix, indien ToeleveringId = ""

- UT aangepast

  1. … 1 more file in changeset.
Aangepast

Aangepast

in constructor steken (wordt nergens gedaan in deze klasse, ik weet het...) als ge u dan niet kunt inhouden, kunt ge hem zelfs injecteren en valideren dat ie wordt opgeroepen indien nodig

in constructor steken (wordt nergens gedaan in deze klasse, ik weet het...)
als ge u dan niet kunt inhouden, kunt ge hem zelfs injecteren en valideren dat ie wordt opgeroepen indien nodig

[UW774] AX OrderUpdate: VerwijderToeleveringenUitBatches oproepen
[UW774] AX OrderUpdate: VerwijderToeleveringenUitBatches oproepen
[UW774] AX OrderUpdate: VerwijderToeleveringenUitBatches oproepen

- SalesOrderIN ToeleveringService wist nu eerst de toeleveringen in Batches. Nadien alsnog controle of er nog toeleveringen in Batches zitten.

[UW767] AX OrderUpdate: controleer ToeleveringenReedsInBatch

- Method ControleerToeleveringen() private gemaakt

- Mock en Fake aangepast

  1. … 2 more files in changeset.
[UW767] AX OrderUpdate: controleer ToeleveringenReedsInBatch

- Method ControleerToeleveringen() gebruikt in VerwijderToeleveringenViaOrder()

- Mock en Fake aangepast

  1. … 2 more files in changeset.
[UW767] AX OrderUpdate: controleer ToeleveringenReedsInBatch

- method GeefToeleveringenReedsInBatch() returns Array i.p.v. List

- BatchService via Injection

- Mock en Fake aangepast

  1. … 2 more files in changeset.
Als het een publieke method moet zijn, dan testen voorzien. Als het een private method moet zijn, dan testen voorzien voor VerwijderToeleveringenViaOrder

Als het een publieke method moet zijn, dan testen voorzien.
Als het een private method moet zijn, dan testen voorzien voor VerwijderToeleveringenViaOrder

[UW767] AX OrderUpdate: controleer ToeleveringenReedsInBatch
[UW767] AX OrderUpdate: controleer ToeleveringenReedsInBatch
[UW767] AX OrderUpdate: controleer ToeleveringenReedsInBatch

- BatchService method ZoekBijhorendeBatchIdVoorToelevering() toegevoegd

  1. … 2 more files in changeset.
Elke class op zich controleert of het order wel ingevuld is.We zouden inderdaad ook de classen niet kunnen oproepen indien het order niet bestaat. Mocht ze vanuit andere code gebruikt worden is dit...

Elke class op zich controleert of het order wel ingevuld is.We zouden inderdaad ook de classen niet kunnen oproepen indien het order niet bestaat.
Mocht ze vanuit andere code gebruikt worden is dit meteen ook in orde, want het heeft geen zin deze code uit te voeren als het order leeg is. En een foutmelding is in dit geval niet nodig.

[UST3177r] VHIP901: ORDER2ADMIN: updates op gedeeltelijk afgewerkte orders. maw geen lijnen updaten die status "in productie" hebben.

- benaming en type van method

Blijkbaar ben ik hier een type vergeten.. is intussen toegevoegd.

Blijkbaar ben ik hier een type vergeten.. is intussen toegevoegd.

inderdaad moet "zet" zijn.

inderdaad moet "zet" zijn.

Ik heb geleerd dat ge niet altijd op cache moogt vertrouwen, dus liever een controle meer dan minder. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Ik heb geleerd dat ge niet altijd op cache moogt vertrouwen, dus liever een controle meer dan minder.

  • More
  • CR-819
  • finished reviewing
Zie veel te veel Quit:$$$Not($IsObject(Order)) staan naar mijn goesting.

Zie veel te veel Quit:$$$Not($IsObject(Order)) staan naar mijn goesting.

  • More
  • CR-817
  • finished reviewing
Geef of Zet?

Geef of Zet?

Kan dit anders zijn zonder dat hierboven een exception gesmeten werd? Indien ja, gebeurd er niets zonder dat het gelogd werd.

Kan dit anders zijn zonder dat hierboven een exception gesmeten werd?
Indien ja, gebeurd er niets zonder dat het gelogd werd.

  • More
  • CR-781
  • finished reviewing