Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-163] [rvTVE] Verkoopfacturen synchronisatie is overbodig geworden. De code en backgroundjob mag...
[ICT-163] [rvTVE] Verkoopfacturen synchronisatie is overbodig geworden. De code en backgroundjob mag...
Deze routine moet nog verwijderd worden samen met de background job

Deze routine moet nog verwijderd worden samen met de background job

Voorlopig heb ik enkel de code die uitgevoerd moet worden in de AX.CARETAKER.mac in commentaar gezet om zeker te zijn dat ik de juiste backgroundjob heb genomen (de beschrijving in het kaartje leek...

Voorlopig heb ik enkel de code die uitgevoerd moet worden in de AX.CARETAKER.mac in commentaar gezet om zeker te zijn dat ik de juiste backgroundjob heb genomen (de beschrijving in het kaartje leek mij niet juist, dus heb mij gebaseerd op de titel van het kaartje en de bijhorende backgroundjob dat ik hierbij gevonden heb).

DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

my point exactly (zie comment). Afzonderen naar apart klaske dat getest kan worden

my point exactly (zie comment).
Afzonderen naar apart klaske dat getest kan worden

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

ClassMethod!

ClassMethod!

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (ju...

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (juist wat parameters zetten enzo) en dan was het geen probleem dat het niet in een niet-testbare classmethod steekt. Met al de extra logica die er ondertussen is bijgekomen is dat eigenlijk niet goed meer.

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
Done!

Done!

try-catch toevoegen en lokaal afhandelen (warning ofzo, maar geen error verder throwen)

try-catch toevoegen en lokaal afhandelen (warning ofzo, maar geen error verder throwen)

misschien nog wat testjes voor het geval de bon zélf niet bestaat

misschien nog wat testjes voor het geval de bon zélf niet bestaat

Best niet in de constructor zetten, maar in een aparte method. In dit concreet geval maakt dat natuurlijk niet echt uit, maar 't is "good practice".

Best niet in de constructor zetten, maar in een aparte method. In dit concreet geval maakt dat natuurlijk niet echt uit, maar 't is "good practice".

[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:
[VRB561] Voorraad: InventTransID, ItemID, ConfigID in PRHIST bewaren:
  • More
  • CR-820
  • finished reviewing
[UST3301] VHIP Migratie: 0602VrijgegProducten: Migratie aliassen - deel 2 - bestaande prod en hun...
[UST3301] VHIP Migratie: 0602VrijgegProducten: Migratie aliassen - deel 2 - bestaande prod en hun...
  • More
  • CR-796
  • finished reviewing
[UST3300] VHIP Migratie: 0609BehoeftePlanning: Afzonderen VeiligheidsVoorraad:
[UST3300] VHIP Migratie: 0609BehoeftePlanning: Afzonderen VeiligheidsVoorraad:
Jo, Is het "MIN Extradagen" of "PLUS" extra dagen? want in kanban staat "... dagen bijtellen" . Deze opmerking zonder hier echt verder over na te denken. Als je dit zelf effe kritisch bekijkt, of ...

Jo,
Is het "MIN Extradagen" of "PLUS" extra dagen? want in kanban staat "... dagen bijtellen" .

Deze opmerking zonder hier echt verder over na te denken. Als je dit zelf effe kritisch bekijkt, of afstemt met analyst/opdrachtgever, dan is het voor mij goed genoeg

  • More
  • CR-750
  • finished reviewing