Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
When in the update hooks, we run a node_load() before running print_pdf_update_7203(), print_pdf_node_load() will attempt to load the database column "size", however this has not been created yet.
The following issues refer to this problem as well:
#1617654: Print doc
#2297701: Need to remove breadcrumbs and URL from Print pages
Comment | File | Size | Author |
---|---|---|---|
#2 | 2446939-print-update7203-2.patch | 997 bytes | Jorrit |
#1 | print-2446939-1.patch | 628 bytes | stefan.r |
Comments
Comment #1
stefan.r CreditAttribution: stefan.r commentedUpgrading priority as it can make a database update break due to a fatal error, preventing other update hooks from running.
Comment #2
Jorrit CreditAttribution: Jorrit at nCode for DOM Digital Online Media GmbH commentedI think executing
db_field_exists()
on every node load is a bit too expensive, it executes a query every time it is called.The attached patch handles PDOExceptions while executing the query, which makes sure that the code executes properly without the columns.
Comment #3
froboyI just updated an old site from
7.x-1.3
to7.x-2.2
and this patch allowed me to proceed successfully. It's pretty straightforward, didn't seem to take too long, and works. LGTM.Comment #4
Jorrit CreditAttribution: Jorrit at nCode for DOM Digital Online Media GmbH commentedDid you test patch #1 or patch #2?
Comment #6
jcnventura CreditAttribution: jcnventura at Wunder commentedThanks everyone!
Comment #7
jcnventura CreditAttribution: jcnventura at Wunder commented