This is a list of all comments for CR-2673. Review Summary: No summary ---------------------------------------- File: APPS/Verpakking/CubeIQRequestBuilder.cls.xml Revision Comment by Tommy Hebb on 19 December 2023, 15:05 defect http://subversion02:8060/cru/CR-2673#c4258 Graag de èchte instantiëren en niet van de DomeinContext afnemen. Revision Comment by Tommy Hebb on 19 December 2023, 15:27 defect http://subversion02:8060/cru/CR-2673#c4259 Lekker muggeziften... dimmekes altijd bovenaan in de method zetten, vóór de setjes. Revision Comment by Tommy Hebb on 19 December 2023, 15:29 defect http://subversion02:8060/cru/CR-2673#c4260 Moet deze in comment blijven staan? Mag die niet gewoon weg dan? Minstens een comment erachter zetten over het waarom de lijn in comment staat :-) Revision Comment by Tommy Hebb on 19 December 2023, 15:44 defect http://subversion02:8060/cru/CR-2673#c4262 Best diene Partialloadonfloor hier ook nog eens defaulten op 0, want hij wordt in deze method rechtstreeks gebruikt enerzijds en anderzijds wordt hij als parameter in de aanroepende method toch ook al gedefaulted op 0, dus mag consistent zijn, zoals gedaan is bij MinFillPercentage. Kan anders gevaar opleveren als deze private method ooit nog door een andere method gecalled wordt ook, waarin dan deze parameter nergens gedefaulted staat. (= undefined at runtime) Revision Comment by Tommy Hebb on 19 December 2023, 15:39 defect http://subversion02:8060/cru/CR-2673#c4261 Korttekst = string (gotta love Cache en types ;-) ) ---------------------------------------- File: APPS/Verpakking/BoxDataRepository.cls.xml Revision Comment by Tommy Hebb on 19 December 2023, 14:56 defect http://subversion02:8060/cru/CR-2673#c4257 Die Dummy ook buiten de While loop dimmen graag ---------------------------------------- File: APPS/Verpakking/DoosService.cls.xml Revision Comment by Tommy Hebb on 19 December 2023, 16:05 defect http://subversion02:8060/cru/CR-2673#c4263 Moet/mag deze lijn in comment blijven staan? Graag comment erachter met meer duiding. ---------------------------------------- File: APPS/Verpakking/OrgaluxBonOptimizer.cls.xml Revision Comment by Tommy Hebb on 19 December 2023, 17:02 defect http://subversion02:8060/cru/CR-2673#c4269 Typo :-) Revision Comment by Tommy Hebb on 19 December 2023, 16:12 defect http://subversion02:8060/cru/CR-2673#c4264 Graag de èchte instantiëren en niet van de DomeinContext afnemen. Revision Comment by Tommy Hebb on 19 December 2023, 16:13 defect http://subversion02:8060/cru/CR-2673#c4265 Aangezien ge hier beide mogelijkheden voorziet (injecteren of de echte gebruiken) moet ge dat ook bij de parameter duidelijk maken door die als lege string te defaulten, zoals bij de andere parameters gedaan is. Op die manier, als iemand deze klasse instantieert ergens, ziet die persoon gelijk aan de parameterdefinitie dat die niet per se moet meegegeven worden (en dat dan een nieuwe instantie voorzien is door deze constructor). Revision Comment by Tommy Hebb on 19 December 2023, 16:23 defect http://subversion02:8060/cru/CR-2673#c4266 Is het niet gevaarlijk die al uit de repo te gaan verwijderen terwijl er nog processen (de calls hieronder) in gang gaan gezet worden, die misschien nog gaan crashen? Moet deze verwijder niet op het einde van deze method uitgevoerd worden (vlak voor het loggen dat hij ermee klaar is)? Reply by Peter Vercoutter on 17 January 2024, 15:02 > Nee, het is best van deze data op te kuisen. Kan in het > slechtste geval nog vervuild zijn van de vorige iteraties > over de bonnummers Revision Comment by Tommy Hebb on 19 December 2023, 16:35 defect http://subversion02:8060/cru/CR-2673#c4267 Hier gebeurt wel heel erg veel op 1 lijn hé :-) Tenminste het product verdient wel een eigen lijn vind ik en dan nog liefst met een ProductRolAPI die in de constructor genewed wordt (en tevens niet van de DomeinContext afgenomen wordt ;-) ) Revision Comment by Tommy Hebb on 19 December 2023, 16:52 defect http://subversion02:8060/cru/CR-2673#c4268 Deze houdt toch risico in, denk ik, want je geeft normaal een object terug van type DS.Prod.OptiBox.BoxDataMetID, maar de logica laat toe dat dit object undefined is en op de volgende lijn van het stuk code dat deze private method gebruikt wordt er gebruik gemaakt van een property van dit object. Je komt ermee weg omdat de repo daar met lege strings overweg kan, maar 't is niet helemaal correct zoals het hier afgehandeld wordt. Minimaal zou je een leeg object moeten teruggeven, of je moet er nog een "Bestaat..." of "Heeft..." tussen moeten steken. Reply by Peter Vercoutter on 17 January 2024, 15:09 > Dit is net het herophalen van data waarvan we al zeker zijn > dat bestaat. We hebben in een vorige stap die data opgezocht. > Nu willen we die nog eens gaan uitlezen uit de DB. Revision Comment by Tommy Hebb on 19 December 2023, 17:13 defect http://subversion02:8060/cru/CR-2673#c4270 Ook hier gebeurt wel heel erg veel op 1 lijn hé... Tenminste het product verdient wel een eigen lijn vind ik en dan nog liefst met een ProductRolAPI die in de constructor genewed wordt (en tevens niet van de DomeinContext afgenomen wordt ) Revision Comment by Tommy Hebb on 19 December 2023, 17:16 defect http://subversion02:8060/cru/CR-2673#c4271 OrderAPI newen in constructor graag. --- ID: CR-2673 http://subversion02:8060/cru/CR-2673 Title: [ICT-3590] Optimalisatie orgalux bon Statement of Objectives: State: Closed Summary: Author: Peter Vercoutter Moderator: Peter Vercoutter Reviewers: (0 active, 1 completed*) Tommy Hebb (*)