blitze / phpBB-ext-sitemaker

model/menus/entity/item.php

Doc Comments +4 added lines, -1 removed lines

@@ -184,7 +184,7 @@ 
 	 * Checks if a url is an extension route
 	 *
 	 * @param string $item_url
-	 * @return true|false
+	 * @return boolean
 	 */
 	private function is_extension_route($item_url)
 	{
@@ -195,6 +195,9 @@ 
 		return ($host || $extension || is_dir($item_url) || $is_absolute_path) ? false : true;
 	}
 
+	/**
+	 * @param string $item_url
+	 */
 	private function add_board_url($item_url)
 	{
 		return $this->board_url . ($item_url[0] === '/' ? '' : '/') . $item_url;

Spacing +4 added lines, -4 removed lines

@@ -136,7 +136,7 @@ 
 	 */
 	public function set_item_icon($icon)
 	{
-		$this->item_icon = ($icon) ? trim($icon) . ' ' : '';
+		$this->item_icon = ($icon) ? trim($icon).' ' : '';
 		return $this;
 	}
 
@@ -146,14 +146,14 @@ 
 	 */
 	public function set_item_url($item_url)
 	{
-		$this->item_url = str_replace($this->board_url . '/', '', $item_url);
+		$this->item_url = str_replace($this->board_url.'/', '', $item_url);
 
 		// to make things uniform and easy to switch between mod_rewrite_enabled or not without
 		// having to edit menu items, we add app.php/ for all extension routes
 		if (strpos($item_url, 'app.php') === false && $this->is_extension_route($item_url))
 		{
 			$this->item_url = ltrim($this->item_url, './');
-			$this->item_url = '/app.php/' . $this->item_url;
+			$this->item_url = '/app.php/'.$this->item_url;
 		}
 
 		return $this;
@@ -197,6 +197,6 @@ 
 
 	private function add_board_url($item_url)
 	{
-		return $this->board_url . ($item_url[0] === '/' ? '' : '/') . $item_url;
+		return $this->board_url.($item_url[0] === '/' ? '' : '/').$item_url;
 	}
 }