Not Compatible with CCK 6.x-2.1

+1
subscribing

subscribing

This seems to be an affect from CCK 6.x 2.0rc7 (http://drupal.org/node/306274). Not quite sure where it went wrong given the list of changes/fixes. My guess would be here:

The admin forms (field creation, field edition...) have been renamed to comply with usual form
naming conventions. Modules and custom code that rely on those form ids through hook_form_alter() or
drupal_execute() will need to be updated.

Since I really have no background in programming, maybe someone else can give a nudge. Perhaps for an immediate solution, rollback to CCK 6.x 2.0rc6 (http://drupal.org/node/295736).

+1

After further investigation I believe to have tracked down the line to change. Please forgive my lack of programming knowledge.

I changed line #58 from

  if ($form_id == '_content_admin_field') {

to

  if ($form_id == 'content_field_edit_form') {

Hope this is a step in the right direction.

txcrew

@txcrew: does this work for you? I couldn't find where the module settings are.

@drupaloSa

Note: I am using 6.x version of comment CCK.

Yes, this does work for me.

I had to make the changes in #5 in comment_cck.module in order to see the checkbox on the field settings page (admin/content/node-type/MY_CONTENT_TYPE/fields/MY_FIELD_NAME

Sorry, i forget an extra underscore in the code. Suggested change in #5 works for CCK 6.x 2.0rc7.

Confirmed

Works for me, thanks!

working with #5 on line 58 in module
thanks :)

cheers!!
sumitk

Patch attached based on #5

Changing title and prio.

Well, looks good. Please commit.

Tested and confirmed patch in #12.

Changing priority to critical.. as this module is unusable without this modification.

patch in #12 works for me too...

had same problem.
implemented change in comment #5.
Works now.

M.M.

wonder if this could go in a new release? plan to use this module in an installation profile and thus a working module out of the box would be nice...

This patch conflicts with #295478: Double appearance of checkbox "Allow comments to alter this field". I've attached a new version to that issue, so the patch in #12 should be applied first.

But yes, rolling this into a new release would rock, as this module currently is unusable without it.

I've written a new, unified patch that fixes this issue along with a few others. It's listed under #420092: Bring comment_cck up-to-date, combine patches and can (should) be applied in place of the patch above.

Committed. Thanks!

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