Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[CODE STYLE] Simple code style improvement #7399

Merged
merged 2 commits into from Jul 10, 2015
Merged

[CODE STYLE] Simple code style improvement #7399

merged 2 commits into from Jul 10, 2015

Conversation

ghost
Copy link

@ghost ghost commented Jul 9, 2015

Removed redundant nesting by using early return.

[CODE STYLE] Simple code style improvement
b627dff 
Removed redundant nesting by using early return.
@zero-24
Copy link
Contributor

zero-24 commented Jul 10, 2015

@nonumber can you fix the Travis CS Issue?

FILE: /home/travis/build/joomla/joomla-cms/libraries/joomla/language/text.php
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
 305 | ERROR | Whitespace found at end of line
--------------------------------------------------------------------------------
UPGRADE TO PHP_CODESNIFFER 2.0 TO FIX ERRORS AUTOMATICALLY
--------------------------------------------------------------------------------

@ghost
Copy link
Author

ghost commented Jul 10, 2015

Done :)

@Kubik-Rubik Kubik-Rubik added the RTC This Pull Request is Ready To Commit label Jul 10, 2015
@Kubik-Rubik Kubik-Rubik added this to the Joomla! 3.4.4 milestone Jul 10, 2015
@Kubik-Rubik
Copy link
Member

Thank you @nonumber! Merged.

Kubik-Rubik added a commit that referenced this pull request Jul 10, 2015
@Kubik-Rubik
Merge pull request #7399 from nonumber/patch-6
1955566 
[CODE STYLE] Simple code style improvement
@Kubik-Rubik Kubik-Rubik merged commit 1955566 into joomla:staging Jul 10, 2015
@ghost ghost deleted the patch-6 branch July 21, 2015 16:28
@zero-24 zero-24 removed the RTC This Pull Request is Ready To Commit label Oct 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Joomla! 3.4.4
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@zero-24 @Kubik-Rubik @joomla-cms-bot