library install from composer instead of pear over php5.4

satomif commented 6 years ago

[fix]ライブラリの依存解決をpearからcomposerに変更, php7.2はpear/mdb2ライブラリが対応していないため一旦除外 [fix]composerのautoloaderに切り替え [delete]xhprofを削除 [fix]pearライブラリのstaticなメソッドにstatic修飾子がないので呼び元で抑制 [fix]constructorがクラス名になっていたのを修正 [fix]クラス内の要素チェック追加 [add]phpunit.xml.dist追加 [add]travis.yml追加 [fix]親クラスからextendしたオーバライド関数の引数の数をあわせるように修正

koriym commented 6 years ago

[fix]error suppression https://github.com/bearsaturday/BEAR.Saturday/pull/50/commits/4341e0e76449c5b4a46e33a09f74d876ac9dc539 https://github.com/bearsaturday/BEAR.Saturday/pull/50/commits/4341e0e76449c5b4a46e33a09f74d876ac9dc539

"@"で全て抑制してしまうより以下のコードを使うのはどうでしょうか？

$reporting = error_reporting( E_ALL | ~E_STRICT );  // E_STRICT無視
// PEARのstatic コールなどをここで
error_reporting( $old_flags ); // E_STRICT復帰

呼び出し先での処理でエラーが出ると発券困難になるのを防げます。

koriym commented 6 years ago

他は良さそうです！

bearsaturday / BEAR.Saturday

library install from composer instead of pear over php5.4 #50