improved generation annotation of properties

[uldisn]

Signed-off-by: uldisn uldisnelsons@gmail.com

Q	A
Is bugfix?	yes
New feature?	yes
Breaks BC?	no
Tests pass?	yes
Fixed issues	-

New features

1. added type float (MySql: decimal, float, money, double)

2. If column value can be null, property type for example for string column set like this

/**
 * @property string|null $car_number
 */

Bug

For MySql table

CREATE TABLE `csc_car_unload` (
  `id` int(10) unsigned NOT NULL AUTO_INCREMENT,
  `task_id` smallint(5) unsigned NOT NULL,
  `delivery_id` int(10) unsigned NOT NULL,
  PRIMARY KEY (`id`),
  KEY `task_id` (`task_id`),
  KEY `delivery_id` (`delivery_id`),
  CONSTRAINT `csc_car_unload_ibfk_1` FOREIGN KEY (`task_id`) REFERENCES `csc_task` (`id`),
  CONSTRAINT `csc_car_unload_ibfk_2` FOREIGN KEY (`delivery_id`) REFERENCES `cmd_delivery` (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=5948 DEFAULT CHARSET=latin1

Properties $id and $delivery_id type is generated as string, but must be integer

/**
 * This is the base-model class for table "csc_car_unload".
 *
 * @property string $id
 * @property integer $task_id
 * @property string $delivery_id
 *
 * @property \coalmar\screen\models\CscTask $task
 * @property \coalmar\screen\models\CmdDelivery $delivery
 */

[yii-bot]

Thank you for putting effort in the improvement of the Yii framework.
We have reviewed your pull request.

Unfortunately a use case is missing. It is required to get a better understanding of the pull request and helps us to determine the necessity and applicability of the suggested change to the framework.

Could you supply us with a use case please? Please be as detailed as possible and show some code!

Thanks!

This is an automated comment, triggered by adding the label pr:missing usecase.

[uldisn]

Updated: #416 (comment)

[samdark]

Looks OK. Please add lines for changelog.md and I'll merge it.

[uldisn]

Looks OK. Please add lines for changelog.md and I'll merge it.

Done

[samdark]

Merged. Thank you!