Download & Extend
Voting API » Issues

Only the update_6100() hook was using schema API, here's a patch for the rest of the goodness.

Posted by hailu on September 18, 2008 at 5:17pm
Project:Voting API
Version:6.x-2.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

I went through and updated 'update' hooks 2 and 4 through 9 to properly use schema API.

This is my first time touching the API, so a look over would be greatly appreciated.

AttachmentSize
votingapi_schema_api_update.patch12.21 KB
Login or register to post comments

Comments

#1

Posted by hailu on September 18, 2008 at 5:28pm

Ouch! There was a parse error in the last patch, please disregard the original attachment.

Here is the correct patch.

AttachmentSize
votingapi_schema_api_update.patch 13.18 KB

#2

Posted by eaton on October 9, 2008 at 9:13pm
Status:needs review» fixed

Committed the second version. Some modules (like CCK) are abandoning older update functions in the 6.x versions. In the future that might be something to consider, but for now this helps. Thanks!

#3

Posted by Anonymous (not verified) on October 23, 2008 at 9:28pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.