You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Jacopo Cappellato <ti...@sastau.it> on 2006/11/27 14:48:31 UTC
Issue with html specific code in ModelFormField
In the ModelFormField class, two getDescription(...) methods returns the
string " " if no description is found.
IMO this is not a correct because it is html specific and in fact is
causing problems when the form is rendered as PDF.
What is the best way to fix this?
Maybe changing " " to " "?
Jacopo
Re: Issue with html specific code in ModelFormField
Posted by Jacopo Cappellato <ti...@sastau.it>.
Thanks David,
following your suggestion, in rev 479931 I've replaced the " "
string with the empty "" one.
Jacopo
David E Jones wrote:
>
> I guess we could even just return an empty String, but space or empty
> seems fine either way.
>
> -David
>
>
> On Nov 27, 2006, at 6:48 AM, Jacopo Cappellato wrote:
>
>> In the ModelFormField class, two getDescription(...) methods returns
>> the string " " if no description is found.
>> IMO this is not a correct because it is html specific and in fact is
>> causing problems when the form is rendered as PDF.
>> What is the best way to fix this?
>> Maybe changing " " to " "?
>>
>> Jacopo
Re: Issue with html specific code in ModelFormField
Posted by David E Jones <jo...@undersunconsulting.com>.
I guess we could even just return an empty String, but space or empty
seems fine either way.
-David
On Nov 27, 2006, at 6:48 AM, Jacopo Cappellato wrote:
> In the ModelFormField class, two getDescription(...) methods
> returns the string " " if no description is found.
> IMO this is not a correct because it is html specific and in fact
> is causing problems when the form is rendered as PDF.
> What is the best way to fix this?
> Maybe changing " " to " "?
>
> Jacopo