Skip to content

Simple CS fix#3

Merged
alex7r merged 1 commit intoalex7r:detach_observerfrom
wojsmol:11037-cs
Jul 6, 2016
Merged

Simple CS fix#3
alex7r merged 1 commit intoalex7r:detach_observerfrom
wojsmol:11037-cs

Conversation

@wojsmol
Copy link
Copy Markdown

@wojsmol wojsmol commented Jul 6, 2016

Pull Request for joomla#11037

@alex7r alex7r merged commit 78d6575 into alex7r:detach_observer Jul 6, 2016
@alex7r
Copy link
Copy Markdown
Owner

alex7r commented Jul 6, 2016

Thanks

@wojsmol wojsmol deleted the 11037-cs branch July 6, 2016 18:16
alex7r pushed a commit that referenced this pull request Dec 18, 2016
… (joomla#12292)

* Cleanups, fixes and a bit of optimizations for site/components batch #3

- com_content

Note: This is a single commit bundling all types of changes, since PR joomla#12261 which had detailed commits, was rejected as a whole

* Ch-Ch-Ch-Changes!

Made some changes as pointed out by @andrepereiradasilva

* A bit more...

* Revert

* Removed empty function, as there is a fallback. Change made according to comment from @wilsonge

* Included @andrepereiradasilva's suggestions

* Inserting whitespace before php closing tag
alex7r pushed a commit that referenced this pull request Jan 28, 2017
* Replace unnecessary double quotes

* And one unnecessary semicolon... ;)

* CS Fix + removal of unnecessary space

* some more double quote replacements

* Changes, based on @shur's comments
alex7r pushed a commit that referenced this pull request Jan 28, 2017
* Some improvements in tests #3:
- call static methods correctly

* Fix T_PAAMAYIM_NEKUDOTAYIM (for all PHP 5.x)

* Remove forgotten call

* Changed a few things after conversation with @mbabker

* Fixes according to @andrepereiradasilva's comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants