Comments

snufkin’s picture

Only that it should be done. I haven't looked ad D6 CCK yet, so I can't make any predictions. Help on this update would be most welcome.

zbricoleur’s picture

I would be willing to help. What can I do?

snufkin’s picture

we need to port the module to CCK 2, for Drupal 6 (and potentially for Drupal 5, if there is a need for that).

sjpatrick’s picture

StatusFileSize
new16.63 KB
new2.26 KB
new898 bytes
new166 bytes

Here is a re-write of ExplainField Widget for CCK 2 on Drupal 6. As best I can tell from my own testing, I'm pretty certain it is feature complete compared to the 5.x-1 release. I've done nothing at all toward adding or changing anything explicitly from the 5.x-2-dev code. Minor tweaks I *did* make on my own include:

  • All javascript code has been moved into a separate .js file.
  • ExplainField text field's label is now optional.
  • Said text field is set to display as an actual textfield where rows is set to 1; otherwise it's a textarea. (I needed this to accomodate a particularly odd custom theme that I inherited.)
  • In text fields where a maximum character count is set, the user can optionally be notified of this in the field's help text. (This notice would be appended to any already-defined help text/description for the field.)
  • Likewise, the form error message delivered when this count is exceeded is verbose regarding the expected and actual character counts. (This is hard-coded in; this can be rather easily made optional if this behavior is found objectionable.)
snufkin’s picture

Wow awesome. I'll test it over the weekend and commit. Separating the JS is a good move, makes the whole code a lot cleaner.

aaronbauman’s picture

Works like a charm for me.
Thanks for the module and the port.

snufkin’s picture

sjpatrick if you want i can give you cvs access so you can commit the Drupal 6 port not depending on my free time.

sjpatrick’s picture

StatusFileSize
new17.24 KB
new2.24 KB
new543 bytes
new144 bytes

snufkin, thank you, I would very much like that. I've put in a application for contributions cvs access and I'll let you know when that has come through.

In the meantime, I discovered that my previous release was not acting correctly in the case where the option of unlimited multiples was selected. I moved over to the Drupal behaviors model and that appears to have fixed the problem.

These new files include this fix and a small amount of code cleanup.

AjK’s picture

sjpatrick CVS access approved and added to this project.

snufkin’s picture

thanks AjK! sjpatrick i dont know if you are familiar with CVS, to help a bit I just created the DRUPAL-6--1 branch of the module, where you can commit your code.

you can check it out and start coding with

export CVSROOT=:pserver:sjpatrick@cvs.drupal.org:/cvs/drupal-contrib
cvs login
cvs -z6 checkout -d explainfield-6--1 -r DRUPAL-6--1 contributions/modules/explainfield/
snufkin’s picture

Status: Active » Fixed

I am closing the issue as the Drupal 6 code has been committed. The 6.x-1.x-dev tarball will be out shortly thanks to sjpatrick.

Status: Fixed » Closed (fixed)

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