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

Fixed add/edit click quick access on 2 system controllers #6601

Merged
merged 3 commits into from Oct 10, 2016

Conversation

Projects
None yet
2 participants
@aleeks
Contributor

aleeks commented Oct 6, 2016

Questions Answers
Branch? develop
Description? Fixed link quick access
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket?
How to test? See after
  • Check all links on sf controllers (product form or catalog)
  • Check all links on legacy controllers
  • Add a new link in sf controller
  • Add a new link in legacy controller
  • See if new links are ok on sf & legacy controllers
}
$quickLink = $this->getQuickLink($url);
return (isset($quickLink) && $quickLink === ($this->getQuickLink($_SERVER['REQUEST_URI'])));

This comment has been minimized.

@antoin-m

antoin-m Oct 10, 2016

Contributor

👍

@antoin-m

antoin-m Oct 10, 2016

Contributor

👍

Show outdated Hide outdated classes/QuickAccess.php Outdated
@antoin-m

This comment has been minimized.

Show comment
Hide comment
@antoin-m

antoin-m Oct 10, 2016

Contributor

Thank you! :D

Contributor

antoin-m commented Oct 10, 2016

Thank you! :D

@antoin-m antoin-m merged commit 29d6e30 into PrestaShop:develop Oct 10, 2016

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@aleeks aleeks deleted the aleeks:quick_access branch Oct 10, 2016

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