-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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] Sitemap filename can't exceed 32 characters #13937 #21553
[Backport] Sitemap filename can't exceed 32 characters #13937 #21553
Conversation
Hi @irajneeshgupta. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
/** | ||
* Save sitemap controller. | ||
*/ | ||
class Save extends \Magento\Sitemap\Controller\Adminhtml\Sitemap implements HttpPostActionInterface |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @irajneeshgupta. I believe HttpPostActionInterface is not the case here since this interface was introduced within a scope of 2.3.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rogyar ,
I will modify it .
Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, check the failing tests and test your solution on 2.2-develop branch manually. Thank you
Hi @rogyar, thank you for the review. |
@irajneeshgupta , I am closing this PR now due to inactivity. |
Hi @irajneeshgupta, thank you for your contribution! |
Original Pull Request
#20044
Description (*)
The Sitemap filename length validation was not added , hence added
\Magento\Framework\Validator\StringLength
max 32 length validation for sitemap filename and it will not trim filename exceeding 32 chars. Client side validation in filename field of form is also added to fix this issue'validate-length maximum-length-32'
. UpdatedsaveTest
unit test case.Fixed Issues (if relevant)
Contribution checklist (*)