12/12 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

------ ---------------------------------------------------------------------------
Line drush/delete_all.drush.inc
------ ---------------------------------------------------------------------------
199 Call to deprecated function node_type_get_types():
in Drupal 8.x, will be removed before Drupal 9.0.
Use \Drupal\node\Entity\NodeType::loadMultiple().
291 Call to deprecated function entity_get_bundles():
in Drupal 8.x-dev and will be removed before Drupal 9.0.0. Use
\Drupal\Core\Entity\EntityTypeBundleInfoInterface::getBundleInfo() for a
single bundle, or
\Drupal\Core\Entity\EntityTypeBundleInfoInterface::getAllBundleInfo() for
all bundles.
------ ---------------------------------------------------------------------------

------ ----------------------------------------------------------------------
Line includes/delete_all.content.batch.inc
------ ----------------------------------------------------------------------
83 Call to deprecated function drupal_set_message():
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
------ ----------------------------------------------------------------------

------ ----------------------------------------------------------------------
Line includes/delete_all.entities.batch.inc
------ ----------------------------------------------------------------------
62 Call to deprecated function drupal_set_message():
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
------ ----------------------------------------------------------------------

------ ----------------------------------------------------------------------
Line includes/delete_all.user.batch.inc
------ ----------------------------------------------------------------------
53 Call to deprecated function user_load():
iin Drupal 8.0.0 and will be removed before Drupal 9.0.0. Use
Drupal\user\Entity\User::load().
82 Call to deprecated function drupal_set_message():
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
------ ----------------------------------------------------------------------

------ ---------------------------------------------------------------------------
Line src/Commands/DeleteAllCommands.php
------ ---------------------------------------------------------------------------
Class Drush\Commands\DrushCommands not found and could not be autoloaded.
------ ---------------------------------------------------------------------------

------ ----------------------------------------------------------------------
Line src/Form/ContentDelete.php
------ ----------------------------------------------------------------------
111 Call to deprecated function drupal_set_message():
in Drupal 8.5.0 and will be removed before Drupal 9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
------ ----------------------------------------------------------------------

[ERROR] Found 8 errors

CommentFileSizeAuthor
#16 3074029_16-Drupal-9-compatibility.patch6.95 KBmsuthars
#14 3074029_14-Drupal-9-compatibility.patch6.95 KBmsuthars
#8 Screenshot 2020-04-18 at 3.13.44 PM.png620.72 KBmsuthars
#8 3074029_8-Drupal-9-compatibility.patch6.97 KBmsuthars
#7 3074029_7.patch3.63 KBshimpy
#2 delete_all-3074029-1.patch3.93 KBjoy29
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joy29 created an issue. See original summary.

joy29’s picture

joy29 CreditAttribution: joy29 at Srijan | A Material+ Company commented
FileSize
delete_all-3074029-1.patch3.93 KB

patch for drupal 9 compatibility

joy29’s picture

joy29 CreditAttribution: joy29 at Srijan | A Material+ Company commented
Assigned: joy29 » Unassigned
Status: Active » Needs review
piyuesh23’s picture

piyuesh23 CreditAttribution: piyuesh23 at QED42 commented
Issue tags: +DIACWApril2020
dipakmdhrm’s picture

dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer and at QED42 commented
Status: Needs review » Needs work 
  1. +++ b/drush/delete_all.drush.inc
@@ -288,7 +290,7 @@ function drush_delete_all_delete_entities() {
+    $bundle_definitions = EntityTypeBundleInfoInterface::getAllBundleInfo($entity_type_options);

    Use entity type storage service to get list of bundles.

msuthars’s picture

msuthars
Location Pune
CreditAttribution: msuthars as a volunteer and at QED42 for Drupal India Association commented
Assigned: Unassigned » msuthars
shimpy’s picture

shimpy
Location India
CreditAttribution: shimpy at UniMity Solutions Pvt Limited for Drupal India Association commented
Status: Needs work » Needs review
FileSize
3074029_7.patch3.63 KB

Revised for with drupal 9 compatibility with deprecated functions

msuthars’s picture

msuthars
Location Pune
CreditAttribution: msuthars as a volunteer and at QED42 for Drupal India Association commented
FileSize
3074029_8-Drupal-9-compatibility.patch6.97 KB
Screenshot 2020-04-18 at 3.13.44 PM.png620.72 KB

Patch for Drupal 9 compatibility.

Resolved below deprecated issue:

msuthars’s picture

msuthars
Location Pune
CreditAttribution: msuthars as a volunteer and at QED42 for Drupal India Association commented
Assigned: msuthars » Unassigned
dipakmdhrm’s picture

dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer and at QED42 for Drupal India Association commented
Assigned: Unassigned » dipakmdhrm
dipakmdhrm’s picture

dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer and at QED42 for Drupal India Association commented

Great work on the path @msuthars!

Just a small change needed:

+++ b/drush.services.yml
@@ -3,3 +3,6 @@ services:
+      arguments:
+        - '@entity_type.manager'
+        - '@entity_type.bundle.info'

This causes syntax error when enabling the module.
arguments should not be a part of 'tags', it should be in line with tags and should follow a different syntax.
Example syntax from ctools.services.yml:

arguments: ['@form_builder', '@event_dispatcher', '@renderer']

dipakmdhrm’s picture

dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer and at QED42 for Drupal India Association commented
Status: Needs review » Needs work
msuthars’s picture

msuthars
Location Pune
CreditAttribution: msuthars as a volunteer and at QED42 for Drupal India Association commented
Assigned: dipakmdhrm » msuthars
msuthars’s picture

msuthars
Location Pune
CreditAttribution: msuthars as a volunteer and at QED42 for Drupal India Association commented
Assigned: msuthars » Unassigned
Status: Needs work » Needs review
FileSize
3074029_14-Drupal-9-compatibility.patch6.95 KB

@dipakmdhrm Thanks for the info, I updated the patch.

msuthars’s picture

msuthars
Location Pune
CreditAttribution: msuthars as a volunteer and at QED42 for Drupal India Association commented
Assigned: Unassigned » msuthars
Status: Needs review » Needs work
msuthars’s picture

msuthars
Location Pune
CreditAttribution: msuthars as a volunteer and at QED42 for Drupal India Association commented
Assigned: msuthars » Unassigned
Status: Needs work » Needs review
FileSize
3074029_16-Drupal-9-compatibility.patch6.95 KB

Latest updated patch for Drupal 9 compatibility.

dipakmdhrm’s picture

dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer and at QED42 for Drupal India Association commented
Status: Needs review » Reviewed & tested by the community

  • dipakmdhrm committed 930146b on 8.x-1.x authored by msuthars 
    Issue #3074029 by msuthars, joy29, dipakmdhrm: Drupal 9 Deprecated Code...
dipakmdhrm’s picture

dipakmdhrm CreditAttribution: dipakmdhrm as a volunteer and at QED42 for Drupal India Association commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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