From dcc1e18ad6b6a761b5d265037c9d0ac7308de459 Mon Sep 17 00:00:00 2001
From: "ohga.ryohei" <ohga.ryohei@withone.co.jp>
Date: Thu, 30 Aug 2018 08:22:17 +0900
Subject: [PATCH] =?UTF-8?q?=E4=B8=8D=E7=94=A8=E6=84=8F=E3=81=ABHash?=
 =?UTF-8?q?=E3=82=92=E4=BD=BF=E7=94=A8=E3=81=97=E3=81=A6=E3=81=84=E3=82=8B?=
 =?UTF-8?q?=E7=AE=87=E6=89=80=E3=82=92=E4=BD=BF=E7=94=A8=E3=81=97=E3=81=AA?=
 =?UTF-8?q?=E3=81=84=E3=82=88=E3=81=86=E3=81=AB=E4=BF=AE=E6=AD=A3?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Controller/PagesController.php     |  4 +++-
 Controller/PagesEditController.php |  5 ++++-
 Model/Behavior/GetPageBehavior.php | 12 +++++++++---
 View/Pages/index.ctp               |  5 ++++-
 4 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/Controller/PagesController.php b/Controller/PagesController.php
index 77a12ef..266d8b6 100644
--- a/Controller/PagesController.php
+++ b/Controller/PagesController.php
@@ -76,7 +76,9 @@ public function index() {
 		$paths = $this->params->params['pass'];
 		$path = implode('/', $paths);
 
-		$spacePermalink = Hash::get($this->request->params, 'spacePermalink', '');
+		$spacePermalink = isset($this->request->params['spacePermalink'])
+			? $this->request->params['spacePermalink']
+			: '';
 		$space = $this->Space->find('first', array(
 			'recursive' => -1,
 			'conditions' => array('permalink' => $spacePermalink, 'id !=' => Space::WHOLE_SITE_ID)
diff --git a/Controller/PagesEditController.php b/Controller/PagesEditController.php
index 5f1412a..e6535a3 100644
--- a/Controller/PagesEditController.php
+++ b/Controller/PagesEditController.php
@@ -114,7 +114,10 @@ public function beforeFilter() {
 	public function index() {
 		$this->__setRedirectUrl();
 		$rooms = $this->Room->children(Current::read('Room.id'), false, 'Room.id', 'Room.rght');
-		$roomIds = Hash::merge(array(Current::read('Room.id')), Hash::extract($rooms, '{n}.Room.id'));
+		$roomIds = [Current::read('Room.id')];
+		foreach ($rooms as $room) {
+			$roomIds[] = $room['Room']['id'];
+		}
 		$this->__prepareIndex($roomIds, []);
 	}
 
diff --git a/Model/Behavior/GetPageBehavior.php b/Model/Behavior/GetPageBehavior.php
index 78020eb..1c6c884 100644
--- a/Model/Behavior/GetPageBehavior.php
+++ b/Model/Behavior/GetPageBehavior.php
@@ -152,6 +152,9 @@ public function getPageIdsWithM17n(Model $model, $pageIds) {
  * @param string $permalink Permalink
  * @param string $spaceId Space id
  * @return array
+ *
+ * 速度改善の修正に伴って発生したため抑制
+ * @SuppressWarnings(PHPMD.NPathComplexity)
  */
 	public function getPageWithFrame(Model $model, $permalink, $spaceId = null) {
 		$model->loadModels([
@@ -220,7 +223,7 @@ public function getPageWithFrame(Model $model, $permalink, $spaceId = null) {
 				'PagesLanguage.language_id' => Current::read('Language.id'),
 			),
 		));
-		$result = Hash::merge($page, $pagesLanguages);
+		$result = $page + $pagesLanguages;
 
 		$pageContainers = $model->PageContainer->find('all', array(
 			'recursive' => -1,
@@ -229,7 +232,10 @@ public function getPageWithFrame(Model $model, $permalink, $spaceId = null) {
 			),
 			'order' => array('container_type' => 'asc'),
 		));
-		$result['PageContainer'] = Hash::extract($pageContainers, '{n}.PageContainer');
+		$result['PageContainer'] = [];
+		foreach ($pageContainers as $pageContainer) {
+			$result['PageContainer'][] = $pageContainer['PageContainer'];
+		}
 		foreach ($result['PageContainer'] as $i => $pageContainer) {
 			$pageContainer['Box'] = $model->Box->getBoxWithFrame($pageContainer['id']);
 			$result['PageContainer'][$i] = $pageContainer;
@@ -260,7 +266,7 @@ public function getTopPageId(Model $model) {
 			)
 		));
 
-		return Hash::get($room, 'Room.page_id_top');
+		return $room['Room']['page_id_top'];
 	}
 
 /**
diff --git a/View/Pages/index.ctp b/View/Pages/index.ctp
index f34d209..4bf5afb 100644
--- a/View/Pages/index.ctp
+++ b/View/Pages/index.ctp
@@ -9,7 +9,10 @@
  * @copyright Copyright 2014, NetCommons Project
  */
 
-$pageContainer = Hash::combine($page['PageContainer'], '{n}.container_type', '{n}');
+$pageContainer = [];
+foreach ($page['PageContainer'] as $item) {
+	$pageContainer[$item['container_type']] = $item;
+}
 echo $this->element('Boxes.render_boxes', array(
 		'boxes' => $pageContainer[Container::TYPE_MAIN]['Box'],
 		'containerType' => Container::TYPE_MAIN