Главная Обзор Помощь
Вход
LBP
/
FressRSS
зеркало из https://github.com/FreshRSS/FreshRSS.git
4
0
Ответвить 0
Файлы Задачи 0 Вики
Просмотр исходного кода

Improve CSS filter to exclude content (#6807)

* Improve CSS filter to exclude content
Improve https://github.com/FreshRSS/FreshRSS/pull/6786
Fix bug when changing the CSS filter and reloading article, because original content was lost.
Fix similar bug with pathEntries.

* Forgotten i18n

* Type check
Alexandre Alapetite 1 год назад
Родитель
fd1b5e9343
Сommit
bea4ed69fc
24 измененных файлов с 45 добавлено и 33 удалено
Разделённый вид Показать статистику Diff
  1. 22 10
      app/Models/Entry.php
  2. 1 1
      app/i18n/cs/sub.php
  3. 1 1
      app/i18n/de/sub.php
  4. 1 1
      app/i18n/el/sub.php
  5. 1 1
      app/i18n/en-us/sub.php
  6. 1 1
      app/i18n/en/sub.php
  7. 1 1
      app/i18n/es/sub.php
  8. 1 1
      app/i18n/fr/sub.php
  9. 1 1
      app/i18n/he/sub.php
  10. 1 1
      app/i18n/hu/sub.php
  11. 1 1
      app/i18n/id/sub.php
  12. 1 1
      app/i18n/it/sub.php
  13. 1 1
      app/i18n/ja/sub.php
  14. 1 1
      app/i18n/ko/sub.php
  15. 1 1
      app/i18n/lv/sub.php
  16. 1 1
      app/i18n/nl/sub.php
  17. 1 1
      app/i18n/oc/sub.php
  18. 1 1
      app/i18n/pl/sub.php
  19. 1 1
      app/i18n/pt-br/sub.php
  20. 1 1
      app/i18n/ru/sub.php
  21. 1 1
      app/i18n/sk/sub.php
  22. 1 1
      app/i18n/tr/sub.php
  23. 1 1
      app/i18n/zh-cn/sub.php
  24. 1 1
      app/i18n/zh-tw/sub.php

+ 22 - 10
app/Models/Entry.php
Просмотреть файл 

@@ -176,7 +176,8 @@ class FreshRSS_Entry extends Minz_Model {
 
 	 * Provides the original content without additional content potentially added by loadCompleteContent().
 
 	 */
 
 	public function originalContent(): string {
 
-		return preg_replace('#<!-- FULLCONTENT start //-->.*<!-- FULLCONTENT end //-->#s', '', $this->content) ?? '';
 
+		return $this->attributeString('original_content') ??
 
+			preg_replace('#<!-- FULLCONTENT start //-->.*<!-- FULLCONTENT end //-->#s', '', $this->content) ?? '';
 
 	}
 
 
 	/**
 
@@ -897,13 +898,16 @@ HTML;
 
 						$originalContent = $this->originalContent();
 
 						switch ($feed->attributeString('content_action')) {
 
 							case 'prepend':
 
+								$this->_attribute('original_content');
 
 								$this->content = $fullContent . $originalContent;
 
 								break;
 
 							case 'append':
 
+								$this->_attribute('original_content');
 
 								$this->content = $originalContent . $fullContent;
 
 								break;
 
 							case 'replace':
 
 							default:
 
+								$this->_attribute('original_content', $originalContent);
 
 								$this->content = $fullContent;
 
 								break;
 
 						}
 
@@ -915,12 +919,12 @@ HTML;
 
 				}
 
 			}
 
 		} elseif (trim($feed->attributeString('path_entries_filter') ?? '') !== '') {
 
+			$originalContent = $this->attributeString('original_content') ?? $this->content;
 
 			$doc = new DOMDocument();
 
 			$utf8BOM = "\xEF\xBB\xBF";
 
-			if (!$doc->loadHTML($utf8BOM . $this->content, LIBXML_NONET | LIBXML_NOERROR | LIBXML_NOWARNING)) {
 
+			if (!$doc->loadHTML($utf8BOM . $originalContent, LIBXML_NONET | LIBXML_NOERROR | LIBXML_NOWARNING)) {
 
 				return false;
 
 			}
 
-			$modified = false;
 
 			$xpath = new DOMXPath($doc);
 
 			$filterednodes = $xpath->query((new Gt\CssXPath\Translator($feed->attributeString('path_entries_filter') ?? '', '//'))->asXPath()) ?: [];
 
 			foreach ($filterednodes as $filterednode) {
 
@@ -928,16 +932,24 @@ HTML;
 
 					continue;
 
 				}
 
 				$filterednode->parentNode->removeChild($filterednode);
 
-				$modified = true;
 
 			}
 
-			if ($modified) {
 
-				$html = $doc->saveHTML();
 
-				if (!is_string($html)) {
 
-					return false;
 
-				}
 
+			$html = $doc->saveHTML($doc->getElementsByTagName('body')->item(0) ?? $doc->firstElementChild);
 
+			if (!is_string($html)) {
 
+				return false;
 
+			}
 
+			$html = preg_replace('%^\s*<body>\s*|\s*</body>\s*$%i', '', $html);
 
+			$this->_attribute('original_content');
 
+			if (is_string($html) && $this->content !== $html) {
 
+				$this->_attribute('original_content', $originalContent);
 
 				$this->content = $html;
 
+				return true;
 
+			}
 
+		} else {
 
+			$originalContent = $this->originalContent();
 
+			if ($originalContent !== $this->content) {
 
+				$this->content = $originalContent;
 
+				return true;
 
 			}
 
-			return $modified;
 
 		}
 
 		return false;
 
 	}

+ 1 - 1
app/i18n/cs/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Přepínač CSS článku na původních webových stránkách',
 
 		'css_path_filter' => array(
 
 			'_' => 'selektor CSS prvků, které mají být odstraněny',
 
-			'help' => 'Selektor CSS může být seznam, například: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'Selektor CSS může být seznam, například: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Popis',
 
 		'empty' => 'Tento kanál je prázdný. Ověřte prosím, zda je stále udržován.',

+ 1 - 1
app/i18n/de/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'CSS-Selektor des Artikelinhaltes auf der Original-Webseite',
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS-Selector für die Elemente, die entfernt werden sollen',
 
-			'help' => 'CSS-Selector könnte eine Liste sein, wie z.B.: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'CSS-Selector könnte eine Liste sein, wie z.B.: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Beschreibung',
 
 		'empty' => 'Dieser Feed ist leer. Bitte stellen Sie sicher, dass er noch gepflegt wird.',

+ 1 - 1
app/i18n/el/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Article CSS selector on original website',	// TODO
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS selector of the elements to remove',	// TODO
 
-			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside</kbd>',	// TODO
 
+			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',	// TODO
 
 		),
 
 		'description' => 'Description',	// TODO
 
 		'empty' => 'This feed is empty. Please verify that it is still maintained.',	// TODO

+ 1 - 1
app/i18n/en-us/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Article CSS selector on original website',	// IGNORE
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS selector of the elements to remove',	// IGNORE
 
-			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside</kbd>',	// IGNORE
 
+			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',	// IGNORE
 
 		),
 
 		'description' => 'Description',	// IGNORE
 
 		'empty' => 'This feed is empty. Please verify that it is still maintained.',	// IGNORE

+ 1 - 1
app/i18n/en/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Article CSS selector on original website',
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS selector of the elements to remove',
 
-			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Description',
 
 		'empty' => 'This feed is empty. Please verify that it is still maintained.',

+ 1 - 1
app/i18n/es/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Ruta a la CSS de los artículos en la web original',
 
 		'css_path_filter' => array(
 
 			'_' => 'Selector CSS de los elementos a remover',
 
-			'help' => 'Un selector CSS puede ser una lista, por ejemplo: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'Un selector CSS puede ser una lista, por ejemplo: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Descripción',
 
 		'empty' => 'La fuente está vacía. Por favor, verifica que siga activa.',

+ 1 - 1
app/i18n/fr/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Sélecteur CSS des articles sur le site d’origine',
 
 		'css_path_filter' => array(
 
 			'_' => 'Sélecteur CSS des éléments à supprimer',
 
-			'help' => 'Un sélecteur CSS peut être une liste comme : <kbd>.footer, .aside</kbd>',
 
+			'help' => 'Un sélecteur CSS peut être une liste comme : <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Description',	// IGNORE
 
 		'empty' => 'Ce flux est vide. Veuillez vérifier qu’il est toujours maintenu.',

+ 1 - 1
app/i18n/he/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'נתיב הCSS של המאמר באתר המקורי',
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS selector of the elements to remove',	// TODO
 
-			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside</kbd>',	// TODO
 
+			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',	// TODO
 
 		),
 
 		'description' => 'תיאור',
 
 		'empty' => 'הזנה זו ריקה. אנא ודאו שהיא עדיין מתוחזקת.',

+ 1 - 1
app/i18n/hu/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Cikk CSS selector az eredeti weblapon',
 
 		'css_path_filter' => array(
 
 			'_' => 'A törlendő elemek CSS selectora',
 
-			'help' => 'Egy CSS selector lehet egy lista például: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'Egy CSS selector lehet egy lista például: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Leírás',
 
 		'empty' => 'Ez a hírforrás üres. Ellenőrizd hogy van e tartalom rajta.',

+ 1 - 1
app/i18n/id/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Article CSS selector on original website',	// TODO
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS selector of the elements to remove',	// TODO
 
-			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside</kbd>',	// TODO
 
+			'help' => 'A CSS selector may be a list such as: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',	// TODO
 
 		),
 
 		'description' => 'Description',	// TODO
 
 		'empty' => 'This feed is empty. Please verify that it is still maintained.',	// TODO

+ 1 - 1
app/i18n/it/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Percorso del foglio di stile CSS del sito di origine',
 
 		'css_path_filter' => array(
 
 			'_' => 'Il selettore CSS degli elementi da rimuovere',
 
-			'help' => 'Il selettore CSS potrebbe essere una lista, ad esempio: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'Il selettore CSS potrebbe essere una lista, ad esempio: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Descrizione',
 
 		'empty' => 'Questo feed non contiene articoli. Per favore verifica il sito direttamente.',

+ 1 - 1
app/i18n/ja/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => '元のWebサイトから記事を抽出するCSSセレクタ',
 
 		'css_path_filter' => array(
 
 			'_' => '要素を削除するCSSセレクタ',
 
-			'help' => 'CSSセレクタは次のようなリストです: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'CSSセレクタは次のようなリストです: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => '説明',
 
 		'empty' => 'このフィードは空です。サイトが運営されているかどうかを確認してみてください。',

+ 1 - 1
app/i18n/ko/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => '웹사이트 상의 글 본문에 해당하는 CSS 경로',
 
 		'css_path_filter' => array(
 
 			'_' => '제거할 요소의 CSS 선택자',
 
-			'help' => 'CSS 선택자는 다음과 같은 목록일 수 있습니다: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'CSS 선택자는 다음과 같은 목록일 수 있습니다: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => '설명',
 
 		'empty' => '이 피드는 비어있습니다. 피드가 계속 운영되고 있는지 확인하세요.',

+ 1 - 1
app/i18n/lv/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Raksta CSS selektors sākotnējā vietnē',
 
 		'css_path_filter' => array(
 
 			'_' => 'Noņemamo elementu CSS selektors',
 
-			'help' => 'CSS selektors var būt saraksts, piemēram.: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'CSS selektors var būt saraksts, piemēram.: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Apraksts',
 
 		'empty' => 'Šī barotne ir tukša. Lūdzu, pārbaudiet, vai tā joprojām tiek uzturēta.',

+ 1 - 1
app/i18n/nl/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'CSS-pad van artikelen op originele website',
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS selector van de elementen om te verwijderen',
 
-			'help' => 'Een CSS selector kan een lijst zijn, zoals: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'Een CSS selector kan een lijst zijn, zoals: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Omschrijving',
 
 		'empty' => 'Deze feed is leeg. Controleer of deze nog actueel is.',

+ 1 - 1
app/i18n/oc/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Selector CSS dels articles sul site d’origina',
 
 		'css_path_filter' => array(
 
 			'_' => 'Selector CSS de l’element de tirar',
 
-			'help' => 'Un selector CSS pòt èsser una lista coma : <kbd>.footer, .aside</kbd>',
 
+			'help' => 'Un selector CSS pòt èsser una lista coma : <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Descripcion',	// IGNORE
 
 		'empty' => 'Aqueste flux es void. Assegurats-vos qu’es totjorn mantengut.',

+ 1 - 1
app/i18n/pl/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Selektor CSS dla wiadomości na pierwotnej stronie',
 
 		'css_path_filter' => array(
 
 			'_' => 'Selektor CSS elementów do usunięcia',
 
-			'help' => 'Selector CSS może być listą, na przykład: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'Selector CSS może być listą, na przykład: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Opis',
 
 		'empty' => 'Ten kanał jest pusty. Należy sprawdzić czy kanał w dalszym ciągu działa.',

+ 1 - 1
app/i18n/pt-br/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Caminho do CSS do artigo no site original',
 
 		'css_path_filter' => array(
 
 			'_' => 'Seletor CSS dos elementos a serem removidos',
 
-			'help' => 'O seletor CSS pode ser uma lista com: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'O seletor CSS pode ser uma lista com: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Descrição',
 
 		'empty' => 'Este feed está vazio. Por favor verifique ele ainda é mantido.',

+ 1 - 1
app/i18n/ru/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'CSS селектор статьи на сайте',
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS селектор элемента для удаления',
 
-			'help' => 'CSS селектор может быть списком как: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'CSS селектор может быть списком как: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Описание',
 
 		'empty' => 'Лента пустая. Пожалуйста, убедитесь, что её до сих пор обслуживают.',

+ 1 - 1
app/i18n/sk/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Pôvodný CSS súbor článku z webovej stránky',
 
 		'css_path_filter' => array(
 
 			'_' => 'CSS selektor elementu na odstránenie',
 
-			'help' => 'CSS selektor môže byť zoznam ako: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'CSS selektor môže byť zoznam ako: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Popis',
 
 		'empty' => 'Tento kanál je prázdny. Overte, prosím, či je ešte spravovaný autorom.',

+ 1 - 1
app/i18n/tr/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => 'Makaleleri kendi CSS görünümü ile göster',
 
 		'css_path_filter' => array(
 
 			'_' => 'Kaldırılacak elemana ait CSS seçicisi',
 
-			'help' => 'CSS seçicisi şu şekilde olabilir: <kbd>.footer, .aside</kbd>',
 
+			'help' => 'CSS seçicisi şu şekilde olabilir: <kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => 'Tanım',
 
 		'empty' => 'Bu akış boş. Lütfen akışın aktif olduğuna emin olun.',

+ 1 - 1
app/i18n/zh-cn/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => '原文的 CSS 选择器',
 
 		'css_path_filter' => array(
 
 			'_' => '需移除元素的 CSS 选择器',
 
-			'help' => '可设置多个 CSS 选择器，例如：<kbd>.footer, .aside</kbd>',
 
+			'help' => '可设置多个 CSS 选择器，例如：<kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => '描述',
 
 		'empty' => '此源为空。请确认它是否正常更新。',

+ 1 - 1
app/i18n/zh-tw/sub.php
Просмотреть файл 

@@ -63,7 +63,7 @@ return array(
 
 		'css_path' => '原文的 CSS 選擇器',
 
 		'css_path_filter' => array(
 
 			'_' => '需移除元素的 CSS 選擇器',
 
-			'help' => '可設置多個 CSS 選擇器，例如：<kbd>.footer, .aside</kbd>',
 
+			'help' => '可設置多個 CSS 選擇器，例如：<kbd>.footer, .aside, p[data-sanitized-class="menu"]</kbd>',
 
 		),
 
 		'description' => '描述',
 
 		'empty' => '此源為空。請確認它是否正常更新。',