• Priority: 0
  • Status: Closed
  • Theme: Venedor
  • Assigned To: Marek
  • Private: No
  • Open Date: 07.10.17, 22:56
  • Opened by: Maciej
  • Closed by: Marek
  • Closed on: 29.12.17, 11:31
  • Reason: Closed
  • Comment: Glad to help!

Ticket #10912 - Price impact in combination - wrong calculations

versions: TS v.1.5.9.6 | venedor v.2.4 | PS v.1.6.1.11

example:
DEFAULT THEME - correct calculation
http://xhdr.eu/panele-led/813-panel-led-24w-o-szerokosci-6-cali-homologacja-led-cree-prostokatny-155mm.html

VENEDOR 2.4 - wrong calculation
https://sklep.jeepmania.pl/pl/panele-led/813-panel-led-24w-o-szerokosci-6-cali-homologacja-led-cree-prostokatny-155mm.html

Price impact for combinations on this product are: 0€ / +1€ / -1 € (1€ is around 4.3 zł)

CORRECT calculations are: 123 zł / 127 zł / 118 zł (works in default theme)
VENEDOR calculates: 123 zł / 141 zł / 104 zł

Comments

Marek 09 Oct 2017, 11:49

Hi Maciej.
So when I change the “WIĄZKA” the price must be changed like this:

  • FLOOD 123 zł
  • COMBO 127 zł
  • SPOT 118 zł

And one more question is that I don’t see product price, but only a message “zapytaj o dostępność”

Maciej 09 Oct 2017, 12:05

Correct price calculation (default theme):
http://xhdr.eu/panele-led/813-panel-led-24w-o-szerokosci-6-cali-homologacja-led-cree-prostokatny-155mm.html

COMBO: 123 zł (impact 0€) - default combination
FLOOD: 127 zł (impact +1€)
SPOT: 118 zł (impact -1€)
— VENEDOR in English (wrong calculations)
https://sklep.jeepmania.pl/en/panele-led/813-led-bar-24w-6-inches-e-mark-led-cree-155mm.html#/beam-combo COMBO: 123 zł
FLOOD: 142 zł
SPOT: 104 zł

I see prices - see attached screenshots …

Marek 09 Oct 2017, 19:05

I need access to your back office to enable force compilation

Maciej 09 Oct 2017, 21:39

Access granted. Details in my profile.

Marek 09 Oct 2017, 22:57

The issue has been fixed. Please check it out

Maciej 09 Oct 2017, 23:39

Thank you … now it works.
So product.tpl and product.js.
something else?
I had to recover some TPL modifications (our own site specific changes) but I maneged to include your fix (your new JS references) in our version of TPL.

Marek 09 Oct 2017, 23:43

Yes, I have updated those two files. You can use your modified tpl file, I’ll add there missed js variables