Hi,
Please support cck role reference, thanks for creating a nice module
joseph
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2011-04-03_1082124-3.patch | 1.09 KB | arhak |
Hi,
Please support cck role reference, thanks for creating a nice module
joseph
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2011-04-03_1082124-3.patch | 1.09 KB | arhak |
Comments
Comment #1
arhak commenteddo you refer to rolereference?
Comment #2
jmolinas commentedyes
Comment #3
arhak commentedit seems pretty similar to userreference & nodereference (if not a total copycat)
nevertheless it deserves throughout testing (which I'm not doing)
to ensure it properly behaves for every possible combination,
i.e. single/multiple values, limited/unlimited values, required/non-required, and all that for each of its widgets
attached patch merely enables it for driven_diff, without any special consideration,
i.e. it just removes the "unsupported field type" constraint and hopes for the best
BTW: I noticed that having a single value, not required, select dropdown widget, doesn't leave an option to select "none"
Comment #4
jmolinas commentedye it behaves like just other cck reference module
thank you very much
Comment #5
arhak commentedwatch out, it doesn't mean they are the same.
nodereference does allow "None" option when the field is not required,
so there are differences between them
and thus, proper tests are due
Comment #6
jmolinas commentedI've tested it, it work no error so far
Comment #7
arhak commented@#6
ok, but it hasn't been committed yet
also, it would require further testing (beyond your particular cases, as stated in #3)