PhpDocController.php 11.4 KB
Newer Older
1 2 3 4 5 6 7 8 9
<?php
/**
 * @link http://www.yiiframework.com/
 * @copyright Copyright (c) 2008 Yii Software LLC
 * @license http://www.yiiframework.com/license/
 */

namespace yii\build\controllers;

10
use Yii;
11 12
use yii\console\Controller;
use yii\helpers\Console;
13
use yii\helpers\FileHelper;
14 15 16

/**
 * PhpDocController is there to help maintaining PHPDoc annotation in class files
17
 *
18 19 20 21 22 23
 * @author Carsten Brandt <mail@cebe.cc>
 * @author Alexander Makarov <sam@rmcreative.ru>
 * @since 2.0
 */
class PhpDocController extends Controller
{
24 25 26 27 28 29 30 31
	public $defaultAction = 'property';

	/**
	 * @var bool whether to update class docs directly. Setting this to false will just output docs
	 * for copy and paste.
	 */
	public $updateFiles = true;

32 33 34
	/**
	 * Generates @property annotations in class files from getters and setters
	 *
35 36 37 38 39 40
	 * Property description will be taken from getter or setter or from an @property annotation
	 * in the getters docblock if there is one defined.
	 *
	 * See https://github.com/yiisoft/yii2/wiki/Core-framework-code-style#documentation for details.
	 *
	 * @param null $root the directory to parse files from. Defaults to YII_PATH.
41
	 */
42
	public function actionProperty($root = null)
43
	{
44
		$except = [];
45
		if ($root === null) {
46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65
			$root = dirname(dirname(YII_PATH));
			Yii::setAlias('@yii/bootstrap', $root . '/extensions/bootstrap');
			Yii::setAlias('@yii/debug', $root . '/extensions/debug');
			Yii::setAlias('@yii/elasticsearch', $root . '/extensions/elasticsearch');
			Yii::setAlias('@yii/gii', $root . '/extensions/gii');
			Yii::setAlias('@yii/jui', $root . '/extensions/jui');
			Yii::setAlias('@yii/redis', $root . '/extensions/redis');
			Yii::setAlias('@yii/smarty', $root . '/extensions/smarty');
			Yii::setAlias('@yii/sphinx', $root . '/extensions/sphinx');
			Yii::setAlias('@yii/swiftmailer', $root . '/extensions/swiftmailer');
			Yii::setAlias('@yii/twig', $root . '/extensions/twig');

			$except = [
				'/apps/',
				'/build/',
				'/docs/',
				'/extensions/composer/',
				'/tests/',
				'/vendor/',
			];
66
		}
67
		$root = FileHelper::normalizePath($root);
Alexander Makarov committed
68
		$options = [
69 70 71 72 73 74 75 76 77
			'filter' => function ($path) {
				if (is_file($path)) {
					$file = basename($path);
					if ($file[0] < 'A' || $file[0] > 'Z') {
						return false;
					}
				}
				return null;
			},
Alexander Makarov committed
78
			'only' => ['.php'],
79
			'except' => array_merge($except, [
80
				'BaseYii.php',
81
				'Yii.php',
82
				'/views/',
83 84
				'/requirements/',
				'/gii/generators/',
85
			]),
Alexander Makarov committed
86
		];
87
		$files = FileHelper::findFiles($root, $options);
88
		$nFilesTotal = 0;
89
		$nFilesUpdated = 0;
90 91 92 93
		foreach ($files as $file) {
			$result = $this->generateClassPropertyDocs($file);
			if ($result !== false) {
				list($className, $phpdoc) = $result;
94
				if ($this->updateFiles) {
95 96 97 98 99 100 101 102 103 104
					if ($this->updateClassPropertyDocs($file, $className, $phpdoc)) {
						$nFilesUpdated++;
					}
				} elseif (!empty($phpdoc)) {
					$this->stdout("\n[ " . $file . " ]\n\n", Console::BOLD);
					$this->stdout($phpdoc);
				}
			}
			$nFilesTotal++;
		}
105

106
		$this->stdout("\nParsed $nFilesTotal files.\n");
107
		$this->stdout("Updated $nFilesUpdated files.\n");
108 109
	}

110 111
	public function globalOptions()
	{
Alexander Makarov committed
112
		return array_merge(parent::globalOptions(), ['updateFiles']);
113 114
	}

115
	protected function updateClassPropertyDocs($file, $className, $propertyDoc)
116
	{
117 118 119 120 121
		$ref = new \ReflectionClass($className);
		if ($ref->getFileName() != $file) {
			$this->stderr("[ERR] Unable to create ReflectionClass for class: $className loaded class is not from file: $file\n", Console::FG_RED);
		}

122
		if (!$ref->isSubclassOf('yii\base\Object') && $className != 'yii\base\Object') {
123
			$this->stderr("[INFO] Skipping class $className as it is not a subclass of yii\\base\\Object.\n", Console::FG_BLUE, Console::BOLD);
124 125 126
			return false;
		}

127 128 129 130 131 132
		$oldDoc = $ref->getDocComment();
		$newDoc = $this->cleanDocComment($this->updateDocComment($oldDoc, $propertyDoc));

		$seenSince = false;
		$seenAuthor = false;

133
		// TODO move these checks to different action
134
		$lines = explode("\n", $newDoc);
135 136 137
		if (trim($lines[1]) == '*' || substr(trim($lines[1]), 0, 3) == '* @') {
			$this->stderr("[WARN] Class $className has no short description.\n", Console::FG_YELLOW, Console::BOLD);
		}
138
		foreach ($lines as $line) {
139 140 141 142 143 144 145 146 147 148 149 150 151 152
			if (substr(trim($line), 0, 9) == '* @since ') {
				$seenSince = true;
			} elseif (substr(trim($line), 0, 10) == '* @author ') {
				$seenAuthor = true;
			}
		}

		if (!$seenSince) {
			$this->stderr("[ERR] No @since found in class doc in file: $file\n", Console::FG_RED);
		}
		if (!$seenAuthor) {
			$this->stderr("[ERR] No @author found in class doc in file: $file\n", Console::FG_RED);
		}

153
		if (trim($oldDoc) != trim($newDoc)) {
154

155 156 157
			$fileContent = explode("\n", file_get_contents($file));
			$start = $ref->getStartLine() - 2;
			$docStart = $start - count(explode("\n", $oldDoc)) + 1;
158

Alexander Makarov committed
159
			$newFileContent = [];
160
			$n = count($fileContent);
161
			for ($i = 0; $i < $n; $i++) {
162
				if ($i > $start || $i < $docStart) {
163
					$newFileContent[] = $fileContent[$i];
164 165
				} else {
					$newFileContent[] = trim($newDoc);
166
					$i = $start;
167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186
				}
			}

			file_put_contents($file, implode("\n", $newFileContent));

			return true;
		}
		return false;
	}

	/**
	 * remove multi empty lines and trim trailing whitespace
	 *
	 * @param $doc
	 * @return string
	 */
	protected function cleanDocComment($doc)
	{
		$lines = explode("\n", $doc);
		$n = count($lines);
187
		for ($i = 0; $i < $n; $i++) {
188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206
			$lines[$i] = rtrim($lines[$i]);
			if (trim($lines[$i]) == '*' && trim($lines[$i + 1]) == '*') {
				unset($lines[$i]);
			}
		}
		return implode("\n", $lines);
	}

	/**
	 * replace property annotations in doc comment
	 * @param $doc
	 * @param $properties
	 * @return string
	 */
	protected function updateDocComment($doc, $properties)
	{
		$lines = explode("\n", $doc);
		$propertyPart = false;
		$propertyPosition = false;
207
		foreach ($lines as $i => $line) {
208 209 210 211 212 213 214
			if (substr(trim($line), 0, 12) == '* @property ') {
				$propertyPart = true;
			} elseif ($propertyPart && trim($line) == '*') {
				$propertyPosition = $i;
				$propertyPart = false;
			}
			if (substr(trim($line), 0, 10) == '* @author ' && $propertyPosition === false) {
215
				$propertyPosition = $i - 1;
216 217 218 219 220 221 222
				$propertyPart = false;
			}
			if ($propertyPart) {
				unset($lines[$i]);
			}
		}
		$finalDoc = '';
223
		foreach ($lines as $i => $line) {
224 225 226 227 228 229
			$finalDoc .= $line . "\n";
			if ($i == $propertyPosition) {
				$finalDoc .= $properties;
			}
		}
		return $finalDoc;
230 231 232 233
	}

	protected function generateClassPropertyDocs($fileName)
	{
234 235
		$phpdoc = "";
		$file = str_replace("\r", "", str_replace("\t", " ", file_get_contents($fileName, true)));
236 237 238
		$ns = $this->match('#\nnamespace (?<name>[\w\\\\]+);\n#', $file);
		$namespace = reset($ns);
		$namespace = $namespace['name'];
239
		$classes = $this->match('#\n(?:abstract )?class (?<name>\w+)( extends .+)?( implements .+)?\n\{(?<content>.*)\n\}(\n|$)#', $file);
240 241 242 243 244 245

		if (count($classes) > 1) {
			$this->stderr("[ERR] There should be only one class in a file: $fileName\n", Console::FG_RED);
			return false;
		}
		if (count($classes) < 1) {
246
			$interfaces = $this->match('#\ninterface (?<name>\w+)( extends .+)?\n\{(?<content>.+)\n\}(\n|$)#', $file);
247 248 249 250 251
			if (count($interfaces) == 1) {
				return false;
			} elseif (count($interfaces) > 1) {
				$this->stderr("[ERR] There should be only one interface in a file: $fileName\n", Console::FG_RED);
			} else {
252 253 254 255 256 257 258 259
				$traits = $this->match('#\ntrait (?<name>\w+)\n\{(?<content>.+)\n\}(\n|$)#', $file);
				if (count($traits) == 1) {
					return false;
				} elseif (count($traits) > 1) {
					$this->stderr("[ERR] There should be only one class/trait/interface in a file: $fileName\n", Console::FG_RED);
				} else {
					$this->stderr("[ERR] No class in file: $fileName\n", Console::FG_RED);
				}
260
			}
261 262 263 264
			return false;
		}

		$className = null;
265
		foreach ($classes as &$class) {
266

267
			$className = $namespace . '\\' . $class['name'];
268

269
			$gets = $this->match(
270
				'#\* @return (?<type>[\w\\|\\\\\\[\\]]+)(?: (?<comment>(?:(?!\*/|\* @).)+?)(?:(?!\*/).)+|[\s\n]*)\*/' .
271 272 273
				'[\s\n]{2,}public function (?<kind>get)(?<name>\w+)\((?:,? ?\$\w+ ?= ?[^,]+)*\)#',
				$class['content']);
			$sets = $this->match(
274
				'#\* @param (?<type>[\w\\|\\\\\\[\\]]+) \$\w+(?: (?<comment>(?:(?!\*/|\* @).)+?)(?:(?!\*/).)+|[\s\n]*)\*/' .
275 276
				'[\s\n]{2,}public function (?<kind>set)(?<name>\w+)\(\$\w+(?:, ?\$\w+ ?= ?[^,]+)*\)#',
				$class['content']);
277 278 279 280 281 282
			// check for @property annotations in getter and setter
			$properties = $this->match(
				'#\* @(?<kind>property) (?<type>[\w\\|\\\\\\[\\]]+)(?: (?<comment>(?:(?!\*/|\* @).)+?)(?:(?!\*/).)+|[\s\n]*)\*/' .
				'[\s\n]{2,}public function [g|s]et(?<name>\w+)\(((?:,? ?\$\w+ ?= ?[^,]+)*|\$\w+(?:, ?\$\w+ ?= ?[^,]+)*)\)#',
				$class['content']);
			$acrs = array_merge($properties, $gets, $sets);
283

Alexander Makarov committed
284
			$props = [];
285 286 287
			foreach ($acrs as &$acr) {
				$acr['name'] = lcfirst($acr['name']);
				$acr['comment'] = trim(preg_replace('#(^|\n)\s+\*\s?#', '$1 * ', $acr['comment']));
Alexander Makarov committed
288
				$props[$acr['name']][$acr['kind']] = [
289 290
					'type' => $acr['type'],
					'comment' => $this->fixSentence($acr['comment']),
Alexander Makarov committed
291
				];
292 293 294
			}

			ksort($props);
295

296 297 298
			if (count($props) > 0) {
				$phpdoc .= " *\n";
				foreach ($props as $propName => &$prop) {
299 300
					$docline = ' * @';
					$docline .= 'property'; // Do not use property-read and property-write as few IDEs support complex syntax.
301
					$note = '';
302
					if (isset($prop['get']) && isset($prop['set'])) {
303 304 305 306
						if ($prop['get']['type'] != $prop['set']['type']) {
							$note = ' Note that the type of this property differs in getter and setter.'
								  . ' See [[get'.ucfirst($propName).'()]] and [[set'.ucfirst($propName).'()]] for details.';
						}
307
					} elseif (isset($prop['get'])) {
308 309 310 311 312 313 314 315 316 317 318 319 320 321
						// check if parent class has setter defined
						$c = $className;
						$parentSetter = false;
						while($parent = get_parent_class($c)) {
							if (method_exists($parent, 'set' . ucfirst($propName))) {
								$parentSetter = true;
								break;
							}
							$c = $parent;
						}
						if (!$parentSetter) {
							$note = ' This property is read-only.';
//							$docline .= '-read';
						}
322
					} elseif (isset($prop['set'])) {
323 324 325 326 327 328 329 330 331 332 333 334 335 336
						// check if parent class has getter defined
						$c = $className;
						$parentGetter = false;
						while($parent = get_parent_class($c)) {
							if (method_exists($parent, 'set' . ucfirst($propName))) {
								$parentGetter = true;
								break;
							}
							$c = $parent;
						}
						if (!$parentGetter) {
							$note = ' This property is write-only.';
//							$docline .= '-write';
						}
337 338
					} else {
						continue;
339 340 341 342 343 344 345 346 347
					}
					$docline .= ' ' . $this->getPropParam($prop, 'type') . " $$propName ";
					$comment = explode("\n", $this->getPropParam($prop, 'comment') . $note);
					foreach ($comment as &$cline) {
						$cline = ltrim($cline, '* ');
					}
					$docline = wordwrap($docline . implode(' ', $comment), 110, "\n * ") . "\n";

					$phpdoc .= $docline;
348 349 350 351
				}
				$phpdoc .= " *\n";
			}
		}
Alexander Makarov committed
352
		return [$className, $phpdoc];
353 354 355 356
	}

	protected function match($pattern, $subject)
	{
Alexander Makarov committed
357
		$sets = [];
358 359 360 361 362 363
		preg_match_all($pattern . 'suU', $subject, $sets, PREG_SET_ORDER);
		foreach ($sets as &$set)
			foreach ($set as $i => $match)
				if (is_numeric($i) /*&& $i != 0*/)
					unset($set[$i]);
		return $sets;
364 365 366 367 368
	}

	protected function fixSentence($str)
	{
		// TODO fix word wrap
369 370 371
		if ($str == '')
			return '';
		return strtoupper(substr($str, 0, 1)) . substr($str, 1) . ($str[strlen($str) - 1] != '.' ? '.' : '');
372 373 374 375
	}

	protected function getPropParam($prop, $param)
	{
376
		return isset($prop['property']) ? $prop['property'][$param] : (isset($prop['get']) ? $prop['get'][$param] : $prop['set'][$param]);
377 378
	}
}