Patch to not require Checkbox Validate module for Legal module

gsutcliffe - November 10, 2008 - 19:44
Project:Legal
Version:5.x-1.9
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:won't fix
Description

I always preferred not installing modules unless absolutely necessary. This patch will remove the dependency for the Checkbox Validate module.

#1

gsutcliffe - November 10, 2008 - 19:54

Bug in the patch. It was checking the form before being submitted as well. Obviously bad. So here is the new patch. Is there anyway I can change the status? I meant to save it as "code needs review".

AttachmentSize
legal.module.patch 876 bytes
legal.info_.patch 538 bytes

#2

EvanDonovan - March 23, 2009 - 19:39
Status:patch (to be ported)» needs review

Status changed. Does this patch simply include the code for the Checkbox Validate module or does it implement this functionality a different way?

#3

sun - July 4, 2009 - 10:26
Status:needs review» won't fix

As long as the bug in Drupal core exists, checkbox_validate module is required. Duplicating code is the worst you can do. This won't fix.

 
 

Drupal is a registered trademark of Dries Buytaert.