The patch removes an unused node_load in og_save_subscription(). The extra node_load actually causes a problem, that I haven't completely tracked down. But since the node_load isn't need I hope it can just be removed.

The problem occurs when node_submit() is called via drupal_execute to auto-submit a CCK content type that is og-enabled. When this happens, nodapi insert gets called which goes down this path and calls og_save_subscription, which calls node_load, which loads the form it's saving, which somehow messes up the node that is being saved.

As I said, I haven't completely tracked it down yet, ... but the og but is pretty simple ... there's an extraneous node_load being called, and it should be removed.

CommentFileSizeAuthor
extra-node-load.patch881 bytesdouggreen

Comments

moshe weitzman’s picture

Status: Needs review » Fixed

committed. thx.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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