Fix CS in generated migrations (remove comma from param name in docbl…

…ock)
CakeDC · Oct 16, 2014 · d3a3af7 · d3a3af7
1 parent abc4c92
commit d3a3af7
Show file tree

Hide file tree

Showing 8 changed files with 22 additions and 22 deletions.
diff --git a/Config/Migration/001_init_migrations.php b/Config/Migration/001_init_migrations.php
@@ -37,7 +37,7 @@ class InitMigrations extends CakeMigration {
 /**
  * Before migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function before($direction) {
@@ -47,11 +47,11 @@ public function before($direction) {
 /**
  * After migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function after($direction) {
 		return true;
 	}
 
-}
+}
diff --git a/Config/Migration/002_convert_version_to_class_names.php b/Config/Migration/002_convert_version_to_class_names.php
@@ -47,7 +47,7 @@ class ConvertVersionToClassNames extends CakeMigration {
 /**
  * Before migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  * @throws InternalErrorException
  */
@@ -68,7 +68,7 @@ public function before($direction) {
 /**
  * After migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function after($direction) {
@@ -104,7 +104,7 @@ public function needsUpgrade() {
  * Check if every plugin is loaded/reachable, we need access to them
  *
  * @throws MissingPluginException
- * @return void 
+ * @return void
  */
 	public function checkPlugins() {
 		$types = Hash::extract($this->records, '{n}.' . $this->Version->Version->alias . '.type');

diff --git a/Config/Migration/003_increase_class_name_length.php b/Config/Migration/003_increase_class_name_length.php
@@ -33,7 +33,7 @@ class IncreaseClassNameLength extends CakeMigration {
 /**
  * Before migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function before($direction) {
@@ -43,11 +43,11 @@ public function before($direction) {
 /**
  * After migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function after($direction) {
 		return true;
 	}
 
-}
+}
diff --git a/Console/Command/Templates/migration.ctp b/Console/Command/Templates/migration.ctp
@@ -31,7 +31,7 @@ class <?php echo $class; ?> extends CakeMigration {
 /**
  * Before migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function before($direction) {
@@ -41,7 +41,7 @@ class <?php echo $class; ?> extends CakeMigration {
 /**
  * After migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function after($direction) {

diff --git a/Console/Templates/classes/migration.ctp b/Console/Templates/classes/migration.ctp
@@ -31,7 +31,7 @@ class <?php echo $class; ?> extends CakeMigration {
 /**
  * Before migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function before($direction) {
@@ -41,7 +41,7 @@ class <?php echo $class; ?> extends CakeMigration {
 /**
  * After migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function after($direction) {

diff --git a/Lib/CakeMigration.php b/Lib/CakeMigration.php
@@ -126,7 +126,7 @@ class CakeMigration extends Object {
 /**
  * Before migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function before($direction) {
@@ -136,7 +136,7 @@ public function before($direction) {
 /**
  * After migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function after($direction) {
@@ -219,7 +219,7 @@ public function __construct($options = array()) {
 /**
  * Run migration
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Status of the process
  * @throws MigrationException
  */

diff --git a/Test/test_app/Plugin/TestMigrationPlugin/Config/Migration/001_schema_dump.php b/Test/test_app/Plugin/TestMigrationPlugin/Config/Migration/001_schema_dump.php
@@ -21,7 +21,7 @@ class M4af6d40056b04408808500cb58157726 extends CakeMigration {
 /**
  * Before migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function before($direction) {
@@ -31,11 +31,11 @@ public function before($direction) {
 /**
  * After migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function after($direction) {
 		return true;
 	}
 
-}
+}
diff --git a/...ugin/TestMigrationPlugin/Config/Migration/002_another_migration_plugin_test_migration.php b/...ugin/TestMigrationPlugin/Config/Migration/002_another_migration_plugin_test_migration.php
@@ -21,7 +21,7 @@ class AnotherMigrationPluginTestMigration extends CakeMigration {
 /**
  * Before migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function before($direction) {
@@ -31,11 +31,11 @@ public function before($direction) {
 /**
  * After migration callback
  *
- * @param string $direction, up or down direction of migration process
+ * @param string $direction Direction of migration process (up or down)
  * @return boolean Should process continue
  */
 	public function after($direction) {
 		return true;
 	}
 
-}
+}