Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #1203 : BcBaserHelper::getDescription() がトップページでサイト基本説明文を取得できない問題を修正 #1204

Merged
merged 1 commit into from
Jul 2, 2019

Conversation

tnagatomi
Copy link
Contributor

@tnagatomi tnagatomi commented Jul 2, 2019

fixes #1203

BcBaserHelper::getDescription() がトップページでサイト基本説明文を取得できない問題を修正しました。

$this->request->params['Content']['site_root'] ではなく isHome() を利用してトップページかどうか判定するように変更しました。

ご確認よろしくお願いします。

@tnagatomi tnagatomi changed the title Fix #1203 : BcBaserHelper::getDescription() がトップページでサイト基本説明文を取得できない問題を修正 fixes #1203 : BcBaserHelper::getDescription() がトップページでサイト基本説明文を取得できない問題を修正 Jul 2, 2019
@tnagatomi tnagatomi changed the title fixes #1203 : BcBaserHelper::getDescription() がトップページでサイト基本説明文を取得できない問題を修正 Fix #1203 : BcBaserHelper::getDescription() がトップページでサイト基本説明文を取得できない問題を修正 Jul 2, 2019
@tnagatomi
Copy link
Contributor Author

テストが失敗しているようなので調整します。

@ryuring ryuring merged commit 0e17935 into baserproject:dev-4 Jul 2, 2019
@ryuring
Copy link
Collaborator

ryuring commented Jul 2, 2019

@tommy6073 テストは別の問題で失敗していると思います。マージしました。

@ryuring
Copy link
Collaborator

ryuring commented Jul 2, 2019

@tommy6073 いや、このコミットでの失敗ですね。調整お願いします。

@tnagatomi tnagatomi deleted the dev-4-fix-1203 branch July 2, 2019 07:28
tnagatomi added a commit to tnagatomi/basercms that referenced this pull request Jul 2, 2019
tnagatomi added a commit to tnagatomi/basercms that referenced this pull request Jul 2, 2019
tnagatomi added a commit to tnagatomi/basercms that referenced this pull request Jul 2, 2019
@gondoh gondoh added this to the etc milestone Aug 13, 2019
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.

BcBaserHelper::getDescription() がトップページでサイト基本説明文を取得できていない
3 participants