Skip to content

Backport 7279 to 2.1 #10011

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

Merged
merged 3 commits into from
Jun 23, 2017
Merged

Backport 7279 to 2.1 #10011

merged 3 commits into from
Jun 23, 2017

Conversation

lazyguru
Copy link
Contributor

@lazyguru lazyguru commented Jun 21, 2017

Description

This backports #7279 to 2.1. The version number comparison in the upgrade script and the module.xml vary slightly to allow for other updates to potentially be backported. As this change is non-destructive there should be no issue with a store upgrading from 2.1.x to 2.2 and having these updates run again.

SolsWebdesign and others added 3 commits June 21, 2017 09:07
Bill-to Name and Ship-to Name trancated to 20 characters in backend
Compared the lengths of firstname, middlename and lastname of table "quote_address" with those of "quote" (255, 40 and 255 resp.) and of "sales_order_address" (255, 255 and 255) and with sales_order (128, 128 and 128) and choose to go with 255, 40 and 255 since these are used in "quote" as well and these are the values I know from Magento 1.9 as well. The other values are a bit inconsistent. Tested.
Bill-to Name and Ship-to Name trancated to 20 characters in backend
Compared the lengths of firstname, middlename and lastname of table "quote_address" with those of "quote" (255, 40 and 255 resp.) and of "sales_order_address" (255, 255 and 255) and with sales_order (128, 128 and 128) and choose to go with 255, 40 and 255 since these are used in "quote" as well and these are the values I know from Magento 1.9 as well. The other values are a bit inconsistent. Tested. Updated setup version of the Quote module to 2.0.5, moved all updates into a single if statement.
This allows for other changes that are in develop to be potentially
backported at a later date
@okorshenko okorshenko self-assigned this Jun 21, 2017
@okorshenko okorshenko added this to the June 2017 milestone Jun 21, 2017
@okorshenko okorshenko added the 2.1 label Jun 21, 2017
@okorshenko okorshenko changed the base branch from 2.1 to 2.1-develop June 21, 2017 19:08
@magento-team magento-team merged commit b84e648 into magento:2.1-develop Jun 23, 2017
magento-team pushed a commit that referenced this pull request Jun 23, 2017
@lazyguru lazyguru deleted the backport-7279-to-2_1 branch August 18, 2017 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants