Re: Problem with New FoFormRenderer

Posted by Jacopo Cappellato on Aug 10, 2006; 9:53am
URL: http://ofbiz.116.s1.nabble.com/Problem-with-New-FoFormRenderer-tp170568p170569.html

Christian,

thanks for the patch, it's in svn with rev. 430317
However, I think that the ModelFormField class shouldn't contain html
(or xml) specific encodings and tags... this is something we should fix
sooner or later.
I'm trying to fix other small issues about the PDF generation right now,
so if you see something wrong please send me patches!

Thanks,

Jacopo


Christian Geisert wrote:

> Hi,
>
> I just wanted to have a look at the new FoFormRenderer and got the
> following error:
>
> org.xml.sax.SAXParseException: The entity "nbsp" was referenced, but not
> declared.
>
> A simple fix is to replace   with   (patch attached)
>
>
>
> ------------------------------------------------------------------------
>
> Index: framework/widget/src/org/ofbiz/widget/form/ModelFormField.java
> ===================================================================
> --- framework/widget/src/org/ofbiz/widget/form/ModelFormField.java (Revision 430292)
> +++ framework/widget/src/org/ofbiz/widget/form/ModelFormField.java (Arbeitskopie)
> @@ -1712,7 +1712,7 @@
>                  retVal = modelFormField.getEntry(context);
>              }
>              if (retVal == null || retVal.length() == 0) {
> -                retVal = " ";
> +                retVal = " ";
>              } else if ("currency".equals(type)) {
>                  Locale locale = (Locale) context.get("locale");
>                  if (locale == null) locale = Locale.getDefault();
> @@ -1826,7 +1826,7 @@
>                  retVal = fieldValue;
>              }
>              if (retVal == null || retVal.length() == 0) {
> -                retVal = " ";
> +                retVal = " ";
>              }
>              return retVal;
>          }