Filters have an internal name, like "filter_html", so both $filter->name and {filter}.name make sense, because they refer to a machine name.

Formats, however, use a user-configurable title, like "Filtered HTML". But that value is currently stored in {filter_format}.name instead of .title. Totally confusing.

{filter_format}.name should be renamed to {filter_format}.title.

And while being there, we also want to fix that filter.test file, where the code refers to "filter" instead of "format" all over the place, but meaning "format" in reality.

Comments

sun’s picture

Issue tags: +API clean-up

Tagging for feature freeze.

dropcube’s picture

Moving here some clean-ups I implemented for #562932: {filter_format}.cache is not saved

sun’s picture

Title: Rename {filter_format}.name to title, fix confusion in tests » Rename {filter_format}.name to .title and replace .format with machine-readable .name
Category: bug » task
Priority: Normal » Critical
sun’s picture

sun’s picture

Issue tags: +Security improvements

Hmmm... PHP module could really use this: http://api.drupal.org/api/function/php_install/7

sun’s picture

Version: 7.x-dev » 8.x-dev
Assigned: sun » Unassigned
Priority: Critical » Normal

This is too much for D7. But this is also a bit in conflict with my plans for the total filter system revamp in D8.

dagmar’s picture

sun’s picture

Status: Active » Closed (duplicate)

Totally.

Also, half of this issue is obsolete, since we introduced machine names for text formats in D7 already.