Problem/Motivation

TypeError: Drupal\printable_pdf\Plugin\PrintableFormat\PdfFormat::__construct(): Argument #10 ($public_stream) must be of type Drupal\Core\StreamWrapper\PublicStream, Drupal\s3fs\StreamWrapper\PublicS3fsStream given, called in /var/www/drupal/web/modules/contrib/printable/modules/printable_pdf/src/Plugin/PrintableFormat/PdfFormat.php on line 187 in Drupal\printable_pdf\Plugin\PrintableFormat\PdfFormat->__construct() (line 120 of modules/contrib/printable/modules/printable_pdf/src/Plugin/PrintableFormat/PdfFormat.php).

CommentFileSizeAuthor
#3 printable_support_s3fswrapper.patch2.65 KBkapil.purohit

Issue fork printable-3317347

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

ThirstySix created an issue. See original summary.

kapil.purohit’s picture

kapil.purohit
Primary language English
commented

Below patch to fix this issue. tested and verified successfully... All the best!

kapil.purohit’s picture

kapil.purohit
Primary language English
commented
StatusFileSize
new printable_support_s3fswrapper.patch2.65 KB
bluegeek9’s picture

bluegeek9 commented
Title: PdfFormat::__construct(): Argument #10 ($public_stream) must be of type Drupal\Core\StreamWrapper\PublicStream, Drupal\s3fs\StreamWrapper\PublicS3fsStream » Use interfaces for PdfFormat::__construct()
Issue summary: View changes
Status: Active » Reviewed & tested by the community
Related issues: +#3194744: AliasManagerInterface InlineLinkExtractor::__construct()

bluegeek9 opened merge request !15

Nigel Cunningham made their first commit to this issue’s fork.

nigelcunningham’s picture

nigelcunningham commented
Status: Reviewed & tested by the community » Fixed

Merged; thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

kapil.purohit’s picture

kapil.purohit
Primary language English
commented