Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-3275] [rvTVE] Oud framework voor CubeIQ opkuisen
  1. … 3 more files in changeset.
[ICT-1527] [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)

WDSL externe winkelkar aanpassen, toevoegen HerbestellingBijProbleem (en voorgaande parameters) - ook interne winkelkar aanpassen

    • -2
    • +2
    ./Winkelkar/InterneWinkelkarService.cls.xml
  1. … 1 more file in changeset.
[ICT-1527] [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)

Integratie testen: IngaveTotBestelling: aanpassen voor Haefele DE zodat de tag DAMAGE_BY_TRANSPORTATION_OR_ASSEMBLY ook wordt weergegeven

    • -2
    • +2
    ./Winkelkar/ExterneWinkelkarService.cls.xml
  1. … 1 more file in changeset.
De = $$$ True is in principe niet nodig (en wordt meestal niet gezet), maar daarover kan lang gediscussieerd worden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticon...

De = $$$ True is in principe niet nodig (en wordt meestal niet gezet), maar daarover kan lang gediscussieerd worden Zie de lijn erboven

Nitpicking, maar als het 123 is, is het niet random. Ofwel is het DummyWinkelkarId = $$$Integer als het echt niet belangrijk is, ofwel is het random en gebruik je de randomizer, ofwel moet het echt...

Nitpicking, maar als het 123 is, is het niet random. Ofwel is het DummyWinkelkarId = $$$Integer als het echt niet belangrijk is, ofwel is het random en gebruik je de randomizer, ofwel moet het echt 123 zijn

Kan je hier ##class(vhTest.Utils.APPS.VKP.Offerte).StandaardOfferte() of één van zijn vriendjes gebruiken?

Kan je hier ##class(vhTest.Utils.APPS.VKP.Offerte).StandaardOfferte() of één van zijn vriendjes gebruiken?

Ik vroeg me af of het interessanter is om de test in twee te splitsen zodat ze elk duiudelijk hun situatie testen, waardoor het beter opvalt wat er aan de hand is (door testomschrijving, en door va...

Ik vroeg me af of het interessanter is om de test in twee te splitsen zodat ze elk duiudelijk hun situatie testen, waardoor het beter opvalt wat er aan de hand is (door testomschrijving, en door variatiepunt in test. Een hele hoop van de asserts en opzet zijn dan gemeenschappelijk en kunnen in wat private methods gemikt worden.

Soit, 't is tijd om te stoppen, zal morgen verderdoen

fyi: in plaats van een hele lijst properties te asserten, zou je ook een AssertObject ofzo kunnen gebruiken.

fyi: in plaats van een hele lijst properties te asserten, zou je ook een AssertObject ofzo kunnen gebruiken.

ook hier werken met de fake datetimeapi

ook hier werken met de fake datetimeapi

Als de variabele ECHT geen bal uitmaakt, vervang hem dan door $$$DummyString (of $$$DummyInteger, ...). Dat maakt functioneel geen bal uit, maar maakt het voor de lezer duidelijk dat het echt niets...

Als de variabele ECHT geen bal uitmaakt, vervang hem dan door $$$DummyString (of $$$DummyInteger, ...).
Dat maakt functioneel geen bal uit, maar maakt het voor de lezer duidelijk dat het echt niets uitmaakt

Gebruik voor een vaste datum de DateTimeApi (bv Do ##class(vhTest.Fake.TECH.DateTime.DateTimeAPI).Activeer()) (voorbeelden in andere testen)

Gebruik voor een vaste datum de DateTimeApi (bv Do ##class(vhTest.Fake.TECH.DateTime.DateTimeAPI).Activeer()) (voorbeelden in andere testen)

Beter om de const te vervangen door een personality-enu

Beter om de const te vervangen door een personality-enu

ICT-1527: [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)
ICT-1527: [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)
IsInterneGebruikerToeglaten is altijd false, we zijn vandaag al dag 65468, dus dat tweede deel is altijd false. Eigenlijk mag dat dus weg, want wordt hieronder enkel 'ge-not' en zal dus altijd true...

IsInterneGebruikerToeglaten is altijd false, we zijn vandaag al dag 65468, dus dat tweede deel is altijd false. Eigenlijk mag dat dus weg, want wordt hieronder enkel 'ge-not' en zal dus altijd true zijn daar

vergelijkbare opmerking als SFS kaartje : boyscout-principe toepassen

vergelijkbare opmerking als SFS kaartje : boyscout-principe toepassen

quit als eerste lijn zetten

quit als eerste lijn zetten

dit is voor een groot deel copy-paste van andere code, maar misschien moet je toch even de ' door $$$Not vervangen, de 0/1 door $$$False/$$$True enz casing checken kan ook geen kwaad uitgecommentar...

dit is voor een groot deel copy-paste van andere code, maar misschien moet je toch even de ' door $$$Not vervangen, de 0/1 door $$$False/$$$True enz
casing checken kan ook geen kwaad
uitgecommentarieerde lijnen verwijderen
...

maw: toepassen van het boyscout-principe

$$$True ipv 1 en je kan die quit gerust op de eerste lijn zetten

$$$True ipv 1
en je kan die quit gerust op de eerste lijn zetten

[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave...
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave...
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave in handel-ERP)

- TestFile toevoegen voor nieuwe webservice

- ConfirmOrder endpoint kopiëren (van www01) en refactoren voor Haefele, hier wordt dus de implementatie toegevoegd (en gerefactord) aan cache2010 wat eerder op www01 stond

  1. … 4 more files in changeset.
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave...
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave...
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave in handel-ERP)

- Vorige commit te niet doen

- ConfirmOrder endpoint kopiëren (van www01) en refactoren voor SFS, hier wordt dus de implementatie toegevoegd aan cache2010 wat eerder op www01 stond

    • -2
    • +4
    ./UNI/SFS/OrderServiceInternal.cls.xml
  1. … 8 more files in changeset.
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave in handel-ERP)

- ConfirmOrder hernoemen naar ConfirmOrderOud om zo de oude functionaliteit te behouden

    • -2
    • +2
    ./UNI/SFS/OrderServiceInternal.cls.xml
  1. … 1 more file in changeset.
Commentaar bij errors Vhintra interne winkelkar.