Changeset 8798

Timestamp:

05/15/2019 02:01:45 PM (5 years ago)

Author:

vedjain

Message:

CampTix-Invoices: Fix path for custom logo to include it in PDF

Currently if a custom logo is set for the invoice, it's not included in the generated PDF.

In testing locally, it can be observed that the default logo works and is pulled in as a /absolute/path/to/file.png where as custom logos are ​https://wordcamp.site/path/to/file.png.

The attached patch attempts to use absolute path to the image in the hope that's the problem, invoice generation works before and after for the test environment.

props dion

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/camptix-invoices/includes/class-camptix-addon-invoices.php (modified) (1 diff)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/camptix-invoices/includes/class-camptix-addon-invoices.php

@@ -384 +384 @@
         $logo = CTX_INV_DIR . '/admin/images/wp-community-support.png';
         if ( ! empty( $camptix_opts['invoice-logo'] ) ) {
-            $attachment = wp_get_attachment_image_src( $camptix_opts['invoice-logo'], 'full' );
-            $logo       = $attachment[0];
+            $logo = get_attached_file( $camptix_opts['invoice-logo'] );
         }