Download & Extend
Filter Default » Issues

add support for CCK text fields

Posted by jruz on September 25, 2008 at 9:47pm
Project:Filter Default
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review

Issue Summary

:(

Login or register to post comments

Comments

#1

Posted by chellman on September 25, 2008 at 10:25pm
Priority:critical» normal
Status:active» postponed (maintainer needs more info)

What aspect isn't working for you? A sad emoticon does not a bug report make.

It works just fine here.

#2

Posted by jruz on September 27, 2008 at 4:25pm

sorry,

i have it installed in 3 diferent 6.4 installations y set the default values for every role (editor, admin) but is still is showing the defult input filter selected on every cck field (comments).

im now testing if it has something to do with cck or with self created input filters.

thanks for your time

#3

Posted by chellman on September 27, 2008 at 6:54pm
Category:bug report» feature request

Ah ha, now that's information I can use. Thank you for following up.

I just tested this, and can confirm that this doesn't work with CCK fields, only the body field. I'm changing this to a feature request. It's worth looking into.

#4

Posted by grendzy on October 1, 2008 at 6:29pm
Status:postponed (maintainer needs more info)» needs review

Here's a patch:

AttachmentSize
filter_default_cck_patch.txt 1.09 KB

#5

Posted by grendzy on October 1, 2008 at 6:33pm
Title:not working on 6.4» add support for CCK text fields

#6

Posted by jruz on October 6, 2008 at 4:25pm

works perfect!!!

thanks!

#7

Posted by moshe weitzman on November 19, 2008 at 8:11pm

Seems like this module could recurse through all of $form and change anything that looks like a format field. That would catch excerpt module and many others.

#8

Posted by moshe weitzman on November 20, 2008 at 5:32pm

Here is some code from mollom module that scans for any button on the form. The code needed for filter is similar

<?php
foreach (element_children($form) as $key) {
      // Iterate over the form elements looking for buttons:
      if (isset($form[$key]['#type']) && ($form[$key]['#type'] == 'submit' || $form[$key]['#type'] == 'button')) {
        // do stuff
      }
    }
?>

#9

Posted by chellman on November 20, 2008 at 5:44pm

Thanks, Moshe. I like this idea, and assuming no one else beats me to it, I'll take care of it soon. This is much more appealing than checking for each individual thing explicitly.

#10

Posted by grendzy on November 25, 2008 at 12:02am

I agree a module-agnostic approach would be best. I'm having trouble finding the correct fieldset, though, using this approach.

Actually my patch in #4 is giving me trouble on one site. I'm not sure why, but I needed to remove this conditional:

<?php
if (isset($form[$field->field_name][$element]['#default_value']['format'])) {
?>