DnaCodeService.pas

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4993][ProScan] Melding tonen indien toelevering voor kartonnen pallet is
[ICT-4993][ProScan] Melding tonen indien toelevering voor kartonnen pallet is
Heb me enkel beziggehouden met de .pas bestanden

Heb me enkel beziggehouden met de .pas bestanden

Wordt deze gebruikt? Ik zie niet direct een verwijzing in de huidige klasse naar deze Service (kan er naast kijken). Indien niet in gebruik: uit de uses (gebruikt zinloze resources). Ook de file in...

Wordt deze gebruikt? Ik zie niet direct een verwijzing in de huidige klasse naar deze Service (kan er naast kijken). Indien niet in gebruik: uit de uses (gebruikt zinloze resources). Ook de file in \ProScan\WS\ wegsmijten in dat geval.

Als ie wel gebruikt wordt: dat is een serieuze dependency die geïntroduceerd wordt. Zou het er dan ook uitwerken

[ICT-3732] Manco deel 1: proboxen aan een pallet/LPN linken
[ICT-3732] Manco deel 1: proboxen aan een pallet/LPN linken
Lessons learned: opsplitsen in kleinere kaartjes... Gooi er maar gerust een extra reviewer (JBA) bij indien nodig.

Lessons learned: opsplitsen in kleinere kaartjes...
Gooi er maar gerust een extra reviewer (JBA) bij indien nodig.

je zou de wijzigingen in caché nog moeten toevoegen aan de review

je zou de wijzigingen in caché nog moeten toevoegen aan de review

[ICT-3707] XS Verpakkingsmateriaal - Planscan/ Proscan afmeldapplicatie
[ICT-3707] XS Verpakkingsmateriaal - Planscan/ Proscan afmeldapplicatie
hier kunt ge ook AssertTrue (of AssertFalse) gebruiken.

hier kunt ge ook AssertTrue (of AssertFalse) gebruiken.

Ge kunt u hier natuurlijk afvragen of deze method op deze klasse nog zin heeft tout court. 't Is in een oude naamgeving / package. Waarschijnlijk beter om de code die deze method oproept, rechtstre...

Ge kunt u hier natuurlijk afvragen of deze method op deze klasse nog zin heeft tout court. 't Is in een oude naamgeving / package. Waarschijnlijk beter om de code die deze method oproept, rechtstreeks de ToeleveringService te laten gebruiken. Dan skippen we dit gewoon.

ge hebt bij de oproepende klasse wel testen voorzien die checken of deze wordt opgeroepen, maar er zijn (nog) geen testen voor deze method zelf

ge hebt bij de oproepende klasse wel testen voorzien die checken of deze wordt opgeroepen, maar er zijn (nog) geen testen voor deze method zelf

Hier valt over te discussiëren: Is het niet beter om een gedefinieerd antwoord terug te geven, ook als het een error was? Onder normale omstandigheden wel, omdat je niet altijd weet wat de client g...

Hier valt over te discussiëren: Is het niet beter om een gedefinieerd antwoord terug te geven, ook als het een error was? Onder normale omstandigheden wel, omdat je niet altijd weet wat de client gaat doen bij een niet-geïnitialiseerde variabele. Aangezien je in dit concreet geval de client ook onder controle hebt, kan je ermee wegkomen. Persoonlijk zou ik in DnaCodeAfmeldenResponse een defaultvalue False voorzien voor de nieuwe property.

Delphi is niet case-sensitive, dus het maakt niet zoveel uit: UiContext is met een kleine i

Delphi is niet case-sensitive, dus het maakt niet zoveel uit: UiContext is met een kleine i