Closed (fixed)
Project:
Pollfield
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Jun 2007 at 19:35 UTC
Updated:
8 Mar 2008 at 12:02 UTC
Jump to comment: Most recent file
On delete of the node the pollfield tables do not get cleaned up. The node entries remain in the pollfield and pollfield_votes tables despite a delete of the underlying node.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | pollfield.module.151021.patch | 1.64 KB | jaydub |
Comments
Comment #1
jaydub commentedSee attached patch for a pass at fixing this.
Be advised that this patch includes the same code fixes for issue http://drupal.org/node/224814. Sorry but was more of a pain to split out at the moment.
Comment #2
karens commentedjaydub, you're my new best friend :)
I haven't had any time to spend working on this module with all the working going on with CCK and Date and you clearly have an interest and some understanding about it. Do you have any interest in maintaining, or co-maintaining this module? I'd love to have some help!
Either way, your patches are appreciated, I'll try to get them committed today.
Comment #3
karens commentedFixed in latest commit.
Comment #4
jaydub commentedSure Karen I'd love to be able to help out. Actually that brings me to a question I've had for awhile regarding CCK fields. I've looked high and low but haven't really found much in the way of helpful documentation for those wishing to write their own fields. For a work project I managed to get what I needed done but it would have been smoother with better docs.
Can you point me anywhere?
Comment #5
karens commentedGreat! Your contact tab is turned off so I can't contact you, but use my contact form to contact me and we'll work out the details.
The best documentation for fields is to use the field.php file that comes in the CCK package as an example to start from, and refer back to core and other widely-used CCK fields as examples -- Date, Imagefield, Link, Viewfield, Computed Field, and this one, Pollfield, all do some fairly complex CCK magic and can help you see how to do more complicated things.
Comment #6
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.