[#9184] - Camel case not permitted for repeatable form type field name
- Closed
- 23 Feb 2016
- Medium
- Build: master
- # 9184
Steps to reproduce the issue
use camel case in repeatable form field name and fieldset name
Expected result
modal displays
Actual result
crashes back to template selector.
System information (as much as possible)
works:
<field name="googlefonts"
type="Repeatable"
icon="list"
label="TPL_SERENITY_FONT_SELECTOR"
default='{
"Elements":["body","H1","H2","H3","H4","H5","H6","p","a","i","ul","li"],
"Font":["Helvetica","Royal"],
"Css":["16px"],
"Size":["blah"]
}'>
<fieldset hidden="true" name="googlefonts_modal" repeat="true">
<field name="Elements"
label="TPL_SERENITY_FONT_ELEMENTS_LABEL"
size="12"
filter="html"
type="text" />
<field name="Font"
label="TPL_SERENITY_FONT_"
size="6"
type="text"
filter="raw" />
<field name="Css"
label="TPL_SERENITY_FONT_CSS_LABEL"
type="text"
filter="raw" />
<field name="Size"
label="TPL_SERENITY_FONT_SIZE_LABEL"
size="8"
type="text" />
</fieldset>
</field>Fails:
<field name="googleFonts"
type="Repeatable"
icon="list"
label="TPL_SERENITY_FONT_SELECTOR"
default='{
"Elements":["body","H1","H2","H3","H4","H5","H6","p","a","i","ul","li"],
"Font":["Helvetica","Royal"],
"Css":["16px"],
"Size":["blah"]
}'>
<fieldset hidden="true" name="googleFonts_modal" repeat="true">
<field name="Elements"
label="TPL_SERENITY_FONT_ELEMENTS_LABEL"
size="12"
filter="html"
type="text" />
<field name="Font"
label="TPL_SERENITY_FONT_"
size="6"
type="text"
filter="raw" />
<field name="Css"
label="TPL_SERENITY_FONT_CSS_LABEL"
type="text"
filter="raw" />
<field name="Size"
label="TPL_SERENITY_FONT_SIZE_LABEL"
size="8"
type="text" />
</fieldset>
</field>Additional comments
| Status | New | ⇒ | Information Required |
the xml allows it but it doesn't work properly when you've done so.
Bear
On 2/21/2016 04:59, Brian Teeman wrote:
I pasted both versions into the templateDetails.xml of protostar and
BOTH worked perfectly
_This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at
issues.joomla.org/joomla-cms/9184
https://issues.joomla.org/tracker/joomla-cms/9184.—
Reply to this email directly or view it on GitHub
#9184 (comment).
What doesn't work I can't read your mind.. My quick test worked fine
On 22 Feb 2016 5:47 pm, "Bear" notifications@github.com wrote:
the xml allows it but it doesn't work properly when you've done so.
BearOn 2/21/2016 04:59, Brian Teeman wrote:
I pasted both versions into the templateDetails.xml of protostar and
BOTH worked perfectly
_This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at
issues.joomla.org/joomla-cms/9184
https://issues.joomla.org/tracker/joomla-cms/9184.—
Reply to this email directly or view it on GitHub
<#9184 (comment)
.—
Reply to this email directly or view it on GitHub
#9184 (comment).
nvm i'm an idiot.. its cpanel has this issue..
Bear
On 2/22/2016 12:15, Brian Teeman wrote:
What doesn't work I can't read your mind.. My quick test worked fine
On 22 Feb 2016 5:47 pm, "Bear" notifications@github.com wrote:the xml allows it but it doesn't work properly when you've done so.
BearOn 2/21/2016 04:59, Brian Teeman wrote:
I pasted both versions into the templateDetails.xml of protostar and
BOTH worked perfectly
_This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at
issues.joomla.org/joomla-cms/9184
https://issues.joomla.org/tracker/joomla-cms/9184.—
Reply to this email directly or view it on GitHub—
Reply to this email directly or view it on GitHub—
Reply to this email directly or view it on GitHub
#9184 (comment).
Closed again
| Status | Information Required | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-02-23 22:28:03 |
| Closed_By | ⇒ | brianteeman |
| Labels |
Added:
?
|
||
I pasted both versions into the templateDetails.xml of protostar and BOTH worked perfectly
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9184.