Issue #2601682 by krknth, rakesh.gectcr, anil280988, jhodgdon: Unnecessary @param tags & Missing variable names in block, block_content module

core/modules/block/src/BlockViewBuilder.php

@@ -186,8 +186,6 @@ class BlockViewBuilder extends EntityViewBuilder {
    *   A block config entity ID.
    * @param $view_mode
    *   The view mode the block is being viewed in.
-   * @param $langcode
-   *   The langcode the block is being viewed in.
    *
    * @return array
    *   A render array with a #pre_render callback to render the block.

core/modules/block/src/Tests/Migrate/d6/MigrateBlockTest.php

@@ -61,8 +61,6 @@ class MigrateBlockTest extends MigrateDrupal6TestBase {
    *
    * @param string $id
    *   The block ID.
-   * @param string $module
-   *   The module.
    * @param array $visibility
    *   The block visibility settings.
    * @param string $region

core/modules/block_content/src/Plugin/Block/BlockContentBlock.php

@@ -57,6 +57,13 @@ class BlockContentBlock extends BlockBase implements ContainerFactoryPluginInter
    */
   protected $blockContent;
 
+  /**
+   * The URL generator.
+   *
+   * @var \Drupal\Core\Routing\UrlGeneratorInterface
+   */
+  protected $urlGenerator;
+
   /**
    * Constructs a new BlockContentBlock.
    *
@@ -66,12 +73,14 @@ class BlockContentBlock extends BlockBase implements ContainerFactoryPluginInter
    *   The plugin ID for the plugin instance.
    * @param mixed $plugin_definition
    *   The plugin implementation definition.
-   * @param \Drupal\Core\Block\BlockManagerInterface
+   * @param \Drupal\Core\Block\BlockManagerInterface $block_manager
    *   The Plugin Block Manager.
    * @param \Drupal\Core\Entity\EntityManagerInterface $entity_manager
    *   The entity manager service.
    * @param \Drupal\Core\Session\AccountInterface $account
    *   The account for which view access should be checked.
+   * @param \Drupal\Core\Routing\UrlGeneratorInterface $url_generator
+   *   The URL generator.
    */
   public function __construct(array $configuration, $plugin_id, $plugin_definition, BlockManagerInterface $block_manager, EntityManagerInterface $entity_manager, AccountInterface $account, UrlGeneratorInterface $url_generator) {
     parent::__construct($configuration, $plugin_id, $plugin_definition);

core/modules/block_content/src/Tests/BlockContentTestBase.php

@@ -65,7 +65,7 @@ abstract class BlockContentTestBase extends WebTestBase {
   /**
    * Creates a custom block.
    *
-   * @param string $title
+   * @param bool|string $title
    *   (optional) Title of block. When no value is given uses a random name.
    *   Defaults to FALSE.
    * @param string $bundle

core/modules/block_content/src/Tests/BlockContentTranslationUITest.php

@@ -83,10 +83,10 @@ class BlockContentTranslationUITest extends ContentTranslationUITestBase {
   /**
    * Creates a custom block.
    *
-   * @param string $title
+   * @param bool|string $title
    *   (optional) Title of block. When no value is given uses a random name.
    *   Defaults to FALSE.
-   * @param string $bundle
+   * @param bool|string $bundle
    *   (optional) Bundle name. When no value is given, defaults to
    *   $this->bundle. Defaults to FALSE.
    *