#2358999: Remove usage of drupal_html_to_text() removed the usage of drupal_html_to_text. Now, let's remove the function itself.

CommentFileSizeAuthor
#8 removed-drupal_html_to_text-2359069-8.patch1.31 KBpushpinderchauhan
#3 removed-drupal_html_to_text-2359069-3.patch1.38 KBkeso

Comments

aczietlow’s picture

aczietlow commented
Issue tags: +Novice
keso’s picture

keso commented
Assigned: Unassigned » keso
keso’s picture

keso commented
StatusFileSize
new removed-drupal_html_to_text-2359069-3.patch1.38 KB

This patch removes the reference to drupal_html_to_text(). It will break until patch #2358999 is committed. Leaving set as postponed until #2358999 is completed.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented

Fix the comment ;) it's better use [ # number] whitout spaces for issues number ;)

This patch removes the reference to drupal_html_to_text(). It will break until patch #2358999: Remove usage of drupal_html_to_text() is committed. Leaving set as postponed until #2358999: Remove usage of drupal_html_to_text() is completed.

keso’s picture

keso commented

Ah-ha! Thanks, still learning the ropes over here. :) Will do so in the future.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Postponed » Needs review

Status: Needs review » Needs work

The last submitted patch, 3: removed-drupal_html_to_text-2359069-3.patch, failed testing.

pushpinderchauhan’s picture

pushpinderchauhan commented
Status: Needs work » Needs review
StatusFileSize
new removed-drupal_html_to_text-2359069-8.patch1.31 KB
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 d7d3a69 and pushed to 8.0.x. Thanks!

  • alexpott committed d7d3a69 on 
    Issue #2359069 by er.pushpinderrana, keso | JeroenT: Remove...

Status: Fixed » Closed (fixed)

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