Browse Source

parser: use node->c14n() instead of expecting html in nodeValue

Andrew Dolgov 4 years ago
parent
commit
1383514ad9
2 changed files with 6 additions and 7 deletions
  1. 2 2
      classes/feeditem/atom.php
  2. 4 5
      classes/feeditem/rss.php

+ 2 - 2
classes/feeditem/atom.php

@@ -75,7 +75,7 @@ class FeedItem_Atom extends FeedItem_Common {
 				}
 			}
 
-			return $content->nodeValue;
+			return $content->c14n();
 		}
 	}
 
@@ -95,7 +95,7 @@ class FeedItem_Atom extends FeedItem_Common {
 				}
 			}
 
-			return $content->nodeValue;
+			return $content->c14n();
 		}
 
 	}

+ 4 - 5
classes/feeditem/rss.php

@@ -71,17 +71,16 @@ class FeedItem_RSS extends FeedItem_Common {
 		$contentB = $this->elem->getElementsByTagName("description")->item(0);
 
 		if ($contentA && !$contentB) {
-			return $contentA->nodeValue;
+			return $contentA->c14n();
 		}
 
-
 		if ($contentB && !$contentA) {
-			return $contentB->nodeValue;
+			return $contentB->c14n();
 		}
 
 		if ($contentA && $contentB) {
 			return mb_strlen($contentA->nodeValue) > mb_strlen($contentB->nodeValue) ?
-				$contentA->nodeValue : $contentB->nodeValue;
+				$contentA->c14n() : $contentB->c14n();
 		}
 	}
 
@@ -89,7 +88,7 @@ class FeedItem_RSS extends FeedItem_Common {
 		$summary = $this->elem->getElementsByTagName("description")->item(0);
 
 		if ($summary) {
-			return $summary->nodeValue;
+			return $summary->c14n();
 		}
 	}