POTI-boardのエラーを減らすために htmltemplate.inc編

potiboard.phpに
ini_set( 'error_reporting', E_ALL );
を追加して、すべてのエラーを表示するように設定。
E_DEPRECATEDレベルのエラーが大量に発生しているのを確認しました。
POTI-boardで使っているhtmltemplate.incに問題があるようです。

POTI-board htmltemplate.inc php7 E_DEPRECATED
PHP: PHP 7.0.x で推奨されなくなる機能 - Manual

PHP 4 形式のコンストラクタ
非 static メソッドに対する static 呼び出し
将来のバージョンでは動作しなくなるので今のうちに警告…という事ですね。
PHP 4 形式のコンストラクタ (クラス名と同じ名前のメソッドを定義するもの) が非推奨
__construct() メソッドを実装していれば、この警告は発生しません。
該当する箇所は…。

POTI-board htmltemplate.inc php7 E_DEPRECATED
class StandardParser extends TemplateParser{
// function StandardParser(){
function __construct(){
class htmltemplate{
private $parser;
// the constructor
// private function htmltemplate(){
private function __construct(){
の2箇所でした。
static 宣言されていないメソッドに対する static 呼び出しが非推奨となりました。 将来的には削除される見込みです。
それなら…static 宣言すれば、という事で、

POTI-board htmltemplate.inc php7 E_DEPRECATED
public static function add($tag){
// This method is obsolete.Use addTag() instead.
$inst=&htmltemplate::getInstance();
$inst->parser->add($tag);
}
POTI-board htmltemplate.inc php7 E_DEPRECATED POTI-board htmltemplate.inc php7 E_DEPRECATED

左、htmltemplate.inc書き直し前、右、書き直し後。
とりあえず、エラーがでなくなるところまで何とか。
PHP 5からPHP 7へ移行する際に気を付けるべきポイント (1/4):CodeZine(コードジン)

staticメソッドとして宣言したメソッドをインスタンスメソッドとして実行するにはワーニングなどが表示されないようですが、こちらも良いコードとは言えませんので、もしそのようなコードがあればこの機会に対応することをおすすめします。

インスタンスを使っているのに静的というのはおかしいという事なのでしょうね…。
staticなプロパティの使いどころ - PHPで翻訳三昧

PHP5からは、static を指定したプロパティは
インスタンス化しなくてもアクセス出来るよう変わりました。

php5ではもっと簡単にわかりやすく書ける…みたいです。

GitHub - sakots/poti-kai: POTI-board, the oekaki BBSサ骨さんによるオープンソースプロジェクトです。
phpのプログラムを書き慣れている方なら、おそらくhtmltemplate.incの問題の箇所を修正できると思います…。

どこが問題なのか、コメント行にも書いておきましたのでよろしくお願いいたします。
これから、サ骨さんのところにアップします…。

この記事へのコメント

コメントを書く
お名前:

メールアドレス:

ホームページアドレス:

コメント:

※ブログオーナーが承認したコメントのみ表示されます。