While reviewing #11218: Allow default text formats per role, and integrate text format permissions, @dropcube pointed out that the $value parameter to filter_form() is very confusingly named, since it actually refers to a text format ID.

We should change it to something more clear.

CommentFileSizeAuthor
#1 filter-form-563272-1.patch1.43 KBDavid_Rothstein
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Status: Active » Needs review
FileSize
filter-form-563272-1.patch1.43 KB

Here's a simple patch.

Jody Lynn’s picture

Jody Lynn
she/her
Primary language English
CreditAttribution: Jody Lynn commented
Status: Needs review » Reviewed & tested by the community

Looks good

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)

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