Ticket #972 (closed 新規開発: 修正済)

Opened 10 years ago

Last modified 10 years ago

リファクタリング開発:[管理画面]デザイン管理

Reported by: kotani Owned by: nanasess
Priority: Milestone: EC-CUBE2.11.1
Component: 管理画面 Version: 2.11.0 β
Keywords: Cc:
修正済み: no

Description

▼リファクタリング対象ファイル
/admin/design/LC_Page_Admin_Design.php
/admin/design/LC_Page_Admin_Design_Bloc.php
/admin/design/LC_Page_Admin_Design_CSS.php
/admin/design/LC_Page_Admin_Design_Header.php
/admin/design/LC_Page_Admin_Design_MainEdit.php
/admin/design/LC_Page_Admin_Design_Template.php
/admin/design/LC_Page_Admin_Design_Up_Down.php
/preview/LC_Page_Preview.php


リファクタリングガイドライン
http://svn.ec-cube.net/open_trac/wiki/リファクタリングガイドライン


Change History

comment:1 Changed 10 years ago by miningbrownie

  • Owner changed from somebody to miningbrownie
  • Status changed from new to assigned

ガイドラインに沿う形に修正すればOKという事ですよね。

comment:2 Changed 10 years ago by miningbrownie

design/headerで登録ボタンのmodeがconfirmなのに、actionではregistとして見ているのは既知のバグですか。

comment:3 Changed 10 years ago by miningbrownie

/admin/design/LC_Page_Admin_Design_Header.php
/admin/design/header.tpl

r20275で上記ファイルを修正しました。 ついでに以下の不具合をみつけたので、tplも合わせて修正しました。

  1. 登録できない

登録するボタンを押した際送信されるmodeがconfirmになっていたがactionの中ではregistとして扱われていた。

  1. 拡大した入力枠が元に戻る

拡大ボタンを押した後でも、header_rowとfooter_rowが空のまま送信されていた。

comment:4 Changed 10 years ago by miningbrownie

あと、divタグがdicになっていた箇所も修正しています。

comment:5 Changed 10 years ago by kotani

  • Milestone changed from EC-CUBE2.11.0 リファクタリングProject to EC-CUBE2.11.1(仮)

comment:6 Changed 10 years ago by kotani

  • Summary changed from [管理画面]デザイン管理 to リファクタリング開発:[管理画面]デザイン管理

comment:7 Changed 10 years ago by kotani

  • Owner changed from miningbrownie to somebody
  • Status changed from assigned to new

comment:8 Changed 10 years ago by nanasess

  • Owner changed from somebody to nanasess
  • 修正済み unset
  • Status changed from new to assigned

comment:9 follow-up: ↓ 10 Changed 10 years ago by kajiwara

管理画面>デザイン管理>レイアウト設定>ページを新規入力

より新規でページ追加しようとしても、ページが追加されません。
(編集ページ一覧に追加されません。)
ご確認くださいませ。

comment:10 in reply to: ↑ 9 ; follow-up: ↓ 11 Changed 10 years ago by nanasess

kajiwara への返信

管理画面>デザイン管理>レイアウト設定>ページを新規入力

より新規でページ追加しようとしても、ページが追加されません。
(編集ページ一覧に追加されません。)
ご確認くださいませ。

r20847 で修正してみました

comment:11 in reply to: ↑ 10 Changed 10 years ago by kajiwara

  • Status changed from assigned to closed
  • Resolution set to 修正済

nanasess への返信

kajiwara への返信

管理画面>デザイン管理>レイアウト設定>ページを新規入力

より新規でページ追加しようとしても、ページが追加されません。
(編集ページ一覧に追加されません。)
ご確認くださいませ。

r20847 で修正してみました

修正されていることを確認しました。

一旦正常動作が確認できましたので、クローズします。

comment:12 Changed 10 years ago by nanasess

r20864 で, デザイン管理機能のリファクタリング完了しました

comment:13 follow-up: ↓ 14 Changed 10 years ago by Seasoft

r20863 にて未記載のファイル /html/user_data/packages/default/sql/update_bloc.sql が編集され、フロント PCテンプレートにて、互換性の問題が発生しているようです。
 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=7910&forum=11

comment:14 in reply to: ↑ 13 Changed 10 years ago by Seasoft

どうやら、従来は利用していなかった update_bloc.sql が実行されるようになった点が強いようです。

/html/ 配下に実行ソースを置くのは良くない気がします。

Note: See TracTickets for help on using tickets.