Fix so database_attachments::cleanup() does not remove attachments from oth...

Aleksander Machniak

2015-09-28 ba48318e2c87f4d07982978e539151c1abcf2114

 plugins/database_attachments/database_attachments.php

@@ -13,14 +13,16 @@
 * @author Aleksander Machniak <alec@alec.pl>
 * @version @package_version@
 */
require_once('plugins/filesystem_attachments/filesystem_attachments.php');

require_once INSTALL_PATH . 'plugins/filesystem_attachments/filesystem_attachments.php';

class database_attachments extends filesystem_attachments
{
    // Cache object
    protected $cache;

    // A prefix for the cache key used in the session and in the key field of the cache table
    protected $prefix = "db_attach";
    const PREFIX = "ATTACH";

    /**
     * Save a newly uploaded attachment
@@ -41,9 +43,9 @@
        $status = $cache->write($key, $data);

        if ($status) {
            $args['id'] = $key;
            $args['id']     = $key;
            $args['status'] = true;
            unset($args['path']);
            $args['path']   = null;
        }

        return $args;
@@ -124,8 +126,10 @@
     */
    function cleanup($args)
    {
        $cache = $this->get_cache();
        $cache->remove($args['group'], true);
        // check if cache object exist, it may be empty on session_destroy (#1489726)
        if ($cache = $this->get_cache()) {
            $cache->remove($args['group'], true);
        }
    }

    /**
@@ -149,9 +153,16 @@
            $ttl    = 12 * 60 * 60; // default: 12 hours
            $ttl    = $rcmail->config->get('database_attachments_cache_ttl', $ttl);
            $type   = $rcmail->config->get('database_attachments_cache', 'db');
            $prefix = self::PREFIX;

            // Add session identifier to the prefix to prevent from removing attachments
            // in other sessions of the same user (#1490542)
            if ($id = session_id()) {
                $prefix .= $id;
            }

            // Init SQL cache (disable cache data serialization)
            $this->cache = $rcmail->get_cache($this->prefix, 'db', $ttl, false);
            $this->cache = $rcmail->get_cache($prefix, $type, $ttl, false);
        }

        return $this->cache;