Login  Register

Re: Problem with New FoFormRenderer

Posted by cjhowe on Aug 10, 2006; 9:58am
URL: http://ofbiz.116.s1.nabble.com/Problem-with-New-FoFormRenderer-tp170568p170570.html

jacopo,
have you looked at upgrading fop.jar? I plan on
looking at that in the next couple of weeks.  There
are quite a few things that don't get ignored entirely
in the .2x release that's in svn.

Just dropping in the latest beta doesn't build and
that's what I was going to look into.

--- Jacopo Cappellato <[hidden email]> wrote:

> 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 &nbsp; 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 = "&nbsp;";
> > +                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 = "&nbsp;";
> > +                retVal = " ";
> >              }
> >              return retVal;
> >          }
>