-
-
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
[4.0] Namespace com_fields #16187
[4.0] Namespace com_fields #16187
Conversation
…extensions itself
* | ||
* @since 3.7.0 | ||
* @since 3.0 |
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.
did you mean to change this?
* | ||
* @since 3.7.0 | ||
* @since 1.6 |
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.
did you meant to change this
* | ||
* @since 3.7.0 | ||
* @since 3.0 |
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.
again looks like you shouldnt have changed this
* | ||
* @since 3.7.0 | ||
* @since 1.6 |
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.
3.7.0
* | ||
* @see JModelLegacy | ||
* @since 3.7.0 | ||
* @since 1.6 |
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.
3.7.0
Please review all the version number @SInCE changes - looks like a lot of copy paste errors. I started to comment inline on the code but there were too many |
Thanks @brianteeman, fixed the since params. |
@@ -106,12 +116,12 @@ protected function allowAdd($data = array()) | |||
* | |||
* @return boolean | |||
* | |||
* @since 1.6 | |||
* @since 3.7.0 |
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.
shouldnt this be since 1.6
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.
or maybe it was wrong before
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.
it was wrong before, correctly it should be done against staging.
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.
ok
(title updated to show its for v4) |
Namespace com_fields.
Some extras:
JModelLegacy::getInstance()