gravity-pdf icon indicating copy to clipboard operation
gravity-pdf copied to clipboard

Replace mPDF HTTP Request with wp_remote_get()

Open jakejackson1 opened this issue 2 years ago • 1 comments

Description

Resolve issues loading images / assets when using mPDF's native cURL functionality.

Fixes #1440

Testing instructions

Screenshots

Checklist:

  • [X] I've tested the code.
  • [ ] My code is easy to read, follow, and understand
  • [ ] My code follows the accessibility standards.
  • [ ] My code has proper inline documentation / docblocks.

Additional Comments

jakejackson1 avatar Dec 29 '22 21:12 jakejackson1

Codecov Report

Merging #1441 (157701d) into development (b9671fd) will increase coverage by 17.60%. The diff coverage is n/a.

@@                Coverage Diff                 @@
##             development    #1441       +/-   ##
==================================================
+ Coverage          75.70%   93.30%   +17.60%     
==================================================
  Files                243       57      -186     
  Lines              12826     1464    -11362     
  Branches             370      370               
==================================================
- Hits                9710     1366     -8344     
+ Misses              3108       90     -3018     
  Partials               8        8               
Impacted Files Coverage Δ
src/Helper/Helper_PDF.php
src/Helper/Fields/Field_Tax.php
src/View/html/Settings/tools.php
src/Helper/Helper_Options_Fields.php
src/templates/zadani.php
src/Model/Model_Mergetags.php
src/Helper/Fields/Field_Quantity.php
src/Model/Model_PDF.php
src/Controller/Controller_Settings.php
src/Controller/Controller_Install.php
... and 176 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Dec 29 '22 21:12 codecov[bot]