OpenPNE3 Plugins » opAlbumPlugin

調査¶

• 「公開範囲」という項目名であるべきところが "Public Flag" となっている
  • 修正方針
    

    listMemberSuccess.php にて、i18n翻訳の対象となる文字列の大文字小文字違い。
    <th><?php echo __('Public Flag') ?></th>
    ↓
    <th><?php echo __('Public flag') ?></th>
    

• 「全体に公開」といった項目値であるべきところが "1" となっている
  • 修正方針
    

    listMemberSuccess.php で公開範囲を示すpublic_flagを直接、値で表示している。
    <td colspan="2"><?php echo $album->getPublic_flag() ?></td>
    ↓
    <td colspan="2"><?php echo $album->getPublicFlagLabel() ?></td>
    

• 「2010年06月30日 23:07」という日本語の表記になるべき日付部分が "2010-06-30 23:07:55" となっている
  • 「2010年06月30日 23:07」 とすべきか 「2010年 06月30日 23:07」 とすべきかは一見議論の余地があるが、以下に簡単に示している通り、テンプレート間で表記に差異があり、ここでは 'XDateTimeJa' が適切だと思われるが、異なる見解があればこの限りではない。
    • 'Br' を付けるのは表示されるボックス幅が狭いときなどで、複数行表示することが想定されている場合であるはずだが、ここは一行で表示することを想定している。
  • 修正方針
    

    listMemberSuccess.php で日付を op_format_date() を通さずに表示しようとしている。
    <tr>
    <th><?php echo __('Created at') ?></th>
    <td colspan="2"><?php echo $album->getCreatedAt() ?></td>
    </tr>
    ↓
    <tr>
    <th><?php echo __('Created at') ?></th>
    <td colspan="2"><?php echo op_format_date($album->getCreatedAt(), 'XDateTimeJa') ?></td>
    </tr>
    
    ところで、この日付部分の表示について、
     * listSuccess.php （グローバルナビ：全体のアルバム一覧）
     * showSuccess.php （アルバム1つの表示）
    この2ページ間で op_format_date() の第2引数の値が異なっている。
    前者は 'XDateTimeJa' で 後者は 'XDateTimeJaBr' であり、
    HTMLソース上で改行文字が含まれているか否かの差異と、
    ブラウザ上の表示で「年」と「月日」の間に空白の有無の差異が生じている。
    

  • showSuccess.php に対しても 'XDateTimeJaBr' ではなく 'XDateTimeJa' を用いるようにする修正をこのチケットで扱うことにします。