Stripes

When field-metadata tag writes "type" attribute for a field it misses a comma, rendering json syntactically incorrect

Details

Description

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())

Activity

Hide
asdf cvboi added a comment - 25/Jan/12 8:08 AM

This bug effectively cripples stripes js validation. Voted up.

Show
asdf cvboi added a comment - 25/Jan/12 8:08 AM This bug effectively cripples stripes js validation. Voted up.
Hide
Ben Gunter added a comment - 07/Feb/12 9:48 AM

This is a duplicate of STS-840, which is fixed for 1.5.7.

Show
Ben Gunter added a comment - 07/Feb/12 9:48 AM This is a duplicate of STS-840, which is fixed for 1.5.7.

People

Vote (1)
Watch (1)

Dates

  • Created:
    13/Oct/11 12:04 PM
    Updated:
    07/Feb/12 9:48 AM
    Resolved:
    07/Feb/12 9:48 AM