Posted by James Andres on March 4, 2009 at 10:04pm
Jump to:
| Project: | COPPA |
| Version: | 6.x-2.0-beta1 |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Needing to use the COPPA module in a Drupal 6.x site, I made the following changes to fix a variety of bugs and glitches:
- Added theme handlers to all forms, so theme_* functions will work.
- Fixed 5.x style form arguments, eg: my_form($form_id, $edit)
- Changed many $edit variables to $form, since this is what they are in 6.x
- Removed unnecessary use of unserialize after variable_(get|set)
- Remove extraneous coppa_admin_settings_submit() function, not needed in 6.x
- Fixed missing {table_name} wrappers on some SQL queries.
- Added a permissions string for limiting who sees the "Sign COPPA compliance" menu item
- Changed all true and false to TRUE and FALSE. OK so I'm being a bit picky ;-)
Please let me know if you see any issues with the patch!
| Attachment | Size |
|---|---|
| coppa_d6_fixes-jamesandres.patch | 14.43 KB |
Comments
#1
Hi I'll likely need to use, or build off of this module & patch, and was just wondering if anything had been developed since this patch was created?
#2
Thanks for the patch! I've made a 6.x-2.0-beta1 release based on this and it will appear after the packaging scripts have run.
#3
#4
Automatically closed -- issue fixed for 2 weeks with no activity.