[Backport 2.3 -> 2.2] Fix meta title property & Updated unit test#16311
[Backport 2.3 -> 2.2] Fix meta title property & Updated unit test#16311gelanivishal wants to merge 2 commits intomagento:2.2-developfrom
Conversation
|
Hi @vgelani. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
orlangur
left a comment
There was a problem hiding this comment.
@vgelani please do not rewrite authors of original commits during porting. Feel free to do just a one commit with git commit -c 32e2e4de5921ecb7b13722955298e886ec2b2d59, just that @slackerzz should be properly credited as author and you as author of port will be mentioned as committer.
|
@orlangur Yeah, I got it. I'll take care in next PR. |
|
@vgelani, thanks for your endeavors, @slackerzz will take care of this PR porting. He noticed some changes were missed which resulted in failing unit test. |
Original Pull Request
#11368
Description
If, inside a controller you try to set metadata title with:
you will get an error on frontend:
Recoverable Error: Object of class Magento\Framework\View\Page\Title could not be converted to string...
becouse
processMetadataContentwill return an istance ofMagento\Framework\View\Page\Titleinstead of a string.Fixed Issues (if relevant)
Manual testing scenarios
Scenario 1
executefunction add:Scenario 2
executefunction add:Contribution checklist