User tests: Successful: Unsuccessful:
Thank yoiu for your contribution! Unfortunately, this PR may be difficult to merge as there are many files being modified, each with a different code style issue. Some of the changes are very good, but there's are others that mat delay or prevent this PR from being accepted. I would recommend trying to break this into smaller pull requests, each limited in scope to one code style fix. That will make it much easier to test and merge. Thanks again!
You may also want to take a look at http://joomla.github.io/coding-standards/ to verify some of the style changes.
Hmm,
i used https://github.com/joomla/coding-standards to check the Style of each file, it should match correctly. When i look at the Travis build it looks good.
What should i exactly split or change?
Now this should pass the CodeStyle test that using https://github.com/joomla/coding-standards
Generally speaking: Codestyle PRs should be small so they can be reviewed fast and merged without further tests. Your PR deals with 39 files in a lot of plugins. That's not easy to review.
Either make one PR per plugin or one per codestyle issue. This way scope is limited and it's easy to review (and test if needed).
Important: Don't change any code at all in a Codestyle PR (like echoing instead of directly outputting).
Changing variable names is also bad (even if codestyle says to not use underscores in $_variable
). It's possible that it may introduce BC issues.
When i look at the Travis build it looks good.
Sidenote: Travis doesn't check codestyle.
Okay, then i will split it into the plugins.
Important: Don't change any code at all in a Codestyle PR (like echoing instead of directly
outputting).
Changing variable names is also bad (even if codestyle says to not use underscores in
$_variable). It's possible that it may introduce BC issues.
The Renamings are full refactored, so it shouldn't accour a BC issue. ;)
Pls wait for my next PR's - this should be declined.
The Renamings are full refactored, so it shouldn't accour a BC issue. ;)
Famous last words
And then some crazy guy goes and extends the class and it fails.
I'm closing this PR and wait for the new ones. Thanks already in advance! It's good to see people caring about codestyle.
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-03-18 12:54:01 |
i will fix this