バグ報告 > 管理機能 > 管理画面でファイル名を入力した際に警告がエラーログに記録される |
管理機能
スレッド表示 | 古いものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
snitta |
投稿日時: 2014/2/14 22:50
対応状況: 解決済
|
一人前 登録日: 2013/10/3 居住地: 島根県 投稿: 100 |
Re: 管理画面でファイル名を入力した際に警告がエラーログに記録される コミットありがとうございます。
テストが通り、警告も出ない事を確認できました。 これにて解決済みにマークさせて頂きます。 お時間を割いて頂き重ねてありがとうございました!
|
red |
投稿日時: 2014/2/14 22:28
対応状況: −−−
|
神 登録日: 2010/2/15 居住地: 東京都 投稿: 1568 |
Re: 管理画面でファイル名を入力した際に警告がエラーログに記録される ありがとうございます
修正をコミットしました http://svn.ec-cube.net/open_trac/changeset/23349 warningはPHPUnitに任せればやってくれるんですね、書き直したら上手く行きました〜
|
snitta |
投稿日時: 2014/2/14 20:13
対応状況: −−−
|
一人前 登録日: 2013/10/3 居住地: 島根県 投稿: 100 |
Re: 管理画面でファイル名を入力した際に警告がエラーログに記録される ご確認ありがとうございます。
ご指摘いただいたテスト項目と省略部分を修正したパッチを用意いたしました。 こんな感じでどうでしょうか? r23289ベース: https://gist.github.com/zenith6/8999302 r23348ベース: https://gist.github.com/zenith6/8999156 引用: 出るべきではない警告が出ていることをテストで明示したかったのですが、warningが出てなくてもテストが落ちてしまいますね テスト対象の仕様として警告を出さないのであるならば、@expectedException PHPUnit_Framework_Error_Warning は指定すべきではないと思います。 E_WARNING が出た時は本来PHPUnitが教えてくれるのですがそれをEC-CUBEがハンドリングしているのが問題のような…? 引用: もうsvnなんて使ってられないので。。 自分も3年ぶりに触ってパッチの作り方も忘れていました
|
red |
投稿日時: 2014/2/14 18:45
対応状況: −−−
|
神 登録日: 2010/2/15 居住地: 東京都 投稿: 1568 |
Re: 管理画面でファイル名を入力した際に警告がエラーログに記録される 出るべきではない警告が出ていることをテストで明示したかったのですが、warningが出てなくてもテストが落ちてしまいますね
書き方を考えます Failed asserting that exception of type "PHPUnit_Framework_Error_Warning" is thrown. 1. SC_CheckError::FILE_NAME_CHECK_BY_NOUPLOAD() に対するパッチが含まれていません(こっちがメインですw)。 こちらですが、下記が省略されているからなんでだろうな?という部分と、このテストだけだとファイル名に日本語やスペースが含まれている事がテストされていないと思うのでそこを確認してからコミットしますね
非公式は2人だけらしいです もうsvnなんて使ってられないので。。
|
snitta |
投稿日時: 2014/2/14 17:46
対応状況: −−−
|
一人前 登録日: 2013/10/3 居住地: 島根県 投稿: 100 |
Re: 管理画面でファイル名を入力した際に警告がエラーログに記録される red様、早速のご対応大変ありがとうございます。
引用: テストを追加してコミットしました チェックアウトして確認させて頂きました。 2点ほど修正orご確認をお願いいたします。 1. SC_CheckError::FILE_NAME_CHECK_BY_NOUPLOAD() に対するパッチが含まれていません(こっちがメインですw)。 2. 出るべきではない警告の修正が目的ですので @expectedException は不要ではないでしょうか? 引用: 非公式にgithubにリポジトリを置いてますのでプルリクエストして頂ければそちらからも取り込めると思います おお、次から利用させて頂きます! 今までnanasess様のリポジトリしか知りませんでしたが結構な方が自前で用意されてるんですね
|
red |
投稿日時: 2014/2/14 15:53
対応状況: −−−
|
神 登録日: 2010/2/15 居住地: 東京都 投稿: 1568 |
Re: 管理画面でファイル名を入力した際に警告がエラーログに記録される ありがとうございます
テストを追加してコミットしました Warningが問題ということだったので、PHPUnit_Framework_Error_Warningをつけておきました http://svn.ec-cube.net/open_trac/changeset/23347 修正内容で間違いないことが確認出来ましたら、本体にコミットします 非公式にgithubにリポジトリを置いてますのでプルリクエストして頂ければそちらからも取り込めると思います https://github.com/nobuhiko/EC-CUBE
|
snitta |
投稿日時: 2014/2/13 22:36
対応状況: −−−
|
一人前 登録日: 2013/10/3 居住地: 島根県 投稿: 100 |
管理画面でファイル名を入力した際に警告がエラーログに記録される ------------------------------------------------------------------------------------------
[EC-CUBE] 2.13.1 [レンタルサーバ] ローカルサーバー [OS] Windows 8.1 [PHP] PHP 5.5.9 [データベース] MySQL 5.6.14 [WEBサーバ] Apache 2.4.7 [ブラウザ] Firefox 27.0 [現象] 管理画面の各所フォームでファイル名を入力した際に警告がエラーログに記録される。 ------------------------------------------------------------------------------------------ いつもお世話になっております。 管理画面の「コンテンツ管理>ファイル管理」でフォルダ作成を作成する際に、ファイル名を指定して送信すると以下の警告が data/logs/error.log に記録されます。
原因は SC_CheckError::FILE_NAME_CHECK_BY_NOUPLOAD() 内で使われている '/[\\]/' という正規表現だと考えます。 この問題に対する修正パッチを作成致しましたので、 よろしければ取り込んで頂けないでしょうか? お手数をおかけしますがご検討のほどよろしくお願いいたします。
※追記 該当箇所の単体テストがありませんでしたので用意致しました。
|
スレッド表示 | 古いものから | 前のトピック | 次のトピック | トップ |