I haven't found any traces of filter_process_text() in the current codebase.

CommentFileSizeAuthor
#9 2897936-9.patch1.04 KBsomepal
#2 FilterInterface-refer-non-existent-function-2897936-2.patch1 KBarunkumark

Comments

Chi created an issue. See original summary.

arunkumark’s picture

arunkumark
Location Coimbatore
commented
Status: Active » Needs review
StatusFileSize
new FilterInterface-refer-non-existent-function-2897936-2.patch1 KB

@Chi thanks for rising issue.

In Drupal 8.x we won't have filter_process_text(). If we want to use process text, there is a field format called processed_text. This will help to sanitize the texts.

So I have created a patch for that will helps to remove the improper documentations.

chi’s picture

chi commented

In Drupal 8.x we won't have filter_process_text().

Well, I could not find this function in Drupal 7 as well. Do you happen to know when it was removed?

arunkumark’s picture

arunkumark
Location Coimbatore
commented

Well, I could not find this function in Drupal 7 as well

Not only the Drupal 7, filter_process_text() won't avail on Drupal 6 also.

chi’s picture

chi commented
Status: Needs review » Reviewed & tested by the community

Thanks.

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.0-alpha1 will be released the week of July 31, 2017, which means new developments and disruptive changes should now be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

star-szr’s picture

star-szr
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Needs review
Related issues: +#2217877: Text filters should be able to add #attached, #post_render_cache, and cache tags

Good catch! Maybe these should be changed to reference \Drupal\filter\Plugin\FilterInterface::process() instead of being removed.

It looks like these references were added in #2217877-18: Text filters should be able to add #attached, #post_render_cache, and cache tags (first patch on that issue) and as far as I can tell the function never existed in core.

idebr’s picture

idebr commented
Status: Needs review » Needs work

Updating status to 'Needs work' per #7

somepal’s picture

somepal commented
StatusFileSize
new 2897936-9.patch1.04 KB

No actual call to filter_process_text() found, Done as per #7

somepal’s picture

somepal commented
Status: Needs work » Needs review
idebr’s picture

idebr commented
Status: Needs review » Reviewed & tested by the community

@somepal Thanks for working on this issue!

The changes in the patch change all references of the non-existent function filter_process_text() to \Drupal\filter\Plugin\FilterInterface::process() in line with the suggestion by Cottser in #7

  • xjm committed 3cf53f5 on 8.5.x 
    Issue #2897936 by arunkumark, somepal, Chi, Cottser: check_markup() and...

  • xjm committed a80ef37 on 8.4.x 
    Issue #2897936 by arunkumark, somepal, Chi, Cottser: check_markup() and...
xjm’s picture

xjm
she/her
Primary language English
commented
Version: 8.5.x-dev » 8.4.x-dev
Status: Reviewed & tested by the community » Fixed
Issue tags: -Novice +mobile-novice

I grepped and confirmed these are the only remaining references. Thanks!

Committed to 8.5.x, and backported to 8.4.x as a docs fix.

chi’s picture

chi commented

-Novice +mobile-novice

Out of curiosity, what does mobile-novice stand for?

somepal’s picture

somepal commented

@xjm Thanks. Its my first patch which got committed in a while.

Status: Fixed » Closed (fixed)

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