Closed (duplicate)
Project:
Feeds
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
22 Jul 2010 at 08:27 UTC
Updated:
23 Jul 2012 at 22:49 UTC
Jump to comment: Most recent file
Comments
Comment #1
rehos commentedChanged status to needs review as I already included a patch
Comment #2
hanno commentedI am testing this. Great addition.
One remark the link to the feed in the logs is incorrect, in my situation the link 'feed' becomes
"/node/0"
It is feed 0, but for sure not node/0 :)
Comment #3
hanno commentedFound out what was happening. I am using a standalone form to import a file. The variabele $source->feed_nid is then always 0. To detect the right feed, it is in that case useful to link to import/$source->id
In the code I also propose to check for the right user access for showing messages. See #867054: Suppress status messages to anonymous users when using poormanscron
the snippet:
$message = t('There is no new content.');
}
if (user_access("administer feeds") || user_access("import $source->id feeds"))
drupal_set_message($message);
if ($source->nid ==0)
$link = l('feed '.$source->id, 'import/$source->id');//stand alone form
else
$link = l('feed '.$source->feed_nid, 'node/$source->feed_nid');//feed node
watchdog('feeds', $message, array(), WATCHDOG_INFO, $link);
Comment #4
hanno commented#867054: Suppress status messages to anonymous users when using poormanscron will hopefully be tackled in poormanscron, so #3 user_access patch can be dismissed. $source->feed_nid part still valid.
@rehos any chance to test this?
Comment #5
twistor commented7.x has custom logging. We should backport that to 6.x
See #606612: More detailed log.