From f49b74c5f6ebe57937fb6dfea7d2e917f4680ce9 Mon Sep 17 00:00:00 2001 From: Klaus Weidenbach Date: Fri, 27 Jun 2014 01:30:12 +0200 Subject: Some documentation for include/reddav.php and a new tpl-file. Documented include/reddav.php a bit more to understand what it is about to try to clean up some parts. Still a lot to be done here. I put serveral @todo markers that need to be checked. Moved some more HTML to template file. Changed getETag() return format according to documentation. (ETag MUST be surrounded by double-quotes) --- include/reddav.php | 1019 ++++++++++++++++++++++----------------- view/tpl/cloud_actionspanel.tpl | 23 + 2 files changed, 609 insertions(+), 433 deletions(-) create mode 100644 view/tpl/cloud_actionspanel.tpl diff --git a/include/reddav.php b/include/reddav.php index e03aa20ee..c9b238efb 100644 --- a/include/reddav.php +++ b/include/reddav.php @@ -1,281 +1,333 @@ -= 2.0 + * requires PHP >= 5.4. + */ use Sabre\DAV; require_once('vendor/autoload.php'); - require_once('include/attach.php'); + +/** + * @brief RedDirectory class. + * + * A class that represents a directory. + */ class RedDirectory extends DAV\Node implements DAV\ICollection, DAV\IQuota { private $red_path; private $folder_hash; + // @todo I think this is not used anywhere, we always strip '/cloud' and only use it in debug private $ext_path; private $root_dir = ''; private $auth; private $os_path = ''; - function __construct($ext_path,&$auth_plugin) { + /** + * Sets up the directory node, expects a full path. + * + * @param string $ext_path a full path + * @param $auth_plugin + */ + public function __construct($ext_path, &$auth_plugin) { logger('RedDirectory::__construct() ' . $ext_path, LOGGER_DEBUG); $this->ext_path = $ext_path; - $this->red_path = ((strpos($ext_path,'/cloud') === 0) ? substr($ext_path,6) : $ext_path); - if(! $this->red_path) + // remove "/cloud" from the beginning of the path + $this->red_path = ((strpos($ext_path, '/cloud') === 0) ? substr($ext_path, 6) : $ext_path); + if (! $this->red_path) { $this->red_path = '/'; + } $this->auth = $auth_plugin; $this->folder_hash = ''; - $this->getDir(); - if($this->auth->browser) + if ($this->auth->browser) { $this->auth->browser->set_writeable(); - + } } - function log() { logger('RedDirectory::log() ext_path ' . $this->ext_path, LOGGER_DATA); logger('RedDirectory::log() os_path ' . $this->os_path, LOGGER_DATA); logger('RedDirectory::log() red_path ' . $this->red_path, LOGGER_DATA); } - function getChildren() { - + /** + * @brief Returns an array with all the child nodes. + * + * @throws DAV\Exception\Forbidden + * @return array DAV\INode[] + */ + public function getChildren() { logger('RedDirectory::getChildren() called for ' . $this->ext_path, LOGGER_DATA); - $this->log(); - if(get_config('system','block_public') && (! $this->auth->channel_id) && (! $this->auth->observer)) { + if (get_config('system', 'block_public') && (! $this->auth->channel_id) && (! $this->auth->observer)) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - if(($this->auth->owner_id) && (! perm_is_allowed($this->auth->owner_id,$this->auth->observer,'view_storage'))) { + if (($this->auth->owner_id) && (! perm_is_allowed($this->auth->owner_id, $this->auth->observer, 'view_storage'))) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - $contents = RedCollectionData($this->red_path,$this->auth); + $contents = RedCollectionData($this->red_path, $this->auth); return $contents; } + /** + * @brief Returns a child by name. + * + * + * @throw DAV\Exception\Forbidden + * @throw DAV\Exception\NotFound + * @param string $name + */ + public function getChild($name) { + logger('RedDirectory::getChild(): ' . $name, LOGGER_DATA); - function getChild($name) { - - logger('RedDirectory::getChild : ' . $name, LOGGER_DATA); - - if(get_config('system','block_public') && (! $this->auth->channel_id) && (! $this->auth->observer)) { + if (get_config('system', 'block_public') && (! $this->auth->channel_id) && (! $this->auth->observer)) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - if(($this->auth->owner_id) && (! perm_is_allowed($this->auth->owner_id,$this->auth->observer,'view_storage'))) { + if (($this->auth->owner_id) && (! perm_is_allowed($this->auth->owner_id, $this->auth->observer, 'view_storage'))) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - if($this->red_path === '/' && $name === 'cloud') { + if ($this->red_path === '/' && $name === 'cloud') { return new RedDirectory('/cloud', $this->auth); } $x = RedFileData($this->ext_path . '/' . $name, $this->auth); - if($x) + if ($x) { return $x; + } - throw new DAV\Exception\NotFound('The file with name: ' . $name . ' could not be found'); - + throw new DAV\Exception\NotFound('The file with name: ' . $name . ' could not be found.'); } - function getName() { + public function getName() { logger('RedDirectory::getName returns: ' . basename($this->red_path), LOGGER_DATA); return (basename($this->red_path)); } + /** + * @brief Creates a new file in the directory. + * + * Data will either be supplied as a stream resource, or in certain cases + * as a string. Keep in mind that you may have to support either. + * + * After successful creation of the file, you may choose to return the ETag + * of the new file here. + * + * @throws DAV\Exception\Forbidden + * @param string $name Name of the file + * @param resource|string $data Initial payload + * @return null|string + */ + public function createFile($name, $data = null) { + logger('RedDirectory::createFile(): ' . $name, LOGGER_DEBUG); - - - function createFile($name,$data = null) { - logger('RedDirectory::createFile : ' . $name, LOGGER_DEBUG); - - if(! $this->auth->owner_id) { + if (! $this->auth->owner_id) { logger('createFile: permission denied'); throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - if(! perm_is_allowed($this->auth->owner_id,$this->auth->observer,'write_storage')) { + if (! perm_is_allowed($this->auth->owner_id, $this->auth->observer, 'write_storage')) { logger('createFile: permission denied'); throw new DAV\Exception\Forbidden('Permission denied.'); - return; } $mimetype = z_mime_content_type($name); - - $c = q("select * from channel where channel_id = %d and not (channel_pageflags & %d) limit 1", + $c = q("SELECT * FROM channel WHERE channel_id = %d AND NOT (channel_pageflags & %d) LIMIT 1", intval($this->auth->owner_id), intval(PAGE_REMOVED) - ); - if(! $c) { - logger('createFile: no channel'); + if (! $c) { + logger('RedDirectory::createFile(): no channel'); throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - $filesize = 0; $hash = random_string(); - $r = q("INSERT INTO attach ( aid, uid, hash, creator, filename, folder, flags, filetype, filesize, revision, data, created, edited, allow_cid, allow_gid, deny_cid, deny_gid ) - VALUES ( %d, %d, '%s', '%s', '%s', '%s', '%s', '%s', %d, %d, '%s', '%s', '%s', '%s', '%s', '%s', '%s' ) ", - intval($c[0]['channel_account_id']), - intval($c[0]['channel_id']), - dbesc($hash), + $r = q("INSERT INTO attach ( aid, uid, hash, creator, filename, folder, flags, filetype, filesize, revision, data, created, edited, allow_cid, allow_gid, deny_cid, deny_gid ) + VALUES ( %d, %d, '%s', '%s', '%s', '%s', '%s', '%s', %d, %d, '%s', '%s', '%s', '%s', '%s', '%s', '%s' ) ", + intval($c[0]['channel_account_id']), + intval($c[0]['channel_id']), + dbesc($hash), dbesc($this->auth->observer), - dbesc($name), + dbesc($name), dbesc($this->folder_hash), dbesc(ATTACH_FLAG_OS), - dbesc($mimetype), - intval($filesize), - intval(0), - dbesc($this->os_path . '/' . $hash), - dbesc(datetime_convert()), - dbesc(datetime_convert()), + dbesc($mimetype), + intval($filesize), + intval(0), + dbesc($this->os_path . '/' . $hash), + dbesc(datetime_convert()), + dbesc(datetime_convert()), dbesc($c[0]['channel_allow_cid']), dbesc($c[0]['channel_allow_gid']), dbesc($c[0]['channel_deny_cid']), dbesc($c[0]['channel_deny_gid']) - - ); $f = 'store/' . $this->auth->owner_nick . '/' . (($this->os_path) ? $this->os_path . '/' : '') . $hash; - file_put_contents($f, $data); - $size = filesize($f); + // returns the number of bytes that were written to the file, or FALSE on failure + $size = file_put_contents($f, $data); + // delete attach entry if file_put_contents() failed + if ($size === false) { + logger('RedDirectory::createFile(): file_put_contents() failed for ' . $name, LOGGER_DEBUG); + attach_delete($c[0]['channel_id'], $hash); + return; + } + // returns now $edited = datetime_convert(); - $d = q("update attach set filesize = '%s', edited = '%s' where hash = '%s' and uid = %d limit 1", + // updates entry with filesize and timestamp + $d = q("UPDATE attach SET filesize = '%s', edited = '%s' WHERE hash = '%s' AND uid = %d LIMIT 1", dbesc($size), dbesc($edited), dbesc($hash), intval($c[0]['channel_id']) ); - $e = q("update attach set edited = '%s' where folder = '%s' and uid = %d limit 1", + // update the folder's lastmodified timestamp + $e = q("UPDATE attach SET edited = '%s' WHERE folder = '%s' AND uid = %d LIMIT 1", dbesc($edited), dbesc($this->folder_hash), intval($c[0]['channel_id']) - ); - - $maxfilesize = get_config('system','maxfilesize'); + ); - if(($maxfilesize) && ($size > $maxfilesize)) { - attach_delete($c[0]['channel_id'],$hash); + $maxfilesize = get_config('system', 'maxfilesize'); + if (($maxfilesize) && ($size > $maxfilesize)) { + attach_delete($c[0]['channel_id'], $hash); return; } - $limit = service_class_fetch($c[0]['channel_id'],'attach_upload_limit'); - if($limit !== false) { - $x = q("select sum(filesize) as total from attach where aid = %d ", + // check against service class quota + $limit = service_class_fetch($c[0]['channel_id'], 'attach_upload_limit'); + if ($limit !== false) { + $x = q("SELECT SUM(filesize) AS total FROM attach WHERE aid = %d ", intval($c[0]['channel_account_id']) ); - if(($x) && ($x[0]['total'] + $size > $limit)) { + if (($x) && ($x[0]['total'] + $size > $limit)) { logger('reddav: service class limit exceeded for ' . $c[0]['channel_name'] . ' total usage is ' . $x[0]['total'] . ' limit is ' . $limit); - attach_delete($c[0]['channel_id'],$hash); + attach_delete($c[0]['channel_id'], $hash); return; } } } + /** + * @brief Creates a new subdirectory. + * + * @param string $name the directory to create + * @return void + */ + public function createDirectory($name) { + logger('RedDirectory::createDirectory(): ' . $name, LOGGER_DEBUG); - function createDirectory($name) { - - logger('RedDirectory::createDirectory: ' . $name, LOGGER_DEBUG); - - if((! $this->auth->owner_id) || (! perm_is_allowed($this->auth->owner_id,$this->auth->observer,'write_storage'))) { + if ((! $this->auth->owner_id) || (! perm_is_allowed($this->auth->owner_id, $this->auth->observer, 'write_storage'))) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - $r = q("select * from channel where channel_id = %d and not (channel_pageflags & %d) limit 1", + $r = q("SELECT * FROM channel WHERE channel_id = %d AND NOT (channel_pageflags & %d) LIMIT 1", intval($this->auth->owner_id), intval(PAGE_REMOVED) ); - if($r) { - $result = attach_mkdir($r[0],$this->auth->observer,array('filename' => $name,'folder' => $this->folder_hash)); - if(! $result['success']) - logger('RedDirectory::createDirectory: ' . print_r($result,true), LOGGER_DEBUG); + if ($r) { + $result = attach_mkdir($r[0], $this->auth->observer, array('filename' => $name, 'folder' => $this->folder_hash)); + if (! $result['success']) + logger('RedDirectory::createDirectory(): ' . print_r($result, true), LOGGER_DEBUG); } } - - function childExists($name) { - - if($this->red_path === '/' && $name === 'cloud') { - logger('RedDirectory::childExists /cloud: true', LOGGER_DATA); + /** + * @brief Checks if a child exists. + * + * @param string $name + * @return boolean + */ + public function childExists($name) { + // On /cloud we show a list of available channels. + // @todo what happens if no channels are available? + if ($this->red_path === '/' && $name === 'cloud') { + logger('RedDirectory::childExists() /cloud: true', LOGGER_DATA); return true; } - $x = RedFileData($this->ext_path . '/' . $name, $this->auth,true); - logger('RedFileData returns: ' . print_r($x,true), LOGGER_DATA); - if($x) + $x = RedFileData($this->ext_path . '/' . $name, $this->auth, true); + logger('RedFileData returns: ' . print_r($x, true), LOGGER_DATA); + if ($x) return true; return false; } + /** + * @todo add description of what this function does. + * + * @throw DAV\Exception\NotFound + * @return void + */ function getDir() { - logger('getDir: ' . $this->ext_path, LOGGER_DEBUG); + logger('RedDirectory::getDir(): ' . $this->ext_path, LOGGER_DEBUG); $this->auth->log(); $file = $this->ext_path; - $x = strpos($file,'/cloud'); - if($x === false) + $x = strpos($file, '/cloud'); + if ($x === false) return; - if($x === 0) { - $file = substr($file,6); + if ($x === 0) { + $file = substr($file, 6); } - if((! $file) || ($file === '/')) { + if ((! $file) || ($file === '/')) { return; } - $file = trim($file,'/'); + $file = trim($file, '/'); $path_arr = explode('/', $file); - - if(! $path_arr) - return; + if (! $path_arr) + return; - logger('getDir(): path: ' . print_r($path_arr,true), LOGGER_DEBUG); + logger('RedDirectory::getDir(): path: ' . print_r($path_arr, true), LOGGER_DEBUG); $channel_name = $path_arr[0]; - - $r = q("select channel_id from channel where channel_address = '%s' and not ( channel_pageflags & %d ) limit 1", + $r = q("SELECT channel_id FROM channel WHERE channel_address = '%s' AND NOT ( channel_pageflags & %d ) LIMIT 1", dbesc($channel_name), intval(PAGE_REMOVED) ); - if(! $r) { + if (! $r) { throw new DAV\Exception\NotFound('The file with name: ' . $channel_name . ' could not be found'); - return; } + $channel_id = $r[0]['channel_id']; $this->auth->owner_id = $channel_id; $this->auth->owner_nick = $channel_name; $path = '/' . $channel_name; - $folder = ''; $os_path = ''; - for($x = 1; $x < count($path_arr); $x ++) { - + for ($x = 1; $x < count($path_arr); $x++) { $r = q("select id, hash, filename, flags from attach where folder = '%s' and filename = '%s' and uid = %d and (flags & %d)", dbesc($folder), dbesc($path_arr[$x]), @@ -283,49 +335,58 @@ class RedDirectory extends DAV\Node implements DAV\ICollection, DAV\IQuota { intval(ATTACH_FLAG_DIR) ); - if($r && ( $r[0]['flags'] & ATTACH_FLAG_DIR)) { + if ($r && ( $r[0]['flags'] & ATTACH_FLAG_DIR)) { $folder = $r[0]['hash']; - if(strlen($os_path)) + if (strlen($os_path)) $os_path .= '/'; $os_path .= $folder; $path = $path . '/' . $r[0]['filename']; - } + } } $this->folder_hash = $folder; $this->os_path = $os_path; return; } - - function getLastModified() { - $r = q("select edited from attach where folder = '%s' and uid = %d order by edited desc limit 1", + /** + * @brief Returns the last modification time for the directory, as a unix + * timestamp. + * + * @return int last modification time in UNIX timestamp + */ + public function getLastModified() { + $r = q("SELECT edited FROM attach WHERE folder = '%s' AND uid = %d ORDER BY edited DESC LIMIT 1", dbesc($this->folder_hash), - intval($this->auth->owner_id) + intval($this->auth->owner_id) ); - if($r) - return datetime_convert('UTC','UTC', $r[0]['edited'],'U'); + if ($r) + return datetime_convert('UTC', 'UTC', $r[0]['edited'], 'U'); return ''; } - + /** + * @brief Return quota usage. + * + * Do guests relly see the used/free values from filesystem of the complete store directory? + * + * @return array with used and free values in bytes. + */ public function getQuotaInfo() { - + // values from the filesystem of the complete store/ directory $limit = disk_total_space('store'); $free = disk_free_space('store'); - if($this->auth->owner_id) { - + if ($this->auth->owner_id) { $c = q("select * from channel where channel_id = %d and not (channel_pageflags & %d) limit 1", intval($this->auth->owner_id), intval(PAGE_REMOVED) - ); - $ulimit = service_class_fetch($c[0]['channel_id'],'attach_upload_limit'); + $ulimit = service_class_fetch($c[0]['channel_id'], 'attach_upload_limit'); $limit = (($ulimit) ? $ulimit : $limit); - $x = q("select sum(filesize) as total from attach where aid = %d ", + $x = q("select sum(filesize) as total from attach where aid = %d", intval($c[0]['channel_account_id']) ); $free = (($x) ? $limit - $x[0]['total'] : 0); @@ -335,237 +396,322 @@ class RedDirectory extends DAV\Node implements DAV\ICollection, DAV\IQuota { $limit - $free, $free ); - } +} // class RedDirectory -} +/** + * RedFile class. + * + */ class RedFile extends DAV\Node implements DAV\IFile { private $data; private $auth; private $name; - function __construct($name, $data, &$auth) { + /** + * Sets up the node, expects a full path name. + * + * @param string $name + * @param array $data from attach table + * @param &$auth + */ + public function __construct($name, $data, &$auth) { $this->name = $name; $this->data = $data; $this->auth = $auth; - logger('RedFile::_construct: ' . print_r($this->data,true), LOGGER_DATA); + logger('RedFile::__construct(): ' . print_r($this->data, true), LOGGER_DATA); } - - function getName() { - logger('RedFile::getName: ' . basename($this->name), LOGGER_DEBUG); + /** + * @brief Returns the name of the file. + * + * @return string + */ + public function getName() { + logger('RedFile::getName(): ' . basename($this->name), LOGGER_DEBUG); return basename($this->name); - } + /** + * @brief Renames the file. + * + * @throw DAV\Exception\Forbidden + * @param string $name The new name of the file. + * @return void + */ + public function setName($newName) { + logger('RedFile::setName(): ' . basename($this->name) . ' -> ' . $newName, LOGGER_DEBUG); - function setName($newName) { - logger('RedFile::setName: ' . basename($this->name) . ' -> ' . $newName, LOGGER_DEBUG); - - if((! $newName) || (! $this->auth->owner_id) || (! perm_is_allowed($this->auth->owner_id,$this->auth->observer,'write_storage'))) { + if ((! $newName) || (! $this->auth->owner_id) || (! perm_is_allowed($this->auth->owner_id, $this->auth->observer, 'write_storage'))) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - $newName = str_replace('/','%2F',$newName); + $newName = str_replace('/', '%2F', $newName); - $r = q("update attach set filename = '%s' where hash = '%s' and id = %d limit 1", + $r = q("UPDATE attach SET filename = '%s' WHERE hash = '%s' AND id = %d LIMIT 1", dbesc($this->data['filename']), intval($this->data['id']) ); - } + /** + * @brief Updates the data of the file. + * + * @param resource $data + * @return void + */ + public function put($data) { + logger('RedFile::put(): ' . basename($this->name), LOGGER_DEBUG); + $size = 0; - function put($data) { - logger('RedFile::put: ' . basename($this->name), LOGGER_DEBUG); - - $c = q("select * from channel where channel_id = %d and not (channel_pageflags & %d) limit 1", + // @todo only 3 values are needed + $c = q("SELECT * FROM channel WHERE channel_id = %d AND NOT (channel_pageflags & %d) LIMIT 1", intval(PAGE_REMOVED), intval($this->auth->owner_id) ); - $r = q("select flags, folder, data from attach where hash = '%s' and uid = %d limit 1", + $r = q("SELECT flags, folder, data FROM attach WHERE hash = '%s' AND uid = %d LIMIT 1", dbesc($this->data['hash']), intval($c[0]['channel_id']) ); - if($r) { - if($r[0]['flags'] & ATTACH_FLAG_OS) { + if ($r) { + if ($r[0]['flags'] & ATTACH_FLAG_OS) { $f = 'store/' . $this->auth->owner_nick . '/' . (($r[0]['data']) ? $r[0]['data'] : ''); + // @todo check return value and set $size directly @file_put_contents($f, $data); $size = @filesize($f); - logger('reddav: put() filename: ' . $f . ' size: ' . $size, LOGGER_DEBUG); - } - else { - $r = q("update attach set data = '%s' where hash = '%s' and uid = %d limit 1", + logger('RedFile::put(): filename: ' . $f . ' size: ' . $size, LOGGER_DEBUG); + } else { + $r = q("UPDATE attach SET data = '%s' WHERE hash = '%s' AND uid = %d LIMIT 1", dbesc(stream_get_contents($data)), dbesc($this->data['hash']), intval($this->data['uid']) ); - $r = q("select length(data) as fsize from attach where hash = '%s' and uid = %d limit 1", + $r = q("SELECT length(data) AS fsize FROM attach WHERE hash = '%s' AND uid = %d LIMIT 1", dbesc($this->data['hash']), intval($this->data['uid']) ); - if($r) + if ($r) { $size = $r[0]['fsize']; + } } - } + // returns now() $edited = datetime_convert(); - $d = q("update attach set filesize = '%s', edited = '%s' where hash = '%s' and uid = %d limit 1", + $d = q("UPDATE attach SET filesize = '%s', edited = '%s' WHERE hash = '%s' AND uid = %d LIMIT 1", dbesc($size), dbesc($edited), dbesc($this->data['hash']), intval($c[0]['channel_id']) ); - $e = q("update attach set edited = '%s' where folder = '%s' and uid = %d limit 1", + // update the folder's lastmodified timestamp + $e = q("UPDATE attach SET edited = '%s' WHERE folder = '%s' AND uid = %d LIMIT 1", dbesc($edited), dbesc($r[0]['folder']), intval($c[0]['channel_id']) - ); + ); - $maxfilesize = get_config('system','maxfilesize'); + // @todo do we really want to remove the whole file if an update fails because of maxfilesize or quota? - if(($maxfilesize) && ($size > $maxfilesize)) { - attach_delete($c[0]['channel_id'],$this->data['hash']); + $maxfilesize = get_config('system', 'maxfilesize'); + if (($maxfilesize) && ($size > $maxfilesize)) { + attach_delete($c[0]['channel_id'], $this->data['hash']); return; } - $limit = service_class_fetch($c[0]['channel_id'],'attach_upload_limit'); - if($limit !== false) { + $limit = service_class_fetch($c[0]['channel_id'], 'attach_upload_limit'); + if ($limit !== false) { $x = q("select sum(filesize) as total from attach where aid = %d ", intval($c[0]['channel_account_id']) ); - if(($x) && ($x[0]['total'] + $size > $limit)) { - logger('reddav: service class limit exceeded for ' . $c[0]['channel_name'] . ' total usage is ' . $x[0]['total'] . ' limit is ' . $limit); - attach_delete($c[0]['channel_id'],$this->data['hash']); + if (($x) && ($x[0]['total'] + $size > $limit)) { + logger('RedFile::put(): service class limit exceeded for ' . $c[0]['channel_name'] . ' total usage is ' . $x[0]['total'] . ' limit is ' . $limit); + attach_delete($c[0]['channel_id'], $this->data['hash']); return; } } } - - function get() { - logger('RedFile::get: ' . basename($this->name), LOGGER_DEBUG); + /** + * @brief Returns the raw data. + * + * @return string + */ + public function get() { + logger('RedFile::get(): ' . basename($this->name), LOGGER_DEBUG); $r = q("select data, flags, filename, filetype from attach where hash = '%s' and uid = %d limit 1", dbesc($this->data['hash']), intval($this->data['uid']) ); - if($r) { - $unsafe_types = array('text/html','text/css','application/javascript'); + if ($r) { + // @todo this should be a global definition + $unsafe_types = array('text/html', 'text/css', 'application/javascript'); - if(in_array($r[0]['filetype'],$unsafe_types)) { + if (in_array($r[0]['filetype'], $unsafe_types)) { header('Content-disposition: attachment; filename="' . $r[0]['filename'] . '"'); header('Content-type: text/plain'); } - if($r[0]['flags'] & ATTACH_FLAG_OS ) { + if ($r[0]['flags'] & ATTACH_FLAG_OS ) { $f = 'store/' . $this->auth->owner_nick . '/' . (($this->os_path) ? $this->os_path . '/' : '') . $r[0]['data']; - return fopen($f,'rb'); + return fopen($f, 'rb'); } return $r[0]['data']; } - } - function getETag() { - return $this->data['hash']; + /** + * @brief Returns the ETag for a file. + * + * An ETag is a unique identifier representing the current version of the file. If the file changes, the ETag MUST change. + * The ETag is an arbitrary string, but MUST be surrounded by double-quotes. + * + * Return null if the ETag can not effectively be determined. + * + * @return mixed + */ + public function getETag() { + $ret = null; + if ($this->data['hash']) { + $ret = '"' . $this->data['hash'] . '"'; + } + return $ret; } - - function getContentType() { - $unsafe_types = array('text/html','text/css','application/javascript'); - if(in_array($this->data['filetype'],$unsafe_types)) { + /** + * @brief Returns the mime-type for a file. + * + * If null is returned, we'll assume application/octet-stream + * + * @return mixed + */ + public function getContentType() { + // @todo this should be a global definition. + $unsafe_types = array('text/html', 'text/css', 'application/javascript'); + if (in_array($this->data['filetype'], $unsafe_types)) { return 'text/plain'; } return $this->data['filetype']; } - - function getSize() { + /** + * @brief Returns the size of the node, in bytes. + * + * @return int + */ + public function getSize() { return $this->data['filesize']; } - - function getLastModified() { - return datetime_convert('UTC','UTC',$this->data['edited'],'U'); + /** + * @brief Returns the last modification time for the file, as a unix + * timestamp. + * + * @return int last modification time in UNIX timestamp + */ + public function getLastModified() { + return datetime_convert('UTC', 'UTC', $this->data['edited'], 'U'); } - - function delete() { - if((! $this->auth->owner_id) || (! perm_is_allowed($this->auth->owner_id,$this->auth->observer,'write_storage'))) { + /** + * @brief Delete the current file. + * + * @throw DAV\Exception\Forbidden + * @return void + */ + public function delete() { + if ((! $this->auth->owner_id) || (! perm_is_allowed($this->auth->owner_id, $this->auth->observer, 'write_storage'))) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; } - if($this->auth->owner_id !== $this->auth->channel_id) { - if(($this->auth->observer !== $this->data['creator']) || ($this->data['flags'] & ATTACH_FLAG_DIR)) { + if ($this->auth->owner_id !== $this->auth->channel_id) { + if (($this->auth->observer !== $this->data['creator']) || ($this->data['flags'] & ATTACH_FLAG_DIR)) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; } } - attach_delete($this->auth->owner_id,$this->data['hash']); + attach_delete($this->auth->owner_id, $this->data['hash']); } - -} - +} // class RedFile + + +/** + * @brief Returns an array with viewable channels. + * + * Get a list of RedDirectory objects with all the channels where the visitor + * has view_storage perms. + * + * @todo Is there any reason why this is not inside RedDirectory class? + * + * @param $auth + * @return array containing RedDirectory objects + */ function RedChannelList(&$auth) { - $ret = array(); - $r = q("select channel_id, channel_address from channel where not (channel_pageflags & %d) and not (channel_pageflags & %d) ", + $r = q("SELECT channel_id, channel_address FROM channel WHERE NOT (channel_pageflags & %d) AND NOT (channel_pageflags & %d)", intval(PAGE_REMOVED), intval(PAGE_HIDDEN) ); - if($r) { - foreach($r as $rr) { - if(perm_is_allowed($rr['channel_id'],$auth->observer,'view_storage')) { + if ($r) { + foreach ($r as $rr) { + if (perm_is_allowed($rr['channel_id'], $auth->observer, 'view_storage')) { logger('RedChannelList: ' . '/cloud/' . $rr['channel_address'], LOGGER_DATA); - $ret[] = new RedDirectory('/cloud/' . $rr['channel_address'],$auth); + // @todo can't we drop '/cloud'? It gets stripped off anyway in RedDirectory + $ret[] = new RedDirectory('/cloud/' . $rr['channel_address'], $auth); } } } return $ret; - } -function RedCollectionData($file,&$auth) { - +/** + * @brief TODO what exactly does this function? + * + * Array with all RedDirectory and RedFile DAV\Node items for the given path. + * + * @todo Is there any reason why this is not inside RedDirectory class? Seems only to be used there and we could simplify it a bit there. + * + * @param string $file path to a directory + * @param &$auth + * @returns array DAV\INode[] + */ +function RedCollectionData($file, &$auth) { $ret = array(); - $x = strpos($file,'/cloud'); - if($x === 0) { - $file = substr($file,6); + $x = strpos($file, '/cloud'); + if ($x === 0) { + $file = substr($file, 6); } - if((! $file) || ($file === '/')) { + // return a list of channel if we are not inside a channel + if ((! $file) || ($file === '/')) { return RedChannelList($auth); } - $file = trim($file,'/'); + $file = trim($file, '/'); $path_arr = explode('/', $file); - if(! $path_arr) + if (! $path_arr) return null; $channel_name = $path_arr[0]; - $r = q("select channel_id from channel where channel_address = '%s' limit 1", + $r = q("SELECT channel_id FROM channel WHERE channel_address = '%s' LIMIT 1", dbesc($channel_name) ); - if(! $r) + if (! $r) return null; $channel_id = $r[0]['channel_id']; @@ -579,15 +725,14 @@ function RedCollectionData($file,&$auth) { $errors = false; $permission_error = false; - for($x = 1; $x < count($path_arr); $x ++) { - + for ($x = 1; $x < count($path_arr); $x++) { $r = q("select id, hash, filename, flags from attach where folder = '%s' and filename = '%s' and uid = %d and (flags & %d) $perms limit 1", dbesc($folder), dbesc($path_arr[$x]), intval($channel_id), intval(ATTACH_FLAG_DIR) ); - if(! $r) { + if (! $r) { // path wasn't found. Try without permissions to see if it was the result of permissions. $errors = true; $r = q("select id, hash, filename, flags from attach where folder = '%s' and filename = '%s' and uid = %d and (flags & %d) limit 1", @@ -596,87 +741,85 @@ function RedCollectionData($file,&$auth) { intval($channel_id), intval(ATTACH_FLAG_DIR) ); - if($r) { + if ($r) { $permission_error = true; } break; } - if($r && ( $r[0]['flags'] & ATTACH_FLAG_DIR)) { + if ($r && ($r[0]['flags'] & ATTACH_FLAG_DIR)) { $folder = $r[0]['hash']; $path = $path . '/' . $r[0]['filename']; - } + } } - if($errors) { - if($permission_error) { + if ($errors) { + if ($permission_error) { throw new DAV\Exception\Forbidden('Permission denied.'); - return; - } - else { - throw new DAV\Exception\NotFound('A component of the request file path could not be found'); - return; + } else { + throw new DAV\Exception\NotFound('A component of the request file path could not be found.'); } } // This should no longer be needed since we just returned errors for paths not found - - if($path !== '/' . $file) { + if ($path !== '/' . $file) { logger("RedCollectionData: Path mismatch: $path !== /$file"); return NULL; } - $ret = array(); - $r = q("select id, uid, hash, filename, filetype, filesize, revision, folder, flags, created, edited from attach where folder = '%s' and uid = %d $perms group by filename", dbesc($folder), intval($channel_id) ); - foreach($r as $rr) { + foreach ($r as $rr) { logger('RedCollectionData: filename: ' . $rr['filename'], LOGGER_DATA); - if($rr['flags'] & ATTACH_FLAG_DIR) - $ret[] = new RedDirectory('/cloud' . $path . '/' . $rr['filename'],$auth); - else - $ret[] = new RedFile('/cloud' . $path . '/' . $rr['filename'],$rr,$auth); + if ($rr['flags'] & ATTACH_FLAG_DIR) { + // @todo can't we drop '/cloud'? it gets stripped off anyway in RedDirectory + $ret[] = new RedDirectory('/cloud' . $path . '/' . $rr['filename'], $auth); + } else { + $ret[] = new RedFile('/cloud' . $path . '/' . $rr['filename'], $rr, $auth); + } } return $ret; - } -function RedFileData($file, &$auth,$test = false) { +/** + * @brief TODO What exactly is this function for? + * + * @param string $file + * @param &$auth + * @param boolean $test (optional) enable test mode + */ +function RedFileData($file, &$auth, $test = false) { logger('RedFileData:' . $file . (($test) ? ' (test mode) ' : ''), LOGGER_DEBUG); - - $x = strpos($file,'/cloud'); - if($x === 0) { - $file = substr($file,6); + $x = strpos($file, '/cloud'); + if ($x === 0) { + $file = substr($file, 6); } - if((! $file) || ($file === '/')) { - return new RedDirectory('/',$auth); - + if ((! $file) || ($file === '/')) { + return new RedDirectory('/', $auth); } - $file = trim($file,'/'); + $file = trim($file, '/'); $path_arr = explode('/', $file); - if(! $path_arr) + if (! $path_arr) return null; - $channel_name = $path_arr[0]; - $r = q("select channel_id from channel where channel_address = '%s' limit 1", dbesc($channel_name) ); - if(! $r) + if (! $r) return null; $channel_id = $r[0]['channel_id']; @@ -694,7 +837,7 @@ function RedFileData($file, &$auth,$test = false) { $errors = false; - for($x = 1; $x < count($path_arr); $x ++) { + for ($x = 1; $x < count($path_arr); $x++) { $r = q("select id, hash, filename, flags from attach where folder = '%s' and filename = '%s' and uid = %d and (flags & %d) $perms", dbesc($folder), dbesc($path_arr[$x]), @@ -702,21 +845,19 @@ function RedFileData($file, &$auth,$test = false) { intval(ATTACH_FLAG_DIR) ); - if($r && ( $r[0]['flags'] & ATTACH_FLAG_DIR)) { + if ($r && ( $r[0]['flags'] & ATTACH_FLAG_DIR)) { $folder = $r[0]['hash']; $path = $path . '/' . $r[0]['filename']; } - if(! $r) { + if (! $r) { $r = q("select id, uid, hash, filename, filetype, filesize, revision, folder, flags, created, edited from attach where folder = '%s' and filename = '%s' and uid = %d $perms group by filename limit 1", dbesc($folder), dbesc(basename($file)), intval($channel_id) - ); } - if(! $r) { - + if (! $r) { $errors = true; $r = q("select id, uid, hash, filename, filetype, filesize, revision, folder, flags, created, edited from attach where folder = '%s' and filename = '%s' and uid = %d group by filename limit 1", @@ -724,71 +865,88 @@ function RedFileData($file, &$auth,$test = false) { dbesc(basename($file)), intval($channel_id) ); - if($r) + if ($r) $permission_error = true; - } - } - if($path === '/' . $file) { - if($test) + if ($path === '/' . $file) { + if ($test) return true; // final component was a directory. - return new RedDirectory('/cloud/' . $file,$auth); + return new RedDirectory('/cloud/' . $file, $auth); } - if($errors) { + if ($errors) { logger('RedFileData: not found'); - if($test) + if ($test) return false; - if($permission_error) { + if ($permission_error) { logger('RedFileData: permission error'); throw new DAV\Exception\Forbidden('Permission denied.'); } return; } - if($r) { - if($test) + if ($r) { + if ($test) return true; - if($r[0]['flags'] & ATTACH_FLAG_DIR) - return new RedDirectory('/cloud' . $path . '/' . $r[0]['filename'],$auth); - else - return new RedFile('/cloud' . $path . '/' . $r[0]['filename'],$r[0],$auth); + if ($r[0]['flags'] & ATTACH_FLAG_DIR) { + // @todo can't we drop '/cloud'? it gets stripped off anyway in RedDirectory + return new RedDirectory('/cloud' . $path . '/' . $r[0]['filename'], $auth); + } else { + return new RedFile('/cloud' . $path . '/' . $r[0]['filename'], $r[0], $auth); + } } return false; } -class RedBasicAuth extends Sabre\DAV\Auth\Backend\AbstractBasic { +/** + * RedBasicAuth class. + * + */ +class RedBasicAuth extends DAV\Auth\Backend\AbstractBasic { + + // @fixme mod/cloud.php:61 public $channel_name = ''; + // @fixme mod/cloud.php:62 public $channel_id = 0; + // @fixme mod/cloud.php:63 public $channel_hash = ''; + // @fixme mod/cloud.php:68 public $observer = ''; + // @fixme include/reddav.php:51 public $browser; + // @fixme include/reddav.php:92 public $owner_id; + // @fixme include/reddav.php:283 public $owner_nick = ''; + // @fixme mod/cloud.php:66 public $timezone; - protected function validateUserPass($username, $password) { - + /** + * + * @param string $username + * @param string $password + */ + protected function validateUserPass($username, $password) { - if(trim($password) === '+++') { + if (trim($password) === '+++') { logger('reddav: validateUserPass: guest ' . $username); return true; } require_once('include/auth.php'); - $record = account_verify_password($username,$password); - if($record && $record['account_default_channel']) { + $record = account_verify_password($username, $password); + if ($record && $record['account_default_channel']) { $r = q("select * from channel where channel_account_id = %d and channel_id = %d limit 1", intval($record['account_id']), intval($record['account_default_channel']) ); - if($r) { + if ($r) { $this->currentUser = $r[0]['channel_address']; $this->channel_name = $r[0]['channel_address']; $this->channel_id = $r[0]['channel_id']; @@ -802,15 +960,15 @@ class RedBasicAuth extends Sabre\DAV\Auth\Backend\AbstractBasic { $r = q("select * from channel where channel_address = '%s' limit 1", dbesc($username) ); - if($r) { + if ($r) { $x = q("select * from account where account_id = %d limit 1", intval($r[0]['channel_account_id']) ); - if($x) { - foreach($x as $record) { - if(($record['account_flags'] == ACCOUNT_OK) || ($record['account_flags'] == ACCOUNT_UNVERIFIED) - && (hash('whirlpool',$record['account_salt'] . $password) === $record['account_password'])) { - logger('(DAV) RedBasicAuth: password verified for ' . $username); + if ($x) { + foreach ($x as $record) { + if (($record['account_flags'] == ACCOUNT_OK) || ($record['account_flags'] == ACCOUNT_UNVERIFIED) + && (hash('whirlpool', $record['account_salt'] . $password) === $record['account_password'])) { + logger('(DAV) RedBasicAuth: password verified for ' . $username); $this->currentUser = $r[0]['channel_address']; $this->channel_name = $r[0]['channel_address']; $this->channel_id = $r[0]['channel_id']; @@ -818,24 +976,24 @@ class RedBasicAuth extends Sabre\DAV\Auth\Backend\AbstractBasic { $_SESSION['uid'] = $r[0]['channel_id']; $_SESSION['account_id'] = $r[0]['channel_account_id']; $_SESSION['authenticated'] = true; - return true; - } - } + return true; + } + } } } - logger('(DAV) RedBasicAuth: password failed for ' . $username); - return false; + logger('(DAV) RedBasicAuth: password failed for ' . $username); + return false; } - function setCurrentUser($name) { + public function setCurrentUser($name) { $this->currentUser = $name; } function setBrowserPlugin($browser) { $this->browser = $browser; } - + // internal? loggin function function log() { logger('dav: auth: channel_name ' . $this->channel_name, LOGGER_DATA); logger('dav: auth: channel_id ' . $this->channel_id, LOGGER_DATA); @@ -845,64 +1003,66 @@ class RedBasicAuth extends Sabre\DAV\Auth\Backend\AbstractBasic { logger('dav: auth: owner_nick ' . $this->owner_nick, LOGGER_DATA); } +} // class RedBasicAuth -} +/** + * RedBrowser class. + * + */ class RedBrowser extends DAV\Browser\Plugin { private $auth; function __construct(&$auth) { - $this->auth = $auth; $this->enableAssets = false; - } // The DAV browser is instantiated after the auth module and directory classes but before we know the current // directory and who the owner and observer are. So we add a pointer to the browser into the auth module and vice // versa. Then when we've figured out what directory is actually being accessed, we call the following function // to decide whether or not to show web elements which include writeable objects. - - function set_writeable() { - - if(! $this->auth->owner_id) + if (! $this->auth->owner_id) { $this->enablePost = false; + } - if(! perm_is_allowed($this->auth->owner_id, get_observer_hash(), 'write_storage')) + if (! perm_is_allowed($this->auth->owner_id, get_observer_hash(), 'write_storage')) { $this->enablePost = false; - else + } else { $this->enablePost = true; - + } } /** * Creates the directory listing for the given path. * - * @param String $path + * @todo Why not use RedDirectory for this? + * + * @param string $path which should be displayed */ public function generateDirectoryIndex($path) { - + // (owner_id = channel_id) is visitor owner of this directory? $is_owner = ((local_user() && $this->auth->owner_id == local_user()) ? true : false); if ($this->auth->timezone) date_default_timezone_set($this->auth->timezone); - $version = ''; require_once('include/conversation.php'); - if ($this->auth->owner_nick) - $html = profile_tabs(get_app(),(($is_owner) ? true : false),$this->auth->owner_nick); + if ($this->auth->owner_nick) { + $html = profile_tabs(get_app(), (($is_owner) ? true : false), $this->auth->owner_nick); + } - $files = $this->server->getPropertiesForPath($path,array( + $files = $this->server->getPropertiesForPath($path, array( '{DAV:}displayname', '{DAV:}resourcetype', '{DAV:}getcontenttype', '{DAV:}getcontentlength', '{DAV:}getlastmodified', - ),1); + ), 1); $parent = $this->server->tree->getNodeForPath($path); @@ -912,21 +1072,20 @@ class RedBrowser extends DAV\Browser\Plugin { list($parentUri) = DAV\URLUtil::splitPath($path); $fullPath = DAV\URLUtil::encodePath($this->server->getBaseUri() . $parentUri); - $parentpath['icon'] = $this->enableAssets?''.t('parent').'':''; + $parentpath['icon'] = $this->enableAssets ? '' . t('parent') . '' : ''; $parentpath['path'] = $fullPath; - } + } $f = array(); foreach ($files as $file) { $ft = array(); + $type = null; // This is the current directory, we can skip it if (rtrim($file['href'],'/')==$path) continue; list(, $name) = DAV\URLUtil::splitPath($file['href']); - $type = null; - if (isset($file[200]['{DAV:}resourcetype'])) { $type = $file[200]['{DAV:}resourcetype']->getValue(); @@ -935,31 +1094,31 @@ class RedBrowser extends DAV\Browser\Plugin { foreach ($type as $k=>$v) { // Some name mapping is preferred - switch($v) { - case '{DAV:}collection' : - $type[$k] = t('Collection'); - break; - case '{DAV:}principal' : - $type[$k] = t('Principal'); - break; - case '{urn:ietf:params:xml:ns:carddav}addressbook' : - $type[$k] = t('Addressbook'); - break; - case '{urn:ietf:params:xml:ns:caldav}calendar' : - $type[$k] = t('Calendar'); - break; - case '{urn:ietf:params:xml:ns:caldav}schedule-inbox' : - $type[$k] = t('Schedule Inbox'); - break; - case '{urn:ietf:params:xml:ns:caldav}schedule-outbox' : - $type[$k] = t('Schedule Outbox'); - break; - case '{http://calendarserver.org/ns/}calendar-proxy-read' : - $type[$k] = 'Proxy-Read'; - break; - case '{http://calendarserver.org/ns/}calendar-proxy-write' : - $type[$k] = 'Proxy-Write'; - break; + switch ($v) { + case '{DAV:}collection' : + $type[$k] = t('Collection'); + break; + case '{DAV:}principal' : + $type[$k] = t('Principal'); + break; + case '{urn:ietf:params:xml:ns:carddav}addressbook' : + $type[$k] = t('Addressbook'); + break; + case '{urn:ietf:params:xml:ns:caldav}calendar' : + $type[$k] = t('Calendar'); + break; + case '{urn:ietf:params:xml:ns:caldav}schedule-inbox' : + $type[$k] = t('Schedule Inbox'); + break; + case '{urn:ietf:params:xml:ns:caldav}schedule-outbox' : + $type[$k] = t('Schedule Outbox'); + break; + case '{http://calendarserver.org/ns/}calendar-proxy-read' : + $type[$k] = 'Proxy-Read'; + break; + case '{http://calendarserver.org/ns/}calendar-proxy-write' : + $type[$k] = 'Proxy-Write'; + break; } } $type = implode(', ', $type); @@ -970,21 +1129,21 @@ class RedBrowser extends DAV\Browser\Plugin { if (!$type && isset($file[200]['{DAV:}getcontenttype'])) { $type = $file[200]['{DAV:}getcontenttype']; } - if (!$type) $type = 'Unknown'; + if (!$type) $type = t('Unknown'); - $size = isset($file[200]['{DAV:}getcontentlength'])?(int)$file[200]['{DAV:}getcontentlength']:''; - $lastmodified = ((isset($file[200]['{DAV:}getlastmodified']))? $file[200]['{DAV:}getlastmodified']->getTime()->format('Y-m-d H:i:s') :''); + $size = isset($file[200]['{DAV:}getcontentlength']) ? (int)$file[200]['{DAV:}getcontentlength'] : ''; + $lastmodified = ((isset($file[200]['{DAV:}getlastmodified'])) ? $file[200]['{DAV:}getlastmodified']->getTime()->format('Y-m-d H:i:s') : ''); - $fullPath = DAV\URLUtil::encodePath('/' . trim($this->server->getBaseUri() . ($path?$path . '/':'') . $name,'/')); + $fullPath = DAV\URLUtil::encodePath('/' . trim($this->server->getBaseUri() . ($path ? $path . '/' : '') . $name, '/')); - $displayName = isset($file[200]['{DAV:}displayname'])?$file[200]['{DAV:}displayname']:$name; + $displayName = isset($file[200]['{DAV:}displayname']) ? $file[200]['{DAV:}displayname'] : $name; $displayName = $this->escapeHTML($displayName); $type = $this->escapeHTML($type); $icon = ''; if ($this->enableAssets) { - $node = $this->server->tree->getNodeForPath(($path?$path.'/':'') . $name); + $node = $this->server->tree->getNodeForPath(($path ? $path . '/' : '') . $name); foreach (array_reverse($this->iconMap) as $class=>$iconName) { if ($node instanceof $class) { $icon = ''; @@ -993,13 +1152,13 @@ class RedBrowser extends DAV\Browser\Plugin { } } - $parentHash=""; - $owner=$this->auth->owner_id; - $splitPath = split("/",$fullPath); + $parentHash = ""; + $owner = $this->auth->owner_id; + $splitPath = split("/", $fullPath); if (count($splitPath) > 3) { - for ($i=3; $ifindAttachHash($owner,$parentHash,$attachName); + $attachHash = $this->findAttachHash($owner, $parentHash, $attachName); $parentHash = $attachHash; } } @@ -1008,10 +1167,10 @@ class RedBrowser extends DAV\Browser\Plugin { // put the array for this file together $ft['attachId'] = $this->findAttachIdByHash($attachHash); - $ft['fileStorageUrl'] = substr($fullPath, 0, strpos($fullPath,"cloud/")) ."filestorage/".$this->auth->channel_name; + $ft['fileStorageUrl'] = substr($fullPath, 0, strpos($fullPath, "cloud/")) . "filestorage/" . $this->auth->channel_name; $ft['icon'] = $icon; $ft['attachIcon'] = (($size) ? $attachIcon : ''); - // is global or per item? + // @todo Should this be an item value, not a global one? $ft['is_owner'] = $is_owner; $ft['fullPath'] = $fullPath; $ft['displayName'] = $displayName; @@ -1023,9 +1182,9 @@ class RedBrowser extends DAV\Browser\Plugin { $f[] = $ft; } - // Storage and Quota + // Storage and quota for the account (all channels of the owner of this directory)! $limit = service_class_fetch($owner, 'attach_upload_limit'); - $r = q("select sum(filesize) as total from attach where aid = %d ", + $r = q("SELECT SUM(filesize) AS total FROM attach WHERE aid = %d", intval($this->auth->channel_account_id) ); $used = $r[0]['total']; @@ -1042,45 +1201,50 @@ class RedBrowser extends DAV\Browser\Plugin { round($used / $limit, 1)); } - // quota for template + // prepare quota for template $quota['used'] = $used; $quota['limit'] = $limit; $quota['desc'] = $quotaDesc; $html .= replace_macros(get_markup_template('cloud_directory.tpl'), array( - '$header' => t('Files').": ".$this->escapeHTML($path) . "/", - '$parentpath' => $parentpath, - '$entries' => $f, - '$quota' => $quota - )); + '$header' => t('Files') . ": " . $this->escapeHTML($path) . "/", + '$parentpath' => $parentpath, + '$entries' => $f, + '$quota' => $quota + )); $output = ''; if ($this->enablePost) { - $this->server->broadcastEvent('onHTMLActionsPanel',array($parent, &$output)); + $this->server->broadcastEvent('onHTMLActionsPanel', array($parent, &$output)); } - $html.=$output; + $html .= $output; get_app()->page['content'] = $html; construct_page(get_app()); } - function userReadableSize($size){ + function userReadableSize($size) { + $ret = ""; if (is_numeric($size)) { $incr = 0; $k = 1024; - $unit = array('bytes','KB','MB','GB','TB','PB'); - while(($size / $k) >= 1){ + $unit = array('bytes', 'KB', 'MB', 'GB', 'TB', 'PB'); + while (($size / $k) >= 1){ $incr++; $size = round($size / $k, 2); } - return $size." ".$unit[$incr]; - } else { - return ""; + $ret = $size . " " . $unit[$incr]; } + return $ret; } - public function htmlActionsPanel(DAV\INode $node, &$output) { - + /** + * Creates a form to add new folders and upload files. + * + * @param DAV\INode $node + * @param string &$output + */ + public function htmlActionsPanel(DAV\INode $node, &$output) { //Removed link to filestorage page //if($this->auth->owner_id && $this->auth->owner_id == $this->auth->channel_id) { @@ -1090,70 +1254,59 @@ class RedBrowser extends DAV\Browser\Plugin { // } //} - if (!$node instanceof DAV\ICollection) - return; - - // We also know fairly certain that if an object is a non-extended - // SimpleCollection, we won't need to show the panel either. - - if (get_class($node)==='Sabre\\DAV\\SimpleCollection') - return; - - $output.= ' - - - - - - - -
'.t('Create new folder').'   
- - - -
'.t('Upload file').'   
- - - - -
'; - - } - - /** - * This method takes a path/name of an asset and turns it into url - * suiteable for http access. - * - * @param string $assetName - * @return string - */ - protected function getAssetUrl($assetName) { - return z_root() .'/cloud/?sabreAction=asset&assetName=' . urlencode($assetName); - } - - protected function findAttachHash($owner, $parentHash, $attachName) { - $r = q("select * from attach where uid = %d and folder = '%s' and filename = '%s' order by edited desc limit 1", - intval($owner), dbesc($parentHash), dbesc($attachName) - ); - $hash = ""; - if($r) { - foreach($r as $rr) { - $hash = $rr['hash']; + if (! $node instanceof DAV\ICollection) + return; + + // We also know fairly certain that if an object is a non-extended + // SimpleCollection, we won't need to show the panel either. + if (get_class($node) === 'Sabre\\DAV\\SimpleCollection') + return; + + $output .= replace_macros(get_markup_template('cloud_actionspanel.tpl'), array( + '$folder_header' => t('Create new folder'), + '$folder_submit' => t('Create'), + '$upload_header' => t('Upload file'), + '$upload_submit' => t('Upload') + )); + } + + /** + * This method takes a path/name of an asset and turns it into url + * suiteable for http access. + * + * @param string $assetName + * @return string + */ + protected function getAssetUrl($assetName) { + return z_root() . '/cloud/?sabreAction=asset&assetName=' . urlencode($assetName); + } + + protected function findAttachHash($owner, $parentHash, $attachName) { + $r = q("SELECT * FROM attach WHERE uid = %d AND folder = '%s' AND filename = '%s' ORDER BY edited desc LIMIT 1", + intval($owner), + dbesc($parentHash), + dbesc($attachName) + ); + $hash = ""; + if ($r) { + foreach ($r as $rr) { + $hash = $rr['hash']; + } } + return $hash; } - return $hash; - } - - protected function findAttachIdByHash($attachHash) { - $r = q("select * from attach where hash = '%s' order by edited desc limit 1", - dbesc($attachHash) - ); - $id = ""; - if($r) { - foreach($r as $rr) { - $id = $rr['id']; + + protected function findAttachIdByHash($attachHash) { + $r = q("SELECT * FROM attach WHERE hash = '%s' ORDER BY edited DESC LIMIT 1", + dbesc($attachHash) + ); + $id = ""; + if ($r) { + foreach ($r as $rr) { + $id = $rr['id']; + } } + return $id; } - return $id; - } -} + +} // class RedBrowser diff --git a/view/tpl/cloud_actionspanel.tpl b/view/tpl/cloud_actionspanel.tpl new file mode 100644 index 000000000..69931b292 --- /dev/null +++ b/view/tpl/cloud_actionspanel.tpl @@ -0,0 +1,23 @@ + + + + + + + + + +
{{$folder_header}}    +
+ + + +
+
{{$upload_header}}    +
+ + + + +
+
\ No newline at end of file -- cgit v1.2.3