Skip to content

Conversation

@mage2pratik
Copy link
Contributor

Original Pull Request

#17773
Fix for ProductLink - setterName was incorrectly being set to an ucfirst instead of converting to camel case, which doesn't work as expected on attributes with underscores in their names.

To replicate, add a custom product link attribute in an extension that has an underscore in it. The normal expected setter routines in camel case do not work.

johnorourke and others added 2 commits September 4, 2018 19:30
…rst instead of converting to camel case, which doesn't work for models with underscores
…:snakeCaseToUpperCamelCase to maintain uniformity, and catch an additional case where the same issue occurred
@magento-engcom-team
Copy link
Contributor

Hi @mage2pratik. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

  • @magento-engcom-team give me test instance - deploy test instance based on PR changes
  • @magento-engcom-team give me {$VERSION} instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

@sidolov
Copy link
Contributor

sidolov commented Sep 5, 2018

Hi @mage2pratik , I'm closing this PR as a duplicate for #17772
Thank you for the collaboration!

@sidolov sidolov closed this Sep 5, 2018
@mage2pratik mage2pratik deleted the 2.3-develop-PR-port-17773 branch September 8, 2018 17:30
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.

5 participants