Is there going to be a d6 port of this module? Is there anything I can do to help?

CommentFileSizeAuthor
#7 legal-dev-d6.patch3.7 KBjcfiala

Comments

donBono’s picture

What is the situation with this?

robert castelo’s picture

I've been sent various Legal 6 updates which I'll need to spend time looking through and testing, as well as finishing anything that's not covered.

I'll need to find a weekend to work on this.

There will be a version of Legal mdule for Drupal 6, thank you for your patience.

revival’s picture

... watching ...

chipway’s picture

Hi Roberto,

Any news ?

Thank you

coltrane’s picture

Robert, would you be willing to share here the updates you've gotten or did you want to attack the port yourself? I have some availability and am willing to help out with porting.

Thanks!

robert castelo’s picture

Version: master » 6.x-7.x-dev
Assigned: Unassigned » robert castelo

I've added a development snapshot of work so far (will be available in the next 12 hours).

There's a weird issue whereby checkboxes which are set to required don't trigger a error if not clicked, I noticed that checkboxes created with Profile have the same problem. We'll have to add our own validation code, which I've already done for the 'accept' checkbox on user/registration page.

By the way I've also added a SimpleTest file for testing registration accept/not accepted, works if T&Cs have already been set up (ideally I'd like to automate that bit as well).

Note: The dev version should not be used on a live site.

jcfiala’s picture

StatusFileSize
new3.7 KB

Hey, I tried your dev version and found a few bugs:

A) If you try to set up an account for someone else as admin, and they try to login, they can't successsfully accept the terms.
B) If you try to set up an account for someone else as admin, the checkbox is not disabled.
C) If you register yourself, it writes two rows to legal_accepted, one row with uid 0, the other one properly.

The enclosed patch fixes these problems.

robert castelo’s picture

Status: Active » Closed (fixed)

Closing this as update is well under way.

Please create a new issue to report bugs or provide patches - keep each one seperate to make them easier to deal with.