-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Rename method arguments to use camelCase casing (plugin directory) #31151
Rename method arguments to use camelCase casing (plugin directory) #31151
Conversation
I have tested this item ✅ successfully on c87c887 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31151. |
@@ -93,25 +93,25 @@ public function onContentPrepare($context, &$row, $params, $page = 0) | |||
/** | |||
* Retrieve Contact | |||
* | |||
* @param int $created_by Id of the user who created the contact | |||
* @param int $userId Id of the user who created the article |
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.
* @param int $userId Id of the user who created the article | |
* @param int $userId Id of the user who you want to get the contact data |
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.
Is this correct English though?
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.
I'm sure it is not but it gets closer to what it happening here right?
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.
what about
* @param int $userId Id of the user from whom to get the contact data
I have tested this item ✅ successfully on c87c887 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31151. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31151. |
Partial Pull Request for Issue #30988.
Summary of Changes
Renames some method arguments to use camelCase casing.
Testing Instructions
Thorough code review / Joomla works like before.
Documentation Changes Required
No.