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
Allow plugin names to be up to 60 characters instead of 40 #19795
Conversation
adding plugin name checks with 60 characters limit
extend change log for plugin name
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.
@peterhashair Did you check the other code for required changes? There is at least
matomo/plugins/CoreConsole/Commands/GeneratePlugin.php
Lines 137 to 139 in 1155273
if(strlen($pluginName) > 40) { | |
throw new \RuntimeException('Your plugin name cannot be longer than 40 characters'); | |
} |
which needs to be removed. Not sure if there are other places where similar stuff is checked.
remove strlen check on plugin put onto regex
@sgiehl I couldn't find anywhere else that has a 40 char limit, anywhere I need to be aware? |
Co-authored-by: Stefan Giehl <stefan@matomo.org>
update function name
This PR maybe require other docs to update. |
* adding plugin name checks with 60 characters limit adding plugin name checks with 60 characters limit * extend change log for plugin name extend change log for plugin name * remove strlen check on plugin put onto regex remove strlen check on plugin put onto regex * Update CHANGELOG.md Co-authored-by: Stefan Giehl <stefan@matomo.org> * update function name update function name Co-authored-by: Stefan Giehl <stefan@matomo.org>
Description:
Fixes: #18727
adding plugin name checks with 60 characters limit
Review