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

_isMonospaced not declared #3

Open
NielsKeegel opened this issue Sep 15, 2023 · 1 comment
Open

_isMonospaced not declared #3

NielsKeegel opened this issue Sep 15, 2023 · 1 comment

Comments

@NielsKeegel
Copy link

NielsKeegel commented Sep 15, 2023

You are using _isMonospace and _isMonospaced but only _isMonospace is declared causing the following error to show:
Creation of dynamic property Zend_Pdf_Resource_Font_Simple_Standard_TimesRoman::$_isMonospaced is deprecated

currently fixed it with the following patch:

--- a/vendor/magento/zend-pdf/library/Zend/Pdf/Resource/Font.php
+++ b/vendor/magento/zend-pdf/library/Zend/Pdf/Resource/Font.php
@@ -81,6 +81,12 @@
      * @var boolean
      */
     protected $_isMonospace = false;
+
+    /**
+     * Flag indicating whether or not this font is monospaced.
+     * @var boolean
+     */
+    protected $_isMonospaced = false;
 
     /**
      * The position below the text baseline of the underline (in glyph units).
@@ -527,3 +533,4 @@
         return ceil(($value / $this->_unitsPerEm) * 1000);    // always round up
     }
 }
+
@hostep
Copy link
Contributor

hostep commented Oct 9, 2024

Hey @NielsKeegel, sorry I've only seen this issue now, but could it be that this problem was found with version 1.16.2 or older of this library? I believe it got fixed in 1.16.3 by #1
I'm double checking because 1.16.3 was released about 16 days before your issue report...

Maybe you can confirm this is fixed in version 1.16.3 and then we can close this issue?

Thanks!

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

No branches or pull requests

2 participants