OpenPNE 3

他にクリック出来ない箇所が無いか、軽く調査しました。
クリック出来ない箇所についての調査結果は以下のとおりです。

グローバルナビ¶

• マイホームと最新日記以外すべて

ローカルナビ¶

• ホーム、日記、メッセージ、あしあと以外すべて

フレンドホームのナビ¶

• 日記を読む、メッセージを送る以外すべて

コミュニティのナビ¶

• 掲示板以外すべて

• global
  

  <div id="globalNav">
  <ul>
  <li id="globalNav__homepage"><a href="/">マイホーム</a></li>
  <li id="globalNav__member_search"><a href="/member/search">メンバー検索</a></li>
  <li id="globalNav__community_search"><a href="/community/search">コミュニティ検索</a></li>
  <li id="globalNav__ranking"><a href="/ranking">ランキング</a></li>
  <li id="globalNav__album_list"><a href="/album">アルバム</a></li>
  <li id="globalNav_diary_index"><a href="/diary">日記</a></li>
  <li id="globalNav_blog_index"><a href="/blog">最新Blog</a></li>
  <li id="globalNav__member_config"><a href="/member/config">設定変更</a></li>
  <li id="globalNav__member_invite"><a href="/invite">友人を招待する</a></li>
  <li id="globalNav__member_logout"><a href="/logout">ログアウト</a></li>
  
  </ul>
  </div><!-- globalNav -->
  

• local - myhome
  

  <div id="localNav">
  <ul class="default">
  <li id="default__homepage"><a href="/">ホーム</a></li>
  <li id="default__friend_list"><a href="/friend/list">マイフレンド</a></li>
  <li id="default_message_index"><a href="/message">メッセージ</a></li>
  <li id="default_ashiato_list"><a href="/ashiato/list">あしあと</a></li>
  <li id="default__application"><a href="/application">アプリ</a></li>
  <li id="default__album_list_mine"><a href="/album/listMember">アルバム</a></li>
  <li id="default_diary_listMember"><a href="/diary/listMember">日記</a></li>
  <li id="default_favorite_list"><a href="/favorite/list">お気に入り</a></li>
  <li id="default__member_profile_mine"><a href="/member/profile">プロフィール確認</a></li>
  <li id="default__member_editProfile"><a href="/member/edit/profile">プロフィール編集</a></li>
  </ul>
  </div><!-- localNav -->
  

• local - friend
  

  <div id="localNav">
  <ul class="friend">
  <li id="friend__member_profile"><a href="/member/2">ホーム</a></li>
  <li id="friend__album_list_member"><a href="/album/listMember/2">アルバム</a></li>
  <li id="friend_diary_listMember"><a href="/diary/listMember/2">日記</a></li>
  <li id="friend__friend_list"><a href="/friend/list?id=2">フレンドリスト</a></li>
  <li id="friend_favorite_add"><a href="/favorite/add?id=2">お気に入りに追加</a></li>
  <li id="friend__obj_member_introfriend"><a href="/introfriend/2">紹介文を書く</a></li>
  <li id="friend_message_sendToFriend"><a href="/message/sendToFriend/id/2">メッセージを送る</a></li>
  <li id="friend__application_list"><a href="/application/list/2">アプリ</a></li>
  </ul>
  </div><!-- localNav -->
  

• local - community
  

  <div id="localNav">
  <ul class="community">
  <li id="community__community_home"><a href="/community/9">コミュニティトップ</a></li>
  <li id="community_communityTopic_listCommunity"><a href="/communityTopic/listCommunity/9">トピックリスト</a></li>
  <li id="community_communityEvent_listCommunity"><a href="/communityEvent/listCommunity/9">イベントリスト</a></li>
  <li id="community__community_join"><a href="/community/join?id=9">コミュニティに参加</a></li>
  <li id="community__community_quit"><a href="/community/quit?id=9">コミュニティを退会</a></li>
  </ul>
  </div><!-- localNav -->
  

更新履歴 582fc7187a88f580d326df644394b6d4ea903dc6 で適用されました。

#1004 にて、 fixture でインポートされる各ナビゲーション項目の URL が、内部 URL （デフォルトルートにマッチさせる形式）からルート名を指定したものに置き換わっていますが、既にこの fixture をインポート済みのデータ（要は OpenPNE 3.4 時点で OpenPNE 2 からのアップグレードや新規インストールを完了させた状態のデータ）に格納された URL は置き換わりません（ナビゲーションは出力がキャッシュされるので、従来の表記を放置したところでパフォーマンス上の影響は小さく抑えられるため、このこと自体は特に大きな問題はないと思っています）。

このチケットでおこなわれた 582fc718 の変更は前述のような状態のデータを考慮していません。わかりやすく言うと、 3.4 時点で OpenPNE 2 からのアップグレードをおこなった SNS がこの変更を適用後の OpenPNE にアップデートした場合、このチケットで報告されているものと同じ（ような）問題に遭遇することになると思います。

ですので、 582fc718 の対応方針を採るのであれば、 OpenPNE 3.6 時点の fixture によって作られるデータだけではなく、 3.4 時点の fixture によって作られるデータも考慮するべきです。

また、他の対応方針として、 (S2) の対応方法の一例として挙げられている op_url_to_id() の実装を変更するのも手です。ただ、 op_url_to_id() は opSkinClassicPlugin に限定して使われているわけではなく、このヘルパー関数の実装の変更は他のスキンプラグインやカスタム CSS などに影響する恐れがあります。従って、 op_url_to_id() を変更する場合、このチケットの対応に必要な最低限度の変更にとどめる（どうやって？）など、できるだけ影響を小さくするように配慮することが求められます。

以上、検討をお願いします。

note-12 の指摘について、 582fc718 の修正が 3.4.x 以前の fixture によって作られるデータを考慮できていない点は、このコミット時点で気づいていませんでした（というより、この問題は 3.4.x にも共通する問題だと思っていました）。コミット後に 3.4.x で状況が違うことに気付き、note-7 の内容を調査をしました。

note-12 と note-7 以降の調査結果を踏まえて、改めて修正を行います。

582fc718 は (C1) と (C2) を同時に扱ってしまっており差分の意味が分かりにくいので、一旦これを revert しようと思います。

note-12 で、

従って、 op_url_to_id() を変更する場合、このチケットの対応に必要な最低限度の変更にとどめる（どうやって？）など、できるだけ影響を小さくするように配慮することが求められます。

と言われている通り、他所でこの関数（およびその結果）を使っているところへの影響が無いようにする必要がありますが、 note-9 で、

• (S2): HTML 側の出力（恐らく op_url_to_id() 関数の定義）を修正する（関数定義は変えないで、テンプレート側で、「先頭に _ が付いていたら除去する」といった対応も可能だろうか）

と記述しているように、「ナビゲーションを表示するテンプレート側で先頭に _ が付いていたら除去する」修正がよいのではないかと考えています。

現時点で思いついている修正案を示します。

• 修正前
  

  function op_url_to_id($uri)
  {
    return str_replace(array('/', ',', ';', '~', '?', '@', '&', '=', '+', '$', '%', '#', '!', '(', ')'), '_', $uri);
  }
  
  <li id="globalNav_<?php echo op_url_to_id($nav->uri) ?>">
  

• 修正案(1) : 互換性を保ったまま関数定義を変える（ナビのために ltrim() だけ追加するのは不自然なので trim() にしています）
  

  function op_url_to_id($uri, $isTrim = false)
  {
    $str = str_replace(array('/', ',', ';', '~', '?', '@', '&', '=', '+', '$', '%', '#', '!', '(', ')'), '_', $uri);
    if ($isTrim)
    {
      $str = trim($str, '_');
    }
  
    return $str;
  }
  
  <li id="globalNav_<?php echo op_url_to_id($nav->uri, true) ?>">
  

• 修正案(2) : テンプレート側で文字列処理をする
  

  function op_url_to_id($uri)
  {
    return str_replace(array('/', ',', ';', '~', '?', '@', '&', '=', '+', '$', '%', '#', '!', '(', ')'), '_', $uri);
  }
  
  <li id="globalNav_<?php echo ltrim(op_url_to_id($nav->uri), '_') ?>">
  

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

テスト完了致しました。
問題は発見されなかったため、ステータスをFixed（完了）に致します。