Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Mogelijks is dit een gevaarlijke voor binnenkort standalone matten denk ik, niet? Zou hier nog een 2e conditie bij kunnen eventueel?

Mogelijks is dit een gevaarlijke voor binnenkort standalone matten denk ik, niet? Zou hier nog een 2e conditie bij kunnen eventueel?

[ICT-4627] [rvPVR] VHLOG MES M1 - TBX afbeeldingcode blijft leeg in de orderoptie:
[ICT-4627] [rvPVR] VHLOG MES M1 - TBX afbeeldingcode blijft leeg in de orderoptie:
[ICT-4494] [rvPVR] ProductieDetailblad via Encoway met matten ondersteunen:
[ICT-4494] [rvPVR] ProductieDetailblad via Encoway met matten ondersteunen:
$J zou ik vervangen door '$Job'?

$J zou ik vervangen door '$Job'?

Ergens geeft dit nu wel een verkeerd beeld van deze method; ik zou ervan uitgaan dat het productid enkel wordt bepaald, maar als je dan in de method gaat kijken zie je dat hier ook de vhConfig word...

Ergens geeft dit nu wel een verkeerd beeld van deze method; ik zou ervan uitgaan dat het productid enkel wordt bepaald, maar als je dan in de method gaat kijken zie je dat hier ook de vhConfig wordt opgeslagen

Converter in constructor initialiseren

Converter in constructor initialiseren

Ik zie dit nu pas, maar misschien kan dit in een 'constante' variabale om wat meer context te geven? Iets als 'DefaultCidAX' of iets dergelijks?

Ik zie dit nu pas, maar misschien kan dit in een 'constante' variabale om wat meer context te geven? Iets als 'DefaultCidAX' of iets dergelijks?

[ICT-4440] [rvJBA] VhConfig persisteren in caché na ordersync AX
[ICT-4440] [rvJBA] VhConfig persisteren in caché na ordersync AX
Deze wordt vanzelf opgevangen omdat het gaat om een 'NotSupportedException' :-D (Zie ook de UT)

Deze wordt vanzelf opgevangen omdat het gaat om een 'NotSupportedException' :-D (Zie ook de UT)

Naar deze enu's wordt nog verwezen in APPS.EC.Transport.Service.cls. Moet die ook niet aangepast worden? Zit niet mee in review.

Naar deze enu's wordt nog verwezen in APPS.EC.Transport.Service.cls. Moet die ook niet aangepast worden? Zit niet mee in review.

Naar deze dto's wordt nog verwezen in APPS.EC.Transport.Service.cls. Moet die ook niet aangepast worden? Zit niet mee in review.

Naar deze dto's wordt nog verwezen in APPS.EC.Transport.Service.cls. Moet die ook niet aangepast worden? Zit niet mee in review.

deze heeft er niet zo veel mee te maken zou ik zeggen. Probleem dat die mee in de commit zit?

deze heeft er niet zo veel mee te maken zou ik zeggen. Probleem dat die mee in de commit zit?

Als dit naar productie gaat, zeker ook de caretakerjob via de caché-systeembeheerportaal afzetten

Als dit naar productie gaat, zeker ook de caretakerjob via de caché-systeembeheerportaal afzetten

hier is in principe extra gedrag bijgekomen, dus zou er ook een testje dat moeten weergeven. Kan zijn dat ik niet in genoeg in detail naar de testen gekeken heb ook natuurlijk.

hier is in principe extra gedrag bijgekomen, dus zou er ook een testje dat moeten weergeven. Kan zijn dat ik niet in genoeg in detail naar de testen gekeken heb ook natuurlijk.

If $$$HasLengfth(Werkost) (zie lijn 181)

If $$$HasLengfth(Werkost) (zie lijn 181)

LadeRegistratieIterator ipv LadeRegistraties, dat maakt het lezen van de code duidelijker verderop. "LadeRegistraties" kan een list, een array, een iterator, ... zijn

LadeRegistratieIterator ipv LadeRegistraties, dat maakt het lezen van de code duidelijker verderop. "LadeRegistraties" kan een list, een array, een iterator, ... zijn