The functionality request is simple, after submitting the form redirect to the PDF that has been created by the module.

This is an alternative to downloading the module or only saving the file to a folder somewhere.

I am working on a quick hack for this so I can implement it this week but would be a nice feature where all the user had to do was check a box.

CommentFileSizeAuthor
#7 fillpdf_1572706_redirect_to_pdf_D6.patch1.86 KBalexborsody
#2 fillpdf_1572706_redirect_to_pdf.patch3.77 KBwizonesolutions

Comments

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Version: 7.x-1.5 » 7.x-1.x-dev

Sounds good. Feel free to post your patch as a starting point, and I'll post mine once I've implemented it. It will be a checkbox, and if people want a different path, they can use the destination query string parameter provided by Drupal (which already works). I'll document that under the checkbox.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Status: Active » Needs review
StatusFileSize
new fillpdf_1572706_redirect_to_pdf.patch3.77 KB

Here's a patch implementing this. Try this for me and let me know if it does the job? If you've already moved on from this problem, let me know if you won't be testing and I'll just commit it after a few days.

alexborsody’s picture

alexborsody commented

I am going to test this today thanks.

alexborsody’s picture

alexborsody commented

The patch works great.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Status: Needs review » Reviewed & tested by the community

Awesome! I tested it too, so...RTBC :P I'll commit it by Thursday, since I'll be preparing my DrupalCamp Twin Cities presentation then.

wizonesolutions’s picture

wizonesolutions
he/him
Primary language English
Location Norway
commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Fixed in fdbf8cb for 7.x.

alexborsody’s picture

alexborsody commented
StatusFileSize
new fillpdf_1572706_redirect_to_pdf_D6.patch1.86 KB

here is the patch for Drupal 6

alexborsody’s picture

alexborsody commented

Ported to D6 in commit 1934f3e

alexborsody’s picture

alexborsody commented
Status: Patch (to be ported) » Closed (fixed)

  • Commit fdbf8cb on 7.x-1.x, 7.x-2.x, 7.x-2.x-tests1, 7.x-1.x-ubercartbackporttest by wizonesolutions: 
    Issue #1572706: Add option to redirect directly to PDF.