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

ACCTG TRANS ENTRIES PDF link on invoiceOverview does not work

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: Release Branch 13.07
    • Fix Version/s: 16.11.04
    • Component/s: birt, order
    • Labels:
      None
    • Environment:

      Ubuntu linux 15.04, JVM 7u91-2.6.3-0ubuntu0.15.04.1

      Description

      Whenever I click on the ACCTG TRANS ENTRIES PDF link in the invoice overview screen I get the following error:

      org.ofbiz.webapp.control.RequestHandlerException: Unknown request [InvoiceAcctgTransEntriesPdf]; this request does not exist or cannot be called directly.

      A grep through the full ofbiz repo yields no results for an implementation or freemarker template - only the link itself in accounting menus xml:

      git checkout release13.07

      find . -type f -exec grep -i InvoiceAcctgTransEntriesPdf {} \; -ls
      <link target="InvoiceAcctgTransEntriesPdf">
      528005 72 rw-rw-r- 1 xxx xxx 70103 Dec 17 07:05 ./applications/accounting/widget/AccountingMenus.xml

      1. OFBIZ-6774_16.11.patch
        27 kB
        Ankit Joshi
      2. OFBIZ-6774_trunk.patch
        27 kB
        Ankit Joshi

        Issue Links

          Activity

          Hide
          lektran Scott Gray added a comment -

          These are links to the BIRT component that wasn't included in the 13.07 release. The accounting -> birt dependency shouldn't have been created and these links should be removed from all versions.

          Show
          lektran Scott Gray added a comment - These are links to the BIRT component that wasn't included in the 13.07 release. The accounting -> birt dependency shouldn't have been created and these links should be removed from all versions.
          Hide
          ankit.joshi Ankit Joshi added a comment -

          Agreed Scott Gray. Either this birt links should be removed or this PDF should be added into accounting component itself to remove Accounting -> birt dependency.
          Added a patch with a solution of adding Acctg Trans Entry PDF into accounting component.

          Show
          ankit.joshi Ankit Joshi added a comment - Agreed Scott Gray . Either this birt links should be removed or this PDF should be added into accounting component itself to remove Accounting -> birt dependency. Added a patch with a solution of adding Acctg Trans Entry PDF into accounting component.
          Hide
          arunpati Arun Patidar added a comment -

          Committed in ofbiz framework trunk at revision 1801320.

          Thanks Morten and Ankit Joshi for your contribution.

          Show
          arunpati Arun Patidar added a comment - Committed in ofbiz framework trunk at revision 1801320. Thanks Morten and Ankit Joshi for your contribution.
          Hide
          deepak.dixit Deepak Dixit added a comment -

          Backported to R16.11 at r#1801321

          Show
          deepak.dixit Deepak Dixit added a comment - Backported to R16.11 at r#1801321
          Hide
          arunpati Arun Patidar added a comment -

          Added missing file in ofbiz framework trunk at r#1801359.

          Show
          arunpati Arun Patidar added a comment - Added missing file in ofbiz framework trunk at r#1801359.
          Hide
          deepak.dixit Deepak Dixit added a comment -

          Backported missing file in R16.11 at r#1801361

          Show
          deepak.dixit Deepak Dixit added a comment - Backported missing file in R16.11 at r#1801361

            People

            • Assignee:
              deepak.dixit Deepak Dixit
              Reporter:
              morten@citizenme.com Morten Jensen
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development