trackback_node table only contains nodes that were created after module install

giorgio79 - February 7, 2009 - 14:27
Project:TrackBack
Version:6.x-1.x-dev
Component:Receiving
Category:bug report
Priority:normal
Assigned:Cybergarou
Status:needs review
Description

Hello,

As per subject, I noticed only those nodes are included in the table that were created after the module was installed.

Does this mean, that nodes created before wont be able to receive trackbacks?

Cheers,
G

#1

Cybergarou - March 20, 2009 - 04:28
Category:support request» bug report
Assigned to:Anonymous» Cybergarou

I also noticed this. The table only contains nids for nodes created or modified after the module was installed. I ran a test and determined that you can receive trackbacks even if the old node the trackback is sent to isn't in the table. So from that angle, this isn't a problem.

I will have problems with this though. I'm working on a patch to add views support to the trackback module. Since nid is used to connect trackbacks to the nodes, the missing entries will produce incorrect results or even errors in some instances. I would prefer to keep this table in the mix of what will be available in views so I can do neat things like indicate trackbacks waiting for cron.

Solving this should be pretty simple so I'll add it to my list of things to do when I get back next week.

#2

thePanz - March 20, 2009 - 09:31

Thank you for joining this issue. I've been busy in latest months and I have some projects on the way for next weeks.
Feel free to post a patch/suggestions/code for Trackback module, I'll commit your changes for you.
Thanks

#3

Cybergarou - March 25, 2009 - 01:23
Status:active» needs review

Here's a patch to make sure the trackback_node table is complete.

With this patch, existing nodes are added during installation. For existing installations, the missing nodes are added through an update.

Just a note here, this patch isn't needed to make the Trackback module work correctly the way it is now, but it is required for the patch I'm working on to add views support.

AttachmentSize
trackback-370660.patch 1.5 KB
 
 

Drupal is a registered trademark of Dries Buytaert.