Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Trunk, 13.07.01
    • Fix Version/s: 14.12.01, 13.07.02, 16.11.01
    • Component/s: order
    • Labels:

      Description

      Create a quote (w/o items), click on Report menu entry

      ordermgr/control/QuoteReport?quoteId=CQ10000

      You get this error in log

      2015-02-16 07:58:10,762 |http-bio-8443-exec-3 |FOUserAgent                   |W| "fo:table-body" is missing child elements. Required content model: marker* (table-row+|table-cell+) (See position 202:32)
      2015-02-16 07:58:10,763 |http-bio-8443-exec-3 |ScreenFopViewHandler          |E| Unable to transform FO file: org.apache.fop.apps.FOPException
      java.lang.NullPointerException; Screen XSL:FO text was:
      

      I think when you try to send the report by email you get a related error (did not check). These error has been reported at OFBIZ-6076

      015-02-16 08:00:32,405 |http-bio-8443-exec-9 |FOUserAgent                   |W| The following feature isn't implemented by Apache FOP, yet: table-layout="auto" (on fo:table) (See position 139:40)
      015-02-16 08:00:32,407 |http-bio-8443-exec-9 |FOUserAgent                   |W| The following feature isn't implemented by Apache FOP, yet: table-layout="auto" (on fo:table) (See position 161:81)
      015-02-16 08:00:32,410 |http-bio-8443-exec-9 |FOUserAgent                   |W| The following feature isn't implemented by Apache FOP, yet: table-layout="auto" (on fo:table) (See position 190:39)
      015-02-16 08:00:32,413 |http-bio-8443-exec-9 |FOUserAgent                   |W| "fo:table-body" is missing child elements. Required content model: marker* (table-row+|table-cell+) (See position 209:32)
      015-02-16 08:00:32,414 |http-bio-8443-exec-9 |EmailServices                 |E| Error rendering PDF attachment for email: org.apache.fop.apps.FOPExceptionava.lang.NullPointerException
      org.apache.fop.apps.FOPException
             at org.ofbiz.webapp.view.ApacheFopWorker.transform(ApacheFopWorker.java:160) ~[ofbiz-webapp.jar:?]
             at org.ofbiz.common.email.EmailServices.sendMailFromScreen(EmailServices.java:533) [ofbiz-common.jar:?]
      

      Same error with stable demo, I did not test with R14.12, but I suppose it's there also. Both works in the old demo (R12.04).

        Issue Links

          Activity

          Hide
          deepak.dixit Deepak Dixit added a comment -

          FOP throwing NPE because there is no NPE check on Quote report ft ftl for quoteItems .
          Added has_content check now report with out item generating properly.

          Create separate issue to fix the table-layout="auto" related warning.
          Thanks Jacques for reporting the issue

          Show
          deepak.dixit Deepak Dixit added a comment - FOP throwing NPE because there is no NPE check on Quote report ft ftl for quoteItems . Added has_content check now report with out item generating properly. Create separate issue to fix the table-layout="auto" related warning. Thanks Jacques for reporting the issue
          Hide
          deepak.dixit Deepak Dixit added a comment - - edited

          Fixed in correct hierarchy of fo:table, and if no items found then display message on quote item section.

          Now NPE on send report by mail should be fixed.
          Changes committed at
          Trunk at r#1672856, 1672862
          14.12 at r#1672859, 1672863

          Show
          deepak.dixit Deepak Dixit added a comment - - edited Fixed in correct hierarchy of fo:table, and if no items found then display message on quote item section. Now NPE on send report by mail should be fixed. Changes committed at Trunk at r#1672856, 1672862 14.12 at r#1672859, 1672863
          Hide
          jacques.le.roux Jacques Le Roux added a comment - - edited

          Thanks Deepak!

          R13.07 r1672868

          Show
          jacques.le.roux Jacques Le Roux added a comment - - edited Thanks Deepak! R13.07 r1672868
          Hide
          deepak.dixit Deepak Dixit added a comment -

          Added missing if condition at r#1672869

          Show
          deepak.dixit Deepak Dixit added a comment - Added missing if condition at r#1672869
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Thanks Deepak, indeed I did not spot the 2nd commit :/

          Show
          jacques.le.roux Jacques Le Roux added a comment - Thanks Deepak, indeed I did not spot the 2nd commit :/

            People

            • Assignee:
              deepak.dixit Deepak Dixit
              Reporter:
              jacques.le.roux Jacques Le Roux
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development