Moving a forum topic to another forum by itself is fine, but after checking the box to leave a shadow copy, there is the following error:

warning: in_array() [function.in-array]: Wrong datatype for second argument in \modules\forum.module on line 496.

I think this is all that's needed; making sure that $node->terms has something in it.

CommentFileSizeAuthor
#4 forum_21.patch739 bytesZen
#1 forum.module_12.patch660 byteswebchick
forum.module_11.patch691 byteswebchick

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
StatusFileSize
new forum.module_12.patch660 bytes

Hey, how about a patch that doesn't have debugging info in it? ;)

dries’s picture

dries commented

In what scenario can $terms be null/empty and is that normal?

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

Sorry, I am not sure. The patch was only intended to get rid of the warning... I came across this bug while testing something else and didn't have a chance to dig any deeper than that. I'm not sure if this is caused by a bug deeper in the system. :\

Zen’s picture

Zen commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new forum_21.patch739 bytes

Patch attached - the wrong variable was being accessed.

Straightforward patch - setting to RTC.

Thanks
-K

killes@www.drop.org’s picture

killes@www.drop.org commented
Status: Reviewed & tested by the community » Fixed

comitted.

Anonymous’s picture

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