From 4a8c72f0c48149081a3174668a87cf360ffb45fe Mon Sep 17 00:00:00 2001
From: Alexey Rogachev <arogachev90@gmail.com>
Date: Fri, 26 Nov 2021 03:45:09 +0600
Subject: [PATCH] Normalize todos (#19045)

---
 .php_cs                                     |  2 +-
 tests/framework/db/ActiveQueryTest.php      | 18 +++++++++---------
 tests/framework/widgets/ActiveFieldTest.php |  2 +-
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/.php_cs b/.php_cs
index 8550239..0350dff 100644
--- a/.php_cs
+++ b/.php_cs
@@ -1,7 +1,7 @@
 <?php
 
 if (!class_exists('yii\cs\YiisoftConfig', true)) {
-    // @todo change error message
+    // TODO: change error message
     fwrite(STDERR, "Your php-cs-version is outdated: please upgrade it.\n");
     die(16);
 }
diff --git a/tests/framework/db/ActiveQueryTest.php b/tests/framework/db/ActiveQueryTest.php
index 2fd9fde..50b8a6e 100644
--- a/tests/framework/db/ActiveQueryTest.php
+++ b/tests/framework/db/ActiveQueryTest.php
@@ -53,7 +53,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: tests for internal logic of prepare()
+     * @todo tests for internal logic of prepare()
      */
     public function testPrepare()
     {
@@ -72,7 +72,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: tests for internal logic of populate()
+     * @todo tests for internal logic of populate()
      */
     public function testPopulate_FilledRows()
     {
@@ -83,7 +83,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: tests for internal logic of one()
+     * @todo tests for internal logic of one()
      */
     public function testOne()
     {
@@ -93,7 +93,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: test internal logic of createCommand()
+     * @todo test internal logic of createCommand()
      */
     public function testCreateCommand()
     {
@@ -103,7 +103,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: tests for internal logic of queryScalar()
+     * @todo tests for internal logic of queryScalar()
      */
     public function testQueryScalar()
     {
@@ -113,7 +113,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: tests for internal logic of joinWith()
+     * @todo tests for internal logic of joinWith()
      */
     public function testJoinWith()
     {
@@ -125,7 +125,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: tests for internal logic of innerJoinWith()
+     * @todo tests for internal logic of innerJoinWith()
      */
     public function testInnerJoinWith()
     {
@@ -157,7 +157,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: tests for the regex inside getQueryTableName
+     * @todo tests for the regex inside getQueryTableName
      */
     public function testGetQueryTableName_from_not_set()
     {
@@ -231,7 +231,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
     }
 
     /**
-     * @todo: tests for internal logic of viaTable()
+     * @todo tests for internal logic of viaTable()
      */
     public function testViaTable()
     {
diff --git a/tests/framework/widgets/ActiveFieldTest.php b/tests/framework/widgets/ActiveFieldTest.php
index 201ca38..d4d94d1 100644
--- a/tests/framework/widgets/ActiveFieldTest.php
+++ b/tests/framework/widgets/ActiveFieldTest.php
@@ -78,7 +78,7 @@ EOD;
     }
 
     /**
-     * @todo    discuss|review  Expected HTML shouldn't be wrapped only by the $content?
+     * @todo discuss|review Expected HTML shouldn't be wrapped only by the $content?
      */
     public function testRenderWithCallableContent()
     {