Move My Ideas link to user drop down

Author: iMattPro

controller/base.php

@@ -13,7 +13,6 @@
 use phpbb\auth\auth;
 use phpbb\config\config;
 use phpbb\controller\helper;
-use phpbb\ideas\ext;
 use phpbb\ideas\factory\linkhelper;
 use phpbb\language\language;
 use phpbb\pagination;
@@ -151,8 +150,6 @@ protected function display_common_vars()
 			'S_DISPLAY_SEARCHBOX'	=> $this->auth->acl_get('u_search') && $this->auth->acl_get('f_search', $this->config['ideas_forum_id']) && $this->config['load_search'],
 			'S_SEARCHBOX_ACTION'	=> append_sid("{$this->root_path}search.$this->php_ext"),
 			'S_SEARCH_IDEAS_HIDDEN_FIELDS'	=> build_hidden_fields(['fid' => [$this->config['ideas_forum_id']]]),
-
-			'U_SEARCH_MY_IDEAS' 	=> $this->helper->route('phpbb_ideas_list_controller', ['sort' => ext::SORT_MYIDEAS, 'status' => '-1']),
 		]);
 	}
 }

event/listener.php

@@ -82,6 +82,7 @@ public function __construct(auth $auth, config $config, helper $helper, idea $id
 	public static function getSubscribedEvents()
 	{
 		return array(
+			'core.page_header'							=> 'global_template_vars',
 			'core.viewforum_get_topic_data'				=> 'ideas_forum_redirect',
 			'core.viewtopic_modify_post_row'			=> 'show_post_buttons',
 			'core.viewtopic_modify_page_title'			=> 'show_idea',
@@ -93,6 +94,22 @@ public static function getSubscribedEvents()
 		);
 	}
 
+	/**
+	 * Assign global template variables
+	 *
+	 * @return void
+	 */
+	public function global_template_vars()
+	{
+		if ($this->user->data['is_registered'] && !$this->user->data['is_bot'])
+		{
+			$this->template->assign_var(
+				'U_SEARCH_MY_IDEAS',
+				$this->helper->route('phpbb_ideas_list_controller', ['sort' => ext::SORT_MYIDEAS, 'status' => '-1'])
+			);
+		}
+	}
+
 	/**
 	 * Redirect users from the forum to the Ideas centre
 	 *
@@ -223,7 +240,6 @@ public function show_idea($event)
 			'U_IDEA_VOTE'		=> $this->link_helper->get_idea_link($idea['idea_id'], 'vote', true),
 			'U_IDEA_DUPLICATE'	=> $this->link_helper->get_idea_link($idea['duplicate_id']),
 			'U_IDEA_STATUS_LINK'=> $this->helper->route('phpbb_ideas_list_controller', ['status' => $idea['idea_status']]),
-			'U_SEARCH_MY_IDEAS' => $this->helper->route('phpbb_ideas_list_controller', ['sort' => ext::SORT_MYIDEAS, 'status' => '-1']),
 			'U_TITLE_LIVESEARCH'=> $this->helper->route('phpbb_ideas_livesearch_controller'),
 		));
 

language/en/common.php

@@ -54,7 +54,7 @@
 
 	'LATEST_IDEAS'			=> 'Latest Ideas',
 	'LIST_DUPLICATE'		=> 'Duplicate ideas',
-	'LIST_EGOSEARCH'		=> 'Your ideas',
+	'LIST_EGOSEARCH'		=> 'My ideas',
 	'LIST_IMPLEMENTED'		=> 'Implemented ideas',
 	'LIST_IN_PROGRESS'		=> 'In Progress ideas',
 	'LIST_INVALID'			=> 'Invalid ideas',

styles/prosilver/template/event/navbar_header_profile_list_after.html

@@ -0,0 +1,7 @@
+{% if U_SEARCH_MY_IDEAS %}
+	<li>
+		<a href="{{ U_SEARCH_MY_IDEAS }}" title="{{ lang('LIST_EGOSEARCH') }}" role="menuitem">
+			<i class="icon fa-lightbulb-o fa-fw" aria-hidden="true"></i><span>{{ lang('LIST_EGOSEARCH') }}</span>
+		</a>
+	</li>
+{% endif %}

styles/prosilver/template/event/navbar_header_quick_links_before.html

@@ -106,6 +106,7 @@ public function test_construct()
 	public function test_getSubscribedEvents()
 	{
 		self::assertEquals(array(
+			'core.page_header',
 			'core.viewforum_get_topic_data',
 			'core.viewtopic_modify_post_row',
 			'core.viewtopic_modify_page_title',
@@ -117,6 +118,36 @@ public function test_getSubscribedEvents()
 		), array_keys(\phpbb\ideas\event\listener::getSubscribedEvents()));
 	}
 
+	public function global_template_vars_data()
+	{
+		return [
+			'registered user' => [true, false, true],
+			'unregistered user' => [false, false, false],
+			'is bot user' => [true, true, false],
+			'is bot guest' => [false, true, false],
+		];
+	}
+
+	/**
+	 * @dataProvider global_template_vars_data
+	 */
+	public function test_global_template_vars($is_registered, $is_bot, $expected)
+	{
+		$this->user->data['is_registered'] = $is_registered;
+		$this->user->data['is_bot'] = $is_bot;
+
+		$this->helper->expects($expected ? $this->once() : $this->never())
+			->method('route')
+			->willReturn('phpbb_ideas_list_controller');
+
+		$this->template->expects($expected ? $this->once() : $this->never())
+			->method('assign_var')
+			->with('U_SEARCH_MY_IDEAS', 'phpbb_ideas_list_controller');
+
+		$listener = $this->get_listener();
+		$listener->global_template_vars();
+	}
+
 	public function show_idea_data()
 	{
 		return [