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.

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

Changed executeComponent() to use the factory method of JController (see pull request #193 ) #194

Closed
wants to merge 4 commits into from

Conversation

dgasparri
Copy link

Changed executeComponent() to use the factory method of JController. The parameter $baseClass is added, and passed by renderComponent().

See discussion "Object oriented components , any plan?"
https://groups.google.com/forum/?fromgroups#!topic/joomla-dev-cms/7tjVkzc08Ks

Change-Id: Ie19778dddc492814dc549c54f1a7c45cf80404c3

Duccio Gasparri added 2 commits April 19, 2012 15:14
1) factory()
2) createEvenrinmentArray()
3) setTask($task)

Changed execute($task) that if no task is passed, it executes the
$this->task (which is set also by the factory method)

2012-04-19 by Duccio Gasparri <duccio.gasparri@gmail.com>
Change-Id: I2a4ded062856670e364128f3cb74716b84b23eea
JController. The parameter $baseClass is added, and passed by
renderComponent().

See discussion "Object oriented components , any plan?"
https://groups.google.com/forum/?fromgroups#!topic/joomla-dev-cms/7tjVkzc08Ks


Change-Id: Ie19778dddc492814dc549c54f1a7c45cf80404c3
Duccio Gasparri added 2 commits April 23, 2012 01:41
Change-Id: I9a9b610865ab3be07bed8d1beadd7808019d5840
Change-Id: I90df53483ee46b877721f5fc9abbdd65f035a2a5
beat added a commit to beat/joomla-cms that referenced this pull request Sep 10, 2013
@phproberto
Copy link
Contributor

I'm closing this as it's not mergeable and 1 year old. Feel free to open if you update it and are still interested in get it merged.

Thanks for contributing.

@phproberto phproberto closed this Oct 3, 2013
wilsonge pushed a commit that referenced this pull request Jan 23, 2017
hans2103 added a commit to hans2103/joomla-cms that referenced this pull request Nov 18, 2020
* no margin-bottom when empty

* remove from DOM when empty

* no margin bottom. Next element has margin-top
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