Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#6 3183938-2.patch676 bytesShruthi Shetty M
#2 3183938-1.patch1.54 KBShruthi Shetty M

Issue fork node_clone-3183938

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Shruthi Shetty M created an issue. See original summary.

Shruthi Shetty M’s picture

Shruthi Shetty M CreditAttribution: Shruthi Shetty M at Specbee commented
Status: Active » Needs review
FileSize
3183938-1.patch1.54 KB

Deprecated method strtolower() , entityManager(), drupal_set_message() is got replaced.

doidd made their first commit to this issue’s fork.

larisse’s picture

larisse CreditAttribution: larisse at CI&T commented
Status: Needs review » Needs work

Hi! The patch applies and work ok, but I notice that in line 216 of node_clone.pages.inc file, there is a deprecated function (entityManager()). I think that this should be fixed too.

Shruthi Shetty M’s picture

Shruthi Shetty M CreditAttribution: Shruthi Shetty M at Specbee commented
Assigned: Unassigned » Shruthi Shetty M
Shruthi Shetty M’s picture

Shruthi Shetty M CreditAttribution: Shruthi Shetty M at Specbee commented
Status: Needs work » Needs review
FileSize
3183938-2.patch676 bytes

Hi @larisse.. As per the above comment, in line 216 of node_clone.pages.inc file added a deprecated function for (entityManager()) as
(entityTypeManager()). Please review the patch.