dlsstudios

I wanted to give an update to let you know we have just incorporated carbon copies on all emails as one of the many new features in our Pro version of the plugin which was just released. For more information, please feel free to visit…

https://www.dlssoftwarestudios.com/sign-up-sheets-wordpress-plugin/

Thanks for the feedback. I have added your request to our wishlist of possible future upgrades.

With the latest version 1.0.6 of the Sign-up Sheets plugin, the sign up form fields are now protected against cross site scripting. Please note that SQL Injection was not an issue as all database inserts and updates are properly sanitized.

Thank you for alerting us of this issue. We will have it corrected with the next version and should be releasing it shortly.

Thank you for reporting this. I have just published a fix in plugin version 1.0.5 that corrects the sort feature. After upgrading you should now be able to drag tasks around in any order. If you have any further trouble, please let me know. Thanks!

Excellent! That’s good to hear.

To help debug this further, would you be able to upgrade to the newest version of the plugin v1.0.4? Then in the file sign-up-sheets.php replace line #767 (starts with “return wp_mail…”) with the code below. Then go into Settings > Sign-up Sheets in your WP admin and turn on detailed error messages. This should provide more details on what is actually erroring out.

$result = wp_mail($to, $subject, $message, $headers);

if (!$result) {
    global $ts_mail_errors;
    global $phpmailer;
    if (!isset($ts_mail_errors)) $ts_mail_errors = array();
    if (isset($phpmailer)) $ts_mail_errors[] = $phpmailer->ErrorInfo;
    throw new SUS_Data_Exception(__('Error sending email.'). (($this->detailed_errors === true) ? '.. '.implode(' --- ', $ts_mail_errors) : ''));
}

return $result;

@hermesnl, can you try changing line #247 of data.php (starting with “throw new”) to the code below…

throw new SUS_Data_Exception('Error adding signup. All spots are filled.'. (($this->detailed_errors === true) ? ' Current Signups: '.count($signups).', Total Spots:'.$task->qty : ''));

Then try running your test again with detailed errors on and let me know the error message you get. You should now get a little more data.

We have recently updated our plugin to v1.0.4 and one of the features allows for you to turn on more detailed error messages. I would recommend updating your plugin to the latest version and then in the admin Settings > Sign-up Sheets menu check the box next to Display detailed errors then try and recreate the error you received above and let me know what it says. Hopefully that should provide some insight.

We have just posted an update to the plugin. One of the new features is to be able to turn on detailed error reporting through the admin Settings > Sign-up Sheets menu. Instead of making the code changes above, please feel free to upgrade and turn on the detailed error messages which may provide some clue as to what is going on.

Thank you for reporting this bug. It has now been corrected with version 1.0.4 of the Sign-up Sheets plugin.

I haven’t been able to recreate this issue yet, but if you wouldn’t mind, please make these temporary modifications which will change the error message you are getting to include more details. Then let me know the new error message you receive. If you prefer to send this information privately you can contact us through https://www.dlssoftwarestudios.com/contact-us/

Replace the line I had you comment out earlier in this topic in the file sign-up-sheets.php with…

$return .= '<p class="dls-sus error">'.__('Error adding signup record. Please try again.').' ERROR DETAIL: '.print_r($this->data->err, true).'</p>';

Then, in data.php below the line at the top that says public $tables = array(); add…

public $err = array();

Then replace the entire add_signup() method with the following code

public function add_signup($fields, $task_id)
    {
        $clean_fields = $this->clean_array($fields, 'signup_');
        $clean_fields = array_intersect_key($clean_fields, $this->tables['signup']['allowed_fields']);
        $clean_fields['task_id'] = $task_id;

        // Check if signup spots are filled
        $task = $this->get_task($task_id);
        $signups = $this->get_signups($task_id);
        if (count($signups) >= $task->qty) {
            $this->err['add_signup'][] = 'All spots are filled.';//debug
            return false;
        }

        $result = $this->wpdb->insert('asdf', $clean_fields);
        if ($result === false) $this->err['add_signup'][] = 'Error when inserting signup... (START MYSQL ERROR) '.print_r(mysql_error(), true).' (START MYSQL ERROR)';//debug

        return $result;
    }

A few questions…

1. In your test, did the email you entered actually receive the sign-up sheet confirmation?
2. If no, was the email address valid?
3. Have you had trouble with any other email notifications through that set up of WordPress?

Hi Scott,

Thank you for your feedback! We will add your suggestions to our wish list of possible future improvements.

Yes, this is already possible using the “copy” function! After logging in to your WordPress Admin go to “Sign-up Sheets” on the left and search for the sign-up sheet you would like to copy. Hover your mouse over the row and you will see the action links appear such as “View Sign-ups”, “Edit”, etc. Click the one that says “Copy” and it will automatically copy that sheet with the same name plus “(Copy)” appended at the end. You can then edit it and modify any necessary fields.

If you have any other questions, please let us know.