[ 
http://www.stripesframework.org/jira/browse/STS-852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12541#comment-12541
 ] 

asdf cvboi commented on STS-852:
--------------------------------

This bug effectively cripples stripes js validation. Voted up.

> When field-metadata tag writes "type" attribute for a field it misses a 
> comma, rendering json syntactically incorrect
> ---------------------------------------------------------------------------------------------------------------------
>
>                 Key: STS-852
>                 URL: http://www.stripesframework.org/jira/browse/STS-852
>             Project: Stripes
>          Issue Type: Bug
>          Components: Tag Library
>    Affects Versions: Release 1.5.6
>            Reporter: Carl Krig
>              Labels: field-metadata, taglib, validation
>
> Sample output from <stripes:field-metadata>: ... 
> "transferAmount":{type:"BigDecimal"required:true,ignore:false ...
> Above, the comma is missing before the "required" property.
> Problem is in FieldMetadataTag.java lines 152-157 + 162. Suggested change on 
> 162, mimicking 216;
> <fieldInfo.append("required:").append(data.required())
> >fieldInfo.append(fieldInfo.length() > 0 ? "," : 
> >"").append("required:").append(data.required())

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Keep Your Developer Skills Current with LearnDevNow!
The most comprehensive online learning library for Microsoft developers
is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3,
Metro Style Apps, more. Free future releases when you subscribe now!
http://p.sf.net/sfu/learndevnow-d2d
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to