Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[BSR]
[BSR]
  • More
  • CR-267
  • finished reviewing
!!!! 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.
[UST2305] VHIP_GLS: CancelContainer implementeren

- review opmerkingen (%New() weg om classmethod op te roepen )

  • 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.
[UST2305] VHIP_GLS: CancelContainer implementeren

- review opmerkingen

  1. … 1 more file 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
[UST2305] VHIP_GLS: CancelContainer implementeren

-review opm.

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

  1. … 12 more files in changeset.