Building on the new syntax used to pass arguments to exportable default_hooks, here is a patch that also allows the exportable to specify a different file for a specific function output. For example, placing some exportable functions in the component-specific file, while placing other functions (like alter hooks) into the main features.inc exported file.

CommentFileSizeAuthor
#5 features_different_export_files-1479068-5.patch1.26 KBmpotter
#2 features_different_export_files-1479068-2.patch1.29 KBmpotter
#1 features_different_export_files-1479068-1.patch826 bytesmpotter

Comments

mpotter’s picture

mpotter commented
Status: Active » Needs review
StatusFileSize
new features_different_export_files-1479068-1.patch826 bytes

Here is the patch:

mpotter’s picture

mpotter commented
StatusFileSize
new features_different_export_files-1479068-2.patch1.29 KB

Here is a new version that also prevents Features from creating an empty file when no content was written to it.

rypit’s picture

rypit commented
Status: Needs review » Reviewed & tested by the community

Tested this with patch on a myriad of features: created, diffed, updated and reverted exportables of context, boxes, views, menus, taxonomies and fields. All work as intended.

hefox’s picture

hefox commented 
+++ b/features.export.inc
@@ -250,13 +250,19 @@ function features_export_render($export, $module_name, $reset = FALSE) {
   foreach ($code as $filename => $contents) {
-    $code[$filename] = "<?php\n/**\n * @file\n * {$module_name}.{$filename}.inc\n */\n\n". implode("\n\n", $contents) ."\n";
+    if (count($contents) > 0) {
+      $code[$filename] = "<?php\n/**\n * @file\n * {$module_name}.{$filename}.inc\n */\n\n". implode("\n\n", $contents) ."\n";
+    }
+    else {
+      unset($code[$filename]); // empty file
+    }

The if else could just be simplified as

$code = array_filter($code); before the foreach

Not sure if worth needs working it for that.

mpotter’s picture

mpotter commented
StatusFileSize
new features_different_export_files-1479068-5.patch1.26 KB

Yep, good suggestion. Here is the new patch.

mpotter’s picture

mpotter commented
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to 7179946

mpotter’s picture

mpotter commented
Version: 7.x-1.x-dev » 6.x-1.x-dev

  • mpotter committed 7179946 on 8.x-3.x 
    Issue #1479068 by mpotter: Added Allow exportable code written to...
  • mpotter committed 8cb86ed on 8.x-3.x
    Issue #1479068 by mpotter: Added Allow exportable code written to...