Skip to content

fix: 追加したパブリックルームをトップページとして一番上に移動しても表示されるように修正#509

Merged
akagane99 merged 3 commits intomasterfrom
bugfix
Dec 27, 2018
Merged

fix: 追加したパブリックルームをトップページとして一番上に移動しても表示されるように修正#509
akagane99 merged 3 commits intomasterfrom
bugfix

Conversation

@akagane99
Copy link
Contributor

NetCommons3/NetCommons3#1394

バグ修正のプルリクエストです。
マージできるか、もくもく会で相談してから決めます。

fix: 追加したパブリックルームをトップページとして一番上に移動しても表示されるように修正
@coveralls
Copy link

coveralls commented Dec 13, 2018

Coverage Status

Coverage increased (+0.1%) to 63.563% when pulling 7ed4947 on bugfix into 00312d9 on master.

fix: パブリックルームのトップページ取得は、パブリックルームが複数ありえるため、スペースIDを指定して取得する
@akagane99
Copy link
Contributor Author

@s-nakajima
修正で、'Room.space_idを条件指定するようにしました。
その結果、__getPage($query)if (! empty(Current::$request->params['requested'])) { がtrueだと
Unknown column 'Room.space_id'でエラーになりました。

__getPage($query)if (! empty(Current::$request->params['requested'])) { をしてる理由がよくわからず、どう修正したものかわからないため、アドバイス欲しいと思ってます。

修正内容

		$result = $this->__getPage(array(
			//'recursive' => -1,
			'recursive' => 0,
			'conditions' => array(
				// パブリックルームのトップページ取得は、パブリックルームが複数ありえるため、スペースIDを指定して取得する
				//'Page.root_id' => Space::getRoomIdRoot(Space::PUBLIC_SPACE_ID),
				'Page.room_id' => 'Room.id',
				'Room.space_id' => Space::PUBLIC_SPACE_ID,
				'Page.parent_id NOT' => null,
			),
			'order' => array('Page.sort_key' => 'asc')
		));

エラー

https://travis-ci.org/NetCommons3/NetCommons/jobs/467439563#L727

1) NetCommonsAppControllerBeforeFilterTest::testLanguageConfiguredThroughQuery
PDOException: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'Room.space_id' in 'where clause'

https://travis-ci.org/NetCommons3/NetCommons/jobs/467439563#L761

2) NetCommonsAppControllerBeforeFilterTest::testLanguageConfiguredThroughQuerySession
PDOException: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'Room.space_id' in 'where clause'

https://travis-ci.org/NetCommons3/NetCommons/jobs/467439563#L795

3) NetCommonsAppControllerBeforeFilterTest::testLanguageConfiguredThroughSession
PDOException: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'Room.space_id' in 'where clause'

__getPage($query)if (! empty(Current::$request->params['requested'])) {

if (! empty(Current::$request->params['requested'])) {

とりあえずここまで。

@akagane99
Copy link
Contributor Author

@s-nakajima
https://github.com/NetCommons3/NetCommons/pull/509/files#diff-a7c195297252f68d68c24bccd0b5d80fR317

とりあえず修正しました。
トラビステスト通って、2,3日したらマージしようと思ってます。

@akagane99 akagane99 requested a review from s-nakajima December 25, 2018 08:56
@akagane99 akagane99 merged commit db4e89c into master Dec 27, 2018
@akagane99 akagane99 deleted the bugfix branch December 27, 2018 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants