General question. In D6 FeedAPI I recall that the cron would import feed items for the parent feed even if it was unpublished. Is this the case with Feeds? Are there settings that need to be set?

If it does, is this the code to adjust to check if importer is published? Please advise on how to adjust it (sorry for my lack of programming skills in advance).

function feeds_cron() {
  if ($importers = feeds_reschedule()) {
    foreach ($importers as $id) {
      feeds_importer($id)->schedule();
      $rows = db_query("SELECT feed_nid FROM {feeds_source} WHERE id = :id", array(':id' => $id));
      foreach ($rows as $row) {
        feeds_source($id, $row->feed_nid)->schedule();
      }
    }
    feeds_reschedule(FALSE);
CommentFileSizeAuthor
#4 feeds-skip-unpublished-nodes-1195484-4.patch672 bytessteven.wichers

Comments

hellomobe’s picture

hellomobe commented

I don't have my cron set up yet to test this, but here is my change - feedback welcome.

function feeds_cron() {
  if ($importers = feeds_reschedule()) {
    foreach ($importers as $id) {
      feeds_importer($id)->schedule();
      //comment out original code
      //$rows = db_query("SELECT feed_nid FROM {feeds_source} WHERE id = :id", array(':id' => $id));
	//check to see if node is published 
 $rows = db_query("SELECT feed_nid FROM {feeds_source} as fs LEFT JOIN {node} as n on fs.feed_nid = n.nid WHERE fs.id = :id AND n.status='1'", array(':id' => $id)); 
	  foreach ($rows as $row) {
        feeds_source($id, $row->feed_nid)->schedule();
      }
    }
hellomobe’s picture

hellomobe commented

This isn't working. When I clear the cache it still runs the unpublished feeds. (I really don't understand why it's running when the cache is cleared).

twistor’s picture

twistor commented
Title: Don't import items if parent feed is unpublished » Don't import items if parent feed is unpublished.
Component: Miscellaneous » Code
Category: support » feature

This is doable, but it would have to be done in a backwards-compatible, and configurable way.

steven.wichers’s picture

steven.wichers commented
StatusFileSize
new feeds-skip-unpublished-nodes-1195484-4.patch672 bytes

Here's a patch for #1. I haven't tested if it actually works beyond comparing the result of a count() with and without restricting by node status (there was a difference).

liquidcms’s picture

liquidcms commented
Status: Active » Needs work

first test seems to show that patch in #4 does NOT work. will test more tomorrow.

tyler-durden’s picture

tyler-durden commented

I'm not a programmer so I can't help, but as a user I'd think there should be a way to control the feed of each individual importer. Right now your only option is to delete it to stop it feeding, unless I am missing something. I need this individual control, and unpublishing it to turn it off would make logical sense.

Did this get anywhere? Last post mentions looking into why it didn't work tomorrow, but "tomorrow" was over 19 weeks ago.

jadhavdevendra’s picture

jadhavdevendra commented
Issue summary: View changes

#4 does not work. Is anyone working on this?

jadhavdevendra’s picture

jadhavdevendra commented

Current workaround & assumptions -

  1. This code assumes that Feeds HTTP Fetcher is used as Fetcher
  2. This code assumes that you are writing code only for your importer
function MODULENAME_feeds_before_import(FeedsSource $source) {
  if ($source->id == 'YOUR_FEED_IMPORTER_MACHINE_NAME') {
    $status = db_query('SELECT status from {node} WHERE nid = :nid LIMIT 1', array(':nid' => $source->feed_nid))->fetchField();
    $config = $source->getConfig();
    if ($status == 0) {
      $config['FeedsHTTPFetcher']['source'] = '';
    }
    $source->setConfig($config);
  }
}
bluegeek9’s picture

bluegeek9 commented
Status: Needs work » Closed (outdated)
//www.flaticon.com/free-icons/thank-you Thank you for your contribution!

Unfortunately, Drupal 7 is End of Life and no longer supported. We strongly encourage you to upgrade to a supported version of Drupal.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.