? Success

User tests: Successful: Unsuccessful:

avatar alexvanniel
alexvanniel
21 Sep 2015

Instead of echoing the input field in place, the html code for the input field should be added to the html array otherwise the html code will end up outside of the intended div.

avatar alexvanniel alexvanniel - open - 21 Sep 2015
avatar alexvanniel alexvanniel - change - 21 Sep 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Sep 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 23 Sep 2015
Category Layout
avatar zero-24
zero-24 - comment - 23 Sep 2015

This looks correct but i have no idea to test it? Where do you see / catch this so we can test it ;)


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7923.

avatar alexvanniel
alexvanniel - comment - 23 Sep 2015

It it is rather hard to actually find a usecase, it is more of a clean up than a big bugfix. Although, if the intent here was to echo the field input before the hidden div then the code is a bit odd and the $html declaration should be done lower and after echoing the field input. You might not even see any "problems" in the browser unless you open up a debug window and start looking for hidden input fields/fieldsets and see that the fieldset is outside of the hidden div.

avatar mbabker
mbabker - comment - 23 Sep 2015

I don't know how you'd check it, but pre-patch the contents are rendered and then an empty div (which has inline styling to not be displayed) is rendered; post-patch the contents are inside that div. Code wise it reads fine to me, I just wouldn't know where to start looking at it. Since this is supposed to be rendering hidden fields, I'd say you could remove the $html array and just echo the fields also.

avatar alexvanniel
alexvanniel - comment - 23 Sep 2015

I am not entirely sure it handles hidden fields. It handles hidden fieldsets as far as I know and not necessarily hidden fields. I suspect that removing the hidden div would potentially display fields that are not set to hidden even though the fieldset is declared hidden. Or am I swinging left and right and missing massively?

avatar mbabker
mbabker - comment - 23 Sep 2015

If it's fieldsets then ya, that might be the case.

avatar alexvanniel
alexvanniel - comment - 23 Sep 2015

Granted, the parameters of this ever showing or causing any problems are obscure to say the least :)

avatar wilsonge wilsonge - change - 8 May 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-05-08 14:14:30
Closed_By wilsonge
avatar wilsonge wilsonge - close - 8 May 2016
avatar wilsonge wilsonge - merge - 8 May 2016
avatar wilsonge wilsonge - reference | 68f3f81 - 8 May 16
avatar wilsonge wilsonge - merge - 8 May 2016
avatar wilsonge wilsonge - close - 8 May 2016
avatar wilsonge
wilsonge - comment - 8 May 2016

Merged on review.

avatar wilsonge wilsonge - change - 8 May 2016
Milestone Added:

Add a Comment

Login with GitHub to post a comment