? Error

User tests: Successful: Unsuccessful:

avatar juelisjr
juelisjr
2 Oct 2013

On line 157, the attribute will return true or false, does not need ternary operator

avatar juelisjr juelisjr - open - 2 Oct 2013
avatar PAlexcom
PAlexcom - comment - 6 Oct 2013

@juelisjr Thanks :+1:
Can and needs to be merged.

avatar brianteeman
brianteeman - comment - 13 Oct 2013

Thanks for your contribution - At this time we are only using github as the place to submit code fixes, the actual reporting of issues and testing fixes is still taking place on Joomlacode.

Please can you:

1) Open an item on the Joomlacode tracker in the appropriate area.

CMS Bug Reports: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8103

CMS Feature Requests: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8549

2) After submitting the item to the Joomlacode tracker, add a link to the Joomlacode tracker item here and make sure that you add a link to this GitHub issue or pull request on the joomlacode tracker item.

avatar wilsonge
wilsonge - comment - 16 May 2014

For testers: This PR has now been made against staging at #3596 - thanks again @juelisjr for your contribution!! :)

avatar Bakual
Bakual - comment - 16 May 2014

Closing this then.

avatar Bakual Bakual - close - 16 May 2014

Add a Comment

Login with GitHub to post a comment