This error pops up if Drupal fails to create or otherwise make writable the configured file path for saving PDFs. This patch fixes that by not attempting to redirect to the PDF if it couldn't be saved. It also adds FILE_MODIFY_PERMISSIONS to the file_prepare_directory() call to decrease the chances of it failing unnecessarily.

Patch will be posted in first comment.

CommentFileSizeAuthor
#1 fillpdf-only-redirect-if-possible-1591816-1.patch2.22 KBwizonesolutions

Comments

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Status: Active » Fixed
StatusFileSize
new fillpdf-only-redirect-if-possible-1591816-1.patch2.22 KB

Attaching patch (already committed to 7.x-1.x in 30b8e84).

Status: Fixed » Closed (fixed)

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

  • Commit 30b8e84 on 7.x-1.x, 7.x-2.x, 7.x-2.x-tests1, 7.x-1.x-ubercartbackporttest by wizonesolutions: 
    Issue #1591816: Redirect to file only if successfully saved.