FuelPHP 1.7.2から1.8へバージョンアップ(PHP7.2で出るエラーにも対応)

  • 公開日:2020/5/17
この記事は最終更新日から3年以上が経過しています。

fuelphp 1.7.2 のバージョンアップをすることがあったのでその方法をメモ。

以下のページを参考にすすめることにしました。
インストール – アップグレード – FuelPHP ドキュメント

準備としてFuelPHPドキュメントのダウンロードページから最新のzipファイルをダウンロードします。
Fuel のダウンロード – インストール – FuelPHP ドキュメント

composer.jsonを先ほどダウンロードしたファイルに置き換えて、FuelPHPを設置しているディレクトリに移動してから以下を実行します。

$ php composer.phar self-update
$ php composer.phar update

成功したら、以下のファイルを差分を確認しつつ手動で更新します。
fuel/app/bootstrap.php
public/.htaccess
public/index.php

単純に置き換えてしまうと危険です。例えば、fuel/app/bootstrap.phpであれば以下に追記している場合など、記述が消えてしまうと動かなくなる可能性があります。

\Autoloader::add_classes(array(
    // ここの記述を消してしまわないように!!
));

これで無事にアップデートできたと思いきや、サーバーのPHPが7.2以上の場合にはエラーが出ます。

Warning - get_class() expects parameter 1 to be object, null given in ***\fuel\core\classes\security.php on line 241

FuelPHP 1.8 が PHP 7.2 で変更された修正に対応されていないために出るエラーのようです。FuelPHP 1.9/develop だと対応されているらしいですが、今回は1.8なので別途対応が必要です。

以下を参考にさせていただきました。
PHP7.2でFuelPHPがエラーになるのを何とかする – Qiita

fuel/app/classes/extend/security.phpを以下の内容で作成します。

<?php
class Security extends Fuel\Core\Security
{
  public static function htmlentities($value, $flags = null, $encoding = null, $double_encode = null)
  {
    static $already_cleaned = array();

    is_null($flags) and $flags = \Config::get('security.htmlentities_flags', ENT_QUOTES);
    is_null($encoding) and $encoding = \Fuel::$encoding;
    is_null($double_encode) and $double_encode = \Config::get('security.htmlentities_double_encode', false);

    // Nothing to escape for non-string scalars, or for already processed values
    if (is_null($value) or is_bool($value) or is_int($value) or is_float($value) or in_array($value, $already_cleaned, true))
    {
      return $value;
    }

    if (is_string($value))
    {
      $value = htmlentities($value, $flags, $encoding, $double_encode);
    }
    elseif (is_object($value) and $value instanceOf \Sanitization)
    {
      $value->sanitize();
      return $value;

    }
    elseif (is_array($value) or ($value instanceof \Iterator and $value instanceof \ArrayAccess))
    {
      // Add to $already_cleaned variable when object
      is_object($value) and $already_cleaned[] = $value;

      foreach ($value as $k => $v)
      {
        $value[$k] = static::htmlentities($v, $flags, $encoding, $double_encode);
      }
    }
    elseif ($value instanceof \Iterator or get_class($value) == 'stdClass')
    {
      // Add to $already_cleaned variable
      $already_cleaned[] = $value;

      foreach ($value as $k => $v)
      {
        $value->{$k} = static::htmlentities($v, $flags, $encoding, $double_encode);
      }
    }
    elseif (is_object($value))
    {
      // Check if the object is whitelisted and return when that's the case
      foreach (\Config::get('security.whitelisted_classes', array()) as $class)
      {
        if (is_a($value, $class))
        {
          // Add to $already_cleaned variable
          $already_cleaned[] = $value;

          return $value;
        }
      }

      // Throw exception when it wasn't whitelisted and can't be converted to String
      if ( ! method_exists($value, '__toString'))
      {
        throw new \RuntimeException('Object class "'.get_class($value).'" could not be converted to string or '.
          'sanitized as ArrayAccess. Whitelist it in security.whitelisted_classes in app/config/config.php '.
          'to allow it to be passed unchecked.');
      }

      $value = static::htmlentities((string) $value, $flags, $encoding, $double_encode);
    }

    return $value;
  }
}

fuel/app/bootstrap.phpに以下を追記して拡張したクラスを有効化します。

\Autoloader::add_classes(array(
  'Security' => APPPATH.'classes/extend/security.php',
));

これでエラーも出なくなり、無事にアップデート出来ました。