Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
  • More
  • CR-267
  • finished reviewing
[UST2446] VHIP_GLS : Exceptions opvangen in service methods.

- review opmerkingen : runtime exception afhandeling in aparte klasse.

  1. … 4 more files in changeset.
!!!! Duplicate code , Duplicate Code !!!! Ik zou voorstellen om een klasse te maken die de Runtime Exception Afhandeling doet .

!!!! Duplicate code , Duplicate Code !!!!

Ik zou voorstellen om een klasse te maken die de Runtime Exception Afhandeling doet .

[UST2446] VHIP_GLS : Exceptions opvangen in service methods.
[UST2446] VHIP_GLS : Exceptions opvangen in service methods.
[UST2446] VHIP_GLS : Exceptions opvangen in service methods.

- opvangen niet voorziene excepties + testen aanpassen of bijschrijven.

  1. … 7 more files in changeset.
  • More
  • CR-179
  • finished reviewing
VHIP_GLS : Exceptions opvangen in service methods.

- opvangen exceptions met boodschap voor gebruiker

- andere voorlopig nog verder smijten.

  1. … 5 more files in changeset.
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

- behandelen uitzonderings gevallen : onbestaande glsunitid, of andere dan "Registratie" laaste bericht.

  1. … 10 more files in changeset.
[UST2197]VHIP_GLS: GLS-comp: onafh. maken van Admin-data (transactionele en masterdata)

- VerzendData onder GLSv1 package

  1. … 40 more files in changeset.
[UST2305] VHIP_GLS: CancelContainer implementeren

- afzonderen controle op fout in response : ResponseControleur.cls

  1. … 11 more files in changeset.
[UST2305] VHIP_GLS: CancelContainer implementeren

- initiele versie + test om onderliggende calls te verifieren.

    • -0
    • +51
    ./CancelContainer.cls.xml
  1. … 13 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