Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

In version 1.6.0 the logo isn't shown on the PDF #897

Merged
merged 1 commit into from
Dec 8, 2022
Merged

In version 1.6.0 the logo isn't shown on the PDF #897

merged 1 commit into from
Dec 8, 2022

Conversation

vespino
Copy link
Contributor

@vespino vespino commented Dec 8, 2022

In version 1.6.0 the logo isn't shown on the PDF invoice. When changing line 39 the logo returns.

Description

Related Issue

Motivation and Context

Screenshots (if appropriate):

Pull Request Checklist

  • My code follows the code formatting guidelines.
  • I have an issue ID for this pull request.
  • I selected the corresponding branch.
  • I have rebased my changes on top of the corresponding branch.

Issue Type (Please check one or more)

  • Bugfix
  • Improvement of an existing Feature
  • New Feature

In version 1.6.0 the logo isn't shown on the PDF invoice. When changing line 39 the logo returns.
Copy link

@accesslint accesslint bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are accessibility issues in these changes.

@@ -36,7 +36,7 @@ function invoice_logo_pdf()
$CI = &get_instance();

if ($CI->mdl_settings->setting('invoice_logo')) {
return '<img src="file://' . getcwd() . '/uploads/' . $CI->mdl_settings->setting('invoice_logo') . '" id="invoice-logo">';
return '<img src="' . base_url() . 'uploads/' . $CI->mdl_settings->setting('invoice_logo') . '" id="invoice-logo">';
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This image is missing a text alternative. This is a problem for people using screen readers.

@nielsdrost7 nielsdrost7 changed the base branch from master to development December 8, 2022 15:44
@nielsdrost7 nielsdrost7 merged commit 6770909 into InvoicePlane:development Dec 8, 2022
@nielsdrost7
Copy link
Contributor

Thanks for the PR man! I've merged it into the development branch since we need it for the next version

@vespino vespino deleted the patch-1 branch December 8, 2022 18:54
@nielsdrost7 nielsdrost7 changed the title Update invoice_helper.php In version 1.6.0 the logo isn't shown on the PDF Dec 16, 2022
@stevesong
Copy link

Thanks! This fix worked for me.

@mohamad-supangat
Copy link

mohamad-supangat commented Jun 17, 2023

get someone help me ?

os: alpine 3.18
php: php-fpm 8.1
ip: v1.6.1-beta-2

@nielsdrost7
Copy link
Contributor

nielsdrost7 commented Jun 17, 2023

@mohamad-supangat
Please don't post sensitive information. I had to remove the screenshot.
The support forums are located at https://community.invoiceplane.com/
Create a new topic, post your problem, but remove the url of your website in that problem.

Did you use docker to start your v1.6.1-beta-2? Mention that in your problem on the community forums

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

After updating IP to v1.6.0 and PHP to 8.1 invoices/generate_pdf no longer works (with SVG?)
4 participants