Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1486] Fouten in UT TestData verminderen
[ICT-1486] Fouten in UT TestData verminderen
[ICT-1486] Fouten in UT TestData verminderen

- Bij OnDelete de Reservatie verwijderen, enkel als die nog ingevuld is.

[ICT319] Private properties publiek voor SQL-mogelijkheden: Setten buiten constructor onmogelijk maken, voor properties waarvoor dat tevoren onmogelijk was.
[ICT319] Private properties publiek voor SQL-mogelijkheden: Setten buiten constructor onmogelijk maken, voor properties waarvoor dat tevoren onmogelijk was.
    • -0
    • +22
    ./impl/InvoerReservatieImpl.cls.xml
  1. … 2 more files in changeset.
[ICT] - Re-Invokable gemaakt
    • -7
    • +10
    ./event/InvoerUitvoerLinkEventData.cls.xml
  • More
  • CR-712
  • finished reviewing
idem als overige, het veld reservatie wordt gemaakt via de constructor waarschijnlijk met de bedoeling dat deze niet mag gewijzigd worden. De MaakReservatie is dan ook private. Waarom zouden we dez...

idem als overige, het veld reservatie wordt gemaakt via de constructor waarschijnlijk met de bedoeling dat deze niet mag gewijzigd worden. De MaakReservatie is dan ook private. Waarom zouden we deze property vijrgeven zodat deze rechtstreeks aangepast kan worden vanuit de code, met alle mogelijke gevolgen van dien. Ook hier kan je beter voorkomen dat deze aangepast kan worden.

ProductID wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een proper...

ProductID wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een property weghalen geven we anderen de mogelijkheid ook na creatie van het object, de property te wijzigen. We zouden op zijn minst moeten zorgen dat deze dan readonly komen te staan. of er een foutmelding gegooid wordt bv via een setter, indien er een poging gedaan wordt om het ProductID te wijzigen.

IDem als ProductID : VerwachteInvoer wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu ...

IDem als ProductID :

VerwachteInvoer wordt meegegeven in de constructor en gezet op een private property. Dergelijke code wordt toegepast met de bedoeling een property niet te kunnen wijzigen. Nu we de private van een property weghalen geven we anderen de mogelijkheid ook na creatie van het object, de property te wijzigen. We zouden op zijn minst moeten zorgen dat deze dan readonly komen te staan. of er een foutmelding gegooid wordt bv via een setter, indien er een poging gedaan wordt om het veld te wijzigen.

[ICT319] Properties public ipv private, om in sql fatsoenlijk te kunnen gebruiken.
[ICT319] Properties public ipv private, om in sql fatsoenlijk te kunnen gebruiken.
[ICT319] Properties public ipv private, om in sql fatsoenlijk te kunnen gebruiken.
[DEF499] Doorst. picking: stock niet juist gecontroleerd:

- ongedaan maken van : "enkel de positieve reservaties (geen terugnames) worden meegerekend voor de praktische stock"

    • -2
    • +1
    ./impl/VerwachteUitvoerAPIimpl.cls.xml
  1. … 1 more file in changeset.
[DEF499] Doorst. picking: stock niet juist gecontroleerd:

- cleanup, refactor, cosmetica, ... up front

- maakt gebruik van Query API

- while terug naar if zodoende we de eerste nemen en niet de laatste

    • -1
    • +1
    ./impl/VerwachteUitvoerAPIimpl.cls.xml
VerwachteUitvoerImpl OnDelete() kleine aanpassing --> nog te verbeteren.
[ICT] - Refactoring in event handling - Events hebben vanaf heden Type method
    • -11
    • +0
    ./event/InvoerUitvoerLinkVerbrokenEvent.cls.xml
[ICT] - Refactoring in event handling - Events hebben vanaf heden Type method
    • -11
    • +0
    ./event/InvoerUitvoerLinkAangemaaktEvent.cls.xml
[UW249] - Memory leaks in event system
    • -0
    • +5
    ./event/InvoerUitvoerLinkEventData.cls.xml
[DEF499] Doorst. picking: stock niet juist gecontroleerd:

- Bedrijf als TECH.Enumeration omwille van AutoComplete

- fix: query parameter werkt niet bij IN

- helper methods voorzien voor opzetten data

- base klasse voor VerwachteUitvoerAPI tests

- tests voor GeefAantalGereserveerd gegroepeerd

    • -4
    • +4
    ./impl/VerwachteUitvoerAPIimpl.cls.xml
  1. … 8 more files in changeset.
[DEF499] Doorst. picking: stock niet juist gecontroleerd:

- enkel de positieve reservaties (geen terugnames) worden meegerekend voor de praktische stock

    • -1
    • +2
    ./impl/VerwachteUitvoerAPIimpl.cls.xml
  1. … 1 more file in changeset.
[DEF499] Doorst. picking: stock niet juist gecontroleerd:

- Query api via constructor injection

- maakt gebruik van TECH.Interface

    • -3
    • +14
    ./impl/VerwachteUitvoerAPIimpl.cls.xml
  1. … 7 more files in changeset.
[DEF499] Doorst. picking: stock niet juist gecontroleerd:

- cleanup, refactor, cosmetica, ... up front

- maakt gebruik van Query API

    • -29
    • +49
    ./impl/VerwachteUitvoerAPIimpl.cls.xml
[ICT169] TestVerwachteInvoerBijTransfers

- cleanup cosmetica

    • -105
    • +110
    ./impl/VerwachteInvoerAPIimpl.cls.xml
  1. … 3 more files in changeset.