「メンバーの編集(editmembersettings)」画面でスキン変数<%pluginextras%>がパースされていなかった不具合を修正

[nucleus-jp/nucleus-next.git] / nucleus / libs / AdminActions.php

diff --git a/nucleus/libs/AdminActions.php b/nucleus/libs/AdminActions.php
index 73b1b08..15e62ad 100644 (file)
--- a/nucleus/libs/AdminActions.php
+++ b/nucleus/libs/AdminActions.php
@@ -470,6 +470,7 @@ class AdminActions extends BaseActions
                                'pluginoptions',
                                'defadminskinselect',
                                'defbookmarkletselect',
+                               'pluginextras',
                                );
                                break;
                        case 'forgotpassword':
@@ -700,7 +701,8 @@ class AdminActions extends BaseActions
                if ( $resource->rowCount() > 0 )
                {
                        $template['content'] = 'actionlist';
-                       $this->parser->parse(showlist($resource, 'table', $template, $template_name));
+                       $action_list = showlist($resource, 'table', $template, $template_name);
+                       $this->parser->parse($action_list);
                }
                else
                {
@@ -854,7 +856,8 @@ class AdminActions extends BaseActions
                if ( $resource->rowCount() > 0 )
                {
                        $template['content'] = 'banlist';
-                       $this->parser-parse(showlist($resource, 'table', $template, $template_name));
+                       $ban_list = showlist($resource, 'table', $template, $template_name);
+                       $this->parser-parse($ban_list);
                }
                else
                {
@@ -1213,7 +1216,8 @@ class AdminActions extends BaseActions
                        $template['extra'] = Entity::hsc(_MEMBERS_USESITELANG);
                }
                
-               $this->parser->parse(showlist($query, 'select', $template, $template_name));
+               $skin_select = showlist($query, 'select', $template, $template_name);
+               $this->parser->parse($skin_select);
                return;
        }
        
@@ -1257,7 +1261,8 @@ class AdminActions extends BaseActions
                        $template['extra'] = Entity::hsc(_MEMBERS_USESITELANG);
                }
                
-               $this->parser->parse(showlist($query, 'select', $template, $template_name));
+               $bookmarklet_select = showlist($query, 'select', $template, $template_name);
+               $this->parser->parse($bookmarklet_select);
                return;
        }
        
@@ -1369,7 +1374,8 @@ class AdminActions extends BaseActions
                {
                        $template['content']  = 'categorylist';
                        $template['tabindex'] = 200;
-                       $this->parser->parse(listplug_batchlist('category', $resource, 'table', $template, $template_name));
+                       $category_list = listplug_batchlist('category', $resource, 'table', $template, $template_name);
+                       $this->parser->parse($category_list);
                }
                else
                {
@@ -1511,8 +1517,9 @@ class AdminActions extends BaseActions
                {
                        $template['content']  = 'teamlist';
                        $template['tabindex'] = 10;
-                               
-                       $this->parser->parse(listplug_batchlist('team', $resource, 'table', $template, $template_name));
+                       
+                       $team_list = listplug_batchlist('team', $resource, 'table', $template, $template_name);
+                       $this->parser->parse($team_list);
                }
                else
                {
@@ -1971,7 +1978,8 @@ class AdminActions extends BaseActions
                $template['tabindex'] = 10;
                $template['skinid'] = intRequestVar('skinid');
                $template['skinname'] = $skin->getName();
-               $this->parser->parse(showlist($normal_skintype, 'list_normalskinlist', $template, $template_name));
+               $skin_list = showlist($normal_skintype, 'list_normalskinlist', $template, $template_name);
+               $this->parser->parse($skin_list);
                
                return;
        }
@@ -1993,7 +2001,8 @@ class AdminActions extends BaseActions
                $template['name'] = 'DefaultBlog';
                $template['selected'] = $CONF['DefaultBlog'];
                $template['tabindex'] = 10;
-               $this->parser->parse(showlist($query, 'select', $template, $template_name));
+               $blog_select = showlist($query, 'select', $template, $template_name);
+               $this->parser->parse($blog_select);
                
                return;
        }
@@ -2019,7 +2028,8 @@ class AdminActions extends BaseActions
                $template['selected'] = $blog->getDefaultCategory();
                $template['tabindex'] = 110;
                
-               $this->parser->parse(showlist($query, 'select', $template, $template_name));
+               $category_select = showlist($query, 'select', $template, $template_name);
+               $this->parser->parse($category_select);
                
                return;
        }
@@ -2071,7 +2081,8 @@ class AdminActions extends BaseActions
                $query = sprintf($query, sql_table('skin_desc'));
                $template['tabindex'] = 50;
                
-               $this->parser->parse(showlist($query, 'select', $template, $template_name));
+               $skin_select = showlist($query, 'select', $template, $template_name);
+               $this->parser->parse($skin_select);
                
                return;
        }
@@ -2247,8 +2258,9 @@ class AdminActions extends BaseActions
                {
                        $template['content'] = 'memberlist';
                        $template['tabindex'] = 10;
-                               
-                       $this->parser->parse(listplug_batchlist('member', $resource, 'table', $template, $template_name));
+                       
+                       $member_list = listplug_batchlist('member', $resource, 'table', $template, $template_name);
+                       $this->parser->parse($member_list);
                }
                else
                {
@@ -2405,7 +2417,8 @@ class AdminActions extends BaseActions
                if ( sizeof($aOptions) > 0 )
                {
                        $template['content'] = 'plugoptionlist';
-                       $this->parser->parse(showlist($aOptions, 'table', $template, $template_name));
+                       $option_list = showlist($aOptions, 'table', $template, $template_name);
+                       $this->parser->parse($option_list);
                }
                else
                {
@@ -2611,11 +2624,13 @@ class AdminActions extends BaseActions
                                $pluginfields = array();
                                if ( !in_array($this->skintype, Admin::$adminskin_actions) )
                                {
-                                       $manager->notify('TemplateExtraFields', array('fields' => &$pluginfields));
+                                       $data = array('fields' => &$pluginfields);
+                                       $manager->notify('TemplateExtraFields', $data);
                                }
                                else
                                {
-                                       $manager->notify('AdminTemplateExtraFields', array('fields' => &$pluginfields));
+                                       $data = array('fields' => &$pluginfields);
+                                       $manager->notify('AdminTemplateExtraFields', $data);
                                }
                                
                                foreach ( $pluginfields as $ptkey => $ptvalue )
@@ -2652,7 +2667,8 @@ class AdminActions extends BaseActions
                                                {
                                                        $content = $template[$ptname];
                                                }
-                                               $this->parser->parse(listplug_templateEditRow($content, $ptdesc, $ptname, $help, $tabidx++, $big, $template_name));
+                                               $tempate_textarea = listplug_templateEditRow($content, $ptdesc, $ptname, $help, $tabidx++, $big, $template_name);
+                                               $this->parser->parse($tempate_textarea);
                                                continue;
                                        }
                                }
@@ -2669,7 +2685,8 @@ class AdminActions extends BaseActions
                                {
                                        $content = $template[$typename];
                                }
-                               $this->parser->parse(listplug_templateEditRow($content, $typedesc, $typename, $help, $tabindex, $big, $template_name));
+                               $tempate_textarea = listplug_templateEditRow($content, $typedesc, $typename, $help, $tabindex, $big, $template_name);
+                               $this->parser->parse($tempate_textarea);
                                break;
                }
                
@@ -3155,7 +3172,8 @@ class AdminActions extends BaseActions
         */
        public function parse_inputyesno($name, $checkedval, $tabindex = 0, $value1 = 1, $value2 = 0, $yesval = _YES, $noval = _NO, $isAdmin = 0, $template_name = '')
        {
-               $this->parser->parse(listplug_input_yesno($name, $checkedval, $tabindex, $value1, $value2, $yesval, $noval, $isAdmin, $template_name));
+               $input_yesno = listplug_input_yesno($name, $checkedval, $tabindex, $value1, $value2, $yesval, $noval, $isAdmin, $template_name);
+               $this->parser->parse($input_yesno);
                return;
        }
        
@@ -3591,7 +3609,8 @@ class AdminActions extends BaseActions
                        'tabindex'      => 10000,
                        'selected'      => 0
                );
-               $this->parser->parse(showlist($query, 'select', $template, $template_name));
+               $member_select = showlist($query, 'select', $template, $template_name);
+               $this->parser->parse($member_select);
                return;
        }
        
@@ -3673,12 +3692,14 @@ class AdminActions extends BaseActions
                        case 'member':
                                $id  = intRequestVar('memberid');
                                $mem =& $manager->getMember($id);
-                               $manager->notify('MemberSettingsFormExtras', array('member' => &$mem));
+                               $data = array('member' => &$mem);
+                               $manager->notify('MemberSettingsFormExtras', $data);
                                break;
                        case 'blog':
                                $id  = intRequestVar('blogid');
                                $blg =& $manager->getBlog($id);
-                               $manager->notify('BlogSettingsFormExtras', array('member' => &$blg));
+                               $data = array('member' => &$blg);
+                               $manager->notify('BlogSettingsFormExtras', $data);
                                break;
                        case 'createaccount':
                                $data = array(
@@ -3691,7 +3712,8 @@ class AdminActions extends BaseActions
                                $manager->notify('RegistrationFormExtraFields', $data);
                                break;
                        default:
-                               $manager->notify('GeneralSettingsFormExtras', array());
+                       $data = array();
+                               $manager->notify('GeneralSettingsFormExtras', $data);
                                break;
                }
                return;
@@ -3746,7 +3768,8 @@ class AdminActions extends BaseActions
                $template['content']  = 'pluginlist';
                $template['tabindex'] = 10;
                
-               $this->parser->parse(showlist($query, 'table', $template, $template_name));
+               $plugin_list = showlist($query, 'table', $template, $template_name);
+               $this->parser->parse($plugin_list);
                
                return;
        }
@@ -3929,7 +3952,8 @@ class AdminActions extends BaseActions
                        $templates = & $manager->getTemplate($template_name);
                }
                $pluginExtras = array();
-               $manager->notify('QuickMenu', array('options' => &$pluginExtras));
+               $data = array('options' => &$pluginExtras);
+               $manager->notify('QuickMenu', $data);
                
                $template  = array();
                if ( count($pluginExtras) > 0 )
@@ -3970,7 +3994,8 @@ class AdminActions extends BaseActions
                                        'plugadmintooltip'      => Entity::hsc($aInfo['tooltip']),
                                        'plugadmintitle'        => Entity::hsc($aInfo['title']),
                                );
-                               $this->parser->parse(Template::fill($template['body'], $data));
+                               $body = Template::fill($template['body'], $data);
+                               $this->parser->parse($body);
                        }
                        $this->parser->parse($template['foot']);
                }
@@ -4087,8 +4112,9 @@ class AdminActions extends BaseActions
                                $query = sprintf($query, sql_table('template_desc'));
                                break;
                }
-       
-               $this->parser->parse(showlist($query, 'table', $show, $template_name));
+               
+               $skin_list = showlist($query, 'table', $show, $template_name);
+               $this->parser->parse($skin_list);
                return;
        }
        
@@ -4207,7 +4233,8 @@ class AdminActions extends BaseActions
                $template['tabindex'] = 10;
                $template['content'] = 'skinlist';
                
-               $this->parser->parse(showlist($query, 'table', $template, $template_name));
+               $skin_list = showlist($query, 'table', $template, $template_name);
+               $this->parser->parse($skin_list);
                
                return;
        }
@@ -4290,7 +4317,8 @@ class AdminActions extends BaseActions
                        $template['tabindex'] = 75;
                        $template['skinid'] = $skin->getID();
                        $template['skinname'] = $skin->getName();
-                       $this->parser->parse(showlist($special_skintypes, 'list_specialskinlist', $template, $template_name));
+                       $skin_list = showlist($special_skintypes, 'list_specialskinlist', $template, $template_name);
+                       $this->parser->parse($skin_list);
                }
                else
                {
@@ -4496,7 +4524,8 @@ class AdminActions extends BaseActions
                $template['tabindex'] = 10;
                $template['content'] = 'templatelist';
                
-               $this->parser->parse(showlist($query, 'table', $template, $template_name));
+               $template_list = showlist($query, 'table', $template, $template_name);
+               $this->parser->parse($template_list);
                
                return;
        }
@@ -4588,7 +4617,8 @@ class AdminActions extends BaseActions
                        if ( $resource->rowCount() > 0 )
                        {
                                $template['content'] = 'draftlist';
-                               $this->parser->parse(showlist($resource, 'table', $template, $template_name));
+                               $draft_list = showlist($resource, 'table', $template, $template_name);
+                               $this->parser->parse($draft_list);
                        }
                        else
                        {
@@ -5214,8 +5244,9 @@ class AdminActions extends BaseActions
        public function parse_pagefoot()
        {
                global $action, $member, $manager, $blogid;
-               
-               $manager->notify('AdminPrePageFoot', array('action' => Admin::$action));
+
+               $data = array('action' => Admin::$action);
+               $manager->notify('AdminPrePageFoot', $data);
                
                $content = $this->parser->skin->getContentFromDB('pagefoot');
                if ( !$content )