Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Reverse merged revision(s) 56420 from trunk2010:

Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

revert van de revert omdat mijn wijzigingen wel goed waren, maar de rest van de ventjes niet :)

    • -2
    • +1
    ./Converter/TeOntvangenProductLocker.cls.xml
  1. … 65 more files in changeset.
Reverse merged revision(s) 56414 from trunk2010:

[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...

........

revert omdat mijn xml export blijkbaar nogal verschillend is van de rest van de ventjes hier

    • -1
    • +2
    ./Converter/TeOntvangenProductLocker.cls.xml
  1. … 65 more files in changeset.
[ICT] Aanpassingen als gevolg van een shshsh die ik heb uitgevoerd lokaal...
    • -2
    • +1
    ./Converter/TeOntvangenProductLocker.cls.xml
  1. … 65 more files in changeset.
[UST3558] VHIP904 receptie goederen: alle betrokken producten locken
[UST3558] VHIP904 receptie goederen: alle betrokken producten locken
[UST3558] VHIP904 receptie goederen: alle betrokken producten locken

- Aparte klasse om producten te locken obv AX bericht.

    • -0
    • +36
    ./Converter/TeOntvangenProductLocker.cls.xml
  1. … 7 more files in changeset.
  • More
  • CR-774
  • finished reviewing
zal ne restant zijn. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (overige testen hebben ook wlip.)

zal ne restant zijn.
(overige testen hebben ook wlip.)

Algemeen zou ik lange methods opsplitsen in kleinere. zodat de naam van de method verduidelijkt wat er gebeurd. Dan hoeft commentaar niet. bv ControleerTerugnameMaatwerk, etc ...

Algemeen zou ik lange methods opsplitsen in kleinere. zodat de naam van de method verduidelijkt wat er gebeurd. Dan hoeft commentaar niet. bv ControleerTerugnameMaatwerk, etc ...

Geen idee of het belangrijk is, maar bij Journaal staat er 01234

Geen idee of het belangrijk is, maar bij Journaal staat er 01234

Storages op dto's nog opnieuw toe te voegen.

Storages op dto's nog opnieuw toe te voegen.

[UST3198] VHIP904/VHIP903 : ontvangst: error tekst indien geen klant of leverancier
[UST3198] VHIP904/VHIP903 : ontvangst: error tekst indien geen klant of leverancier
[UST3198] VHIP904: ontvangst: error tekst indien geen klant of leverancier

- blijkbaar iets fout gedaan bij vorige commit

    • -9
    • +0
    ./Converter/InventoryInConverter.cls.xml
    • -0
    • +18
    ./Converter/ConversieBezwarenBepaler.cls.xml
  1. … 9 more files in changeset.
Getter zonder returnwaarde? Ik ga het zo laten.

Getter zonder returnwaarde? Ik ga het zo laten.

  • More
  • CR-497
  • finished reviewing
  • More
  • CR-552
  • finished reviewing
Is dit eigenlijk geen Getter ipv een Zetter ? maar één via byRef

Is dit eigenlijk geen Getter ipv een Zetter ? maar één via byRef

Moeilijk te doen zonder complete integratie test op te zetten. commentaar in InventoryOut convertor gezet met alle converters in het zelfde geval.

Moeilijk te doen zonder complete integratie test op te zetten. commentaar in InventoryOut convertor gezet met alle converters in het zelfde geval.

[ust2726] VHIP904: ontvangst: mapping velden uit XSD bij maken van toelevering

- asserten op response

    • -5
    • +3
    ./Converter/InventoryInConverter.cls.xml
  1. … 3 more files in changeset.
In standaard gedrag word getest of de converter van de terug te geven response opgeroepen wordt. De mock geeft een voorgedefinieerd antwoord terug. Asserten van dit lijkt mij niet direct zinvol hie...

In standaard gedrag word getest of de converter van de terug te geven response opgeroepen wordt. De mock geeft een voorgedefinieerd antwoord terug.
Asserten van dit lijkt mij niet direct zinvol hier, maar wel in de test van die converter.

is al gebeurd, wsl outdated

is al gebeurd, wsl outdated

anders opgelost.

anders opgelost.

Gelieve ook de response te asserten

Gelieve ook de response te asserten

'k zou hier opteren om de LijnId en ProductId mee te geven zodat je duidelijk aan toont dat het over het zelfde product gaat met verschillende LijnId's

'k zou hier opteren om de LijnId en ProductId mee te geven zodat je duidelijk aan toont dat het over het zelfde product gaat met verschillende LijnId's

Gelieve het resultaat duidelijk te maken in de response

Gelieve het resultaat duidelijk te maken in de response

Gelieve in uw test benamingen ook uw gedrag te beschrijven maw het resultaat bvb Test: 2 Lijnen voor zelfde leverancier worden toegevoegd aan de receptie

Gelieve in uw test benamingen ook uw gedrag te beschrijven maw het resultaat

bvb Test: 2 Lijnen voor zelfde leverancier worden toegevoegd aan de receptie