Closed (fixed)
Project:
CCK Select Other
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Reporter:
Created:
13 Feb 2011 at 14:42 UTC
Updated:
23 Mar 2011 at 17:51 UTC
Jump to comment: Most recent file
Check Problem:
1) Add 2 "cck other select" field in the same content type.
2) When display the form, only attached JavaScript function ("cckSelectOther") works in the last "cck other select field".
Apparently the error are in "cck_select_other.module" file (lines 238 to 245 and 265 to 272), when add drupal js settings.
I attach possible patch:
- cck_select_other.module: to work with multiples fields
- cck_select_other.js: to work with multiples fields (optimized code).
Sorry for my english, is very bad :(
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 1060332-js-fixes.patch | 7.49 KB | mradcliffe |
| cck_select_other.module.diff.txt | 1.6 KB | emilianot | |
| cck_select_other.js_.diff.txt | 2.69 KB | emilianot |
Comments
Comment #1
mradcliffeThank you for the patch. I worked on this afternoon, finally, and got it working.
I wrote a basic test, but I cannot assert it via SimpleTest assertions because of SimpleTest limitations. However, I can confirm it works in SimpleTest using debug.
Posting a new patch.
Comment #2
mradcliffeCommitted.