I can't find this checkbox. I'm looking in the fields options and, etc and I don't see it anywhere.

CommentFileSizeAuthor
#12 comment_cck_309367_01.patch641 bytessanduhrs

Comments

arhak’s picture

arhak commented

+1
subscribing

drupaloSa’s picture

drupaloSa commented

subscribing

Anonymous’s picture

Anonymous (not verified) commented
Title: Allow Comments to alter this field - where? » Not Compatible with CCK 6.x 2.0rc7
Category: support » bug
Priority: Normal » Critical

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).

chasz’s picture

chasz commented

+1

Anonymous’s picture

Anonymous (not verified) commented

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.

drupaloSa’s picture

drupaloSa commented

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

Anonymous’s picture

Anonymous (not verified) commented

@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

drupaloSa’s picture

drupaloSa commented
Status: Active » Reviewed & tested by the community

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

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

Confirmed

RoboPhred’s picture

RoboPhred commented

Works for me, thanks!

sumitk’s picture

sumitk commented

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

cheers!!
sumitk

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
StatusFileSize
new comment_cck_309367_01.patch641 bytes

Patch attached based on #5

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
commented
Title: Not Compatible with CCK 6.x 2.0rc7 » Not Compatible with CCK 6.x-2.1
Priority: Critical » Normal

Changing title and prio.

Flying Drupalist’s picture

Flying Drupalist commented

Well, looks good. Please commit.

ultimateboy’s picture

ultimateboy commented
Priority: Normal » Critical

Tested and confirmed patch in #12.

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

criz’s picture

criz
he/him
Location Vienna/Austria
commented

patch in #12 works for me too...

manimejia’s picture

manimejia commented

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

M.M.

criz’s picture

criz
he/him
Location Vienna/Austria
commented

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...

joshuajabbour’s picture

joshuajabbour commented

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.

joshuajabbour’s picture

joshuajabbour commented

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.

mitchell’s picture

mitchell commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

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