Drupal 6 Port please

Comments

jcfiala’s picture

Assigned: Unassigned » jcfiala

There's actually a Drupal 6 port that I'm working on right now - it was necessary for a client's site.

It contains a few enhancements - again, thanks to said client - although it will be backwards compatible with the D5 setup.

prameya’s picture

Could you upload your dev version?

We could start testing it... and maybe speed up the development process.

loze’s picture

subscribing

jcfiala’s picture

I honestly expected to get this out over the holiday, but then ran into problems with my home machine. I'm intending to get something up soon - I'm just not very good with cvs, and it takes a lot of work to figure out how to do what I want every time.

jcfiala’s picture

Version: 5.x-1.0 » 6.x-1.x-dev

Okay, I think I've correctly branched for 6.x, and created a dev release. I entirely intend to do more with this, but the tests don't pass (probably because of 5/6 problems and changes to simpletest) so I'm not going to mark it as a release yet.

Note that I've added something to the code, and I mis-remembered what I added, so naturally my commit message is wrong. There's a new attribute for each tag, called 'closed'. If 'closed' is true, then it means you're dealing with a tag that has two parts, and surrounds something else, like the anchor tag does: <a href="blah">stuff</a>. If 'closed' is false, then it means you're dealing with a standalone tag, like the image or <br /> tags. If 'closed' isn't specified (or is NULL), then the tag will be tried both ways.

I intend to clean up the tests and get them green, once I've got that going then I'll make an official D6 release.

dldege’s picture

I'm working with the dev version and this module looks really promising - nice work. I'll post any issues I find.

deleuje’s picture

subscribing

mtsanford’s picture

subscribing

jcfiala’s picture

The current dev release of the DME is very close to the final version.

It incorporates the context tag that got brought up in another ticket on this module - when you save a node, it sticks a little <dme:context> tag in the start of it that will tell the DME what the nid is when it's processing, which will solve a *bunch* of buggy behavior from earlier versions.

The todo list at this point is:

  • Finish creating the batch process that people can use to insert context tags in all of their existing nodes. It's not part of the install because if you're installing it doesn't matter if none of your nodes have context tags because none of them have dme tags yet either. And, it may be helpful for occasionally repairing data.
  • Make sure teasers are being handled properly - I think if the teaser is separate from the body, instead of being a sub-string, it's not quite working yet.
  • I should do some investigation of cck and see if I can quickly set it up to write the teasers into the cck fields as well that are filtered.

I'm currently planning on getting these issues finished and uploaded this weekend, by which I mean by 3/23/09.

jcfiala’s picture

Just added a new update - it's really stable for 6, and is being used on a client's site which should go live soon. This addition includes changes to the context tag so it doesn't get messed up by wysiwyg, by removing the context tag during the prepare nodeapi step and re-inserting it back in on presave.

jcfiala’s picture

This port is almost complete - I'm adding final features now, and I welcome folks adding any new comments/suggestions/problems.

guillaumeduveau’s picture

sub

jcfiala’s picture

Status: Active » Fixed

I've just published a 6.x-1.0. I realize this is pretty horribly late - sorry folks.

Status: Fixed » Closed (fixed)

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