Closed (duplicate)
Project:
Drupal core
Version:
8.0.x-dev
Component:
filter.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
10 Sep 2009 at 16:02 UTC
Updated:
29 Jul 2014 at 18:27 UTC
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
Comment #1
sunTagging for feature freeze.
Comment #2
dropcube commentedMoving here some clean-ups I implemented for #562932: {filter_format}.cache is not saved
Comment #3
sunChanging scope.
The following issues need to be committed before work on this one can start:
#562908: {filter}.module is not saved
#562694: Text formats should throw an error and refuse to process if a plugin goes missing
#560738: Trying to delete a text format gives "page not found"
Comment #4
sunWhen doing this, we also need to store the "old_name" in the form. See http://api.drupal.org/api/function/user_image_style_delete/7 and http://api.drupal.org/api/function/user_image_style_save/7
Comment #5
sunHmmm... PHP module could really use this: http://api.drupal.org/api/function/php_install/7
Comment #6
sunThis is too much for D7. But this is also a bit in conflict with my plans for the total filter system revamp in D8.
Comment #7
dagmar@sun can we implement this in #1779026: Convert Text Formats to Configuration System ?
Comment #8
sunTotally.
Also, half of this issue is obsolete, since we introduced machine names for text formats in D7 already.