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

ページ管理の編集対象に会員登録(確認ページ)を加える #4428 #4433

Merged
merged 9 commits into from
Dec 2, 2020

Conversation

oywc410
Copy link
Contributor

@oywc410 oywc410 commented Jan 12, 2020

ページ管理の編集対象に会員登録(確認ページ)を加える #4428

dtb_page変更:
edit_typeに3を追加して確認ページとして利用します。
master_page_idは確認ページのIDに設定し、ページ登録時にURLチェックを除外します。

もし仕様問題がない場合はページ一覧のデザインが調整もお願いいたします。

以上、よろしくお願いいたします。

@okazy okazy added affected:admin_template 管理画面テンプレートのDOMに影響のある変更 enhancement 機能追加 Status: needs-review labels Jan 17, 2020
@okazy okazy added this to the 4.0.4 milestone Jan 17, 2020
@okazy okazy modified the milestones: 4.0.4, 4.0.x May 22, 2020
@chihiro-adachi
Copy link
Contributor

chihiro-adachi commented Dec 1, 2020

TwigInitializeListenerで、ページを特定できず(入力画面か確認画面かの判別ができない)、レイアウトが適用されない場合がありました。
https://github.com/EC-CUBE/ec-cube/blob/4.0/src/Eccube/EventListener/TwigInitializeListener.php#L196

以下のように対応しています。

  • 確認ページのルーティングを、entry_confirmのように別ルーティングで定義、ページとレイアウトを紐付けるようにする
  • TwigInitializeListenerの時点では入力画面か確認画面かの判別ができないため、レイアウトの取得はコントローラ側で制御する

@chihiro-adachi chihiro-adachi modified the milestones: 4.0.x, 4.0.6 Dec 1, 2020
*/
final class Version20201127000000 extends AbstractMigration
{
const NAME = 'dtb_page';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

使っていない定数。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

削除しました

)");

$sortNo++;
$this->addSql("INSERT INTO dtb_page_layout (page_id, layout_id, sort_no, discriminator_type) VALUES ($pageId, 2, , $sortNo, 'pagelayout')");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

syntax error

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正しました

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:admin_template 管理画面テンプレートのDOMに影響のある変更 enhancement 機能追加 Status: needs-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants