Closed (fixed)
Project:
Legal
Version:
6.x-7.x-dev
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Reporter:
Created:
16 Mar 2008 at 20:05 UTC
Updated:
25 May 2008 at 17:10 UTC
Jump to comment: Most recent file
Is there going to be a d6 port of this module? Is there anything I can do to help?
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | legal-dev-d6.patch | 3.7 KB | jcfiala |
Comments
Comment #1
donBono commentedWhat is the situation with this?
Comment #2
robert castelo commentedI'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.
Comment #3
revival commented... watching ...
Comment #4
chipway commentedHi Roberto,
Any news ?
Thank you
Comment #5
coltraneRobert, 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!
Comment #6
robert castelo commentedI'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.
Comment #7
jcfiala commentedHey, 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.
Comment #8
robert castelo commentedClosing 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.