This is a list of all comments for CR-2050. Review Summary: No summary ---------------------------------------- File: APPS/Halux/PPS/impl/AansturingActie/common/KlantEtiket/Printer.cls.xml Revision Comment by Tom Vermeulen on 29 April 2020, 14:37 http://subversion02:8060/cru/CR-2050#c3396 Beter een $isObject gebruiken (doet in principe hetzelfde, maar is duidelijker) Btw, wat als er een empty Iterator wordt meegegeven, dan gaat er helemaal niets gebeuren. Willen we dat (open vraag hoor)? Ik neig toch om er nog een .HasNext()-achtig stukje bij te zetten Revision Comment by Tom Vermeulen on 29 April 2020, 14:39 defect http://subversion02:8060/cru/CR-2050#c3397 Ik weet dat dit stuk van KCA komt, maar hier klopt toch iets niet hoor. Stel dat we met 1 product hier binnenkomen, zonder DnaCodeIterator... dan gaat die ..GeefDnaCode ergens een bijna willekeurgie DnaCode uit zijn mouw schudden om dan mee te geven aan een etiket dat Aantal keer afgedrukt kan worden. Dat mag eigenlijk nooit gebeuren! We mogen geen etiketten met gelijke DnaCodes meermaals afdrukken (tenzij natuurlijk eentje kapot was en een gebruiker expliciet dat ene etiket opnieuw afdrukt) Revision Comment by Tom Vermeulen on 29 April 2020, 14:58 defect http://subversion02:8060/cru/CR-2050#c3401 moet ge eigenlijk niet defaulten naar een empty string, want dan gaat uwe reset op de eerste lijn nogal voenken geven (opmerking geldt ook voor op een paar andere plaatsen). Als je echt iets wil defaulten, kan je het proberen met een EmptyIterator. In dit concreet geval is het waarschijnlijk wat overkill, omdat het een interne private method is die (momenteel) enkel opgeroepen kan worden indien DnaCodeIterator wel degelijk ingevuld is. Revision Comment by Tom Vermeulen on 29 April 2020, 14:33 http://subversion02:8060/cru/CR-2050#c3395 hier moet ik eigenlijk zeggen dat we (in nieuwe stukken) niet meer werken met de DomeinContext, maar de klasse zelf gewoon Newen (in de constructor van deze klasse natuurlijk, wat trouwens ook zo zou moeten zijn als je wel via de DomeinContext werkt: gebruikte klassen altijd in de constructor opzetten) Revision Comment by Tom Vermeulen on 29 April 2020, 14:44 defect http://subversion02:8060/cru/CR-2050#c3398 (ben niet in onderliggende code gaat zoeken, misschien weet ge het antwoord direct) lopen we hier niet het risico dat we voor elke DnaCode in de iterator een bestandje krijgen? en ook: als we afdrukken met een dnacode er op, moet het aantal altijd 1 zijn (zie opmerking een beetje hoger) Reply by Laura Landuyt on 30 April 2020, 10:41 > Hier moet aantal dus altijd 1 zijn. Want in initiële print > moet als ..geefLijnAantal > 1 dan moet er eigenlijk per > etiket ook een andere dnacode zijn wat nu niet het geval. Reply by Laura Landuyt on 30 April 2020, 10:44 > Hier nog eens checken of als er dnacodes in zelfde batch > zitten moeten die in dezelfde file geprint worden. dit nog > eens checken. Misschien moet hiervoor nog aanpassing gebeuren > want in print hadetik zij else kant worden wrs gwn de > dnacodes direct in 1 file gezet en geen rekening gehouden met > de dnacodes die in zelfdfe batch zitten en dus samen in 1 > file moeten komen because performatie ---------------------------------------- File: WSimpl/Halux/AAP/AfdrukService.cls.xml Revision Comment by Tom Vermeulen on 29 April 2020, 14:22 defect http://subversion02:8060/cru/CR-2050#c3390 testen van elke method voorzien (die van de probox kunt ge al moven vanuit vhUnitTest.WSimpl.Halux.AAP.DnaCodeService.ProboxEtikettenAfdrukken.Test) Revision Comment by Tom Vermeulen on 29 April 2020, 14:21 http://subversion02:8060/cru/CR-2050#c3389 mag weg vanop de DnaCodeService Revision Comment by Tom Vermeulen on 29 April 2020, 14:26 defect http://subversion02:8060/cru/CR-2050#c3392 nonononono. copy-paste gevalleke van de proboxlogica denk ik :) de product/klantetiketten moeten op vaste printers in de halux-bureau afgedrukt worden. De proboxetiketten worden op printers aan het einde van elke lijn afgedrukt en gebruiken daarom die mapProductieLijnOpPrinter Revision Comment by Tom Vermeulen on 29 April 2020, 14:29 defect http://subversion02:8060/cru/CR-2050#c3393 heeeeeee, hier staat het in commentaar. comments wegsmijten Revision Comment by Tom Vermeulen on 29 April 2020, 14:30 defect http://subversion02:8060/cru/CR-2050#c3394 TODO's doen of comment wegsmijten Revision Comment by Tom Vermeulen on 15 July 2020, 08:09 defect http://subversion02:8060/cru/CR-2050#c3462 #dim resultaat as %String hoe dan ook: in VerwerkKlantEtiketAfdrukken wordt resultaat nergens ingevuld en zal het altijd een lege string zijn, wat een beetje zijn doe voorbijschiet beter een exceptie laten smijten in de lagere regionen Revision Comment by Tom Vermeulen on 15 July 2020, 08:13 defect http://subversion02:8060/cru/CR-2050#c3463 als je dus een exceptie smijt in de lagere stukken, dan kan deze reutemeteut ook weg :) Revision Comment by Tom Vermeulen on 30 April 2020, 08:50 defect http://subversion02:8060/cru/CR-2050#c3405 comments wegdoen of todo implementeren Revision Comment by Tom Vermeulen on 15 July 2020, 08:38 defect http://subversion02:8060/cru/CR-2050#c3466 BatchService injecten en testen voorzien die het gedrag van een true of false antwoord op BestaatBatch illustreren, dan had het gedoe hieronder niet naar boven gekomen :) Revision Comment by Tom Vermeulen on 15 July 2020, 08:35 defect http://subversion02:8060/cru/CR-2050#c3465 bovenstaand stukje is wreed tricky. In principe gebruiken we hoe dan ook geen early quits, want die maken het lezen/volgen van code moeilijker. Wat gebeurt er met ErrorMessage? Niets :) In mijn testgeval gaf BestaatBatch blijkbaar een false en er is niets in resultaat/exception dat daarop wijst. De code geeft zelfs een 200 terug op hoger niveau Revision Comment by Tom Vermeulen on 15 July 2020, 08:14 defect http://subversion02:8060/cru/CR-2050#c3464 wlipkes galore! (zie lager) Revision Comment by Laura Landuyt on 30 April 2020, 11:12 http://subversion02:8060/cru/CR-2050#c3409 private maken Revision Comment by Tom Vermeulen on 15 July 2020, 08:48 defect http://subversion02:8060/cru/CR-2050#c3468 Batch gaat hier altijd de Batch van de laatste DnaCode in de Iterator geven. Waarom is de whileloop nodig? Revision Comment by Tom Vermeulen on 15 July 2020, 08:41 defect http://subversion02:8060/cru/CR-2050#c3467 ook hier beter een exceptie raisen ipv een wlipke ---------------------------------------- File: APPS/Halux/PPS/impl/AansturingActie/common/KlantEtiket/EtiketPrinter.cls.xml Revision Comment by Tom Vermeulen on 29 April 2020, 14:48 defect http://subversion02:8060/cru/CR-2050#c3399 $isObject en desnoods ook een .HasNext() ---------------------------------------- File: WSimpl/Halux/AAP/AfdrukService/EtiketAfdrukkenHelper.cls.xml Revision Comment by Tom Vermeulen on 30 April 2020, 08:43 http://subversion02:8060/cru/CR-2050#c3403 om als lezer exact te weten wat dit eigenlijk allemaal doet, moet je de testen voorzien Vervang de early quit op de derde lijn hoe dan ook al door eentje op de eerste lijn. Quits moeten niet in het midden staan, enkel helemaal bovenaan of onderaan Revision Comment by Tom Vermeulen on 30 April 2020, 08:48 http://subversion02:8060/cru/CR-2050#c3404 Kan je heel deze method eigenlijk niet vervangen door 1 call naar de repository waar je een method "ZoekViaMeerdereDnaCodes" toevoegt die dan een zoekviafilter met een "in" ipv een "=" doet? Just wondering hoor, kan zijn dat dat niet werkt, maar het zou het natuurlijk wel een heel stuk simpeler maken (en geen aparte hoop testen nodig) ---------------------------------------- File: APPS/Halux/Docs/EtiketBuilder.cls.xml Revision Comment by Tom Vermeulen on 29 April 2020, 14:50 defect http://subversion02:8060/cru/CR-2050#c3400 ##class naar constructor DnaCode="" --> $$$HasLength(DnaCode) --- ID: CR-2050 http://subversion02:8060/cru/CR-2050 Title: ICT-1248: PA-UB: Productie afmelden: etiketten printen Statement of Objectives: Uitleg bij Tom te verkrijgen van wat er precies moet worden geimplementeerd Verdere uitleg, zie Confluence pagina State: Review Author: Laura Landuyt Moderator: Laura Landuyt Reviewers: (0 active, 1 completed*) Tom Vermeulen (*)