If tac_lite is also installed the node is cloned successfully, but a warning message is generated:

warning: Illegal offset type in /var/www/html/drupaltest/sites/pada.org/modules/tac_lite/tac_lite.module on line 287.

I'll post more when I have time to evaluate.

CommentFileSizeAuthor
#4 tac_lite_save_patch_4.diff536 bytespwolanin
#2 tac_lite_save_patch_2.diff1.29 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Project: Node clone » Taxonomy Access Control Lite

Yes, I see the same error when testing on a site with tac_lite. However, I think this is a problem with tac_lite. The node is being saved with taxonomy terms as objects, and tac_lite isn't checking for that case.

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new tac_lite_save_patch_2.diff1.29 KB

I modified tac_lite to confrom better to the current code in taxonomy_node_save. This seems to fix the problem, but it should be more thoroughly tested.

pwolanin’s picture

pwolanin commented
Title: Warning when tac_lite is also installed » Error on node_save() if $node->taxonomy contains objects

better title

pwolanin’s picture

pwolanin commented
Priority: Minor » Normal
StatusFileSize
new tac_lite_save_patch_4.diff536 bytes

doh- copied too much from taxonomy module. Attached (simpler) patch should be sufficient.

identity’s picture

identity commented

Not sure if this is related, but same error received when rearranging book pages from the outline interface within Admin > Content area.

RobRoy’s picture

RobRoy commented

This may relate to a bug in core http://drupal.org/node/197532. Also, see my NAT bug where I first posted this http://drupal.org/node/188377.

Dave Cohen’s picture

Dave Cohen commented
Status: Needs review » Fixed

This fix has been in the 5.x version for some time. Never applied to 4.7 (until now). Thanks pwolanin for the patch.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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