Everything with the feed works fine except that there is no link back to the original article. Without this, you can't read the article. Where should it be?

Comments

Ashraf Amayreh’s picture

Assigned: Unassigned » Ashraf Amayreh
Status: Active » Fixed

Please download the new release and let me know if it solves the problem.

tonythemediaguy’s picture

Now the module doesn't work at all. When I add a new feed, I see my category dropdown at the top and at the bottom of the page. So I choose my category I want my feed items to go into on both of the dropdowns, and then when it runs I get an error telling me that it can't find an include file in the feed_handlers folder named the same as the category I chose.

My category was Target (as in Target Dept Store) so it told me it couldn't open feed_handlers/Target.inc because it couldn't be found.

Ashraf Amayreh’s picture

Category: bug » support
Status: Fixed » Closed (works as designed)

When creating a feed, there's one drop down that points to the feed type you're aggregating from. This should be fixed at RSS20 if you're aggregating from RSS 2.0, it seems you mistakingly understood that this is where the items will go under, which is not the case.

As to the terms you'd like to assign your aggregated items to, you need to create vocabularies for your "Feed Items", then you will get more drop downs to assign your aggregated items to when creating feeds. This is probably where you should have added "Target" in. Hope this clears things up. Let me know if you face any problems.

Ashraf Amayreh’s picture

Status: Closed (works as designed) » Closed (fixed)

closing...

peterdeitz’s picture

Assigned: Ashraf Amayreh » peterdeitz
Priority: Normal » Critical
Status: Closed (fixed) » Active

Hi all,

I'm posting my question here instead of creating a new support request. The title of this request pretty much matches my issue.

I installed aggregation and everything seems to be working fine. I've setup about ten feeds, all of which are aggregating correctly.

My problem is that the node titles in the teasers do not link directly to the original articles even though I have checked "Link items to URLs". My understanding is that that selecting this checkbox would convert the destination link of all teaser titles and 'read more' hyperlinks to the original article.
Am I mistaken?

I have also enabled the "aggregation views" module and was expecting to see an option of adding a field with the feed item title linked to the original article. No such luck.

Please advice on the two issues I'm facing:
1) Node titles and 'read more' links in the teaser do not link to original article
2) Aggregation views does not have a field for seamlessly linking back to the original article (short of creating a list view and then modifying the template).

Thank you,
Peter

Ashraf Amayreh’s picture

Status: Active » Closed (fixed)

Actually, clicking "read more" or the node title still points to the article's page, nothing changes here by checking "Link items to URLs".

What does change is that you should find a new hyper link Inside the article's full page view called "original article" that points to the original URL. If you don't see it, then either the feed is lacking original urls, which is unlikely but possible, or the links are not printed out in the theming layer, which is also possible.

As to the aggregation views, this is not yet released, we need to test it a little more. So I really don't know how functional it is. I'll take care to keep a lookout on the issue you raised below when working on it. Let me know if I've answered your questions or if you have anything else in mind.

physiotek’s picture

thanks for the precision about the way it works with the 'link to original' box checked.
i am wondering if i can have the link appears also directly on the teaser?

thanks,
pht3k

physiotek’s picture

Status: Closed (fixed) » Postponed (maintainer needs more info)
vito_swat’s picture

You can have link directly on teasers, but it requires hack.

Change:

function aggregation_link($type, $node = NULL, $teaser = FALSE)
{
	$links = array();
	if (!$teaser && $type == 'node' && $node->type == 'aggregation_item')
		if ($node->link_to_original_url == 'yes' && valid_url($node->url, TRUE))
			$links[] = array(
				'href' => $node->url,
				'title' => t('Original article'),
				'attributes' => array('class' => 'aggregation-link')
			);

	return $links;
} 

to:

function aggregation_link($type, $node = NULL, $teaser = FALSE)
{
	$links = array();
	if ($type == 'node' && $node->type == 'aggregation_item')
		if ($node->link_to_original_url == 'yes' && valid_url($node->url, TRUE))
			$links[] = array(
				'href' => $node->url,
				'title' => t('Original article'),
				'attributes' => array('class' => 'aggregation-link')
			);

	return $links;
} 

There are plans to add additional checkboxes in configuration page for this but we need some more time to finish rewriting the module.

physiotek’s picture

Status: Postponed (maintainer needs more info) » Closed (fixed)

niiiiice!
thanks a lot
i will try this hack as soon the kids will be sleeping

physiotek’s picture

working like a charm :)