Problem/Motivation

Seeing this when saving files on Acquia.

Adding DependencySerializationTrait to the event subscriber cures.

CommentFileSizeAuthor
#10 settings-cannot-be-serialized-10.patch7.47 KBethant
#9 settings-cannot-be-serialized-9.patch7.25 KBethant
#8 settings-cannot-be-serialized-8.patch5.22 KBethant
#7 settings-cannot-be-serialized-7.patch4.11 KBethant
#6 settings-cannot-be-serialized-6.patch3.52 KBethant
#5 settings-cannot-be-serialized-5.patch3.05 KBethant
#4 settings-cannot-be-serialized-4.patch743 bytesethant
#3 settings-cannot-be-serialized-3.patch1.67 KBethant
#2 settings-cannot-be-serialized-2.patch1.2 KBethant
settings-cannot-be-serialized.patch851 bytesethant

Issue fork pdf_metadata-3368688

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

EthanT created an issue. See original summary.

ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-2.patch1.2 KB
ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-3.patch1.67 KB
ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-4.patch743 bytes
ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-5.patch3.05 KB
ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-6.patch3.52 KB
ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-7.patch4.11 KB
ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-8.patch5.22 KB
ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-9.patch7.25 KB
ethant’s picture

ethant
Location Sarasota, Florida
commented
StatusFileSize
new settings-cannot-be-serialized-10.patch7.47 KB
ethant’s picture

ethant
Location Sarasota, Florida
commented

j-barnes made their first commit to this issue’s fork.

j-barnes opened merge request !5

j-barnes’s picture

j-barnes commented

@ethan - Great find, it looks like mpdf needed to have the temp location explicitly set to avoid having it use the vendor location which was causing permission issues on certain environments like Acquia Platform. This lead to a serialization error because the logger function was trying to print out the entire exception.

I've updated the logger to use $e->getMessage() and now pull in the temp location via filesystem and have also cleaned up some of the types.

Thanks again, going to leave this open for a bit for testing.

j-barnes’s picture

j-barnes commented
Status: Active » Needs review
j-barnes’s picture

j-barnes commented

Unfortunately it looks like mPDF uses FPDI which does not work on PDFs above v1.4 (at least the free version). Updating MR to use GhostScript instead which should have increased performance and works on 1.7+.

sunny-lee’s picture

sunny-lee commented
Status: Needs review » Reviewed & tested by the community

PDF metadata patches and modules works with Drupal 10 and PHP 8.1

j-barnes’s picture

j-barnes commented

  • j-barnes committed 4a3cdf9e on 1.0.x 
    Issue #3368688: LogicException: Settings can not be serialized.
j-barnes’s picture

j-barnes commented
Status: Reviewed & tested by the community » Fixed
j-barnes’s picture

j-barnes commented
Status: Fixed » Closed (fixed)