Not Compatible with CCK 6.x 2.0rc7

Miraploy - September 16, 2008 - 20:23
Project:Comment CCK
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:patch (reviewed & tested by the community)
Description

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

#1

arhak - September 17, 2008 - 01:14

+1
subscribing

#2

drupaloSa - September 20, 2008 - 22:28

subscribing

#3

txcrew - September 21, 2008 - 03:45
Title:Allow Comments to alter this field - where?» Not Compatible with CCK 6.x 2.0rc7
Category:support request» bug report
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).

#4

chasz - September 21, 2008 - 04:08

+1

#5

txcrew - September 21, 2008 - 04:17

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

drupaloSa - September 21, 2008 - 11:24

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

#7

txcrew - September 21, 2008 - 14:05

@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

drupaloSa - September 21, 2008 - 14:14
Status:active» patch (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.

#9

agentrickard - September 26, 2008 - 19:10

Confirmed

#10

RoboPhred - September 29, 2008 - 15:15

Works for me, thanks!

#11

sumitk - October 1, 2008 - 11:15

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

cheers!!
sumitk

 
 

Drupal is a registered trademark of Dries Buytaert.