This module needs to be ported to Drupal 6. Patch to follow shortly.

CommentFileSizeAuthor
#1 joomla-486080-1.patch18.05 KBmalclocke

Comments

malclocke’s picture

Status: Active » Needs review
StatusFileSize
new18.05 KB

Patch attached. This patch does pretty much the minimum to bring the module up to d6. I have taken the liberty to change all the queries to use printf formatting, see http://drupal.org/writing-secure-code

The module needs a lot of changes to meet Drupal coding standards, but I have not included these here to make the changes more visible. I will supply a patch to meet coding standards separately.

I will be doing a lot of work on this module in the coming weeks, please let me know if there is anything I can do to help.

Malc

drupalnesia’s picture

Status: Needs review » Patch (to be ported)

malclocke,

I just give you access to "Joomla to Drupal" CVS access.

You can starting to upload your new version for Drupal 6. I suggest not to use dev label, using alpha1 is better. Why? With several devs then hard to determine which version involved when users send bug/request, users must state the date of his installed dev version.

Starting from joomla-6.x-1.0-alpha1 then alpha2 and so on, after that we can release betaX then rcX, until we confidence to give a stable label.

Thanks to contribute!

malclocke’s picture

Status: Patch (to be ported) » Fixed

Committed to HEAD

Status: Fixed » Closed (fixed)

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