If $node has some recursion, defaultcontent_export_node/features_var_export will get in a recursive loop and fail

One issue that causes this is if something like search_cron has indexed the node in the same page load and node uses user/node references, those module may add recursion there

Reproducible using drush site-install openpublic

CommentFileSizeAuthor
#1 2049527-defaultcontent_node_load_refresh-1.patch467 byteshefox
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hefox’s picture

hefox CreditAttribution: hefox commented
Status: Active » Needs review
FileSize
2049527-defaultcontent_node_load_refresh-1.patch467 bytes

Here's a patch that addresses the fact that the node may have been put through node_view via refreshing the load with $reset TRUE

hefox’s picture

hefox CreditAttribution: hefox commented

#2049529: features_var_export is suspectable to recursion is features issue, but imo defaultcontent should not be sending a dirty/recursive node

kenorb’s picture

kenorb CreditAttribution: kenorb commented
Issue summary: View changes
Related issues: +#2049529: features_var_export is suspectable to recursion