There are some exportable components that just don't make sense to export—variables, for example, like site_offline, cron_last, and css_js_query_string, whose values are designed to be transient. Their presence in the list of available components adds clutter, making it easy to overlook uncaptured configuration, and leaves them susceptible to accidental selection. I propose that there should be a mechanism for specifying that certain components be omitted from the list of available components—perhaps a hook of some sort. (There isn't one already, is there?)

CommentFileSizeAuthor
#8 features-options-alter-1999254-7.patch1.02 KBfrankcarey
#4 features-options-alter-1999254-4.patch1.46 KBfrankcarey

Comments

hefox’s picture

hefox commented
Status: Active » Closed (duplicate)

#1153028: menu_links_features_export_options() dies with too many items. just added a settings tab where such functionality has been added.

Angry Dan’s picture

Angry Dan commented
Status: Closed (duplicate) » Active

This is different - looking at the settings tab I can include or exclude all variables, but not specific items.

Looking at the OP it seems to me that we should be able to expand all categories in the list (not just menu) and hide components?

lpalgarvio’s picture

lpalgarvio commented
Issue summary: View changes

we need a way of hiding items that we dont want / arent safe to export

frankcarey’s picture

frankcarey commented
StatusFileSize
new features-options-alter-1999254-4.patch1.46 KB

I just created a module for this, but it requires a simple patch for features to drupal_alter() the options. I'll update this ticket when I get the module posted up, but here is the patch for right now.

frankcarey’s picture

frankcarey commented

And here is the module https://www.drupal.org/project/features_banish , feedback very welcome :)

frankcarey’s picture

frankcarey commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: features-options-alter-1999254-4.patch, failed testing.

frankcarey’s picture

frankcarey commented
Status: Needs work » Needs review
StatusFileSize
new features-options-alter-1999254-7.patch1.02 KB

Fixed paths in patch.

frankcarey’s picture

frankcarey commented

Also, added a documentation page for listing features that should never be exported here https://www.drupal.org/node/2400531

frankcarey’s picture

frankcarey commented
jonhattan’s picture

jonhattan
Primary language Spanish
Location Plasencia
commented
Status: Needs review » Reviewed & tested by the community

That's a great addition.

  • mpotter committed a257eef on 7.x-2.x authored by frankcarey 
    Issue #1999254 by frankcarey: Add ability to ignore arbitrary components
mpotter’s picture

mpotter commented
Status: Reviewed & tested by the community » Fixed

Committed this to a257eef. Feel free to open another issue and propose a patch for Features to exclude obvious stuff from core if you feel it should be part of Features itself rather than features_banish.

frankcarey’s picture

frankcarey commented

Thanks mpotter :) Here is a ticket discussing how features_banish might make sense to be in core features: https://www.drupal.org/node/2505463

Status: Fixed » Closed (fixed)

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