Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-6120

Allow FOP to support <fo:external-graphic/> for -Dportoffset

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 14.12.01, 13.07.02, 16.11.01
    • Component/s: framework
    • Labels:
      None

      Description

      FOP <fo:external-graphic/> images do not appear when portoffset used:

      2015-02-28 18:12:12,249 |http-bio-8445-exec-4 |ApacheFopWorker |I| FOP FontBaseURL: file:/home/anon/workspace/ofbiz/framework/webapp/config/
      2015-02-28 18:12:12,254 |http-bio-8445-exec-4 |FOP |D| target-resolution set to: 72.0dpi (px2mm=0.35277778)
      2015-02-28 18:12:12,593 |http-bio-8445-exec-4 |FOP |E| Error with opening URL 'http://localhost:8080/content/control/stream?contentId=10004': Connection refused
      2015-02-28 18:12:12,616 |http-bio-8445-exec-4 |FOUserAgent |E| Image not found. URI: /content/control/stream?contentId=10004. (See position 40:158)
      2015-02-28 18:12:12,715 |http-bio-8445-exec-4 |FOUserAgent |W| Font "NotoSans,normal,700" not found. Substituting with "NotoSans,normal,400".
      2015-02-28 18:12:12,957 |http-bio-8445-exec-4 |FOUserAgent |W| Line 1 of a paragraph overflows the available area by 567 millipoints. (See position 270:61)
      2015-02-28 18:12:13,001 |http-bio-8445-exec-4 |FOUserAgent |W| Content overflows the viewport of the fo:region-before on page 1 in block-progression direction by 19200 millipoints. (See position 30:58)
      2015-02-28 18:12:13,026 |http-bio-8445-exec-4 |FOUserAgent |E| Image not found. URI: /content/control/stream?contentId=10004. (No context info available)
      2015-02-28 18:12:13,104 |http-bio-8445-exec-4 |ServerHitBin |I| Visit delegatorName=default, ServerHitBin delegatorName=default

      1. OFBIZ-6120.patch
        1 kB
        Christian Carlow

        Activity

        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks to Deepak's report, I fixed the way the FOP baseUrl was set in

        trunk r1677123
        R14.12 r1677124
        R13.07 r1677125

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks to Deepak's report, I fixed the way the FOP baseUrl was set in trunk r1677123 R14.12 r1677124 R13.07 r1677125
        Hide
        jacques.le.roux Jacques Le Roux added a comment - - edited

        Thanks Christian,

        Your improved patch is in
        trunk r1663979
        R14.12 r1663980
        R13.07 r1663981+r1664244

        Show
        jacques.le.roux Jacques Le Roux added a comment - - edited Thanks Christian, Your improved patch is in trunk r1663979 R14.12 r1663980 R13.07 r1663981+r1664244
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks for the tip Christian, I will try to improve.

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks for the tip Christian, I will try to improve.
        Hide
        ofbizzer Christian Carlow added a comment -

        This patch employs FopFactory.setBaseUrl() along with the portOffset from Start class. A better solution might be to extract the port from the baseUrl of fop.xconf and then apply the offset to that but I don't have time to implement that yet.

        Show
        ofbizzer Christian Carlow added a comment - This patch employs FopFactory.setBaseUrl() along with the portOffset from Start class. A better solution might be to extract the port from the baseUrl of fop.xconf and then apply the offset to that but I don't have time to implement that yet.
        Hide
        ofbizzer Christian Carlow added a comment -

        In framework/webapp/config/fop.xconf <base>http://localhost:8080</base> is hardcoded

        Show
        ofbizzer Christian Carlow added a comment - In framework/webapp/config/fop.xconf <base> http://localhost:8080 </base> is hardcoded

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            ofbizzer Christian Carlow
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development