You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Scott Gray (JIRA)" <ji...@apache.org> on 2009/06/22 23:26:07 UTC

[jira] Issue Comment Edited: (OFBIZ-2644)
syntax is incorrect

    [ https://issues.apache.org/jira/browse/OFBIZ-2644?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12722828#action_12722828 ] 

Scott Gray edited comment on OFBIZ-2644 at 6/22/09 2:25 PM:
------------------------------------------------------------

*edit: should even -> shouldn't even

Perhaps it's the actual invalid html that is causing the problem:
{code}
-                    <td><div <#if hasntStarted><div style="color: red;"/></#if>${(productFeatureCatGrpAppl.fromDate)?if_exists}</div></td>
+                    <td><div <#if hasntStarted><div style="color: red;"> </div></#if>${(productFeatureCatGrpAppl.fromDate)?if_exists}</div></td>
{code}
will generate a div with a missing '>'
{code}
<td><div <div style="color: red;"/>${(productFeatureCatGrpAppl.fromDate)?if_exists}</div></td>
{code}
regardless of whether or not <div/> is valid or not, doesn't the div want to wrap the from date text so that it can color it red?  My guess is that there should even be an inner div and the outer one should have the style applied to it, I think that's why there is no closing '>' and also why the first div has a space after it.

      was (Author: lektran):
    Perhaps it's the actual invalid html that is causing the problem:
{code}
-                    <td><div <#if hasntStarted><div style="color: red;"/></#if>${(productFeatureCatGrpAppl.fromDate)?if_exists}</div></td>
+                    <td><div <#if hasntStarted><div style="color: red;"> </div></#if>${(productFeatureCatGrpAppl.fromDate)?if_exists}</div></td>
{code}
will generate a div with a missing '>'
{code}
<td><div <div style="color: red;"/>${(productFeatureCatGrpAppl.fromDate)?if_exists}</div></td>
{code}
regardless of whether or not <div/> is valid or not, doesn't the div want to wrap the from date text so that it can color it red?  My guess is that there should even be an inner div and the outer one should have the style applied to it, I think that's why there is no closing '>' and also why the first div has a space after it.
  
> <div/> syntax is incorrect
> --------------------------
>
>                 Key: OFBIZ-2644
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2644
>             Project: OFBiz
>          Issue Type: Bug
>            Reporter: Harmeet Bedi
>            Priority: Trivial
>             Fix For: SVN trunk
>
>         Attachments: EditCategoryFeatureCats.ftl.diff
>
>
> use to be widespread earlier found this only in one spot in trunk. Attaching patch.
> <div some-attribute="some-value"/> is incorrect syntax
> but 
> <div some-attribute="some-value"></div> is right

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.