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

[-] BO: Fix PSCSX-6208 / Call to undefined method stdClass::setRedire… #3414

Closed
wants to merge 1 commit into
base: 1.6.1.x
from

Conversation

Projects
None yet
3 participants
@PrestaEdit
Contributor

PrestaEdit commented Jul 13, 2015

…ctAfter()

@jnadaud

This comment has been minimized.

Show comment
Hide comment
@jnadaud

jnadaud Jul 17, 2015

Contributor

Hi,

I think this is a better way to fix this issue : #3448
What do you think ?

Best regards.

Contributor

jnadaud commented Jul 17, 2015

Hi,

I think this is a better way to fix this issue : #3448
What do you think ?

Best regards.

@jnadaud jnadaud added Bug and removed pending review labels Jul 17, 2015

@PrestaEdit

This comment has been minimized.

Show comment
Hide comment
@PrestaEdit

PrestaEdit Jul 17, 2015

Contributor

Why not. I think that if you just change "AdminController" by "AdminDashboard" in this PR, it's good.

But, of course, yours is good to. I don't have time to testing it anymore, but at my first read, it's good.

I think you can go with it ;-)

Contributor

PrestaEdit commented Jul 17, 2015

Why not. I think that if you just change "AdminController" by "AdminDashboard" in this PR, it's good.

But, of course, yours is good to. I don't have time to testing it anymore, but at my first read, it's good.

I think you can go with it ;-)

@jnadaud

This comment has been minimized.

Show comment
Hide comment
@jnadaud

jnadaud Jul 17, 2015

Contributor

Not just adminDashboard, but also calling controller run() method in order to display profiling redirection caught :-)

Contributor

jnadaud commented Jul 17, 2015

Not just adminDashboard, but also calling controller run() method in order to display profiling redirection caught :-)

@jnadaud jnadaud closed this Jul 17, 2015

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