バグの報告です。CHECK_DATE2メソッドにおいて、

// 少なくともどれか一つが入力されている。
if ($this->arrParam[$value[1]] > 0 || $this->arrParam[$value[2]] > 0 || $this->arrParam[$value[3]] > 0 || $this->arrParam[$value[4]] >= 0 || $this->arrParam[$value[5]] >= 0) {

このif文の

$this->arrParam[$value[4]] >= 0
と
$this->arrParam[$value[5]] >= 0
の式の値が常に成り立つ。

以下のように空文字が0と解釈されるため。

code:
<?php
if ('' >= 0)
echo 'true';

echo "<br />";
echo (int)'';
?>
result:
true
0

以下の対応を行うとよい。
if ($this->arrParam[$value[1]] > 0 || $this->arrParam[$value[2]] > 0 || $this->arrParam[$value[3]] > 0 || ($this->arrParam[$value[4]] >= 0 && $this->arrParam[$value[4]] != '')
|| ($this->arrParam[$value[5]] >= 0 && $this->arrParam[$value[5]] != '')) {

php:5.2.17
eccube:2.12.4

もしなおすならstrlenなどを使うほうがよい気がしますがCHECK_DATE2とCHECK_DATE3はどこのも使われていないようですし消してしまっていい関数のように見えますね

----------------
EC-CUBEのカスタマイズ、トラブル解決承ります
お気軽にお問い合わせ下さい
https://www.ec-cube.net/integrate/partner/partner.php?partner_id=690

red 様の方針を支持します。(直すにしても、削除するにしても)

----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

デフォルトでは使っていませんが、時と分を入力するようにカスタマイズする場合のエラーチェックで使用するので消すのはどうかと思いますが。

動作の保証もできないような機能を保守し続けるのは、今のプロジェクト体制では難しい気もいたします。

単体テストの自動化も進んでいるので、そちらでカバーする方法もあろうかと思いますが、それでも大きな実装の組み換え時には、開発の足かせとなり得ますし。


とは言え、頻繁に利用されそうな、若しくは近い将来 EC-CUBE 本体でも使われそうな機能を残すのは有意義だと思います。

SC_CheckError::CHECK_DATE2 は、それに該当する様なメソッドなのか否か・・・ 個人的には使った記憶が無い機能なので、あまり有用性は感じておりませんが、一般的にはどうなのですかね。

----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。