SDMON Feature request & issue

[kakenbok]

Hello pepak,

please provide an action hook to enable custom download error messages/screens.
For version 0.21 I have added a single line of code here:

Code Select Expand

protected function ErrorMessage($code, $message, $shortmessage)
{
    // added
    do_action('sdmon_download_error');  
   
    $code = intval($code);
    ...

[pepak]

OK, will do.

Is there a way to pass the arguments ($code, $message, $shortmessage) to the action hook?

[kakenbok]

Apparently YES: http://codex.wordpress.org/Function_Reference/do_action

And this way you catch the hook with arguments:

http://codex.wordpress.org/Function_Reference/add_action

[kakenbok]

I forgot the issue:

It is currently not possible to deselect an item that is hidden from the widget.
The reason is that your condition expects both the hide ids and the show ids.
In the case of a complete deselection the hide ids won't be available.

fix1 (exclusive condition):

Code Select Expand

public function ToolsPanel()
{
    ...
    if ($this->IsAdmin())
    {
        if (isset($_POST['SimpleDownloadMonitor_Update']) && (isset($_POST['SimpleDownloadMonitor_HideIds']) && is_array($_POST['SimpleDownloadMonitor_HideIds']) || isset($_POST['SimpleDownloadMonitor_ShowIds']) && is_array($_POST['SimpleDownloadMonitor_ShowIds'])))
        {
            $this->HideDownloads($_POST['SimpleDownloadMonitor_ShowIds'], $_POST['SimpleDownloadMonitor_HideIds']);
        }
        if (isset($_POST['SimpleDownloadMonitor_Delete']) && isset($_POST['SimpleDownloadMonitor_DeleteIds']) && is_array($_POST['SimpleDownloadMonitor_DeleteIds']))
        ...
and fix2 (test the lists separately):

Code Select Expand

protected function HideDownloads($show_ids, $hide_ids = array())
{
    global $wpdb;
    $downloads = $this->table_downloads();
    if ($show_ids)
    {
        $show_ids = implode(',', array_map('intval', $show_ids));
        $sql = "UPDATE ${downloads} SET hide_from_sidebar=0, last_date=last_date WHERE id IN (${show_ids})";
        // The last_date is used to prevent automatic update to current_timestamp
        $wpdb->query($sql);
                    }
    if ($hide_ids)
    {
        $hide_ids = implode(',', array_map('intval', $hide_ids));
        $sql = "UPDATE ${downloads} SET hide_from_sidebar=1, last_date=last_date WHERE id IN (${hide_ids})";
        // The last_date is used to prevent automatic update to current_timestamp
        $wpdb->query($sql);
    }
    //wp_redirect($_SERVER['REQUEST_URI']);
    //die();
}

[pepak]

OK, will check this out as well.