OpenPNE3 Plugins » opAlbumPlugin

本チケットは、高橋さんに依頼しました。(高橋さんのassign権限を要求中)

変更点確認しました。動作上とくに問題ありませんが、テンプレート (_formEditImage.php) 内でのPHPコード部分につきまして、

<?php
$imageTag = image_tag_sf_image($image->getCoverImage(), array('size' => '120x120'));
if ($image->getFileId()): ?>
...
else:
...
endif;

<?php $imageTag = image_tag_sf_image($image->getCoverImage(), array('size' => '120x120')); ?>
<?php if ($image->getFileId()): ?>
...
<?php else: ?>
...
<?php endif;

序にですが、テンプレートファイルであれば、ファイル全体がPHP処理命令であることは想定していないため、最終行がPHP処理記述であってもPHP処理命令終了区切り子（つまり ?> ）を記述した方が見栄えがよいかもしれません。参考までに関連するコーディング規約を示しておきます。 http://www.openpne.jp/coding-standards-ja/#id3

<?php $imageTag = image_tag_sf_image($image->getCoverImage(), array('size' => '120x120')); ?>
<?php if ($image->getFileId()): ?>
...
<?php else: ?>
...
<?php endif; ?>

なお、以下のような記述よりも note-5 で指摘されている記述の方が好ましい理由を個人的な観点から示します。

1: <?php
2: else:
3:   echo $imageTag; ?><br />
4: %input%<br />

ここで示した 3 行目について、例えば $imageTag の直前にHTML記述を追加する修正を行いたい場合、 else 文全体に差分が生じるような修正となってしまいます。また $imageTag を出力しない修正を行いたい場合、 1 行目で改行してしまっているのが不自然になる可能性があります。

PHP処理記述の目的が異なる場合は、処理が連続していたとしても処理命令区間を独立させたほうがよいと考えています。以下の例は極端かもしれませんが、個人的にはテンプレートに於いては以下のような書き方をしています（もちろん、場所によってはPHP処理の中で sprintf() などを用いて出力した方が見栄えがよい場面もあると思います）。

<?php
// テンプレート内で用いる一時的な変数は冒頭に書く
$var1 = ...;
$var2 = ...;
?>
<?php foreach ($vals as $val): ?>
<?php
// foreach 文ブロック内で一時的に用いる変数を書く
$val_x = ...;
$val_y = ...;
?>
<li><?php echo $val; ?>(<?php echo $val_x; ?>, <?php echo $val_y; ?>)</li>
<?php endforeach; ?>

上記はコーディング規約で定められているわけではないためあくまで参考程度にしていただければと思います。

お二人のご指摘を元に
https://github.com/martini2002jp/opAlbumPlugin/commit/d83416d24cdc09606c23cc9c0528be2ac6f6a8ed
にてテンプレートを再度修正いたしました。

note-4 で行った動作確認を再度おこない、正しく動作していることを確認いたしました。