1.7.7.x#1
Open
Salexber wants to merge 2110 commits intoSalexber:1.7.7.xfrom
Open
Conversation
Fix already selected condition group on safari inside catalog price rules
When the multishop option is activated and at least two shops have been created,
a new panel is displayed at the bottom of module configuration pages, containing a
checkbox labeled "Activate module for this shop context". Checking/unchecking
this checkbox is supposed to enable/disable the current module for the current shop
context.
This does not currently work. Clicking this box reloads the current url, adding
an "enable" parameter with a value of 0 or 1 depending on if it is checked or not. Getting
the status of the checkbox is achieved via the jquery .attr() method, which since jQuery 1.6
cannot be used for attributes like "checked" or "disabled". As a result "enable" is set to 0
wether the box is checked or not.
The solution is to use jQuery's .is(':checked') method instead.
After a bit of research, it seems .prop() is more direct, not requiring to parse the argument. Co-authored-by: atomiix <tbaccelli@gmail.com>
Fix hooks tests (Nightly error)
Fixed Ecotax when editing an Order ((Add/Update)ProductToOrder)
Implement Address Format in the View Order Page
…kbox Fix module activation checkbox in module configuration pages
Fix gift added/removed when editing order (product quantity/shipping address)
Co-authored-by: Progi1984 <progi1984@gmail.com>
Co-authored-by: Progi1984 <progi1984@gmail.com>
Build assets on 1.7.7.x
Check variable before using it
Fixed selector for fetching if the tax is included
Add changelog for 1.7.7.6 (DO NOT MERGE)
Co-authored-by: PululuK <pululuandre@hotmail.com>
Use _DB_PREFIX_ constant instead of ps_ in SQL queries
Bump version to 1.7.7.7
Name has no reason to live in admin_orders_display_customization_imge_route
Define version to 1778
…round Product combination price edit : Use blur instead of onkeyup to avoid price flip
Use null coalescing operator to fix notice
Changelog for 1.7.7.8
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
18 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Every detail helps: versions, browser/server configuration, specific module/theme, etc. Feel free to add more information below this table.