Currently, the text formats and the filters they use are created on system_install(). This should be a responsibility of install profiles, not system module. Install profiles should be able to create the text format they want, select which is the default format, and choose which roles may use the text format.

The attached patch move the installation code from system.install to default.install

CommentFileSizeAuthor
profile-install-formats.patch5.03 KBdropcube

Comments

dropcube’s picture

dropcube commented

Requires #556832: Add text format API to manipulate formats

Status: Needs review » Needs work

The last submitted patch failed testing.

mfer’s picture

mfer commented

@dropcube - there are 2 install profiles shipped with core. They should be installed in both of them.

rickvug’s picture

rickvug commented

The makes a lot of sense. Any chance of this still making it into D7?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs work » Fixed

Fixed as part of #626024: filter_list_format() hits database too often / filter_format_save() doesn't save all filters

Status: Fixed » Closed (fixed)
Issue tags: -FilterSystemRevamp, -install profile

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

Issue tags: +FilterSystemRevamp, +install profile

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.