Fix path_entries encoding (#4823)

* Fix path_entries encoding
#fix https://github.com/FreshRSS/FreshRSS/issues/4815

* Fix preview

3 files changed, 9 insertions, 2 deletions

diff --git a/app/Models/Category.php b/app/Models/Category.php
index e5da764d3..c4ca12fd3 100644
--- a/app/Models/Category.php
+++ b/app/Models/Category.php
@@ -54,6 +54,7 @@ class FreshRSS_Category extends Minz_Model {
 	public function kind(): int {
 		return $this->kind;
 	}
+	/** @return string HTML-encoded name of the category */
 	public function name(): string {
 		return $this->name;
 	}
diff --git a/app/Models/Entry.php b/app/Models/Entry.php
index 12cef68a5..47fcf3b4a 100644
--- a/app/Models/Entry.php
+++ b/app/Models/Entry.php
@@ -184,6 +184,7 @@ class FreshRSS_Entry extends Minz_Model {
 		return null;
 	}
 
+	/** @return string HTML-encoded link of the entry */
 	public function link(): string {
 		return $this->link;
 	}
@@ -589,10 +590,10 @@ class FreshRSS_Entry extends Minz_Model {
 				$this->content = $entry->content();
 			} else {
 				try {
-					// l’article n’est pas en BDD, on va le chercher sur le site
+					// The article is not yet in the database, so let’s fetch it
 					$fullContent = self::getContentByParsing(
 						htmlspecialchars_decode($this->link(), ENT_QUOTES),
-						$feed->pathEntries(),
+						htmlspecialchars_decode($feed->pathEntries(), ENT_QUOTES),
 						$feed->attributes()
 					);
 					if ('' !== $fullContent) {
diff --git a/app/Models/Feed.php b/app/Models/Feed.php
index 32b22edf2..f24ec1884 100644
--- a/app/Models/Feed.php
+++ b/app/Models/Feed.php
@@ -145,6 +145,7 @@ class FreshRSS_Feed extends Minz_Model {
 	public function name($raw = false): string {
 		return $raw || $this->name != '' ? $this->name : preg_replace('%^https?://(www[.])?%i', '', $this->url);
 	}
+	/** @return string HTML-encoded URL of the Web site of the feed */
 	public function website(): string {
 		return $this->website;
 	}
@@ -157,6 +158,7 @@ class FreshRSS_Feed extends Minz_Model {
 	public function priority(): int {
 		return $this->priority;
 	}
+	/** @return string HTML-encoded CSS selector */
 	public function pathEntries(): string {
 		return $this->pathEntries;
 	}
@@ -192,6 +194,7 @@ class FreshRSS_Feed extends Minz_Model {
 		return $this->ttl;
 	}
 
+	/** @return mixed attribute (if $key is not blank) or array of attributes, not HTML-encoded */
 	public function attributes($key = '') {
 		if ($key == '') {
 			return $this->attributes;
@@ -301,6 +304,7 @@ class FreshRSS_Feed extends Minz_Model {
 	public function _priority($value) {
 		$this->priority = intval($value);
 	}
+	/** @param string $value HTML-encoded CSS selector */
 	public function _pathEntries(string $value) {
 		$this->pathEntries = $value;
 	}
@@ -320,6 +324,7 @@ class FreshRSS_Feed extends Minz_Model {
 		$this->mute = $value < self::TTL_DEFAULT;
 	}
 
+	/** @param mixed $value Value, not HTML-encoded */
 	public function _attributes(string $key, $value) {
 		if ($key == '') {
 			if (is_string($value)) {