#2361797: Remove usage of drupal_strlen()
In this issue, let's remove the function itself.

CommentFileSizeAuthor
#5 2361799-5.patch737 bytesPalashvijay4O
#1 2361797-1.patch17.03 KBPalashvijay4O

Comments

Palashvijay4O’s picture

Palashvijay4O commented
StatusFileSize
new 2361797-1.patch17.03 KB

Test patch .

Palashvijay4O’s picture

Palashvijay4O commented
Status: Postponed » Needs review
rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Status: Needs review » Postponed

@Palashvijay4 first must be fixed #2361797: Remove usage of drupal_strlen(), to remove this function.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Postponed » Needs review
Palashvijay4O’s picture

Palashvijay4O commented
StatusFileSize
new 2361799-5.patch737 bytes

Previous patch was made on obsolete code . This is the correct one .

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

This issue is a prioritized change (removal of deprecated code) as per #2350615: [policy, no patch] What changes can be accepted during the Drupal 8 beta phase? and it's benefits outweigh any disruption. Committed 0287c86 and pushed to 8.0.x. Thanks!

  • alexpott committed 0287c86 on 
    Issue #2361799 by Palashvijay4O | rpayanm: Remove drupal_strlen.
rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Assigned: rpayanm » Unassigned

Status: Fixed » Closed (fixed)

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