openrat-cms

OpenRat Content Management System
git clone http://git.code.weiherhei.de/openrat-cms.git
Log | Files | Refs | README

commit 9b3b3e4906db17e84abec581d7cccac50fdb2f2f
parent 5786d589b8030126639c4906591080f1ad01cb32
Author: Jan Dankert <develop@jandankert.de>
Date:   Sat,  6 Mar 2021 02:50:20 +0100

New: Enable actions for guest users.

Diffstat:
Mmodules/cms/action/AliasAction.class.php | 2--
Mmodules/cms/action/FileAction.class.php | 2--
Mmodules/cms/action/FolderAction.class.php | 2--
Mmodules/cms/action/ImageAction.class.php | 2--
Mmodules/cms/action/LinkAction.class.php | 2--
Mmodules/cms/action/ObjectAction.class.php | 6++++--
Mmodules/cms/action/PageAction.class.php | 2--
Mmodules/cms/action/TextAction.class.php | 2--
Mmodules/cms/action/UrlAction.class.php | 2--
Mmodules/cms/model/BaseObject.class.php | 7++-----
10 files changed, 6 insertions(+), 23 deletions(-)

diff --git a/modules/cms/action/AliasAction.class.php b/modules/cms/action/AliasAction.class.php @@ -37,8 +37,6 @@ use cms\model\Alias; */ class AliasAction extends ObjectAction { - public $security = Action::SECURITY_USER; - /** * @var Alias */ diff --git a/modules/cms/action/FileAction.class.php b/modules/cms/action/FileAction.class.php @@ -40,8 +40,6 @@ use util\Upload; */ class FileAction extends ObjectAction { - public $security = Action::SECURITY_USER; - /** * @var File */ diff --git a/modules/cms/action/FolderAction.class.php b/modules/cms/action/FolderAction.class.php @@ -37,8 +37,6 @@ use util\Upload; class FolderAction extends ObjectAction { - public $security = Action::SECURITY_USER; - /** * @var Folder */ diff --git a/modules/cms/action/ImageAction.class.php b/modules/cms/action/ImageAction.class.php @@ -14,8 +14,6 @@ use util\Html; */ class ImageAction extends FileAction { - public $security = Action::SECURITY_USER; - protected $image; /** diff --git a/modules/cms/action/LinkAction.class.php b/modules/cms/action/LinkAction.class.php @@ -33,8 +33,6 @@ use util\Html; */ class LinkAction extends ObjectAction { - public $security = Action::SECURITY_USER; - /** * @var Link */ diff --git a/modules/cms/action/ObjectAction.class.php b/modules/cms/action/ObjectAction.class.php @@ -28,7 +28,7 @@ use util\Session; class ObjectAction extends BaseAction { - public $security = Action::SECURITY_USER; + public $security = Action::SECURITY_GUEST; private $objectid; @@ -50,7 +50,9 @@ class ObjectAction extends BaseAction $baseObject->objectLoad(); $this->setBaseObject( $baseObject ); - } + + $this->checkRight( Permission::ACL_READ ); + } protected function setBaseObject( $baseObject ) { diff --git a/modules/cms/action/PageAction.class.php b/modules/cms/action/PageAction.class.php @@ -32,8 +32,6 @@ use util\Session; class PageAction extends ObjectAction { - public $security = Action::SECURITY_USER; - /** * @var Page */ diff --git a/modules/cms/action/TextAction.class.php b/modules/cms/action/TextAction.class.php @@ -30,8 +30,6 @@ use language\Messages; */ class TextAction extends FileAction { - public $security = Action::SECURITY_USER; - /** * @var Text */ diff --git a/modules/cms/action/UrlAction.class.php b/modules/cms/action/UrlAction.class.php @@ -32,8 +32,6 @@ use language\Messages; */ class UrlAction extends ObjectAction { - public $security = Action::SECURITY_USER; - /** * @var Url */ diff --git a/modules/cms/model/BaseObject.class.php b/modules/cms/model/BaseObject.class.php @@ -838,8 +838,6 @@ SQL $stmt = Db::sql( <<<SQL UPDATE {{object}} SET parentid = {parentid}, - lastchange_date = {time} , - lastchange_userid = {userid} , filename = {filename}, valid_from = {validFrom}, valid_to = {validTo}, @@ -857,8 +855,6 @@ SQL $user = \util\Session::getUser(); $this->lastchangeUser = $user; $this->lastchangeDate = Startup::now(); - $stmt->setInt ('time' , $this->lastchangeDate ); - $stmt->setInt ('userid' , $this->lastchangeUser->userid ); $stmt->setString('filename' , $this->filename ); $stmt->setString('settings' , $this->settings ); $stmt->setInt ('validFrom', $this->validFromDate ); @@ -888,8 +884,9 @@ SQL $user = \util\Session::getUser(); $this->lastchangeUser = $user; $this->lastchangeDate = Startup::now(); + $userid = $this->lastchangeUser ? $this->lastchangeUser->userid : null; - $sql->setInt ('userid' ,$this->lastchangeUser->userid ); + $sql->setIntOrNull('userid' ,$userid ); $sql->setInt ('objectid',$this->objectid ); $sql->setInt ('time' ,$this->lastchangeDate );