This is a list of all comments for CR-1921. Review Summary: No summary General Comment by Laura Landuyt on 16 January 2020, 13:35 http://subversion02:8060/cru/CR-1921#c2920 - Naamgevingen verduidelijken - Geen engelse stukken vb VerzendwijzeLONG.. - {} plaatsen volgens cache standaard - Zie ook confluence notities voor dit ticket ---------------------------------------- File: AXimpl/Dynamics/InventoryOut/EventHandler/PakbonMagGeboektWorden.cls.xml Revision Comment by Laura Landuyt on 16 January 2020, 13:47 http://subversion02:8060/cru/CR-1921#c2924 // var hernoemen naar GeneratePakbonMagGeboektWordenResult en ook het vereder gebruik , argument in methode aanpassen ---------------------------------------- File: AXimpl/Dynamics/InventoryOut/BatchJobPakbonMagGeboektWorden.cls.xml Revision Comment by Jo Willems on 18 March 2020, 11:00 http://subversion02:8060/cru/CR-1921#c3228 Repository maken over Flow.Uitlevering.Hoofding ? encapsulaties gebruiken waar het kan? overweeg met PRA om te zien of het de investering waard is. Indien niet gebruik maken van encapsulatie: nu wordt 2 x $Order uitgevoerd voor één uitlevering. zo ook voor de volgende whiles Dat kan beter Na de rest van de code te bekijken : Repository maken (resulteert in één iterator en testbaarheid of meer dan één uitlevering correct afgehandeld worden. Revision Comment by Laura Landuyt on 16 January 2020, 13:37 http://subversion02:8060/cru/CR-1921#c2921 VerwerkRecord hernoemen naar VerwerkBon (dit is iets duidelijker) Revision Comment by Jo Willems on 18 March 2020, 18:23 defect http://subversion02:8060/cru/CR-1921#c3234 Dit gaat alleen voor de eerste bon werken niet meer voor de 2de. Revision Comment by Jo Willems on 18 March 2020, 17:54 http://subversion02:8060/cru/CR-1921#c3231 Wat doen we met Data? Revision Comment by Jo Willems on 18 March 2020, 17:56 http://subversion02:8060/cru/CR-1921#c3232 Private? Revision Comment by Jo Willems on 18 March 2020, 17:57 http://subversion02:8060/cru/CR-1921#c3233 ik zou dit toch over een paar lijnen spreiden, gaat wat leesbaarder worden dan. Revision Comment by Jo Willems on 18 March 2020, 10:57 http://subversion02:8060/cru/CR-1921#c3227 Maak hier een specifieke method voor waarvan de naam betekenis heeft. 63 zegt niet veel. cfr andere opmerking op de impl hiervan. ---------------------------------------- File: AXimpl/Dynamics/InventoryOut/EventHandler/PickingCompleet/PakbonMagGeboektWorden.cls.xml Revision Comment by Laura Landuyt on 16 January 2020, 13:51 http://subversion02:8060/cru/CR-1921#c2927 // GeefBon --> Geef klant --> via objecten oproepen, niet rechtstreeks via de global Revision Comment by Laura Landuyt on 16 January 2020, 13:50 http://subversion02:8060/cru/CR-1921#c2926 //$Not van maken in plaats van ..VerzendwijzeUitgesteldPakbonBoekenAx(bon, klant) = $$$False Revision Comment by Laura Landuyt on 16 January 2020, 13:49 http://subversion02:8060/cru/CR-1921#c2925 een else kant toevoegen met een log else { // log toevoegen dat event uitgesteld zal worden verzonden } Revision Comment by Laura Landuyt on 16 January 2020, 13:53 http://subversion02:8060/cru/CR-1921#c2930 Geen engelstalige stukken in naamgeving Code wordt overbodig want GeefBon gebruiken en dan verder ook Bon.GeefVerzendwijze Revision Comment by Laura Landuyt on 16 January 2020, 13:52 http://subversion02:8060/cru/CR-1921#c2929 // GeefVerzendwijze methode gebruiken uit apps.common.dataservice ipv zelf rechtstreeks %OpenId te doen Revision Comment by Laura Landuyt on 16 January 2020, 13:51 http://subversion02:8060/cru/CR-1921#c2928 //$Not van maken ipv ' ---------------------------------------- File: vhUnitTest/AXimpl/Dynamics/InventoryOut/BatchJobPakbonMagGeboektWorden/Test.cls.xml Revision Comment by Jo Willems on 19 March 2020, 11:05 http://subversion02:8060/cru/CR-1921#c3235 test of meerdere uitleveringen goed afgehandeld worden, cfr opm repository over flow.uitlevering.hoofding ---------------------------------------- File: AXimpl/Dynamics/InventoryOut/EventHandler/PakbonMagGeboektWorden/AanduidenAxBerichtPakbonMagGeboektWordenVerzonden.cls.xml Revision Comment by Laura Landuyt on 16 January 2020, 13:38 http://subversion02:8060/cru/CR-1921#c2922 ctor toevoegen en injecteren van de geeforderapiservice niet via context Revision Comment by Laura Landuyt on 16 January 2020, 13:39 http://subversion02:8060/cru/CR-1921#c2923 Niet gewoon gebruik maken van de globals maar de gemapte objecten gebruiken zoals Geefbon Daaraan een methode toevoegen als en Zetvlag(new) De zet vlag zal dan onderstaande set piece bevatten ---------------------------------------- File: AXimpl/Dynamics/InventoryOut/EventHandler/PakbonMagGeboektWordenHelper.cls.xml Revision Comment by Jo Willems on 18 March 2020, 16:52 http://subversion02:8060/cru/CR-1921#c3229 grrrr.... Revision Comment by Jo Willems on 18 March 2020, 16:53 http://subversion02:8060/cru/CR-1921#c3230 grrrrr... ---------------------------------------- File: DOM/VKP/Bon.cls.xml Revision Comment by Jo Willems on 18 March 2020, 10:42 http://subversion02:8060/cru/CR-1921#c3226 cfr andere opmerking in implementatie ---------------------------------------- File: DOM/VKP/impl/DataM/DataMBonAbstract.cls.xml Revision Comment by Jo Willems on 18 March 2020, 08:50 defect http://subversion02:8060/cru/CR-1921#c3225 Lijkt me 2 mooie methods om private te zetten en dan in specifiek genaamde methods voor speciefieke gegevens te gebruiken. Nu staat er ergens anders in de code een getal 63 te blinken.... --- ID: CR-1921 http://subversion02:8060/cru/CR-1921 Title: [ICT-1078] Fase 2 sturen bericht dat pakbon mag geboekt worden Statement of Objectives: State: Review Author: Laura Landuyt Moderator: Laura Landuyt Reviewers: (0 active, 1 completed*) Jo Willems (*)