DnaCodeRepository.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5581] Veel orders blijven vastzitten in buffer

- Method verplaatst van DnaCodeRepository naar DnaCodeService

- Unit tests aangemaakt

  1. … 8 more files in changeset.
Deze method op APPS.Halux.AAP.DnaCodeService zetten en van daaruit de repo oproepen. En eigenlijk ook testjes voorzien.

Deze method op APPS.Halux.AAP.DnaCodeService zetten en van daaruit de repo oproepen.

En eigenlijk ook testjes voorzien.

[ICT-5581] Veel orders blijven vastzitten in buffer
[ICT-5581] Veel orders blijven vastzitten in buffer
[ICT-5581] Veel orders blijven vastzitten in buffer

- Extra method aangemaakt in de DnaCodeService die nagaat of een ingekorte PalletCode (met 8 karakters) afkomstig is van een A/B pallet.

- In ReceptieProductie gebruik gemaakt van deze method om de locatie van de nog te verwerken buffer te veranderen naar die van de locatie van de al bestaande pallet, enkel in geval het om A/B gaat.

  1. … 3 more files in changeset.
De lijn hieronder is een uitbreiding op deze en omvat deze case volledig + de A/B case.

De lijn hieronder is een uitbreiding op deze en omvat deze case volledig + de A/B case.

[ICT-4933] [rvTVE] Batch met meerdere SST zorgt voor foutieve dnacodes bij SST

- Reviewopmerking: Query verfijnen om false positives te voorkomen

  1. … 3 more files in changeset.
gaat dit nooit meer $$$True mogen zijn (geen idee hoor, just wondering)?

gaat dit nooit meer $$$True mogen zijn (geen idee hoor, just wondering)?

dé moment om te laten zien dat ik wel naar gekeken heb: 'Do' ipv 'do' (hieronder nog ergens)

dé moment om te laten zien dat ik wel naar gekeken heb: 'Do' ipv 'do' (hieronder nog ergens)

Hierin zit dus ook het verwijderen van alles wat met LPN-code te maken heet (aan de Caché-kant). De tests zouden hopelijk duidelijk moeten maken wat de PalletIDBepaler allemaal moet doen.

Hierin zit dus ook het verwijderen van alles wat met LPN-code te maken heet (aan de Caché-kant). De tests zouden hopelijk duidelijk moeten maken wat de PalletIDBepaler allemaal moet doen.

[ICT-4246] Manco: Backend ProScan
[ICT-4246] Manco: Backend ProScan
Is het niet veiliger om hier een ZoekViaToeleveringnSequentienummerEnRecent van te maken? sequentienummers van sommige ladetypes beginnen elke keer weer op 1 kwestie van geen false positives te h...

Is het niet veiliger om hier een ZoekViaToeleveringnSequentienummerEnRecent van te maken?

sequentienummers van sommige ladetypes beginnen elke keer weer op 1

kwestie van geen false positives te hebben

[ICT-4933] [rvTVE] Batch met meerdere SST zorgt voor foutieve dnacodes bij SST
[ICT-4933] [rvTVE] Batch met meerdere SST zorgt voor foutieve dnacodes bij SST
[ICT-4933] [rvTVE] Batch met meerdere SST zorgt voor foutieve dnacodes bij SST

- Het afdrukken van klantetiketten SST zorgde voor een aanpassing van de sequetienummers in de dnacodetabel wat niet gewenst is. Dit zorgde voor een verdere verkeerde verwerking

  1. … 4 more files in changeset.
zelfde opmerking als bij andere review: check zeker performantie van deze query. Btw: in Caché kunt ge ook de "[" operator gebruiker (='contains'), dan moet ge niet met die '%' werken (idem bij de...

zelfde opmerking als bij andere review: check zeker performantie van deze query. Btw: in Caché kunt ge ook de "[" operator gebruiker (='contains'), dan moet ge niet met die '%' werken

(idem bij de andere queries natuurlijk)

gaat deze zoek nog even snel gaan? een "=" gaat indexed zoeken, maar een "like" gaat mogelijks natural door de data. Het gaat hier wel om een tabel met enkele miljoenen records. Zeker dubbelchecken...

gaat deze zoek nog even snel gaan? een "=" gaat indexed zoeken, maar een "like" gaat mogelijks natural door de data. Het gaat hier wel om een tabel met enkele miljoenen records. Zeker dubbelchecken! (idem voor de andere queries natuurlijk)

Gisteren hier nog even met WV over gebabbeld en we zouden hier toch 'adviseren' om de grotere private methods in (een) aparte helperklasse(s) te steken, elk met hun eigen testen. Op die manier zijn...

Gisteren hier nog even met WV over gebabbeld en we zouden hier toch 'adviseren' om de grotere private methods in (een) aparte helperklasse(s) te steken, elk met hun eigen testen. Op die manier zijn de verschillende codepaden beter te testen (ipv sommige testen in de UT van deze klasse)
Dit valt onder het "Single Responsibility" principe. Ik ga dat hier niet allemaal verder uittypen, we praten of bellen wel

opeetcode!!! zeker aanpassen

opeetcode!!! zeker aanpassen