Improve comments and add checking $theme type.

Author: alxgh

DatePicker.php

@@ -4,6 +4,7 @@
 
 use yii\helpers\Json;
 use yii\helpers\Html;
+use yii\base\InvalidConfigException;
 use yii\widgets\InputWidget;
 /**
  * Jalali date & time.
@@ -16,13 +17,26 @@ class DatePicker extends InputWidget
 	 * @var array Date picker options.
 	 */
 	public $clientOptions = ['formatDate' => "YYYY/0M/0D"];
-
+	/**
+	 * @var string Theme name.
+	 *			   Date picker currently have 2 theme : dark, default.
+	 */
 	public $theme = 'default';
+	/**
+	 * @inheritdoc
+	 */
 	public function init()
 	{
 		parent::init();
+		if(! is_string($this->theme)) {
+			throw new InvalidConfigException(
+				"Theme name must be a string."
+			);
+		}
 	}
-	
+	/**
+     * Executes the widget.
+     */
 	function run()
 	{
 		echo $this->renderInput();
@@ -44,7 +58,7 @@ function registerDarkThemeAsset()
 		DatePickerDarkThemeAsset::register($this->getView());
 	}
 	/**
-	 * Register datepicker assets without into view.
+	 * Register datepicker assets without css into view.
 	 */
 	function registerNoThemeAsset()
 	{
@@ -67,24 +81,25 @@ function renderInput()
 	 */
 	function renderJsCode()
 	{
+		//Jquery plugin name.
 		$name = 'persianDatepicker';
 
 		$id = $this->options['id'];
-
+		// Set theme name in clientOptions.
 		$this->clientOptions['theme'] = $this->theme;
-
+		// Build onSelect event.
 		if(! isset($this->clientOptions['onSelect'])) {
 			$this->clientOptions['onSelect'] = "function(){
 				$('#$id').trigger('change');
 			}";
 		}
-
+		// Register theme assets.
 		if(in_array($this->theme, ['default', 'dark'])) {
 			$this->{"register" . ucfirst($this->theme) . "ThemeAsset"}();
 		} else {
 				$this->registerNoThemeAsset();
 		}
-
+		// Add events.
 		$onSelect = $this->clientOptions['onSelect'];
 
 		if(isset($this->clientOptions['onShow'])) {
@@ -116,7 +131,7 @@ function renderJsCode()
 			$options .= '}';
 		}
 		$js = "jQuery('#$id').$name($options);";
-		
+		// Register js code.
 		$this->getView()->registerJs($js);
 	}
 }