WS.AX.Transport.GLS.v01.RegistrationService.txt

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2574] - Telefoon nummer / provincie toevoegen in Leveradres

- testen van niet actieve code fixen.

    • -0
    • +1
    ./WS.AX.Transport.GLS.v01.RegistrationService.txt
[ICT-2532] langgoed -> wegwerppallet
[ICT-2532] langgoed -> wegwerppallet
[ICT-2532] langgoed -> wegwerppallet

- neven effect op niet gebruikte webservice.

    • -0
    • +1
    ./WS.AX.Transport.GLS.v01.RegistrationService.txt
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

- falende testen omdat nu op het einde geen extra CRLF wordt toegevoegd, zoals vroeger (foutief) gebeurde : er zat ook geen CRLF in de originele stream

    • -1
    • +1
    ./WS.AX.Transport.GLS.v01.RegistrationService.txt
  1. … 104 more files in changeset.
[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
[DEF735] TECH.StreamUtils::Replace voegt extra CRLF toe indien stream > 32000
  • More
  • CR-234
  • finished reviewing
[UST2240]VHIP: GLS: interface beschikbaar maken voor BizTalk
[UST2240]VHIP: GLS: interface beschikbaar maken voor BizTalk
  • More
  • CR-162
  • finished reviewing
Response uitbouwen op het einde van de code.. Niet we doen iets .. we bouwen den Response op en we doen nog iets en we quiten de response. Beter we doen iets , we doen nog iets , we bouwen de res...

Response uitbouwen op het einde van de code..

Niet we doen iets .. we bouwen den Response op en we doen nog iets en we quiten de response.

Beter we doen iets , we doen nog iets , we bouwen de response op en we quiten die

$$$If structuur gebruiken..

$$$If structuur gebruiken..

[UST2241]VHIP_GLS: RegisterContainer implementeren
[UST2241]VHIP_GLS: RegisterContainer implementeren
De parcel en unit id zijn gebaseerd op zelfde veld uit de verzend data. ze moeten dus de zelfde waarde hebben... idem voor de t&t url

De parcel en unit id zijn gebaseerd op zelfde veld uit de verzend data. ze moeten dus de zelfde waarde hebben...
idem voor de t&t url

  • More
  • CR-163
  • finished reviewing
Assert methods mogen geprivatiseerd worden

Assert methods mogen geprivatiseerd worden

Ik zou toch opteren voor andere data zodoende er een verschil tussen de GLSParcelNumber als de GLSUnitNumber zodat we zeker zijn we het juiste veld overgekopieerd hebben. Idem dito voor de TrackAnd...

Ik zou toch opteren voor andere data zodoende er een verschil tussen de GLSParcelNumber als de GLSUnitNumber zodat we zeker zijn we het juiste veld overgekopieerd hebben.
Idem dito voor de TrackAndTraceUrlVoorKlant en dienen voor VH

Er zal wel ergens vhTest.Utils.APPS.TRANSP.dto.LeveringAdres).EenBelg staan vermoed ik .. en als het nog niet zo is .. maak ze maar http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wik...

Er zal wel ergens vhTest.Utils.APPS.TRANSP.dto.LeveringAdres).EenBelg staan vermoed ik .. en als het nog niet zo is .. maak ze maar

??? waarom deze specifieke en niet de algemene ???

??? waarom deze specifieke en niet de algemene ???