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

Opened 11 years ago

Last modified 11 years ago

[フロント]ガイド/エラー画面

Reported by: kotani Owned by: yomoro
Priority: Milestone: EC-CUBE2.11.0 リファクタリングProject
Component: フロント Version: 2.11.0 β
Keywords: Cc:
修正済み:

Description

▼リファクタリング対象ファイル
・当サイトについて
/abouts/LC_Page_Abouts.php

・特定商取引
/order/LC_Page_Order.php

・ご利用ガイド
/guide/LC_Page_Guide.php
/guide/LC_Page_Guide_About.php
/guide/LC_Page_Guide_Charge.php
/guide/LC_Page_Guide_Kiyaku.php
/guide/LC_Page_Guide_Privacy.php
/guide/LC_Page_Guide_Usage.php

・エラー画面
/error/LC_Page_Error.php
/error/LC_Page_Error_DispError.php
/error/LC_Page_Error_SystemError.php


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


Change History

comment:1 Changed 11 years ago by yomoro

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

comment:2 Changed 11 years ago by yomoro

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

comment:3 Changed 11 years ago by AMUAMU

  • Status changed from closed to reopened
  • Resolution 修正済 deleted

LC_Page_Error_DispError が正常に動いていない為、差し戻します。
SC_Utils_Ex::sfDispError($type) メソッドからの呼び出しがメインとなりますので、こちらにて従来通りの挙動で動くようにお願いします。

comment:4 follow-up: ↓ 5 Changed 11 years ago by kotani

#961 ([管理画面]ログイン/ホーム)
の動作確認時に見つかった下記不具合、原因は LC_Page_Error_DispError にあるとのことで本チケットを差し戻しております。

▼不具合内容
管理画面ログインページで、空のまま「LOGIN」ボタンをクリックするなど、エラーを発生させた場合、エラー画面が正しく表示されない。

comment:5 in reply to: ↑ 4 Changed 11 years ago by yomoro

kotani への返信

#961 ([管理画面]ログイン/ホーム)
の動作確認時に見つかった下記不具合、原因は LC_Page_Error_DispError にあるとのことで本チケットを差し戻しております。

▼不具合内容
管理画面ログインページで、空のまま「LOGIN」ボタンをクリックするなど、エラーを発生させた場合、エラー画面が正しく表示されない。

梶原様にはご報告済みですが、 LC_Page_Error_DispError.php自体にはこれといって怪しい部分が見当たらない ので、継承元クラスの処理とか"どこかのページから呼び出された時だけ"とかい う面倒臭いケースかもしれないです。

comment:6 Changed 11 years ago by AMUAMU

r20135 から問題が発生していることを確認しました。 AdminView?に関する問題かと

comment:7 Changed 11 years ago by shutta

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

r20259 で修正してみました。

Note: See TracTickets for help on using tickets.