[TYPO3-ttnews] ReSOLVED: RSS Feed: link to external URLs duplicates "http://"

[Frans Saris]

Hi Dominik,

your solution doesn't make sense.

Explanation:
   when http:// is found return the value else return the value.

Thats the same as do nothing.

The problem is that strpos returns 0 because http:// starts at position 0.

I think is has to be:

$exturl = trim(eregi('^http://', $row['ext_url']) ? $row['ext_url'] :
'http://'.$row['ext_url']);

groeten
Frans

2007/7/1, Dominik Lindner <d-nik at indynews.net>:
>
> Hi, Dominik (?!)
>
> I have resolved the problem by changing line 1115 of the
> pi/class.tx_ttnews.php from
>
> $exturl =
>
> trim(strpos($row['ext_url'],'http://')?$row['ext_url']:'http://'.$row['ext_url']);
>
> to:
>
> $exturl =
> trim(strpos($row['ext_url'],'http://')?$row['ext_url']:$row['ext_url']);
>
> Greetings,
>
> D.
>
>
> Dominik Lindner schrieb:
> > Hi,
> >
> > I've reported the issue to the bugtracker.
> > There is another bug concerning the external links within the rss-feed
> > which i cannot confirm.
> >
> > I can't find any solution in the web, nor can I find anybody facing the
> > same problem.
> >
> >
> > Dominik
> >
> > Ingo Renner schrieb:
> >
> >> Dominik Lindner wrote:
> >>> Hello list,
> >>>
> >>> first of all thanks to all contributers for this great list!
> >>>
> >>> I'm using the rss (2.0) -Feature of tt_news (v. 2.5.0) and finally the
> >>> realurl-implementation works.
> >>>
> >>> Unfortunately the links to external news are shown as
> >>> "http://http://external-website.com" within the rss-feed page.
> >>>
> >>> Wihtin the normal page-type the links are rendered correctly, but with
> >>> "type=100" ist does not work.
> >> did you check the bugtracker whether this has been reported yet?
> >>
> >> if not, please do so... (check first, and then file a bug ;-)
> >>
> >>
> >> Ingo
> >>
> _______________________________________________
> TYPO3-project-tt-news mailing list
> TYPO3-project-tt-news at lists.netfielders.de
> http://lists.netfielders.de/cgi-bin/mailman/listinfo/typo3-project-tt-news
>