Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Deze comment/opmerking is belangrijke informatie http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Deze comment/opmerking is belangrijke informatie

[ICT-2627][rvTVE] CARB Biztalk wijzigingen
[ICT-2627][rvTVE] CARB Biztalk wijzigingen
FYI : Bij een volgende uitbreiding van deze If-else, zullen we best de impl afzonderen in aparte method (of klasse). Bvb. wanneer Revego (of eenvoudig maatwerk) erbij komt, kan de eerste if zijn : ...

FYI :
Bij een volgende uitbreiding van deze If-else, zullen we best de impl afzonderen in aparte method (of klasse).
Bvb. wanneer Revego (of eenvoudig maatwerk) erbij komt, kan de eerste if zijn :

if Not(GeefFamilie() = "Lade") then ... else ... 


Of als je dat nu reeds wil doen, feel free

FYI: We hadden samen bekeken, in welke klasse we Mat-materiaal best zouden toevoegen. Het is nu pas, tijdens review, dat me opvalt dat deze klasse (met "...Omschrijving" in de naam) NIET taalafhank...

FYI:
We hadden samen bekeken, in welke klasse we Mat-materiaal best zouden toevoegen.
Het is nu pas, tijdens review, dat me opvalt dat deze klasse (met "...Omschrijving" in de naam) NIET taalafhankelijk is, maar dat is dus eerder toevallig, omdat LBX, TAX en TBX in alle talen hetzelfde is.

Het had zeker niet verkeerd geweest om deze klasse verder uit te breiden met vertaling en MatMateriaal. My bad!
Voorlopig zullen we het hierbij laten. De huidige oplossing is zeker ook oké.

[ICT-2707] HTTP post voor matten uitbreiden
[ICT-2707] HTTP post voor matten uitbreiden
Is uw probleem natuurlijk, ge moet lezen wat er staat...

Is uw probleem natuurlijk, ge moet lezen wat er staat...

Bij AssertEquals() is het derde argument enkel zinvol als je daar een uitleg wil schrijven. Met Asserts zoals hierboven lijkt het al gauw op $$$AssertEquals(StreamSize, "") Dat staat er natuur...

Bij AssertEquals() is het derde argument enkel zinvol als je daar een uitleg wil schrijven.

Met Asserts zoals hierboven lijkt het al gauw op

 	$$$AssertEquals(StreamSize, "") 

Dat staat er natuurlijk niet, maar mijn dislectisch brein interpretteert dit wel zo :-\
Ook met de andere Asserts in deze klasse heb ik moeite om snel te begrijpen wat er getest wordt.

Wil je de lege Comments in de Asserts (alle in de hele klasse) verwijderen? Uw voorganger heeft niet proper gedaan http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticon...

Wil je de lege Comments in de Asserts (alle in de hele klasse) verwijderen?

Uw voorganger heeft niet proper gedaan
Vandaar mijn vraag of je hier het Boyscout principe wil toepassen? Merci Jo.

Deze assert (van uw voorganger) kan VEEL specifieker en veiliger : Do $$$AssertEquals(Stream.Size, StreamSize + (9*25 )) ; i.e. "*" komt 9x per lijn voor, er zijn 25 lijnen. Wil je deze aanpa...

Deze assert (van uw voorganger) kan VEEL specifieker en veiliger :

	Do $$$AssertEquals(Stream.Size, StreamSize + (9*25 ))   ; i.e. "*" komt 9x per lijn voor, er zijn 25 lijnen.  

Wil je deze aanpassen? Thx.

Beter : Replace naar "Line ... AA" of naar "Line ... 00" , want "Line ... 99" bestaat reeds vooraleer iets ge-replaced wordt --> werkt wel, maar is een beetje misleidend. (wellicht ook in de volgen...

Beter :
Replace naar "Line ... AA" of naar "Line ... 00" , want "Line ... 99" bestaat reeds vooraleer iets ge-replaced wordt --> werkt wel, maar is een beetje misleidend.
(wellicht ook in de volgende testmethods).

[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
Hier ook... KUL killen please http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Hier ook... KUL killen please

KUL killen please http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

KUL killen please

[ICT350] Tests die klanten maken: schijfruimte-probl: tests onafh van echte klanten, of CRM-event...
[ICT350] Tests die klanten maken: schijfruimte-probl: tests onafh van echte klanten, of CRM-event...
Die wordt niet gebruikt, staat in commentaar. Nog af te stemmen met PRA > [UST2317] File system pad voor AX DocGen PDF's. commit 42048

Die wordt niet gebruikt, staat in commentaar. Nog af te stemmen met PRA > [UST2317] File system pad voor AX DocGen PDF's. commit 42048

  • More
  • CR-285
  • finished reviewing
Als er dan toch een configItem gebruikt wordt gelieve een duidelijker naamgeving AX_DOCGEN_[EXTRA_INFO_TER_DUIDING]

Als er dan toch een configItem gebruikt wordt gelieve een duidelijker naamgeving AX_DOCGEN_[EXTRA_INFO_TER_DUIDING]

Dit is inderdaad niet correct als definitieve code. Ik heb de laatste commit ook toegevoegd. De code hierboven was een tijdelijke implementatie voor cnext. Die wilden een path en bestandsnaam terug...

Dit is inderdaad niet correct als definitieve code. Ik heb de laatste commit ook toegevoegd. De code hierboven was een tijdelijke implementatie voor cnext. Die wilden een path en bestandsnaam terugkrijgen bij de call, onafhankelijk van hun input. De code in commentaar is uiteindelijk geactiveerd en fixed path is verwijderd voor productie.
-> AX_DOCGEN config is niet nodig tot nader order van PRA.(is voorzien door PRA, maar heeft momenteel geen nut.)

  • More
  • CR-285
  • resumed reviewing
Ik deel WV zijn mening : tenzij zo afsproken met ADU's

Ik deel WV zijn mening : tenzij zo afsproken met ADU's