CacheAdminA_trunk2010/WS/EC/Unishop/InternationaleZending

impl

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 2
Last Commit: 03 May 17
Commits this week: 0
Total Lines of Code (LoC): 0
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wednesday 17 May 2017
2:07 pm

Kurt Callebaut

12:11 pm

Kurt Callebaut

zie hierboven

zie hierboven

12:11 pm

Kurt Callebaut

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebru...

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebruiker"()
Ik heb in ieder geval een test toegevoegd die controleert dat er geen verzendingmogelijkheden berekend worden voor standaard klanten.

11:38 am

Kurt Callebaut

aangezien deploy wordt er geen aanpassing doorgevoerd.

aangezien deploy wordt er geen aanpassing doorgevoerd.

9:45 am

Tom Vermeulen

  • More
  • CR-227
  • finished reviewing
9:44 am

Tom Vermeulen

Wat met klanten zonder transportkeuze?

Wat met klanten zonder transportkeuze?

9:44 am

Tom Vermeulen

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

9:43 am

Tom Vermeulen

ExactAantalKeer(1) om intentie duidelijk te maken

ExactAantalKeer(1) om intentie duidelijk te maken

9:42 am

Tom Vermeulen

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

9:40 am

Tom Vermeulen

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

9:02 am

Tom Vermeulen

beter de vhTest.Fake.TECH.Mail ?

beter de vhTest.Fake.TECH.Mail ?

8:51 am

Tom Vermeulen

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?

8:40 am

Tom Vermeulen

testje voor 0 palletten toevoegen

testje voor 0 palletten toevoegen

8:28 am

Tom Vermeulen

$$$Not

$$$Not

Tuesday 16 May 2017
2:29 pm

Kurt Callebaut

[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven
Monday 21 Nov 2016
9:46 am

KurtC

Reverse merged revision(s) 37918-37917 from trunk2010: (WIJZIGINGEN ONGEDAAN GEMAAKT ! WEGWERPPALLET ALTIJD OP TUSSENETIKET ALS VERZENDWIJZE DHL IS, DUS LOS VAN UNISHOP VERZENDINGMOGELIJKHEDEN EN OPTIBOX PALLETBEREKENINGEN)

[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- ColliDragerType wordt bepaald en opgeslagen bij zendingdetails.

(beter zou zijn als de volledige flow doorlopen wordt om dit te bepalen. Dwz bij request om palletten te bepalen reeds meegeven welke palletten / dragers er ondersteund worden voor een bepaalde klant (bv Hein). Deze waarden meegeven naar de optibox en het resultaat van de optibox bepaalt dan ColliDragerType dat via verzendingmogelijkheden, offerte,... op het etiket komt. Nog niet doorgevoerd omdat er voor andere klanten nog geen nood is voor extra info op etiket.)

........

[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- Overbodige code is verwijderd uit test.

........

Options
    • -19
    • +0
    ./ZendingDetails.cls.xml
  2. … 9 more files in changeset.
8:54 am

KurtC

[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- ColliDragerType wordt bepaald en opgeslagen bij zendingdetails.

(beter zou zijn als de volledige flow doorlopen wordt om dit te bepalen. Dwz bij request om palletten te bepalen reeds meegeven welke palletten / dragers er ondersteund worden voor een bepaalde klant (bv Hein). Deze waarden meegeven naar de optibox en het resultaat van de optibox bepaalt dan ColliDragerType dat via verzendingmogelijkheden, offerte,... op het etiket komt. Nog niet doorgevoerd omdat er voor andere klanten nog geen nood is voor extra info op etiket.)

Options
    • -0
    • +19
    ./ZendingDetails.cls.xml
  2. … 9 more files in changeset.
Tuesday 08 Nov 2016
7:56 am

KurtC

[UST1914] Klant Hein: altijd in 40€-DHL-modus draaien

- Kostprijsbepaling -> testserver. Testen nog na te zien.

Options
    • -3
    • +3
    ./ZendingDetails.cls.xml
Wednesday 26 Oct 2016
1:47 pm

KurtC

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Sessie voor LandenBepaler met klanten array

Options
    • -2
    • +2
    ./LandenBepalerFactory.cls.xml
  2. … 7 more files in changeset.
12:26 pm

KurtC

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- PersonalityID en KlantID in Sessie object aangepast voor VerkoopprijsBerekenaarFactory.

Options
    • -2
    • +2
    ./VerkoopprijsBerekenaarFactory.cls.xml
    • -1
    • +1
    ./ZendingDetails.cls.xml
  3. … 4 more files in changeset.
11:59 am

KurtC

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- PersonalityID en KlantID in Sessie object aangepast voor TransportFirmaBepalerFactory.

Options
    • -2
    • +2
    ./TransportFirmaBepalerFactory.cls.xml
    • -12
    • +6
    ./ZendingDetails.cls.xml
  3. … 14 more files in changeset.
Thursday 20 Oct 2016
1:10 pm

KurtC

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Refact verkoopprijs naar TransportKost (servicecontract webservice voor intracto.)

- #dims toegevoegd waar nodig.

Options
    • -2
    • +2
    ./ZendingDetails.cls.xml
  2. … 4 more files in changeset.
12:57 pm

KurtC

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Beschikbarelanden refactored -> opsplitsen Van Hoecke klanten (tax export) en Hein klant en personality.

- Beschikbarelanden refactored -> Personality en klantID worden meegegeven vanuit de ExterneVerkoopService.

- Hein klant en personality geeft enkel land DE op de webshop.

- Beschikbare landen unishop losgekoppeld van Beschikbare landen in de offerteBerekenaar voor DHL.

Options
    • -0
    • +36
    ./LandenBepalerFactory.cls.xml
  2. … 20 more files in changeset.
Wednesday 19 Oct 2016
1:58 pm

KurtC

[UST1914] Klant Hein: altijd in 40€-DHL-modus draaien

- PersonalityID en KlantID moeten niet meer opgehaald worden om transportfirmas en verkoopprijzen te bepalen,

- PersonalityID en KlantID worden meegegeven vanuit ExterneVerkoopService.

(-> Later, misschien, mogelijks, eventueel nog de 4 parameters in de method calls te vervangen door een object.)

Options
    • -18
    • +4
    ./ZendingDetails.cls.xml
  2. … 5 more files in changeset.
1:25 pm

KurtC

[UST1914] Klant Hein: altijd in 40€-DHL-modus draaien

- Testen aangepast voor onderstaande wijziging.

- VerkoopprijsBerekenaarFactory en TransportFirmaBepalerFactory aangepast met variabelen personalityID en KlantID.

- Methods nog op te roepen om personality en klant te bepalen en in te vullen. (zendingdetails)

Options
    • -9
    • +6
    ./TransportFirmaBepalerFactory.cls.xml
    • -2
    • +29
    ./ZendingDetails.cls.xml
    • -11
    • +8
    ./VerkoopprijsBerekenaarFactory.cls.xml
  4. … 5 more files in changeset.
12:02 pm

KurtC

[UST1914] Klant Hein: altijd in 40€-DHL-modus draaien

- refact ivm injecten van beide klassen transportfirmabepaler en verkoopprijsberekenaar in zendingdetails.

Options
    • -1
    • +1
    ./TransportFirmaBepalerFactory.cls.xml
    • -9
    • +9
    ./ZendingDetails.cls.xml
    • -1
    • +1
    ./VerkoopprijsBerekenaarFactory.cls.xml
  4. … 5 more files in changeset.
11:47 am

KurtC

[UST1914] Klant Hein: altijd in 40€-DHL-modus draaien

- ahdv Hein klant ook transportfirma en kostprijs bepalen.

- Deze (dubbele) code om personality en klant te bepalen moet vervangen worden door variabelen in de method call.

- Test voor klant hein moet nog gebeuren, na refact "variabelen in method call" , hiervoor is een aanpassing nodig ivm injecten van beide klassen transportfirmabepaler en verkoopprijsberekenaar in zendingdetails.

Options
    • -3
    • +21
    ./TransportFirmaBepalerFactory.cls.xml
    • -2
    • +21
    ./VerkoopprijsBerekenaarFactory.cls.xml
Tuesday 18 Oct 2016
3:03 pm

KurtC

[UST1914] Klant Hein: altijd in 40€-DHL-modus draaien

- Improvements

Options
    • -0
    • +1
    ./TransportFirmaBepalerFactory.cls.xml
  2. … 1 more file in changeset.
Thursday 13 Oct 2016
9:48 am

KurtC

[UST1886] Tekst /Titel meegeven met verzendmogelijkheden naar webshop.

- Refact / rename Verzendmogelijkheidlist

- niet gebruikte code, DummyService verwijderd.

Options
    • -2
    • +2
    ./ZendingDetails.cls.xml
  2. … 10 more files in changeset.
8:43 am

KurtC

[UST1884] [rvJCL] Hein - Transportkost offertedata (domeinobject) en verzenditems (persistent) opsplitsen aankoop en verkoop

- refactoring na review.

-> testnaam verduidelijkt.

-> Hernoemen verzendmogelijkheid.

Options
    • -58
    • +61
    ./ZendingDetails.cls.xml
  2. … 3 more files in changeset.