Any word on when this might be available for D7? This is a great module we use in D6 and we are anxious to migrate. Thanks for all of your efforts!

Comments

Roulion’s picture

subscribe

keha3912’s picture

subscribe

koppie’s picture

Subscribing!

pukku’s picture

Hi! I'm hoping to start looking at Drupal 7 in the next few weeks. As this module is relatively simple (well, mostly), this will hopefully be one of the first I work on...

Ricky

d.jankovics’s picture

subscribe

pukku’s picture

Version: 6.x-2.0-beta1 » 7.x-2.x-dev
Assigned: Unassigned » pukku

Hi! I've created a first test at a port to D7. There are three things to note.

1. This is my first D7 module.

2. The style and title options now require quotes around their parameters. Double-quotes, actually. I am working to relax this a bit, but my guess is that they will still require some sort of quote from now on. This change may start to affect the D6 version as well, as there is a fatal bug in the parsing regex that I need to figure out.

3. I'm not sure about the cache problem. I'm still trying to figure out how to make sure that the collapse javascript library is loaded. Right now I can't seem to get it not loaded, but I haven't spent a lot of time on it. If anyone can tell me for sure that it is not loaded, and thus the collapsing doesn't happen, that would be great.

Thanks,
Ricky

pukku’s picture

Actually, I just changed the code that caused issue 2 above. The method of handling the tag is now much more robust, and will also try to fix missing quotes. If there is a quote in the title or style option though, things will not work the way you want them to.

pukku’s picture

Please test the beta that I've posted (it should show up in a few hours).

pukku’s picture

Status: Active » Fixed

7.x-2.1 released.

Status: Fixed » Closed (fixed)

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