Active
Project:
Tumblr Connect
Version:
7.x-1.6
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
22 Mar 2012 at 05:21 UTC
Updated:
5 Dec 2012 at 00:12 UTC
What an awesome module. Thank you for sharing it!
I've tracked this down bug to lines 135 & 145 in tumblr_connect.module. Seems to be a problem with the Tumblr api, our side or theirs. Perhaps someone else know's the solution? In the meantime, I'm gonna look around myself...
Comments
Comment #1
mpaler commentedThe reason updating was failing is because I had revisions enabled on my node type. When the node is updated in drupal, the node->vid is always set to the next node_revisions auto increment before it is passed to getByNode(). This causes a mismatch with the last vid saved to tumblr_connect_node.vid
Rather than mess with trying to figure out the correct vid of the node being edited, I suggest we simply pass in the $nid to the getByNode. I'm assuming there will always only be one unique row per node in the tumblr_connect_node table -- so I'm not sure we ever have to narrow the search by vid.
Simply removing vid from the query in getByNode fixes it for me:
But fixing this the right way gets its fingers into a few different places so I want to confirm this direction before I write up a full patch.
Comment #2
mattwmc commentedI'm getting the same error.
Comment #3
socialnicheguru commentedI am getting the error in Drupal 7.
I have enabled the module but not configured it yet.
Comment #3.0
socialnicheguru commentedgive some thanks!