CommunicatieService.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UW665] GLS bijkomende logging om sneller te zien wat er met een bepaalde zending gebeurd is.

- toplevel calls

- type van bericht dat verzonden gaat worden

  1. … 1 more file in changeset.
[UW493] Parcels verzenden faalt nog gedeeltelijk.
[UW493] Parcels verzenden faalt nog gedeeltelijk.
[UW493] Parcels verzenden faalt nog gedeeltelijk.

- Logging weg

[UW493] Parcels verzenden faalt nog gedeeltelijk.

- Logging toevoegen. Vastelling is dat een rit 215 parcels / leveringen heeft en dat er de dag voordien bij het afsluiten van een transport er maar 102 releases doorgestuurd zijn naar GLS. maw van de overige hebben we gegevens tekort voor track and trace. We hebben ook geen foutmelding gezien. het doorsturen is gewoon gestopt volgens de logging.

-> dus meer logging om te achterhalen of dit gaat om timeout ? onvolledige lijst ? breekt de iteratie af of niet ???

-> het gaat niet om de laatste parcels die geregistreerd zijn, maar willekeurig. Er zijn ook parcels van vm of bv 3u nm die niet gereleased zijn, terwijl er ook parcels gereleased zijn van smorgens en savonds.

  • More
  • CR-179
  • finished reviewing
Het is te hopen dat de andere code dat ook doet !? Gelieve hiervoor een betere omschrijving te bedenken. Een korte samenvatting van het wat gedrag is m.a.w. en niet hoe maar wat er juist gedaan wo...

Het is te hopen dat de andere code dat ook doet !?

Gelieve hiervoor een betere omschrijving te bedenken. Een korte samenvatting van het wat gedrag is m.a.w. en niet hoe maar wat er juist gedaan wordt.

Beter een $$$Not(BevatFout) zodoende 0 en "" hetzelfde terug geven.

Beter een $$$Not(BevatFout) zodoende 0 en "" hetzelfde terug geven.

  • More
  • CR-161
  • finished reviewing
macro $$$false gebruiken ipv 0

macro $$$false gebruiken ipv 0

Gelieve de bewuste id in de foutmelding op te nemen en indien mogelijk ook welke type bericht het dan wel was, kwestie om zo duidelijk mogelijke foutmelding te kunnen geven. ps : Enkel eigennamen o...

Gelieve de bewuste id in de foutmelding op te nemen en indien mogelijk ook welke type bericht het dan wel was, kwestie om zo duidelijk mogelijke foutmelding te kunnen geven.
ps : Enkel eigennamen of het begin van een zin begint met een hoofdletter

White space http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

White space

GeefVerzendData zegt weinig over de wat voor verzend data het is ? Als het echt niet uitmaakt gelieve dan een ##class(vhTest.Dummy) te gebruiken als het wel van belang is gelieve dan een iets duide...

GeefVerzendData zegt weinig over de wat voor verzend data het is ? Als het echt niet uitmaakt gelieve dan een ##class(vhTest.Dummy) te gebruiken als het wel van belang is gelieve dan een iets duidelijkere benaming te kiezen.

Utils mogen classmethods zijn

Utils mogen classmethods zijn

Gelieve het gedrag en het gewenste resultaat iets duidelijk in de test te vermelden zodoende er kan gecontroleerd worden dat de test setup wel is volgens het vooropgestelde gedrag. De volgende stap...

Gelieve het gedrag en het gewenste resultaat iets duidelijk in de test te vermelden zodoende er kan gecontroleerd worden dat de test setup wel is volgens het vooropgestelde gedrag. De volgende stap is dat dat de code voldoet aan het vooropgesteld gedrag van de test.

Lijkt mij beter om VerzendData uit te breiden met BevatFout en Foutmelding zodoende er maar één object moet doorgegeven worden. Idem dito voor de CancelContainer..

Lijkt mij beter om VerzendData uit te breiden met BevatFout en Foutmelding zodoende er maar één object moet doorgegeven worden. Idem dito voor de CancelContainer..

?? mag deze method niet weg daar de response controleur kan gebruikt worden ??

?? mag deze method niet weg daar de response controleur kan gebruikt worden ??

[UST2305] VHIP_GLS: CancelContainer implementeren
[UST2305] VHIP_GLS: CancelContainer implementeren
[UST2305] VHIP_GLS: CancelContainer implementeren

- afzonderen controle op fout in response : ResponseControleur.cls

  1. … 11 more files in changeset.
Verzekeren dat er nergens data zit die niet verwijst naar niet GLSv1 als dat zo is deploy klasse maken om die data te corrigeren.

Verzekeren dat er nergens data zit die niet verwijst naar niet GLSv1 als dat zo is deploy klasse maken om die data te corrigeren.

Lijkt mij beter om deze klasse om te vormen naar een vhTest.Utils klasse .. kwestie van een beetje conformiteit in de code base te krijgen

Lijkt mij beter om deze klasse om te vormen naar een vhTest.Utils klasse .. kwestie van een beetje conformiteit in de code base te krijgen

AX testen zou ik niet samen steken met andere testen.. De zaken duidelijk van elkaar gescheiden houden lijkt mij wel handig in dit concept zodoende we makkelijk de zaken kunnen opkuisen eens de zak...

AX testen zou ik niet samen steken met andere testen.. De zaken duidelijk van elkaar gescheiden houden lijkt mij wel handig in dit concept zodoende we makkelijk de zaken kunnen opkuisen eens de zaken effectief in AX zitten.

auwch

auwch

Factory is niet testbaar daar er een classmethod gebruikt wordt . . -> depedency ook naar boven brengen er nu een andere klasse gebruikt wordt om dto klassen te maken.

Factory is niet testbaar daar er een classmethod gebruikt wordt . . -> depedency ook naar boven brengen er nu een andere klasse gebruikt wordt om dto klassen te maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

GlsTools verplaatsen naar constructor -> depedencies zichtbaar maken.

Foutmelding zal hoogstwaarschijnlijk is niet mogelijk moeten zijn. Foutmelding dan ook beetje uniformiseren en de waarde tussen ' plaatsen

Foutmelding zal hoogstwaarschijnlijk is niet mogelijk moeten zijn. Foutmelding dan ook beetje uniformiseren en de waarde tussen ' plaatsen