Project

General

Profile

Bug(バグ) #2850

「コミュニティ管理者の承認が必要」な状態でコミュニティ参加申請時を開いてもメッセージを記入するフォームが表示されない

Added by Yuya Watanabe over 9 years ago. Updated over 9 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2012-02-28
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
[QA]バグ通知済:
No
3.8 で発生するか:
Unknown (未調査)

Description

概要

「コミュニティ管理者の承認が必要」な状態でコミュニティ参加申請時(community/join?id=コミュニティID)を開いてもメッセージを記入するフォームが表示されない.

原因

OpenPNE 本体側での修正 #550 「Typo in community config: "policy" is misspelled "poricy" (コミュニティ設定で policy を poricy と間違っている箇所がある)」
でDBの列名が変更されたはずだが,opMessagePlugin ではまだ修正されていないため.

この修正は OpenPNE 3.5 以降のもののため, BP として 対応する必要はないはず.

lib/opMessagePluginObserver.class.php

 90       $community = $arguments['actionInstance']->community;
 91       if ('close' !== $community->getConfig('register_poricy'))
 92       {
 93         return false;
 94       }
...
111     else
112     {
113       $community = $arguments['actionInstance']->community;
114       if ('close' !== $community->getConfig('register_poricy'))
115       {
116         // Injected message field is not useful in this community
117         unset($arguments['actionInstance']->form['message']);
118       }
119     }

修正案

diff --git a/lib/opMessagePluginObserver.class.php b/lib/opMessagePluginObserver.class.php
index 287b680..7c4ef43 100644
--- a/lib/opMessagePluginObserver.class.php
+++ b/lib/opMessagePluginObserver.class.php
@@ -99,7 +99,7 @@ class opMessagePluginObserver
     if ($arguments['result'] == sfView::SUCCESS)
     {
       $community = $arguments['actionInstance']->community;
-      if ('close' !== $community->getConfig('register_poricy'))
+      if ('close' !== $community->getConfig('register_policy'))
       {
         return false;
       }
@@ -122,7 +122,7 @@ class opMessagePluginObserver
     else
     {
       $community = $arguments['actionInstance']->community;
-      if ('close' !== $community->getConfig('register_poricy'))
+      if ('close' !== $community->getConfig('register_policy'))
       {
         // Injected message field is not useful in this community
         unset($arguments['actionInstance']->form['message']);

Related issues

Related to OpenPNE 3 - Bug(バグ) #550: Typo in community config: "policy" is misspelled "poricy" (コミュニティ設定で policy を poricy と間違っている箇所がある) Fixed(完了) 2010-01-31

Associated revisions

Revision dd3328a0 (diff)
Added by Maki Takahashi over 9 years ago

(fixes #2850) fixed typo that changed "register poricy" to "register policy"

History

#1 Updated by Yuya Watanabe over 9 years ago

  • Description updated (diff)

#2 Updated by Maki Takahashi over 9 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Maki Takahashi

#3 Updated by Maki Takahashi over 9 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#4 Updated by Yuya Watanabe over 9 years ago

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

レビューOKです.

#5 Updated by Yuma Sakata over 9 years ago

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

テストOKです。

Also available in: Atom PDF