Skip to content
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

[Backport] Fixed set template syntax issue #16832

Merged

Conversation

gelanivishal
Copy link
Contributor

Original Pull Request

#15825

Description

I have added module name on a template file as per Magento standard coding format.

Fixed Issues (if relevant)

N/A

Manual testing scenarios

I have changed all Magneto Bundle module template. so You can check after login on Magento Dashboard.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@sidolov
Copy link
Contributor

sidolov commented Jul 15, 2018

@gelanivishal , please, revert extra changes from Magento/Bundle/Block/Catalog/Product/View/Type/Bundle/Option/Multi.php

@gelanivishal
Copy link
Contributor Author

@sidolov I have reverted back changes as requested. Please review at once.

@magento-engcom-team
Copy link
Contributor

Hi @gelanivishal. Thank you for your contribution.
We will aim to release these changes as part of 2.1.15.
Please check the release notes for final confirmation.

magento-engcom-team added a commit that referenced this pull request Aug 28, 2019
…it returns a store specific value even if there is no default value #23369

 - Merge Pull Request #23369 from semajeg/magento2:16382-getAttributeRawValue-no-default
 - Merged commits:
   1. 01f1f15
   2. 64184b3
   3. 35c5185
   4. 9ae33c8
   5. 8c4bfa6
   6. 7b0e528
   7. c34f83c
   8. 4d380bf
   9. 540f03e
magento-engcom-team pushed a commit that referenced this pull request Aug 28, 2019
…it returns a store specific value even if there is no default value #23369
magento-engcom-team added a commit that referenced this pull request Aug 28, 2019
…op expedited

Accepted Community Pull Requests:
 - #24253: Add tip for upper limit of the Email Text Length Limit field range (by @TomashKhamlai)
 - #24252: Correct the return of type of function _columnBrake in module-theme block topMenu.php (by @Krielkip)
 - #23369: #16832 - Reworked query in getAttributeRawValue so that it returns a store specific value even if there is no default value (by @semajeg)
 - #24249: Resolve Mass Delete Subscribers doesn't have "confirm" message issue24248 (by @edenduong)
 - #24242: "Save Product" with the invalid data must scroll to the error message issue24241 (by @edenduong)
 - #24341: Correct spelling (by @ravi-chandra3197)
 - #24327: Update Magento_AuthorizenetCardinal ReadMe (by @diazwatson)
 - #24328: Update Magento_AuthorizenetGraphQl ReadMe (by @diazwatson)
 - #24326: Update Magento_AuthorizenetAcceptjs Readme (by @diazwatson)
 - #24278: Update Magento_AsynchronousOperations ReadMe (by @diazwatson)
 - #24265: Correction in the shipping method if it is empty it no longer shows t� (by @wbeltranc)
 - #18678: Throw exception to prevent infinite loop caused by third party code. (by @convenient)


Fixed GitHub Issues:
 - #24251: Module-Theme: TopMenu invalid type of value returned (reported by @Krielkip) has been fixed in #24252 by @Krielkip in 2.3-develop branch
   Related commits:
     1. 60ea2be
     2. e9dd493

 - #16382: Product resource getAttributeRawValue function does not return anything if there is no default value (reported by @VincentMarmiesse) has been fixed in #23369 by @semajeg in 2.3-develop branch
   Related commits:
     1. 01f1f15
     2. 64184b3
     3. 35c5185
     4. 9ae33c8
     5. 8c4bfa6
     6. 7b0e528
     7. c34f83c
     8. 4d380bf
     9. 540f03e

 - #24248: Mass Delete Subscribers doesn't have "confirm" message (reported by @edenduong) has been fixed in #24249 by @edenduong in 2.3-develop branch
   Related commits:
     1. 01edbdc

 - #24241: "Save Product" with the invalid data must scroll to the error message (reported by @edenduong) has been fixed in #24242 by @edenduong in 2.3-develop branch
   Related commits:
     1. 157e1cd

 - #19853: Shipping method undefined issue (reported by @namratavora) has been fixed in #24265 by @wbeltranc in 2.3-develop branch
   Related commits:
     1. 10f880c
     2. 775eab4
     3. d8c8574
     4. 86228e9
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