using the edit/add form to create a node to be used in a block, the values get stored by _nodesinblock_save, simply because the $node->nodesinblock_enable was on the form.

However if you create an unpublished node and later on try to publish this node by using a combination of node_load and node_save. The function_nodesinblock_save doesn't get called since $node->nodeinblock_enable is never added to the node object.

Why not just add _nodesisblock_load to an $op = load in hook_nodeapi?
This will provide the options to later update the node with values and update the block that node is in at the time time. Also you won't need to call _nodesisblock_load in seperate functions anymore, the values always exists in the node.

Also there is an error in the name of the function.. it should be _nodesinblock_load not _nodesisblock_load.

You can Assign it to me if you like, i'll add the patch when I get time.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TidalWave’s picture

The promised patch

TidalWave’s picture

Status: Active » Needs review