Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-2989

TCPDF Wrapper unnecessarily loads TCPDF library file

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 4.4.0-b1
    • Fix Version/s: 4.5.0-b1
    • Component/s: None
    • Labels:
      None

      Description

      The TcpdfWrapper class for invoice templates attempts to load a TCPDF file that doesn't exist from the vendors directory. However, it is unnecessary to load the file in any case since it is loaded automatically by composer. See https://www.blesta.com/forums/index.php?/topic/12353-new-tcpdf-class-with-tecnickcom/

      1. Remove the attempt to load the TCPDF file from the top of the TcpdfWrapper class

        Issue Links

          Activity

          tyson Tyson Phillips (Inactive) created issue -
          tyson Tyson Phillips (Inactive) made changes -
          Field Original Value New Value
          Link This issue relates to CORE-2830 [ CORE-2830 ]
          tyson Tyson Phillips (Inactive) made changes -
          Sprint 4.5.0 Sprint 5 [ 77 ]
          tyson Tyson Phillips (Inactive) made changes -
          Rank Ranked higher
          Automated transition triggered when Tyson Phillips (Inactive) created a branch in Stash -
          Status Open [ 1 ] In Progress [ 3 ]
          Automated transition triggered when Tyson Phillips (Inactive) created pull request #597 in Stash -
          Status In Progress [ 3 ] In Review [ 5 ]
          Resolution Fixed [ 1 ]
          tyson Tyson Phillips (Inactive) made changes -
          Description The TcpdfWrapper class for invoice templates attempts to load a TCPDF file that doesn't exist from the vendors directory. However, it is unnecessary to load the file in any case since it is loaded automatically by composer.

          # Remove the attempt to load the TCPDF file from the top of the TcpdfWrapper class
          The TcpdfWrapper class for invoice templates attempts to load a TCPDF file that doesn't exist from the vendors directory. However, it is unnecessary to load the file in any case since it is loaded automatically by composer. See https://www.blesta.com/forums/index.php?/topic/12353-new-tcpdf-class-with-tecnickcom/

          # Remove the attempt to load the TCPDF file from the top of the TcpdfWrapper class
          tyson Tyson Phillips (Inactive) made changes -
          Sprint 4.5.0 Sprint 5 [ 77 ] 4.5.0 Sprint 5, 4.5.0 Sprint 6 [ 77, 78 ]
          tyson Tyson Phillips (Inactive) made changes -
          Rank Ranked higher
          Automated transition triggered when Tyson Phillips (Inactive) merged pull request #597 in Stash -
          Status In Review [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              tyson Tyson Phillips (Inactive)
              Reporter:
              tyson Tyson Phillips (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                31/Jan/19

                Agile