New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BOOM-1223] [Again!!] Undefined $urls index in ajax rendered template #10638

Closed
b0tm1nd opened this Issue Sep 21, 2018 · 8 comments

Comments

4 participants
@b0tm1nd
Copy link

b0tm1nd commented Sep 21, 2018

Describe the bug
The bug described in http://forge.prestashop.com/browse/BOOM-1223 and supposed to be fixed by #8856 still present in 1.7.4.1 version.

Additionnal information
PrestaShop version: 1.7.4.1

@khouloudbelguith

This comment has been minimized.

Copy link
Contributor

khouloudbelguith commented Sep 21, 2018

Hi @b0tm1nd,

Could you please add steps to reproduce the issue in a new comment.

Best regards, Khouloud

@b0tm1nd

This comment has been minimized.

@b0tm1nd

This comment has been minimized.

@khouloudbelguith

This comment has been minimized.

Copy link
Contributor

khouloudbelguith commented Sep 21, 2018

@eternoendless @mickaelandrieu what do you think of this issue?

@b0tm1nd

This comment has been minimized.

Copy link

b0tm1nd commented Sep 21, 2018

You can put say {$urls.theme_assets} somewhere in your /themes/yourtheme/modules/ps_shoppingcart/ps_shoppingcart.tpl and just load this in the browser https://yourdomain.com/module/ps_shoppingcart/ajax

In debug mode it will give 500 error with undefined $urls, in normal mode it will be just empty in place of the var.

@khouloudbelguith

This comment has been minimized.

Copy link
Contributor

khouloudbelguith commented Sep 21, 2018

Hi @b0tm1nd,

Thanks for this clarifications.
I manage to reproduce this issue with PS1.7.4.2
image
We will see how to fix it.

Best regards, Khouloud

@khouloudbelguith khouloudbelguith added Bug Minor CO TBR and removed NMI labels Sep 21, 2018

@marionf marionf added To Do and removed TBR labels Sep 26, 2018

@marionf marionf removed their assignment Sep 26, 2018

@marionf marionf added this to To do in PrestaShop 1.7.5 via automation Sep 27, 2018

@marionf marionf moved this from To do to Up next in PrestaShop 1.7.5 Sep 27, 2018

@marionf marionf removed the To Do label Oct 1, 2018

@marionf marionf moved this from Up next to To be reviewed in PrestaShop 1.7.5 Oct 1, 2018

@eternoendless

This comment has been minimized.

Copy link
Member

eternoendless commented Oct 1, 2018

Hi there,

As it happens, the original bug had been fixed in the core but not in the ps_shoppingcart module.

@marionf marionf added this to the 1.7.5.0 milestone Oct 1, 2018

@marionf marionf moved this from To be reviewed to Done in PrestaShop 1.7.5 Oct 1, 2018

@eternoendless

This comment has been minimized.

Copy link
Member

eternoendless commented Oct 1, 2018

Fixed in v2.0.2

@marionf marionf closed this Oct 1, 2018

@eternoendless eternoendless added the Fixed label Oct 1, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment