Support » Plugin: WP Document Revisions » [Plugin: WP Document Revisions] Documents have WordPress HTML appended to them

  • Resolved Lynxy

    (@thelynxy)


    Ben Balter, thank you for the wonderful plugin! It is great, save one flaw: when downloading a document, the plugin returns control over to WordPress after outputting the file. This results in the full HTML data of the site being appended to the file! While it does not affect any of the browsers I’ve tested, I have had complaints from clients showing a bunch of binary data within a page.

    I have a fix for this though! I have it applied on my client’s site, and hope you will incorporate it into future updates. The fix entails changing return; to exit; at the end of the method serve_file() in wp-document-revisions.php. Simple enough!

    Thank you

    http://wordpress.org/extend/plugins/wp-document-revisions/

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author Ben Balter

    (@benbalter)

    Sorry for the trouble there. That should be a relatively, recent problem as I know that code was changed to make automated testing easier.

    I have to test it, but I suspect that moving the serve_file filter from single_template to template_include should fix the problem.

    Although template single would in theory be a bit more efficient, depending on the set up, I realize now it could in theory trip one of the other conditions in wp-includes/template-loader.php.

    Exiting isn’t explicitly necessary, as template-loader should be the last file executed.

    I’ve opened up a bug ticket and hope to follow up with a fix.

    Plugin Author Ben Balter

    (@benbalter)

    Could I trouble you to give the development version (zip) a try? I believe the issue should be resolved.

    Plugin Author Ben Balter

    (@benbalter)

    Marking the post as resolved (development version) and look for a stable release soon.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘[Plugin: WP Document Revisions] Documents have WordPress HTML appended to them’ is closed to new replies.