Project

General

Profile

Actions

Bug(バグ) #2850

closed

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

Added by Yuya Watanabe almost 13 years ago. Updated almost 13 years ago.

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

100%

Estimated time:
3.6 で発生するか:
Yes (はい)
[QA]バグ通知済:
No
3.8 で発生するか:

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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Yuya Watanabe almost 13 years ago

  • Description updated (diff)
Actions #2

Updated by Maki Takahashi almost 13 years ago

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

Updated by Maki Takahashi almost 13 years ago

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

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

Actions #4

Updated by Yuya Watanabe almost 13 years ago

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

レビューOKです.

Actions #5

Updated by Yuma Sakata almost 13 years ago

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

テストOKです。

Actions

Also available in: Atom PDF