diff --git a/src/QUI/Bricks/Controls/Banner.php b/src/QUI/Bricks/Controls/Banner.php index 67fa4e692a8c586e19a01ec65491fea7106edc34..7339c0e2b6723086ced394782d3e251e8aaf494c 100644 --- a/src/QUI/Bricks/Controls/Banner.php +++ b/src/QUI/Bricks/Controls/Banner.php @@ -34,7 +34,7 @@ public function __construct($attributes = array()) dirname(__FILE__).'/Banner.css' ); - parent::setAttributes($attributes); + parent::__construct($attributes); } /** @@ -52,4 +52,4 @@ public function getBody() return $Engine->fetch(dirname(__FILE__).'/Banner.html'); } -} \ No newline at end of file +} diff --git a/src/QUI/Bricks/Controls/SideBox1.php b/src/QUI/Bricks/Controls/SideBox1.php index 2f40224ca530acd2f142ab5776ea1251e8e524a8..450c7b7f2ad6c8bd5bf3bf343a95d94fb39142a5 100644 --- a/src/QUI/Bricks/Controls/SideBox1.php +++ b/src/QUI/Bricks/Controls/SideBox1.php @@ -61,7 +61,7 @@ public function getBody() */ protected function getSite() { - $Project = $this->_getProject(); + $Project = $this->getProject(); $site = $this->getAttribute('site'); if (is_numeric($site)) { diff --git a/src/QUI/Bricks/Events.php b/src/QUI/Bricks/Events.php index 1cd3eb7b3b3d20875bb22444071b166e1aaf77af..fdc7067f76a25099fb2ae9e76f337f92815460c3 100644 --- a/src/QUI/Bricks/Events.php +++ b/src/QUI/Bricks/Events.php @@ -23,15 +23,14 @@ class Events * * @param Site|Edit $Site */ - static function onSiteSave($Site) + public static function onSiteSave($Site) { QUI\Rights\Permission::checkPermission('quiqqer.bricks.assign'); + $areas = $Site->getAttribute('quiqqer.bricks.areas'); + $areas = json_decode($areas, true); - $areas = $Site->getAttribute( 'quiqqer.bricks.areas' ); - $areas = json_decode( $areas, true ); - - if ( !$areas || empty( $areas ) ) { + if (!$areas ||empty($areas)) { return; } @@ -39,25 +38,24 @@ static function onSiteSave($Site) // get inharitance areas $Project = $Site->getProject(); - $projectAreas = $Manager->getAreasByProject( $Project ); - $projectTable = QUI::getDBProjectTableName( Manager::TABLE_CACHE, $Project ); + $projectAreas = $Manager->getAreasByProject($Project); + $projectTable = QUI::getDBProjectTableName(Manager::TABLE_CACHE, $Project); - foreach ( $projectAreas as $area ) - { - if ( !$area[ 'inheritance' ] ) { + foreach ($projectAreas as $area) { + if (!$area['inheritance']) { continue; } - if ( !isset( $areas[ $area[ 'name' ] ] ) ) { + if (!isset($areas[$area['name']])) { continue; } - if ( empty( $areas[ $area[ 'name' ] ] ) ) { + if (empty($areas[$area['name']])) { continue; } - $bricks = $areas[ $area[ 'name' ] ]; + $bricks = $areas[$area['name']]; // clear area and new data set QUI::getDataBase()->delete($projectTable, array( @@ -66,8 +64,7 @@ static function onSiteSave($Site) )); // check if deactivated - if ( isset( $bricks[ 0 ] ) && isset( $bricks[ 0 ][ 'deactivate' ] ) ) - { + if (isset($bricks[0]) && isset($bricks[0]['deactivate'])) { QUI::getDataBase()->insert($projectTable, array( 'id' => $Site->getId(), 'area' => $area[ 'name' ], @@ -78,19 +75,21 @@ static function onSiteSave($Site) } - foreach ( $bricks as $brick ) - { + foreach ($bricks as $brick) { $customFields = array(); - if (isset($brick['customfields']) && is_string($brick['customfields'])) { + if (isset($brick['customfields']) + && is_string($brick['customfields'])) { $customFields = json_decode($brick['customfields'], true); } - if (isset($brick['customfields']) && is_array($brick['customfields'])) { + if (isset($brick['customfields']) + && is_array($brick['customfields'])) { $customFields = $brick['customfields']; } - if (!isset($customFields['inheritance']) || !(int)$customFields['inheritance']) { + if (!isset($customFields['inheritance']) + || !(int)$customFields['inheritance']) { continue; } @@ -107,13 +106,12 @@ static function onSiteSave($Site) * Event : on smarty init * add new brickarea function */ - static function onSmartyInit($Smarty) + public static function onSmartyInit($Smarty) { // {brickarea} - if ( !isset( $Smarty->registered_plugins['function'] ) || - !isset( $Smarty->registered_plugins['function']['brickarea'] ) - ) - { + if (!isset($Smarty->registered_plugins['function']) + || !isset($Smarty->registered_plugins['function']['brickarea']) + ) { $Smarty->registerPlugin("function", "brickarea", "\QUI\Bricks\Events::brickarea"); } } @@ -124,15 +122,14 @@ static function onSmartyInit($Smarty) * @param Array $params - function parameter * @param \Smarty */ - static function brickarea($params, $smarty) + public static function brickarea($params, $smarty) { - if ( !isset( $params['Site'] ) || !isset( $params['area'] ) ) - { - if ( !isset( $params['assign'] ) ) { + if (!isset($params['Site']) || !isset($params['area'])) { + if (!isset($params['assign'])) { return array(); } - $smarty->assign( $params['assign'], array() ); + $smarty->assign($params['assign'], array()); return; } @@ -142,12 +139,12 @@ static function brickarea($params, $smarty) $Site = $params['Site']; $area = $params['area']; - $result = $BricksManager->getBricksByArea( $area, $Site ); + $result = $BricksManager->getBricksByArea($area, $Site); - if ( !isset( $params['assign'] ) ) { + if (!isset($params['assign'])) { return $result; } - $smarty->assign( $params['assign'], $result ); + $smarty->assign($params['assign'], $result); } -} \ No newline at end of file +} diff --git a/src/QUI/Bricks/Manager.php b/src/QUI/Bricks/Manager.php index 4a3a7b781f3fa95f1eb8d68243b8b03fbe7dbe7b..3f08fcbabd9eefb2db1ca3655eb016f3492d529b 100644 --- a/src/QUI/Bricks/Manager.php +++ b/src/QUI/Bricks/Manager.php @@ -32,27 +32,27 @@ class Manager * * @var array */ - protected $_bricks = array(); + protected $bricks = array(); /** * Initialized brick manager * * @var null */ - static $_BrickManager = null; + public static $BrickManager = null; /** * Return the global QUI\Bricks\Manager * * @return Manager */ - static function init() + public static function init() { - if (is_null(self::$_BrickManager)) { - self::$_BrickManager = new QUI\Bricks\Manager(true); + if (is_null(self::$BrickManager)) { + self::$BrickManager = new QUI\Bricks\Manager(true); } - return self::$_BrickManager; + return self::$BrickManager; } /** @@ -254,8 +254,8 @@ public function getAvailableBricks() */ public function getBrickById($id) { - if (isset($this->_bricks[$id])) { - return $this->_bricks[$id]; + if (isset($this->bricks[$id])) { + return $this->bricks[$id]; } $data = QUI::getDataBase()->fetch(array( @@ -270,10 +270,10 @@ public function getBrickById($id) throw new QUI\Exception('Brick not found'); } - $this->_bricks[$id] = new Brick($data[0]); - $this->_bricks[$id]->setAttribute('id', $id); + $this->bricks[$id] = new Brick($data[0]); + $this->bricks[$id]->setAttribute('id', $id); - return $this->_bricks[$id]; + return $this->bricks[$id]; } /**