Not Compatible with CCK 6.x-2.1
Flying Drupalist - September 16, 2008 - 20:23
| Project: | Comment CCK |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed |
Description
I can't find this checkbox. I'm looking in the fields options and, etc and I don't see it anywhere.

#1
+1
subscribing
#2
subscribing
#3
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:
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).
#4
+1
#5
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
#6
@txcrew: does this work for you? I couldn't find where the module settings are.
#7
@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
#8
Sorry, i forget an extra underscore in the code. Suggested change in #5 works for CCK 6.x 2.0rc7.
#9
Confirmed
#10
Works for me, thanks!
#11
working with #5 on line 58 in module
thanks :)
cheers!!
sumitk
#12
Patch attached based on #5
#13
Changing title and prio.
#14
Well, looks good. Please commit.
#15
Tested and confirmed patch in #12.
Changing priority to critical.. as this module is unusable without this modification.
#16
patch in #12 works for me too...
#17
had same problem.
implemented change in comment #5.
Works now.
M.M.
#18
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...
#19
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.
#20
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.
#21
Committed. Thanks!
#22
Automatically closed -- issue fixed for 2 weeks with no activity.