\Drupal::service('router.builder')->rebuild() call should be avoided in classes, Use dependency injection instead in drupal Database

I have Created a patch to fix this issue.

+  /**
+   * Constructs the FakeLoginConfigForm.
+   *
+   * @param \Drupal\Core\Routing\RouteBuilderInterface $router_builder
+   *   The route builder.

The description for a constructor must start with Constructs a new followed by the class name (including its namespace), and end with object.

+  /**
+   * @param ContainerInterface $container
+   *
+   * @return static
+   */
+  public static function create(ContainerInterface $container) {

That is not the documentation comment for a method inherited from a parent class or defined in an interface.

Updated patch #2 by addressing #3, please review it.

Thanks!

+  /**
+   * Constructs a new FakeLoginConfigForm object.
+   *

The description for a constructor must start with Constructs a new followed by the class name (including its namespace), and end with object.

Let me work on #5.

@apaderno, The description contains all the points which you have described.
Please let us know what desc you are expecting.

There is no namespace in FakeLoginConfigForm.

Updated patch #4 by addressing #8, please review it.

Thanks!

+  /**
+   * Constructs a new Drupal\fakelogin\Form\FakeLoginConfigForm object.
+   *
+   * @param \Drupal\Core\Routing\RouteBuilderInterface $router_builder
+   *   The route builder.
+   */

That description contains the class namespace too. The previous change was different.

+  /**
+   * Constructs a new FakeLoginConfigForm object.
+   *
+   * @param \Drupal\Core\Routing\RouteBuilderInterface $router_builder
+   *   The route builder.
+   */

FakeLoginConfigForm alone does not include the class namespace.

Since these changes are already applied in another issue, this issue is a duplicate of that issue.