Project

General

Profile

Bug(バグ) #2286

opDiaryPluginをはずすと携帯でログイン出来なくなる

Added by pnetan   about 8 years ago. Updated almost 4 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2011-07-21
Due date:
% Done:

100%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

Description

説明:http://sns.openpne.jp/communityTopic/7363 より転記

トピック

opDiaryPluginをはずすと携帯でログイン出来なくなります。
The route "diary_new" does not exist.

「opDiaryPluginをはずすと携帯でログイン出来なくなる」問題についてコメントが付いていましたので、それも転記します。

管理画面(pc_backend)>ナビゲーション設定>携帯版ナビ でURLにdiaryのついたナビを全部消してください。

pc版はデータが残っていても平気な手当てがされているのですが、モバイル版は手当てがなくてここでエラーになります。
修正版pull requestしてるのですがお忙しいようで、なかなか取り込まれないですね^^;

Related issues

Related to OpenPNE 3 - Backport(バックポート) #2307: opDiaryPluginをはずすと携帯でログイン出来なくなる Fixed(完了) 2011-07-27
Related to OpenPNE 3 - Backport(バックポート) #2687: opDiaryPluginをはずすと携帯でログイン出来なくなる Fixed(完了) 2011-07-21 2012-01-12

Associated revisions

Revision 1aa9b1c0
Added by Yuya Watanabe about 8 years ago

Merge remote branch 'remotes/77web/add_accessibility_check_to_navi_in_mobile' (fixes #2286)

Revision b494468e (diff)
Added by Yuya Watanabe about 8 years ago

fixed for coding standard (fixes #2286)

History

#1 Updated by pnetan   about 8 years ago

  • Project changed from OpenPNE 3 to opDiaryPlugin
  • [QA]バグ通知済 set to No

#2 Updated by Rimpei Ogawa about 8 years ago

  • Project changed from opDiaryPlugin to OpenPNE 3

#3 Updated by Rimpei Ogawa about 8 years ago

本体側の問題ですのでプロジェクト移動しました。

77web さんからのプルリクエストがあります。

モバイルのナビに存在するルート名かどうかのチェックを追加
https://github.com/openpne/OpenPNE3/pull/14

#4 Updated by Kousuke Ebihara about 8 years ago

  • Priority changed from Normal(通常) to High(高め)
  • Target version set to OpenPNE 3.7.0
  • 3.6 で発生するか set to Yes

#5 Updated by Shingo Yamada about 8 years ago

  • 360対象 set to RC1

#6 Updated by Shingo Yamada about 8 years ago

  • Assignee set to Yuya Watanabe

#7 Updated by wa ta about 8 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 1aa9b1c00eed87a6bf6581291344417ad1707e89 で適用されました。

#8 Updated by wa ta about 8 years ago

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

#9 Updated by Yuya Watanabe about 8 years ago

redmine内でdiffが表示されないようなので,github内のマージを行ったコミットのURLを記載しておきます.

https://github.com/openpne/OpenPNE3/commit/1aa9b1c00eed87a6bf6581291344417ad1707e89

#10 Updated by Yuma Sakata about 8 years ago

  • 360対象 deleted (RC1)

#11 Updated by Yuma Sakata about 8 years ago

  • 360対象 set to RC1

#12 Updated by Kousuke Ebihara almost 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#13 Updated by Minoru Takai almost 8 years ago

pull request のコミットからはこのチケットが辿れないため、検索用にコメントをしておきます。

-    $list[] = link_to($nav->getCaption(), $nav->getUri());
+    if(op_is_accessible_url($nav->getUri()))
+    {
+      $list[] = link_to($nav->getCaption(), $nav->getUri());
+    }
-function op_is_accessable_url($uri)

+function op_is_accessible_url($uri)

#14 Updated by Fumie Toyooka almost 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

#15 Updated by kaoru n almost 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

Also available in: Atom PDF