bresciauc

Thanks Ashley — I appreciate you adding the suggestion to the list.

I’ll make sure I use the download link whenever I originate the email. Often, our users just copy the link as rendered in their browser, which is how the much longer links can end up in emails and other documents.

Cheers, Dan

Thanks for the response Frankie!

The users had an Editor role, so the role-less hypothesis probably does not hold in this case. I’ve now re-activated Stream on that site, and will let you know if the errors start to crop up again. If none pop up in the next week or two, then we may need to file this one as a ‘Could not reproduce’ and close the ticket.

Really liking Stream by the way, which I’m using on another multisite as well, without any issues.

Just in case I introduced an error when copying code to the functions.php file, here is the current code.

/**
* Custom functions to allow Delightful Downloads (https://www.ads-software.com/plugins/delightful-downloads/) to open PDF files in a tab.
* DD is used to managed PDF documents on the Brescia web sites. Hopefully these functions will go into the DD plugin some day...
*/
function download_custom_redirect( $download_id ) {

    // Get path
    $download_url = get_post_meta( $download_id, '_dedo_file_url', true );

    // Send to file
    header( "Location: $download_url" );
    exit();

}
add_action( 'ddownload_download_before', 'download_custom_redirect', 20, 1 );

function download_new_tab( $styles ) {

	$styles['new_tab'] = array(
 		'name'		=> __( 'New Tab', 'ddownload' ),
 		'format'	=> '<a href="%url%" title="%text%" target="_blank" rel="nofollow">%text%</a>'
	);

	return $styles;
}
add_filter( 'dedo_get_styles', 'download_new_tab' );

I get the same 404 when I try this URL: https://bresciau.ca/wp-content/uploads/sites/8/delightful-downloads/2014/07/TheFOURRulesforResumeWriting3.pdf

One more thought… we just started using a CDN. Could this be be a possible factor?

Hi Ashley. I wonder if the issue may have something to do with the fact that the plug-in is being used on a sub-directory multisite, though there woulds also have to be some other factor in play.

As an example the direct URL of one of the downloads is https://bresciau.ca/life/wp-content/uploads/sites/8/delightful-downloads/2014/07/TheFOURRulesforResumeWriting3.pdf

The actual file is located at /public_html/wp-content/uploads/sites/8/delightful-downloads/2014/07/TheFOURRulesforResumeWriting3.pdf and has 644 permissions.

Can you see anything in the relationship between the URL and the path that might give us a clue as to what is up?

Hi Ashley — A followup question:

My new downloads are not working, giving a file-not-found error. I’m trying to figure out why…

In the functions.php code above I see this line:

‘name’ => __( ‘New Tab’, ‘delightful-downloads’ ),

In Download Settings the default Download Address directory is ‘ddownload’.

In the file system of my multisite, the directories are named as shown below:

‘/public_html/wp-content/uploads/sites/8/delightful-downloads’ with 755 permissions.

I have two questions about this:

Should all three of these locations be the same? ‘delightful-downloads’ or ‘ddownload’, but not a mix of both?

Also, since the issue seems to have started after a migration to a new server and new IP number yesterday, should I be raising it with my hosting service, as perhaps being permissions- or server-related?

Thanks — Dan

PS: We just donated $30 as a thanks for your hero-level support of this great plugin!

Thanks — Deployed and working well!

Hi Ashley,

The revised code works, but unfortunately it overwrites the %title% with a generic ‘Download’ if %text% does not exist. For this to work for our site, which has a lot of existing downloads with no %text% set, it would have to default to %title% if %text% does not exist.

Cheers, Dan [PS: The PayPal link is not working.]

Hi Ashley — Is it possible to make text customization work with the ‘New Tab’ format?

Currently doing something like this:
[ddownload id=”446″ style=”new_tab” text=”Download this file…”]
does not display the custom text as the link text.

Also… do you have a tip jar or similar where Delightful Download fans can chip in a bit of cash to thank you for the great support you give this plugin?

Just a follow-up note that I’m using Ashley’s code above, and it is working fine on my test server. I expect I’ll go into production with it exactly as-is. Thanks again!

Thanks Ashley — I really appreciate you giving us this great start!

You might find that this is a feature worth adding to a future release future as a configurable plugin option.

I’d guess that 20% or so of DD’s potential users are looking for a well-organized way to manage PDFs that open in the browser so they can maintain a very simple user experience for both authors and users.

(In our case we are a university, so we have a lot of informational PDFs, and I am really liking Delightful Downloads as a clean and easy way to manage these files.)

I’ll see what I can do with the sample code, and will post any modifications we do in case others find it useful. Cheers, Dan