warning: array_merge() [function.array-merge]: Argument #2 is not an array in /modules/comment_cck/comment_cck.module on line 106.

adding "dvm($comment_fields);" to line 105 shows the variable to be NULL when submitting valid data.

CommentFileSizeAuthor
#1 comment_cck_fieldgroup_refactor.patch3.39 KBa_c_m

Comments

a_c_m’s picture

a_c_m commented
Title: Does not save updated CCK info in comment? » Missing fieldgroup module causes updates to fail
Status: Active » Needs review
StatusFileSize
new comment_cck_fieldgroup_refactor.patch3.39 KB

Attached is a patch that fixes this, it also re-factors some of the code in comment_cck_comment() to avoid code duplication.

a_c_m’s picture

a_c_m commented
Status: Needs review » Fixed

Committed - http://drupal.org/cvs?commit=241264

nashINATOR’s picture

nashINATOR commented
Status: Fixed » Needs work

This patch is causing the wrong comment to show the update message in:

For example if update was made to a cck field during creating comment #53 then the update will show up in #52 (one before)