I have checked module by drupal check and found below errors.

------ ----------------------------------------------------------------
Line fac.install
------ ----------------------------------------------------------------
15 Call to deprecated function file_unmanaged_delete_recursive():
in Drupal 8.7.0, will be removed before Drupal 9.0.0.
Use \Drupal\Core\File\FileSystemInterface::deleteRecursive().
------ ----------------------------------------------------------------

------ ----------------------------------------------------------------
Line fac.module
------ ----------------------------------------------------------------
115 Call to deprecated function file_unmanaged_delete_recursive():
in Drupal 8.7.0, will be removed before Drupal 9.0.0.
Use \Drupal\Core\File\FileSystemInterface::deleteRecursive().
------ ----------------------------------------------------------------

------ ----------------------------------------------------------------
Line src/Controller/FacController.php
------ ----------------------------------------------------------------
108 Call to deprecated function file_prepare_directory():
in Drupal 8.7.0, will be removed before Drupal 9.0.0.
Use \Drupal\Core\File\FileSystemInterface::prepareDirectory().
110 Call to deprecated function file_unmanaged_save_data():
in Drupal 8.7.0, will be removed before Drupal 9.0.0.
Use \Drupal\Core\File\FileSystemInterface::saveData().
------ ----------------------------------------------------------------

------ ----------------------------------------------------------------
Line src/Entity/FacConfig.php
------ ----------------------------------------------------------------
301 Call to deprecated function file_unmanaged_delete_recursive():
in Drupal 8.7.0, will be removed before Drupal 9.0.0.
Use \Drupal\Core\File\FileSystemInterface::deleteRecursive().
310 Call to deprecated function file_unmanaged_delete():
in Drupal 8.7.0, will be removed before Drupal 9.0.0.
Use \Drupal\Core\File\FileSystemInterface::delete().
------ ----------------------------------------------------------------

------ ----------------------------------------------------------------
Line src/Form/FacConfigDisableConfirmForm.php
------ ----------------------------------------------------------------
53 Call to deprecated function file_unmanaged_delete_recursive():
in Drupal 8.7.0, will be removed before Drupal 9.0.0.
Use \Drupal\Core\File\FileSystemInterface::deleteRecursive().
------ ----------------------------------------------------------------

------ -------------------------------------------------------------------------
Line tests/src/Unit/HashServiceTest.php
------ -------------------------------------------------------------------------
24 Call to deprecated method getMock() of class Drupal\Tests\UnitTestCase:
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Tests\PhpunitCompatibilityTrait::createMock() instead.
27 Call to deprecated method getMock() of class Drupal\Tests\UnitTestCase:
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Tests\PhpunitCompatibilityTrait::createMock() instead.
57 Call to deprecated method getMock() of class Drupal\Tests\UnitTestCase:
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Tests\PhpunitCompatibilityTrait::createMock() instead.
------ -------------------------------------------------------------------------

[ERROR] Found 10 errors

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

B N Pandey created an issue. See original summary.

B N Pandey’s picture

This is the patch for issue found by drupal check.

B N Pandey’s picture

Assigned: B N Pandey » Unassigned
Status: Active » Needs review

Marty2081’s picture

Status: Needs review » Fixed

Thanks for the patch. I've fixed some more deprecations and code style issues. Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.