Smarty4.3 を使用する

[nanasess]

• Smarty4.2 4.3 の検証
• システム要件は PHP7.4以降に変更が必要
• s/SmartyBC/Smarty/ の変更のみで問題ないっぽい
• PHP8.3でも問題なし
  • PHP7.4未満と、PHP7.4以上のパッケージを分ける必要がありそう

[nobuhiko]

世界のウミウシにも反映させよ😆

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.20%. Comparing base (9fb10ef) to head (0846c96).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #615      +/-   ##
==========================================
- Coverage   55.21%   55.20%   -0.01%     
==========================================
  Files          75       75              
  Lines        8913     8912       -1     
==========================================
- Hits         4921     4920       -1     
  Misses       3992     3992              

Flag	Coverage Δ
tests	55.20% <100.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[seasoftjapan]

master を PHP 8 (8.1以降?) で動かして、テンプレート構文を誤ると、例外処理がコケる様子でした。

Fatal error(E_COMPILE_ERROR): Type of SmartyCompilerException::$line must be int (as in class Exception) on [/**/data/vendor/smarty/smarty/libs/sysplugins/smartycompilerexception.php(8)]

これかな。smarty-php/smarty#713

この PR をマージすると、ちゃんと直りますね。

[nanasess]

Smarty4.3 でも問題なさそう

[nanasess]

以下をざっと見た感じ、SC_View() クラスの更新のみで Smarty5 も動作するのではないかと思われる
https://smarty-php.github.io/smarty/5.x/upgrading/

#681 の対応にあわせて、 Smarty5 の導入も検討する

[nanasess]

#854 がマージされましたので、 WIP 外しました