Browse Source

Changements syntaxiques mineurs aledeg/filters

https://github.com/marienfressinaud/FreshRSS/pull/352
Évite quelques appels de fonctions, et affectations inutiles
Alexandre Alapetite 12 years ago
parent
commit
a280bb1013
4 changed files with 17 additions and 19 deletions
  1. 1 0
      CHANGELOG
  2. 1 1
      app/i18n/en.php
  3. 1 1
      app/i18n/fr.php
  4. 14 17
      app/layout/nav_menu.phtml

+ 1 - 0
CHANGELOG

@@ -26,6 +26,7 @@
 	* Permet de modifier la description et l’adresse d’un flux RSS ainsi que le site Web associé
 	* Permet de modifier la description et l’adresse d’un flux RSS ainsi que le site Web associé
 	* Nouveau raccourci pour ouvrir/fermer un article (‘c’ par défaut)
 	* Nouveau raccourci pour ouvrir/fermer un article (‘c’ par défaut)
 	* Boutons pour effacer les logs et pour purger les vieux articles
 	* Boutons pour effacer les logs et pour purger les vieux articles
+	* Nouveaux filtres d’affichage : seulement les articles favoris, et seulement les articles lus
 * SQL :
 * SQL :
 	* Nouveau moteur de recherche, aussi accessible depuis la vue mobile
 	* Nouveau moteur de recherche, aussi accessible depuis la vue mobile
 		* Mots clefs de recherche “intitle:”, “inurl:”, “author:”
 		* Mots clefs de recherche “intitle:”, “inurl:”, “author:”

+ 1 - 1
app/i18n/en.php

@@ -44,7 +44,7 @@ return array (
 	'rss_view'			=> 'RSS feed',
 	'rss_view'			=> 'RSS feed',
 	'show_all_articles'		=> 'Show all articles',
 	'show_all_articles'		=> 'Show all articles',
 	'show_not_reads'		=> 'Show only unread',
 	'show_not_reads'		=> 'Show only unread',
-	'show_read'				=> 'Show only read',
+	'show_read'			=> 'Show only read',
 	'show_favorite'			=> 'Show favorites',
 	'show_favorite'			=> 'Show favorites',
 	'older_first'			=> 'Oldest first',
 	'older_first'			=> 'Oldest first',
 	'newer_first'			=> 'Newer first',
 	'newer_first'			=> 'Newer first',

+ 1 - 1
app/i18n/fr.php

@@ -44,7 +44,7 @@ return array (
 	'rss_view'			=> 'Flux RSS',
 	'rss_view'			=> 'Flux RSS',
 	'show_all_articles'		=> 'Afficher tous les articles',
 	'show_all_articles'		=> 'Afficher tous les articles',
 	'show_not_reads'		=> 'Afficher les non lus',
 	'show_not_reads'		=> 'Afficher les non lus',
-	'show_read'				=> 'Afficher les lus',
+	'show_read'			=> 'Afficher les lus',
 	'show_favorite'			=> 'Afficher les favoris',
 	'show_favorite'			=> 'Afficher les favoris',
 	'older_first'			=> 'Plus anciens en premier',
 	'older_first'			=> 'Plus anciens en premier',
 	'newer_first'			=> 'Plus récents en premier',
 	'newer_first'			=> 'Plus récents en premier',

+ 14 - 17
app/layout/nav_menu.phtml

@@ -103,21 +103,21 @@
 				$url_output = $url;
 				$url_output = $url;
 				$actual_view = Minz_Request::param('output', 'normal');
 				$actual_view = Minz_Request::param('output', 'normal');
 			?>
 			?>
-			<?php if($actual_view != 'normal') { ?>
+			<?php if($actual_view !== 'normal') { ?>
 			<li class="item">
 			<li class="item">
 				<?php $url_output['params']['output'] = 'normal'; ?>
 				<?php $url_output['params']['output'] = 'normal'; ?>
 				<a class="view_normal" href="<?php echo Minz_Url::display ($url_output); ?>">
 				<a class="view_normal" href="<?php echo Minz_Url::display ($url_output); ?>">
 					<?php echo Minz_Translate::t ('normal_view'); ?>
 					<?php echo Minz_Translate::t ('normal_view'); ?>
 				</a>
 				</a>
 			</li>
 			</li>
-			<?php } if($actual_view != 'reader') { ?>
+			<?php } if($actual_view !== 'reader') { ?>
 			<li class="item">
 			<li class="item">
 				<?php $url_output['params']['output'] = 'reader'; ?>
 				<?php $url_output['params']['output'] = 'reader'; ?>
 				<a class="view_normal" href="<?php echo Minz_Url::display ($url_output); ?>">
 				<a class="view_normal" href="<?php echo Minz_Url::display ($url_output); ?>">
 					<?php echo Minz_Translate::t ('reader_view'); ?>
 					<?php echo Minz_Translate::t ('reader_view'); ?>
 				</a>
 				</a>
 			</li>
 			</li>
-			<?php } if($actual_view != 'global') { ?>
+			<?php } if($actual_view !== 'global') { ?>
 			<li class="item">
 			<li class="item">
 				<?php $url_output['params']['output'] = 'global'; ?>
 				<?php $url_output['params']['output'] = 'global'; ?>
 				<a class="view_normal" href="<?php echo Minz_Url::display ($url_output); ?>">
 				<a class="view_normal" href="<?php echo Minz_Url::display ($url_output); ?>">
@@ -130,41 +130,38 @@
 
 
 			<?php
 			<?php
 				$url_state = $url;
 				$url_state = $url;
-				$url_state['params']['state'] = 'all';
-				if (strcmp($this->state, $url_state['params']['state']) <> 0) {
+				if ($this->state !== 'all') {
+					$url_state['params']['state'] = 'all';
 			?>
 			?>
 			<li class="item">
 			<li class="item">
 				<a class="print_all" href="<?php echo Minz_Url::display ($url_state); ?>">
 				<a class="print_all" href="<?php echo Minz_Url::display ($url_state); ?>">
 					<?php echo Minz_Translate::t ('show_all_articles'); ?>
 					<?php echo Minz_Translate::t ('show_all_articles'); ?>
 				</a>
 				</a>
 			</li>
 			</li>
-			<?php } ?>
-
 			<?php
 			<?php
-				$url_state['params']['state'] = 'not_read';
-				if (strcmp($this->state, $url_state['params']['state']) <> 0) {
+				}
+				if ($this->state !== 'not_read') {
+					$url_state['params']['state'] = 'not_read';
 			?>
 			?>
 			<li class="item">
 			<li class="item">
 				<a class="print_non_read" href="<?php echo Minz_Url::display ($url_state); ?>">
 				<a class="print_non_read" href="<?php echo Minz_Url::display ($url_state); ?>">
 					<?php echo Minz_Translate::t ('show_not_reads'); ?>
 					<?php echo Minz_Translate::t ('show_not_reads'); ?>
 				</a>
 				</a>
 			</li>
 			</li>
-			<?php } ?>
-
 			<?php
 			<?php
-				$url_state['params']['state'] = 'read';
-				if (strcmp($this->state, $url_state['params']['state']) <> 0) {
+				}
+				if ($this->state !== 'read') {
+					$url_state['params']['state'] = 'read';
 			?>
 			?>
 			<li class="item">
 			<li class="item">
 				<a class="print_read" href="<?php echo Minz_Url::display ($url_state); ?>">
 				<a class="print_read" href="<?php echo Minz_Url::display ($url_state); ?>">
 					<?php echo Minz_Translate::t ('show_read'); ?>
 					<?php echo Minz_Translate::t ('show_read'); ?>
 				</a>
 				</a>
 			</li>
 			</li>
-			<?php } ?>
-
 			<?php
 			<?php
-				$url_state['params']['state'] = 'favorite';
-				if (strcmp($this->state, $url_state['params']['state']) <> 0) {
+				}
+				if ($this->state != 'favorite') {
+					$url_state['params']['state'] = 'favorite';
 			?>
 			?>
 			<li class="item">
 			<li class="item">
 				<a class="print_read" href="<?php echo Minz_Url::display ($url_state); ?>">
 				<a class="print_read" href="<?php echo Minz_Url::display ($url_state); ?>">