http://redmine.openpne.jp/
http://redmine.openpne.jp/favicon.ico
2011-05-06T13:16:51Z
OpenPNE Issue Tracking System
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=8119
2011-05-06T13:16:51Z
Masato Nagasawa
nagasawa@tejimaya.com
<ul><li><strong>ステータス</strong> を <i>New(新規)</i> から <i>Accepted(着手)</i> に変更</li></ul>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=8120
2011-05-06T13:44:35Z
Masato Nagasawa
nagasawa@tejimaya.com
<ul></ul><p>テスト</p>
<table>
<tr>
<td>区分</td>
<td>期待結果</td>
<td>結果</td>
</tr>
<tr>
<td>携帯側のコミュニティ副管理者を降格画面で「いいえ」を選択</td>
<td>メンバー管理画面へ戻る</td>
<td>○</td>
</tr>
<tr>
<td>PC側のコミュニティ副管理者を降格画面で「いいえ」を選択</td>
<td>メンバー管理画面へ戻る</td>
<td>○</td>
</tr>
</table>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=8127
2011-05-06T14:46:03Z
Masato Nagasawa
nagasawa@tejimaya.com
<ul><li><strong>ステータス</strong> を <i>Accepted(着手)</i> から <i>Pending Review(レビュー待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>0</i> から <i>50</i> に変更</li></ul><p>更新履歴 <a class="changeset" title="fixed moved get parameter to input tag (fixes #2058)" href="http://redmine.openpne.jp/projects/op3/repository/revisions/032f641badd65e02400a432dd28a5baef1819bc0">032f641badd65e02400a432dd28a5baef1819bc0</a> で適用されました。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9106
2011-06-27T05:21:36Z
Naoya Tozuka
tozuka@tejimaya.com
<ul><li><strong>担当者</strong> を削除 (<del><i>Masato Nagasawa</i></del>)</li></ul>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9107
2011-06-27T06:10:23Z
Naoya Tozuka
tozuka@tejimaya.com
<ul></ul><p>修正箇所確認しました。OKです。</p>
<a name="メモ副管理者から降格に対応する英文の誤りについて"></a>
<h3 >【メモ】「副管理者から降格」に対応する英文の誤りについて<a href="#メモ副管理者から降格に対応する英文の誤りについて" class="wiki-anchor">¶</a></h3>
<p>今回指摘されている問題とは異なりますが、「副管理者から降格」に対応する英文<pre>
"Do you demotion %0% from this %community%'s sub-administrator?" // %0% には副管理者の名前が、%community% にはコミュニティ名が入ります。
</pre>において demotion は名詞なので、これは demote に置き換えるべきと思われます。</p>
<p>修正すべき箇所は、</p>
<ul>
<li>apps/mobile_frontend/i18n/messages.ja.xml</li>
<li>apps/mobile_frontend/modules/community/templates/memberManageSuccess.php</li>
<li>apps/mobile_frontend/modules/community/templates/removeSubAdminInput.php</li>
<li>apps/pc_frontend/i18n/messages.ja.xml</li>
<li>apps/pc_frontend/modules/community/templates/memberManageSuccess.php</li>
<li>apps/pc_frontend/modules/community/templates/removeSubAdminInput.php</li>
</ul>
<p>になります。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9108
2011-06-27T06:30:25Z
Shingo Yamada
s.yamada@tejimaya.com
<ul><li><strong>担当者</strong> を <i>Naoya Tozuka</i> にセット</li></ul>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9109
2011-06-27T06:32:42Z
Naoya Tozuka
tozuka@tejimaya.com
<ul></ul><blockquote>
<p>takai: @tozuka その間違った文言のコミットに対して、修正すべきは戸塚さんが指摘した文言のみでしょうか。もし間違った文言が他にもあるのであれば全面的に修正した方が好ましい気がしました。間違った文言のコミットは <a class="changeset" title="added an ability for community's sub-administrator (refs #228)" href="http://redmine.openpne.jp/projects/op3/repository/revisions/43ae2a74d62274a26820484f89f34214551ddfd2">43ae2a74</a> らしいですね(masterブランチ)。 <a class="external" href="http://redmine.openpne.jp/issues/228">http://redmine.openpne.jp/issues/228</a> の4個目のコミット。</p>
<p>tozuka: @takai なるほど( <a class="changeset" title="added an ability for community's sub-administrator (refs #228)" href="http://redmine.openpne.jp/projects/op3/repository/revisions/43ae2a74d62274a26820484f89f34214551ddfd2">43ae2a74</a> みました)。「副管理者から降格」に射影される英文が2つあるので両方辿ってみます。</p>
</blockquote>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9110
2011-06-27T06:56:11Z
Naoya Tozuka
tozuka@tejimaya.com
<ul></ul><p><del>更新履歴 commit:"054285bf3cd48d0f963e8becc51914e24afcc13a" で適用されました。</del> → <a class="changeset" title="corrected some text to use 'demote' instead of 'demotion' where the verb is required; (fixes #2058)" href="http://redmine.openpne.jp/projects/op3/repository/revisions/043cb4ac14f4bc76616ff02c7db1d2c8d1392975">043cb4ac</a> に統合のため削除</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9111
2011-06-27T06:56:11Z
Naoya Tozuka
tozuka@tejimaya.com
<ul></ul><p><del>更新履歴 commit:"dd065c59f1674bbf5bf0d26e7f87609c10838587" で適用されました。</del> → <a class="changeset" title="corrected some text to use 'demote' instead of 'demotion' where the verb is required; (fixes #2058)" href="http://redmine.openpne.jp/projects/op3/repository/revisions/043cb4ac14f4bc76616ff02c7db1d2c8d1392975">043cb4ac</a> に統合のため削除</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9112
2011-06-27T07:00:07Z
Naoya Tozuka
tozuka@tejimaya.com
<ul></ul><p>高井さんの指摘を受け、 <a class="changeset" title="corrected some text to use 'demote' instead of 'demotion' where the verb is required; (fixes #2058)" href="http://redmine.openpne.jp/projects/op3/repository/revisions/043cb4ac14f4bc76616ff02c7db1d2c8d1392975">043cb4ac</a> にて他の間違った文言も修正しました。</p>
<p>レビューお願いします。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9119
2011-06-27T08:49:18Z
Minoru Takai
main.coeurl@gmail.com
<ul><li><strong>ステータス</strong> を <i>Pending Review(レビュー待ち)</i> から <i>Rejected(差し戻し)</i> に変更</li></ul><a name="このチケットに対して"></a>
<h3 >このチケットに対して<a href="#このチケットに対して" class="wiki-anchor">¶</a></h3>
<p>このチケット自体をレビューします。</p>
<blockquote>
<p>3. 適当なメンバーで「退会させる」を押下:/community/removeSubAdmin/id/19/member_id/2</p>
</blockquote>
<p>これは『3. 副管理者に対して「副管理者から降格」を押下』の誤りでしょうか。</p>
<blockquote>
<a name="原因"></a>
<h3 >原因<a href="#原因" class="wiki-anchor">¶</a></h3>
<p>get の form のaction では action="URL?id=○○" のパラメータが引き継げないので URL? ページに遷移してしまう</p>
</blockquote>
<p>『get の form のaction では action="URL?id=○○" のパラメータが引き継げない』この理由・原因について可能であれば、コメントかチケットに記述してください。修正方針が妥当かどうか、他の箇所について修正する必要がないかを判断するのに有用です。</p>
<a name="note-3-での修正内容について"></a>
<h3 >note-3 での修正内容について<a href="#note-3-での修正内容について" class="wiki-anchor">¶</a></h3>
<blockquote>
<a name="修正内容"></a>
<h3 >修正内容<a href="#修正内容" class="wiki-anchor">¶</a></h3>
<p>inputタグでパラメータを引き継ぐように修正。</p>
</blockquote>
<p>この修正方針が妥当であれば、修正内容は問題ないと思います。</p>
<pre>
- 'no_url' => url_for('@community_memberManage?id='.$community->getId()),
+ 'no_url' => url_for('@community_memberManage'),
'no_method' => 'get',
+ 'no_form' => '<input type="hidden" name="id" value="'.$community->getId().'"/>',
)) ?>
</pre>
<p>PC版の修正について、追加している input 要素タグの末尾のスラッシュの直前に空白がありませんが、本質的な問題ではないためこの修正内容で問題ないと思います。</p>
<a name="note-10-での修正内容について"></a>
<h3 >note-10 での修正内容について<a href="#note-10-での修正内容について" class="wiki-anchor">¶</a></h3>
<p>修正内容自体はOKです。</p>
<p>チケット自体に文言について修正する旨が記述されていないため、note-3 の修正に併せて、文言についての修正も扱ったことが分かるようチケットを書き換える必要がありそうです。</p>
<p>また、バックポートチケットでも同じ修正を取り込むか、あるいは取り込まないことが分かるようになっているべきです。3.4.x 向けのBPチケット <a class="issue tracker-4 status-12 priority-4 priority-default closed" title="Backport(バックポート): コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる (Invalid(無効))" href="http://redmine.openpne.jp/issues/2063">#2063</a> は、この親チケットが完了する前にクローズされているため、 note-10 での修正は <a class="issue tracker-4 status-12 priority-4 priority-default closed" title="Backport(バックポート): コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる (Invalid(無効))" href="http://redmine.openpne.jp/issues/2063">#2063</a> の派生チケット <a class="issue tracker-1 status-12 priority-4 priority-default closed" title="Bug(バグ): 特定の端末によってはフォームのaction属性内のパラメータが引き継げずにエラー画面に遷移する (Invalid(無効))" href="http://redmine.openpne.jp/issues/2093">#2093</a> で扱うようにした方がよいかと思います。</p>
<p>一旦、担当者に差し戻します。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9126
2011-06-27T12:15:33Z
Minoru Takai
main.coeurl@gmail.com
<ul><li><strong>ステータス</strong> を <i>Rejected(差し戻し)</i> から <i>Accepted(着手)</i> に変更</li><li><strong>担当者</strong> を <i>Naoya Tozuka</i> から <i>Minoru Takai</i> に変更</li></ul><p>一先ず、チケットの概要(現象・原因・修正内容)を書き換えました。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9132
2011-06-28T05:21:14Z
Minoru Takai
main.coeurl@gmail.com
<ul></ul><p>action 属性値のクエリストリングを input 要素に渡す方針から、ルーティング定義を適切なものにする修正方針に改めます。また、原因を見れば明らかですが、この問題はルーティング定義が適切でないことに因るものであり、 <a class="issue tracker-1 status-12 priority-4 priority-default closed" title="Bug(バグ): Becomes an access error when selected the "no" button in the member of community coercion leave f... (Invalid(無効))" href="http://redmine.openpne.jp/issues/1543">#1543</a> と同一の問題(つまり <a class="issue tracker-1 status-12 priority-4 priority-default closed" title="Bug(バグ): Becomes an access error when selected the "no" button in the member of community coercion leave f... (Invalid(無効))" href="http://redmine.openpne.jp/issues/1543">#1543</a> と <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug(バグ): コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる (Fixed(完了))" href="http://redmine.openpne.jp/issues/2058">#2058</a> の問題が同時に解消するもの)です。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9139
2011-06-28T08:19:00Z
Minoru Takai
main.coeurl@gmail.com
<ul></ul><p>note-10 の文言に関する修正についてですが、チケットの関連をより分かりやすくするために別チケット対応としました。文言の修正については <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug(バグ): コミュニティの副管理者降格画面の文言が不適切である (Fixed(完了))" href="http://redmine.openpne.jp/issues/2250">#2250</a> で扱います。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9145
2011-06-28T09:25:31Z
Minoru Takai
main.coeurl@gmail.com
<ul><li><strong>題名</strong> を <i>コミュニティの副管理者降格画面(/community/removeSubAdmin/id/:id/member_id/:id)で「いいえ」を選ぶとアクセスエラーになる</i> から <i>コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる</i> に変更</li></ul>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9146
2011-06-28T09:45:08Z
Minoru Takai
main.coeurl@gmail.com
<ul><li><strong>ステータス</strong> を <i>Accepted(着手)</i> から <i>Pending Review(レビュー待ち)</i> に変更</li></ul><p>更新履歴 <a class="changeset" title="(fixed #2058) redefined routing url of @community_memberManage to contain id parameter into url." href="http://redmine.openpne.jp/projects/op3/repository/revisions/81b179d454f13ffed2dc02ccf4aa72905cb777ca">81b179d454f13ffed2dc02ccf4aa72905cb777ca</a> で適用されました。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9153
2011-06-28T10:53:38Z
Naoya Tozuka
tozuka@tejimaya.com
<ul><li><strong>ステータス</strong> を <i>Pending Review(レビュー待ち)</i> から <i>Pending Testing(テスト待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>50</i> から <i>70</i> に変更</li></ul><p>新しい修正方針に基づいた修正を確認しました。レビューOKです。<br />チケットの整理どうもありがとうございます。</p>
OpenPNE 3 - Bug(バグ) #2058: コミュニティのメンバー管理画面に戻るためのYes/Noフォームで「いいえ」を選ぶとアクセスエラーになる
http://redmine.openpne.jp/issues/2058?journal_id=9440
2011-07-08T11:15:02Z
Minoru Takai
main.coeurl@gmail.com
<ul><li><strong>ステータス</strong> を <i>Pending Testing(テスト待ち)</i> から <i>Fixed(完了)</i> に変更</li><li><strong>進捗率</strong> を <i>70</i> から <i>100</i> に変更</li></ul><a name="動作テスト"></a>
<h3 >動作テスト<a href="#動作テスト" class="wiki-anchor">¶</a></h3>
<p>実装時に動作テストを行っており、原理的にもこの修正による副作用はないものと判断しています。</p>
<p>このチケットで挙げられた当初の問題点については改善できていることが確認できており、修正内容も妥当であると判断しているため、動作上も問題がないと判断します。</p>
<p>完了とします。</p>