wip #44

LaraPalCom · Dec 14, 2015 · afd2151 · afd2151
1 parent dce91e2
commit afd2151
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 28 deletions.
diff --git a/src/Roumen/Feed/Feed.php b/src/Roumen/Feed/Feed.php
@@ -130,36 +130,22 @@ public function render($format = null, $cache = null, $key = null)
         if (empty($this->link)) $this->link = Config::get('application.url');
         if (empty($this->pubdate)) $this->pubdate = date('D, d M Y H:i:s O');
 
-        $this->pubdate = $this->formatDate($this->pubdate, $format);
-
         foreach($this->items as $k => $v)
         {
+            $this->items[$k]['title'] = html_entity_decode(strip_tags($this->items[$k]['title']));
             $this->items[$k]['pubdate'] = $this->formatDate($this->items[$k]['pubdate'], $format);
         }
 
         $channel = [
-            'title'=>$this->title,
-            'description'=>$this->description,
-            'logo' => $this->logo,
-            'icon' => $this->icon,
-            'link'=>$this->link,
-            'pubdate'=>$this->pubdate,
-            'lang'=>$this->lang
+            'title'         =>  html_entity_decode(strip_tags($channel['title'])),
+            'description'   =>  $this->description,
+            'logo'          =>  $this->logo,
+            'icon'          =>  $this->icon,
+            'link'          =>  $this->link,
+            'pubdate'       =>  $this->formatDate($this->pubdate, $format),
+            'lang'          =>  $this->lang
         ];
 
-        if ($format == 'rss')
-        {
-            $channel['title'] = html_entity_decode(strip_tags($channel['title']));
-            $channel['description'] = html_entity_decode(strip_tags($channel['description']));
-
-            foreach($this->items as $k => $v)
-            {
-                // escaping & in description, based on http://stackoverflow.com/questions/1328538/how-do-i-escape-ampersands-in-xml
-                $this->items[$k]['description'] = str_replace('&', '&amp;amp;', html_entity_decode(strip_tags($this->items[$k]['description'])));
-                $this->items[$k]['title'] = html_entity_decode(strip_tags($this->items[$k]['title']));
-            }
-        }
-
         $viewData = [
             'items'         => $this->items,
             'channel'       => $channel,

diff --git a/src/views/atom.blade.php b/src/views/atom.blade.php
@@ -1,7 +1,7 @@
 {!! '<'.'?'.'xml version="1.0" encoding="UTF-8" ?>' !!}
 <feed xmlns="http://www.w3.org/2005/Atom"<?php foreach($namespaces as $n) echo " ".$n; ?>>
-    <title type="html">{!! $channel['title'] !!}</title>
-    <subtitle type="html">{!! $channel['description'] !!}</subtitle>
+    <title type="text">{!! $channel['title'] !!}</title>
+    <subtitle type="html"><![CDATA[{!! $channel['description'] !!}]]></subtitle>
     <link href="{{ Request::url() }}"></link>
     <id>{{ $channel['link'] }}</id>
     <link rel="alternate" type="text/html" href="{{ Request::url() }}" ></link>
@@ -16,9 +16,9 @@
 @foreach($items as $item)
         <entry>
             <author>
-                <name><![CDATA[{!! $item['author'] !!}]]></name>
+                <name>{{ $item['author'] }}</name>
             </author>
-            <title type="html"><![CDATA[{!! $item['title'] !!}]]></title>
+            <title type="text">{!! $item['title'] !!}></title>
             <link rel="alternate" type="text/html" href="{{ $item['link'] }}"></link>
             <id>{{ $item['link'] }}</id>
             <summary type="html"><![CDATA[{!! $item['description'] !!}]]></summary>

diff --git a/src/views/rss.blade.php b/src/views/rss.blade.php
@@ -1,9 +1,9 @@
 {!! '<'.'?'.'xml version="1.0" encoding="UTF-8" ?>' !!}
 <rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom" xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:media="http://search.yahoo.com/mrss/"<?php foreach($namespaces as $n) echo " ".$n; ?>>
     <channel>
-        <title><![CDATA[{{ $channel['title'] }}]]></title>
+        <title>{!! $channel['title'] !!}</title>
         <link>{{ Request::url() }}</link>
-        <description>{{ $channel['description'] }}</description>
+        <description><![CDATA[{!! $channel['description'] !!}]]></description>
         <atom:link href="{{ $channel['link'] }}" rel="self"></atom:link>
 @if (!empty($channel['logo']))
         <image>