I'm not entirely sure why simplefeed_link uses 'href' => 'feeds/refresh/' . $node->vid. Is this a typo? If it's not, why break from tradition and expose vids in the URL scheme, when the rest of core uses nids?

Comments

FiReaNGeL’s picture

FiReaNGeL commented

If I remember well I coded the manual refresh part and it was one of my first contributions, so yeah, just a mistake on my part. Didn't catch it cause I never found a use for the revision system. Just change it to nid :)

csevb10’s picture

csevb10 commented
Status: Active » Fixed

It now uses nid as would be expected.

m3avrck’s picture

m3avrck commented

Indeed, see this commit which fixes it entirely and improves performance: http://drupal.org/cvs?commit=100084

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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