Please activate JavaScript in your browser.
?
Pending
Success
continuous-integration/drone
the build was successful
Details
Success
continuous-integration/travis-ci/pr
The Travis CI build passed
Details
Pending
JTracker/HumanTestResults
Human Test Results: 1 Successful 0 Failed.
Details
User tests:
Successful:
Unsuccessful:
Summary of Changes
This PR simplifies com_fields code a bit by changing the useage of JComponentHelper::getComponent('com_foo')->params
to JComponentHelper::getParams('com_foo')
Just proper use of existing API
Testing Instructions
Test that the com_fields links in the sidebar appear as expected.
Test that the backend menu items under "Content" for com_fields appear as expected.
Documentation Changes Required
None
Bakual
-
open
-
3 Nov 2016
joomla-cms-bot
-
change
-
3 Nov 2016
joomla-cms-bot
-
change
-
3 Nov 2016
Category
⇒
Administration
Components
Modules
Title
Differences
…
Use JComponentHelper::getParams()
[com_fields] Use JComponentHelper::getParams() instead of ComponentHelper::getComponent('com_foo')->params
Title
Differences
…
Use JComponentHelper::getParams()
[com_fields] Use JComponentHelper::getParams() instead of ComponentHelper::getComponent('com_foo')->params
andrepereiradasilva
-
test_item
-
3 Nov 2016
-
Tested successfully
wilsonge
-
change
-
3 Nov 2016
Title
Differences
…
[com_fields] Use JComponentHelper::getParams() instead of ComponentHelper::getComponent('com_foo')->params
[com_fields] Use JComponentHelper::getParams() instead of J ComponentHelper::getComponent('com_foo')->params
wilsonge
-
edited
-
3 Nov 2016
wilsonge
-
change
-
3 Nov 2016
Title
Differences
…
[com_fields] Use JComponentHelper::getParams() instead of ComponentHelper::getComponent('com_foo')->params
[com_fields] Use JComponentHelper::getParams() instead of J ComponentHelper::getComponent('com_foo')->params
wilsonge
-
merge
-
3 Nov 2016
wilsonge
-
change
-
3 Nov 2016
Status
Pending
⇒
Fixed in Code Base
Closed_Date
0000-00-00 00:00:00
⇒
2016-11-03 14:40:58
Closed_By
⇒
wilsonge
wilsonge
-
close
-
3 Nov 2016
wilsonge
-
merge
-
3 Nov 2016
wilsonge
-
close
-
3 Nov 2016
wilsonge
-
close
-
3 Nov 2016
wilsonge
-
change
-
3 Nov 2016
Add a Comment
Login with GitHub to post a comment
Tested it successfully, but got the missing SHA error.