refactored based on comments (2nd round)

framework/yii/bootstrap/Nav.php

@@ -8,7 +8,7 @@
 namespace yii\bootstrap;
 
 use yii\base\InvalidConfigException;
-use yii\helpers\base\ArrayHelper;
+use yii\helpers\ArrayHelper;
 use yii\helpers\Html;
 
 /**
@@ -22,12 +22,12 @@ use yii\helpers\Html;
  *         array(
  *             'label' => 'Home',
  *             'url' => '/',
- *             'options' => array(...),
+ *             'linkOptions' => array(...),
  *             'active' => true,
  *         ),
  *         array(
  *             'label' => 'Dropdown',
- *             'dropdown' => array(
+ *             'items' => array(
  *                  array(
  *                      'label' => 'DropdownA',
  *                      'url' => '#',
@@ -52,26 +52,25 @@ class Nav extends Widget
 	 * @var array list of items in the nav widget. Each array element represents a single
 	 * menu item with the following structure:
 	 *
-	 * ```php
+	 * - label: string, required, the nav item label.
-	 * array(
+	 * - url: optional, the item's URL. Defaults to "#".
-	 *     // required, the menu item label.
+	 * - linkOptions: array, optional, the HTML attributes of the item's link.
-	 *     'label' => 'Nav item label',
+	 * - options: array, optional, the HTML attributes of the item container (LI).
-	 *     // optional, the URL of the menu item. Defaults to "#"
+	 * - active: boolean, optional, whether the item should be on active state or not.
-	 *     'url'=> '#',
+	 * - items: array, optional, the configuration of specify the item's dropdown menu. You can optionally set this as
-	 *     // optional, the HTML options of the URL.
+	 *   a string (ie. `'items'=> Dropdown::widget(array(...))`
-	 *     'urlOptions' => array(...),
+	 *   - important: there is an issue with sub-dropdown menus, and as of 3.0, bootstrap won't support sub-dropdown.
-	 *     // optional the HTML attributes of the item container (LI).
-	 *     'options' => array(...),
-	 *     // optional, an array of [[Dropdown]] widget items so to display a dropdown menu on the tab header.
-	 *     // important: there is an issue with sub-dropdown menus, and as of 3.0, bootstrap won't support sub-dropdown
-	 *     // @see https://github.com/twitter/bootstrap/issues/5050#issuecomment-11741727
-	 *     'dropdown'=> array(...)
-	 * )
-	 * ```
 	 *
-	 * Optionally, you can also use a plain string instead of an array element.
+	 * **Note:** Optionally, you can also use a plain string instead of an array element.
+	 *
+	 * @see https://github.com/twitter/bootstrap/issues/5050#issuecomment-11741727
+	 * @see [[Dropdown]]
 	 */
 	public $items = array();
+	/**
+	 * @var boolean whether the nav items labels should be HTML-encoded.
+	 */
+	public $encodeLabels = true;
 
 
 	/**
@@ -79,6 +78,7 @@ class Nav extends Widget
 	 */
 	public function init()
 	{
+		parent::init();
 		$this->addCssClass($this->options, 'nav');
 	}
 
@@ -88,6 +88,7 @@ class Nav extends Widget
 	public function run()
 	{
 		echo $this->renderItems();
+		$this->registerPlugin('dropdown');
 	}
 
 	/**
@@ -117,21 +118,26 @@ class Nav extends Widget
 		if (!isset($item['label'])) {
 			throw new InvalidConfigException("The 'label' option is required.");
 		}
-		$label = $item['label'];
+		$label = $this->encodeLabels ? Html::encode($item['label']) : $item['label'];
-		$url = ArrayHelper::getValue($item, 'url', '#');
 		$options = ArrayHelper::getValue($item, 'options', array());
-		$urlOptions = ArrayHelper::getValue($item, 'urlOptions', array());
+		$dropdown = ArrayHelper::getValue($item, 'items');
-		$dropdown = null;
+		$url = Html::url(ArrayHelper::getValue($item, 'url', '#'));
+		$linkOptions = ArrayHelper::getValue($item, 'linkOptions', array());
+
+		if(ArrayHelper::getValue($item, 'active')) {
+			$this->addCssClass($options, 'active');
+		}
 
-		// does it has a dropdown widget?
+		if ($dropdown !== null) {
-		if (isset($item['dropdown'])) {
+			$linkOptions['data-toggle'] = 'dropdown';
-			$urlOptions['data-toggle'] = 'dropdown';
 			$this->addCssClass($options, 'dropdown');
 			$this->addCssClass($urlOptions, 'dropdown-toggle');
 			$label .= ' ' . Html::tag('b', '', array('class' => 'caret'));
-			$dropdown = Dropdown::widget(array('items' => $item['dropdown'], 'clientOptions' => false));
+			$dropdown = is_string($dropdown)
+				? $dropdown
+				: Dropdown::widget(array('items' => $item['items'], 'clientOptions' => false));
 		}
 
-		return Html::tag('li', Html::a($label, $url, $urlOptions) . $dropdown, $options);
+		return Html::tag('li', Html::a($label, $url, $linkOptions) . $dropdown, $options);
 	}
 }

framework/yii/bootstrap/NavBar.php

@@ -8,7 +8,7 @@
 namespace yii\bootstrap;
 
 use yii\base\InvalidConfigException;
-use yii\helpers\base\ArrayHelper;
+use yii\helpers\ArrayHelper;
 use yii\helpers\Html;
 
 /**
@@ -18,6 +18,7 @@ use yii\helpers\Html;
  *
  * ```php
  * echo NavBar::widget(array(
+ *     'brandLabel' => 'NavBar Test',
  *     'items' => array(
  *         // a Nav widget
  *         array(
@@ -33,17 +34,18 @@ use yii\helpers\Html;
  *                     ),
  *                     array(
  *                         'label' => 'Dropdown',
- *                         // configure a dropdown menu
+ *                         'content' => new Dropdown(array(
- *                         'dropdown' => array(
+ *                             'items' => array(
- *                             array(
+ *                                 array(
- *                                 'label' => 'DropdownA',
+ *                                     'label' => 'DropdownA',
- *                                 'url' => '#',
+ *                                     'url' => '#',
- *                             ),
+ *                                 ),
- *                             array(
+ *                                 array(
- *                                 'label' => 'DropdownB',
+ *                                     'label' => 'DropdownB',
- *                                 'url' => '#'
+ *                                     'url' => '#'
- *                             ),
+ *                                 ),
- *                         )
+ *                             )
+ *                         ),
  *                     ),
  *                 )
  *             ),
@@ -62,7 +64,20 @@ use yii\helpers\Html;
  */
 class NavBar extends Widget
 {
-	public $brand;
+	/**
+	 * @var string the text of the brand.
+	 * @see http://twitter.github.io/bootstrap/components.html#navbar
+	 */
+	public $brandLabel;
+	/**
+	 * @param array|string $url the URL for the brand's hyperlink tag. This parameter will be processed by [[Html::url()]]
+	 * and will be used for the "href" attribute of the brand link. Defaults to site root.
+	 */
+	public $brandRoute = '/';
+	/**
+	 * @var array the HTML attributes of the brand link.
+	 */
+	public $brandOptions = array();
 	/**
 	 * @var array list of menu items in the navbar widget. Each array element represents a single
 	 * menu item with the following structure:
@@ -83,6 +98,10 @@ class NavBar extends Widget
 	 * Optionally, you can also use a plain string instead of an array element.
 	 */
 	public $items = array();
+	/**
+	 * @var string the generated brand url if specified by [[brandLabel]]
+	 */
+	protected $brand;
 
 
 	/**
@@ -91,7 +110,10 @@ class NavBar extends Widget
 	public function init()
 	{
 		parent::init();
+		$this->clientOptions = false;
 		$this->addCssClass($this->options, 'navbar');
+		$this->addCssClass($this->brandOptions, 'brand');
+		$this->brand = Html::a($this->brandLabel, $this->brandRoute, $this->brandOptions);
 	}
 
 	/**
@@ -102,19 +124,20 @@ class NavBar extends Widget
 		echo Html::beginTag('div', $this->options);
 		echo $this->renderItems();
 		echo Html::endTag('div');
+		$this->getView()->registerAssetBundle('yii/bootstrap');
 	}
 
 	/**
+	 * Renders the items.
 	 * @return string the rendering items.
 	 */
 	protected function renderItems()
 	{
 		$items = array();
-
 		foreach ($this->items as $item) {
 			$items[] = $this->renderItem($item);
 		}
-		$contents =implode("\n", $items);
+		$contents = implode("\n", $items);
 		if (self::$responsive === true) {
 			$this->getView()->registerAssetBundle('yii/bootstrap/collapse');
 			$contents =
@@ -127,6 +150,7 @@ class NavBar extends Widget
 		} else {
 			$contents = $this->brand . "\n" . $contents;
 		}
+
 		return Html::tag('div', $contents, array('class' => 'navbar-inner'));
 	}
 
@@ -143,11 +167,11 @@ class NavBar extends Widget
 			return $item;
 		}
 		$config = ArrayHelper::getValue($item, 'options', array());
-		$config['class'] = ArrayHelper::getValue($item, 'class', 'yii\bootstrap\Nav');
+		$config['clientOptions'] = false;
-		$widget = \Yii::createObject($config);
+
-		ob_start();
+		$class = ArrayHelper::getValue($item, 'class', 'yii\bootstrap\Nav');
-		$widget->run();
+
-		return ob_get_clean();
+		return $class::widget($config);
 	}
 
 	/**
@@ -160,10 +184,10 @@ class NavBar extends Widget
 		for ($i = 0; $i < 3; $i++) {
 			$items[] = Html::tag('span', '', array('class' => 'icon-bar'));
 		}
-		return Html::tag('a', implode("\n", $items), array(
+		return Html::a(implode("\n", $items), null, array(
 			'class' => 'btn btn-navbar',
 			'data-toggle' => 'collapse',
 			'data-target' => 'div.navbar-collapse',
-		)) . "\n";
+		));
 	}
 }