Commit 74aaa4f0 by Antonio Ramirez

code style fixes and refactored as stated on comments

parent fc783984
...@@ -8,8 +8,8 @@ ...@@ -8,8 +8,8 @@
namespace yii\bootstrap; namespace yii\bootstrap;
use Yii; use Yii;
use yii\base\Model;
use yii\helpers\Html; use yii\helpers\Html;
use yii\widgets\ActiveForm;
/** /**
* TypeAhead renders a typehead bootstrap javascript component. * TypeAhead renders a typehead bootstrap javascript component.
...@@ -18,12 +18,12 @@ use yii\widgets\ActiveForm; ...@@ -18,12 +18,12 @@ use yii\widgets\ActiveForm;
* *
* ```php * ```php
* echo TypeAhead::widget(array( * echo TypeAhead::widget(array(
* 'form' => $form, * 'form' => $form,
* 'model' => $model, * 'model' => $model,
* 'attribute' => 'country', * 'attribute' => 'country',
* 'pluginOptions' => array( * 'pluginOptions' => array(
* 'source' => array('USA', 'ESP'), * 'source' => array('USA', 'ESP'),
* ), * ),
* )); * ));
* ``` * ```
* *
...@@ -31,10 +31,10 @@ use yii\widgets\ActiveForm; ...@@ -31,10 +31,10 @@ use yii\widgets\ActiveForm;
* *
* ```php * ```php
* echo TypeAhead::widget(array( * echo TypeAhead::widget(array(
* 'name' => 'country', * 'name' => 'country',
* 'pluginOptions' => array( * 'pluginOptions' => array(
* 'source' => array('USA', 'ESP'), * 'source' => array('USA', 'ESP'),
* ), * ),
* )); * ));
*``` *```
* *
...@@ -45,11 +45,6 @@ use yii\widgets\ActiveForm; ...@@ -45,11 +45,6 @@ use yii\widgets\ActiveForm;
class TypeAhead extends Widget class TypeAhead extends Widget
{ {
/** /**
* @var ActiveForm the form that the TypeAhead field is associated with. If no form is associated with the widget
* then the id will be used instead
*/
public $form;
/**
* @var \yii\base\Model the data model that this field is associated with * @var \yii\base\Model the data model that this field is associated with
*/ */
public $model; public $model;
...@@ -64,50 +59,32 @@ class TypeAhead extends Widget ...@@ -64,50 +59,32 @@ class TypeAhead extends Widget
public $name; public $name;
/** /**
* Initializes the widget. * Renders the widget
* Renders the input field.
*/
public function init()
{
parent::init();
echo "\n" . $this->renderField() . "\n";
}
/**
* Registers the plugin.
*/ */
public function run() public function run()
{ {
echo "\n" . $this->renderField() . "\n";
$this->registerPlugin('typeahead'); $this->registerPlugin('typeahead');
} }
/** /**
* Renders the TypeAhead field. If [[form]] has been specified then it will render an active field. * Renders the TypeAhead field. If [[model]] has been specified then it will render an active field.
* Please, note that function will only check whether the form has been set, model and attributes will not. * If [[model]] is null or not from an [[Model]] instance, then the field will be rendered according to
* If [[form]] is null not from an [[ActiveForm]] instance, then the field will be rendered according to * the [[name]] attribute.
* the `name` key setting of [[options]] array attribute.
* @return string the rendering result * @return string the rendering result
* @throws InvalidParamException when none of the required attributes are set to render the textInput. That is, * @throws InvalidParamException when none of the required attributes are set to render the textInput. That is,
* if [[form]], [[model]] and [[attribute]] are not set, then [[name]] is required. * if [[model]] and [[attribute]] are not set, then [[name]] is required.
*/ */
public function renderField() public function renderField()
{ {
if ($this->form instanceof ActiveForm) { if ($this->model instanceof Model && $this->attribute !== null) {
$this->options['id'] = $this->id = Html::getInputId($this->model, $this->attribute); $this->options['id'] = $this->id = Html::getInputId($this->model, $this->attribute);
return Yii::createObject( return Html::activeTextInput($this->model, $this->attribute, $this->options);
array(
'class' => 'yii\widgets\ActiveField',
'model' => $this->model,
'attribute' => $this->attribute,
'form' => $this->form,
)
)->textInput();
} }
if ($this->name === null) if ($this->name === null) {
{
throw new InvalidParamException( throw new InvalidParamException(
get_class($this) . ' must specify "form", "model" and "attribute" or "name" property values' get_class($this) . ' must specify "form", "model" and "attribute" or "name" property values'
); );
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment