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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aczietlow’s picture

Issue tags: +Novice
keso’s picture

Assigned: Unassigned » keso
keso’s picture

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

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

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

JeroenT’s picture

Status: Postponed » Needs review

Status: Needs review » Needs work

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

er.pushpinderrana’s picture

Status: Needs work » Needs review
FileSize
1.31 KB
JeroenT’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

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.