OpenPNE 3

更新履歴 8c58d3b41c050abe4727acdb3143ed7bcc2c4280 で適用されました。

op_image_path() についてフィードバックします。

1. スキンプラグイン名のみを保存すれば十分¶

スキンプラグインのパスは必要なく、プラグイン名のみを保存し、sf_web_dir/opSkin***Plugin/images/$source が存在するかどうかを確認するようにすれば、もっとシンプルになると思います。

パターンマッチもないので glob() も不要です。

2. strpos() の引数間違い¶

1142       if (strpos('.', $path[count($path) - 1]))

strpos() の引数の順番が逆です。と、おそらく !strpos() としたかった場面ではないでしょうか。

3. 文字列連結の間違い¶

1144         $pattern .= $pattern.'.png';

これだと、 $pattern = $pattern.$pattern.'.png' になってしまいます。

更新履歴 640900818758410b3dafcb2b4b3a9cce3a8a6040 で適用されました。

Skinプラグインの判定は、他の箇所では正規表現を使っていないので合わせてください。

lib/config/opApplicationConfiguration.class.php
91:      if (0 === strpos($pluginName, 'opSkin'))

lib/plugin/opPlugin.class.php
135:    return (0 === strpos($this->name, 'opSkin'));

lib/helper/opUtilHelper.php
1147:        if (preg_match('/^opSkin.*Plugin$/', $plugin))

更新履歴 8077965f79759000bb300a60817d025d5e650c69 で適用されました。

$source からファイル名を返すのには basename() が使えます（#1160 と同じ指摘でこちらに書き忘れていました）

_compute_public_path() が参考になります。

396   if (false === strpos(basename($source), '.'))
397   {
398     $source .= '.'.$ext;
399   }

更新履歴 6ac2ac610df11512ef52eed030f9d21f800b8b22 で適用されました。