This is a great module. Just one question -- would it be possible to add an option so that we can move the CCK fields above the comment form itself?

CommentFileSizeAuthor
#5 243943_comment_cck_form.patch537 byteschristefano

Comments

amariotti’s picture

amariotti commented

This definitely is a must...

Anonymous’s picture

Anonymous (not verified) commented

I achieved this by changing the line:

$form['comment_filter']['comment_cck']['#weight'] = 50;
to
$form['comment_filter']['comment_cck']['#weight'] = -50;

notice the '-'

This should be an option that is configurable through the admin UI

amariotti’s picture

amariotti commented

Way to go. You'd think that having it above the comment field would be where everyone would want it. Might just be me though. I think that module has some good potential. If I could get a bug fixed then I would be ready to roll with my project management site.

--Andrew

samhassell’s picture

samhassell commented

+1 for this suggestion - it mimicks casetracker, which a lot of people will be moving from.

not everyone can write a hook_form_alter().

christefano’s picture

christefano commented
Priority: Normal » Critical
Status: Active » Needs review
StatusFileSize
new 243943_comment_cck_form.patch537 bytes
samhassell’s picture

samhassell commented

I tested christefano's patch under 5.12. it works fine.

christefano’s picture

christefano commented

samhassell: Cool! My patch is just txcrew's proposed change in the form of a patch, though. Please mark this "reviewed & tested" if you think this should be in the next release.

samhassell’s picture

samhassell commented
Status: Needs review » Reviewed & tested by the community

ah true! well it's good so marking reviewed & tested.

this change actually makes the module more like Drupal.org, so it must be a good thing, right?

mitchell’s picture

mitchell commented
Version: 5.x-1.0 » 6.x-1.x-dev
Status: Reviewed & tested by the community » Fixed

Committed to 6.x. Thanks!

Status: Fixed » Closed (fixed)

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