normadize

The fix is to change .text(n[0]) to .html(n[0]) in scripts.js line 220 in here:

$.fn.wpcf7RefillQuiz = function(quiz) {
	return this.each(function() {
		var form = $(this);

		$.each(quiz, function(i, n) {
			form.find(':input[name="' + i + '"]').clearFields();
			form.find(':input[name="' + i + '"]').siblings('span.wpcf7-quiz-label').html(n[0]);
			form.find('input:hidden[name="_wpcf7_quiz_answer_' + i + '"]').attr('value', n[1]);
		});
	});
};

For anyone interested, if you want your uploaded files to be accessible after upload via a URL, and also to have that URL in the email sent by WPCF7, then here’s a solution I used in my plugin.

If you don’t use classes, then replace the array with just the function name. Note that this is for one file only, use a foreach() loop if your form has multiple upload files.

You may also want to change the chmod() permissions to suit your server. In my case, SuPHP requires 644 or else it denies access.

class MyPlugin
{
    /**
     * Relative to the WP uploads dir.
     */
    public $dir_uploaded = 'foo';

    function __construct ()
    {
        add_action('wpcf7_before_send_mail', array($this, 'WPCF7_BeforeSendMail'));
    }

    /**
     * Fetch uploaded files and modify email body to point to uploaded file url.
     * @param WPCF7_ContactForm $cf7
     */
    public function WPCF7_BeforeSendMail (WPCF7_ContactForm $cf7)
    {
        if (empty($cf7->uploaded_files['fileupload']))
            return;
        $dest = wp_upload_dir();
        $dest_dir = $dest['basedir'].$this->dir_uploaded;
        $dest_url = $dest['baseurl'].$this->dir_uploaded;
        wp_mkdir_p($dest_dir);
        $dest_file = wp_unique_filename($dest_dir, basename($cf7->uploaded_files['fileupload']));
        $dest_path = "$dest_dir/$dest_file";
        @rename($cf7->uploaded_files['fileupload'], $dest_path);
        // avoid SuPHP denying access since WPCF7 hardcodes 0400 (...)
        chmod($dest_path, 0644);
        $cf7->mail['body'] = "Uploaded: $dest_url/$dest_file\n\n" . $cf7->mail['body'];
    }
}

Cheers.

The wpcf7_before_send_mail hook worked just fine. I had to move the file to another directory though so that CF7 won’t delete it.

You may still want to consider including something like this as a core feature (big file uploads as download URL in the email).

One small thing: I’m using SuPHP in a setup that requires 0644 for files and and 0755 for dirs. CF7 forces files to be 0400 and I understand the reason behind it, but it introduces some difficulty in SuPHP setups.

Cheers, and keep up the good work.

Your current code is not any more secure. More importantly, you should leave the choice to the user, not making the decision for them.

I have a need for this for an internal project. As far as i can tell, currently you

– require an extension (not all files have one)
– force an extension filter
– are not willing to even consider this since you marked this as resolved

Those are not very healthy choices. We are talking about an admin option so security wise it’s fine and you can still leave the default to require an extension.

I see only benefits from adding an option like filetyperegex: for the expert users.

Got it. It was the suphp config file which had a umask of 0077. Edit /etc/suphp/suphp.conf (or wherever you keep yours) and set umask=0022 to get 644 files and 755 dirs.

Surprising that I haven’t seen this solution in these threads, although other people discussed suphp, or maybe I just missed it.

It might be worth adding this info in WPSC … for the Admin Panel test, if it can’t read back the .html files then it could tell the user some possible causes. Currently it just says “failed to read”.

Cheers

The solution is not to check the post type — this plugin should be available on all post types

But Oshouki’s fix doesn’t check the post type, and the dropbown box is indeed available for all post types. His fix prevents multiple copies of the same dropdown box, which is the problem I reported. Your fix works too, thanks.

Ok. I’m posting a fix here too, just in case others are interested in patching their Pods too. In pick.php line 383 in PodsField_Pick::data(). Append after:

$result[ $search_data->field_index ] = trim( $result[ $search_data->field_index ] );

the line:

if ( $search_data->field_index == 'post_title' )
    $result[ 'post_title' ] = apply_filters( 'the_title', $result[ 'post_title' ] );

It’s crude, I know, but you get the point.