Issue #2532646 by TR: "@return boolean" is wrong, use "@return bool"

core/includes/install.core.inc

@@ -1350,7 +1350,7 @@ function install_download_translation(&$install_state) {
  *   original name. If the path contains a filename as well, that one will be
  *   used instead.
  *
- * @return boolean
+ * @return bool
  *   TRUE on success, FALSE on failure.
  */
 function install_retrieve_file($uri, $destination) {

core/lib/Drupal/Component/Datetime/DateTimePlus.php

@@ -508,7 +508,7 @@ public static function prepareArray($array, $force_valid_date = FALSE) {
    * @param array $array
    *   An array of datetime values keyed by date part.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the datetime parts contain valid values, otherwise FALSE.
    */
   public static function checkArray($array) {

core/lib/Drupal/Component/Gettext/PoItem.php

@@ -147,7 +147,7 @@ function setPlural($plural) {
   /**
    * Get if the translation has plural values.
    *
-   * @return boolean $plural
+   * @return bool
    */
   function isPlural() {
     return $this->_plural;

core/lib/Drupal/Component/PhpStorage/FileStorage.php

@@ -224,7 +224,7 @@ public function deleteAll() {
    * @param string $path
    *   A string containing either a file or directory path.
    *
-   * @return boolean
+   * @return bool
    *   TRUE for success or if path does not exist, FALSE in the event of an
    *   error.
    */

core/lib/Drupal/Core/Condition/ConditionInterface.php

@@ -52,7 +52,7 @@ interface ConditionInterface extends ExecutableInterface, PluginFormInterface, C
   /**
    * Determines whether condition result will be negated.
    *
-   * @return boolean
+   * @return bool
    *   Whether the condition result will be negated.
    */
   public function isNegated();

core/lib/Drupal/Core/Config/Schema/TypedConfigInterface.php

@@ -24,7 +24,7 @@ interface TypedConfigInterface extends TraversableTypedDataInterface {
   /**
    * Determines whether the data structure is empty.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the data structure is empty, FALSE otherwise.
    */
   public function isEmpty();

core/lib/Drupal/Core/Config/StorageInterface.php

@@ -156,7 +156,7 @@ public function listAll($prefix = '');
    *   (optional) The prefix to search for. If omitted, all configuration
    *   objects that exist will be deleted.
    *
-   * @return boolean
+   * @return bool
    *   TRUE on success, FALSE otherwise.
    */
   public function deleteAll($prefix = '');

core/lib/Drupal/Core/DrupalKernel.php

@@ -1402,7 +1402,7 @@ public static function validateHostname(Request $request) {
    * @param array $host_patterns
    *   The array of trusted host patterns.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the Host header is trusted, FALSE otherwise.
    *
    * @see https://www.drupal.org/node/1992030

core/lib/Drupal/Core/Entity/ContentEntityFormInterface.php

@@ -58,7 +58,7 @@ public function getFormLangcode(FormStateInterface $form_state);
    * @param \Drupal\Core\Form\FormStateInterface $form_state
    *   The current state of the form.
    *
-   * @return boolean
+   * @return bool
    *   Returns TRUE if the entity form language matches the entity one.
    */
   public function isDefaultFormLangcode(FormStateInterface $form_state);

core/lib/Drupal/Core/Field/FieldItemInterface.php

@@ -151,7 +151,7 @@ public function __set($property_name, $value);
    * @param $property_name
    *   The name of the property to get; e.g., 'title' or 'name'.
    *
-   * @return boolean
+   * @return bool
    *   Returns TRUE if the property exists and is set, FALSE otherwise.
    */
   public function __isset($property_name);

core/lib/Drupal/Core/Plugin/DefaultPluginManager.php

@@ -336,7 +336,7 @@ protected function alterDefinitions(&$definitions) {
   /**
    * Determines if the provider of a definition exists.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if provider exists, FALSE otherwise.
    */
   protected function providerExists($provider) {

core/lib/Drupal/Core/Queue/QueueInterface.php

@@ -89,7 +89,7 @@ public function deleteItem($item);
    * @param $item
    *   The item returned by \Drupal\Core\Queue\QueueInterface::claimItem().
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the item has been released, FALSE otherwise.
    */
   public function releaseItem($item);

core/lib/Drupal/Core/Session/SessionManager.php

@@ -152,7 +152,7 @@ public function start() {
   /**
    * Forcibly start a PHP session.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the session is started.
    */
   protected function startNow() {

core/lib/Drupal/Core/TypedData/ComplexDataInterface.php

@@ -94,7 +94,7 @@ public function toArray();
   /**
    * Determines whether the data structure is empty.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the data structure is empty, FALSE otherwise.
    */
   public function isEmpty();

core/lib/Drupal/Core/TypedData/ListInterface.php

@@ -25,7 +25,7 @@ interface ListInterface extends TraversableTypedDataInterface, \ArrayAccess, \Co
   /**
    * Determines whether the list contains any non-empty items.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the list is empty, FALSE otherwise.
    */
   public function isEmpty();

core/modules/comment/src/Tests/CommentNonNodeTest.php

@@ -180,7 +180,7 @@ function postComment(EntityInterface $entity, $comment, $subject = '', $contact
    * @param bool $reply
    *   Boolean indicating whether the comment is a reply to another comment.
    *
-   * @return boolean
+   * @return bool
    *   Boolean indicating whether the comment was found.
    */
   function commentExists(CommentInterface $comment = NULL, $reply = FALSE) {
@@ -201,7 +201,7 @@ function commentExists(CommentInterface $comment = NULL, $reply = FALSE) {
   /**
    * Checks whether the commenter's contact information is displayed.
    *
-   * @return boolean
+   * @return bool
    *   Contact info is available.
    */
   function commentContactInfoAvailable() {

core/modules/comment/src/Tests/CommentTestBase.php

@@ -182,7 +182,7 @@ public function postComment($entity, $comment, $subject = '', $contact = NULL, $
    * @param bool $reply
    *   Boolean indicating whether the comment is a reply to another comment.
    *
-   * @return boolean
+   * @return bool
    *   Boolean indicating whether the comment was found.
    */
   function commentExists(CommentInterface $comment = NULL, $reply = FALSE) {
@@ -323,7 +323,7 @@ public function setCommentSettings($name, $value, $message, $field_name = 'comme
   /**
    * Checks whether the commenter's contact information is displayed.
    *
-   * @return boolean
+   * @return bool
    *   Contact info is available.
    */
   function commentContactInfoAvailable() {

core/modules/content_translation/src/Tests/ContentTranslationSettingsTest.php

@@ -231,7 +231,7 @@ function testAccountLanguageSettingsUI() {
    * @param array $edit
    *   An array of values to submit to the content translation settings page.
    *
-   * @return boolean
+   * @return bool
    *   TRUE if the assertion succeeded, FALSE otherwise.
    */
   protected function assertSettings($entity_type, $bundle, $enabled, $edit) {

core/modules/migrate/src/Plugin/migrate/destination/EntityFile.php

@@ -121,7 +121,7 @@ public function import(Row $row, array $old_destination_id_values = array()) {
    * @param integer $replace
    *  FILE_EXISTS_REPLACE (default) or FILE_EXISTS_RENAME.
    *
-   * @return boolean
+   * @return bool
    *  TRUE on success, FALSE on failure.
    */
   protected function writeFile($source, $destination, $replace = FILE_EXISTS_REPLACE) {
@@ -164,7 +164,7 @@ protected function getOverwriteMode(Row $row) {
    * @param string $uri
    *  The URI or path.
    *
-   * @return boolean|string
+   * @return string|false
    *  The directory component of the path or URI, or FALSE if it could not
    *  be determined.
    */
@@ -187,7 +187,7 @@ protected function getDirectory($uri) {
    * @param string $destination
    *  The destination URI.
    *
-   * @return boolean
+   * @return bool
    *  TRUE if the source and destination URIs refer to the same physical path,
    *  otherwise FALSE.
    */
@@ -210,7 +210,7 @@ protected function isLocationUnchanged($source, $destination) {
    * @param string $uri
    *  The URI or path to test.
    *
-   * @return boolean
+   * @return bool
    */
   protected function isLocalUri($uri) {
     $scheme = $this->fileSystem->uriScheme($uri);

core/modules/system/src/SystemManager.php

@@ -95,7 +95,7 @@ public function __construct(ModuleHandlerInterface $module_handler, EntityManage
   /**
    * Checks for requirement severity.
    *
-   * @return boolean
+   * @return bool
    *   Returns the status of the system.
    */
   public function checkRequirements() {

core/modules/tracker/tracker.module

@@ -138,7 +138,7 @@ function tracker_cron() {
  * @param int $account
  *   The account ID to check.
  *
- * @return boolean
+ * @return bool
  *   TRUE if a user is accessing tracking info for their own account and
  *   has permission to access the content.
  *

core/scripts/migrate-db.sh

@@ -286,7 +286,7 @@ function restore_table($path, Connection $connection) {
  * @param string $path
  *  The path to the dump file.
  *
- * @return boolean
+ * @return bool
  */
 function table_is_valid($path) {
   // The call to rtrim() is important, since we need to extract a specific