mirror of
https://github.com/FreshRSS/FreshRSS.git
synced 2026-05-19 05:44:39 -04:00
Update title and get parameter correctly.
See https://github.com/marienfressinaud/FreshRSS/issues/634
This commit is contained in:
@@ -4,7 +4,6 @@
|
||||
* This class handles main actions of FreshRSS.
|
||||
*/
|
||||
class FreshRSS_index_Controller extends Minz_ActionController {
|
||||
private $nb_not_read_cat = 0;
|
||||
|
||||
public function indexAction() {
|
||||
// TODO: update the context with information from request.
|
||||
@@ -18,30 +17,6 @@ class FreshRSS_index_Controller extends Minz_ActionController {
|
||||
|
||||
return;
|
||||
|
||||
$this->view->nb_favorites = $entryDAO->countUnreadReadFavorites();
|
||||
$this->view->nb_not_read = FreshRSS_CategoryDAO::CountUnreads($this->view->cat_aside, 1);
|
||||
$this->view->currentName = '';
|
||||
|
||||
$this->view->get_c = '';
|
||||
$this->view->get_f = '';
|
||||
|
||||
$get = Minz_Request::param('get', 'a');
|
||||
$getType = $get[0];
|
||||
$getId = substr($get, 2);
|
||||
if (!$this->checkAndProcessType($getType, $getId)) {
|
||||
Minz_Log::debug('Not found [' . $getType . '][' . $getId . ']');
|
||||
Minz_Error::error(404);
|
||||
return;
|
||||
}
|
||||
|
||||
// mise à jour des titres
|
||||
$this->view->rss_title = $this->view->currentName . ' | ' . Minz_View::title();
|
||||
Minz_View::prependTitle(
|
||||
($this->nb_not_read_cat > 0 ? '(' . formatNumber($this->nb_not_read_cat) . ') ' : '') .
|
||||
$this->view->currentName .
|
||||
' · '
|
||||
);
|
||||
|
||||
// On récupère les différents éléments de filtrage
|
||||
$this->view->state = Minz_Request::param('state', FreshRSS_Context::$conf->default_view);
|
||||
$state_param = Minz_Request::param('state', null);
|
||||
@@ -115,57 +90,6 @@ class FreshRSS_index_Controller extends Minz_ActionController {
|
||||
}
|
||||
}
|
||||
|
||||
/*
|
||||
* Vérifie que la catégorie / flux sélectionné existe
|
||||
* + Initialise correctement les variables de vue get_c et get_f
|
||||
* + Met à jour la variable $this->nb_not_read_cat
|
||||
*/
|
||||
private function checkAndProcessType($getType, $getId) {
|
||||
switch($getType) {
|
||||
case 'a':
|
||||
$this->view->currentName = _t('your_rss_feeds');
|
||||
$this->nb_not_read_cat = $this->view->nb_not_read;
|
||||
$this->view->get_c = $getType;
|
||||
return true;
|
||||
case 's':
|
||||
$this->view->currentName = _t('your_favorites');
|
||||
$this->nb_not_read_cat = $this->view->nb_favorites['unread'];
|
||||
$this->view->get_c = $getType;
|
||||
return true;
|
||||
case 'c':
|
||||
$cat = isset($this->view->cat_aside[$getId]) ? $this->view->cat_aside[$getId] : null;
|
||||
if ($cat === null) {
|
||||
$catDAO = new FreshRSS_CategoryDAO();
|
||||
$cat = $catDAO->searchById($getId);
|
||||
}
|
||||
if ($cat) {
|
||||
$this->view->currentName = $cat->name();
|
||||
$this->nb_not_read_cat = $cat->nbNotRead();
|
||||
$this->view->get_c = $getId;
|
||||
return true;
|
||||
} else {
|
||||
return false;
|
||||
}
|
||||
case 'f':
|
||||
$feed = FreshRSS_CategoryDAO::findFeed($this->view->cat_aside, $getId);
|
||||
if (empty($feed)) {
|
||||
$feedDAO = FreshRSS_Factory::createFeedDao();
|
||||
$feed = $feedDAO->searchById($getId);
|
||||
}
|
||||
if ($feed) {
|
||||
$this->view->currentName = $feed->name();
|
||||
$this->nb_not_read_cat = $feed->nbNotRead();
|
||||
$this->view->get_f = $getId;
|
||||
$this->view->get_c = $feed->category();
|
||||
return true;
|
||||
} else {
|
||||
return false;
|
||||
}
|
||||
default:
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* This action displays the normal view of FreshRSS.
|
||||
*/
|
||||
@@ -175,11 +99,19 @@ class FreshRSS_index_Controller extends Minz_ActionController {
|
||||
return;
|
||||
}
|
||||
|
||||
$catDAO = new FreshRSS_CategoryDAO();
|
||||
$entryDAO = FreshRSS_Factory::createEntryDao();
|
||||
try {
|
||||
$this->updateContext();
|
||||
} catch (Minz_Exception $e) {
|
||||
Minz_Error::error(404);
|
||||
}
|
||||
|
||||
$this->view->categories = $catDAO->listCategories();
|
||||
$this->view->categories = FreshRSS_Context::$categories;
|
||||
|
||||
$title = FreshRSS_Context::$name;
|
||||
if (FreshRSS_Context::$get_unread > 0) {
|
||||
$title = '(' . FreshRSS_Context::$get_unread . ') · ' . $title;
|
||||
}
|
||||
Minz_View::prependTitle($title . ' · ');
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -193,8 +125,13 @@ class FreshRSS_index_Controller extends Minz_ActionController {
|
||||
|
||||
Minz_View::appendScript(Minz_Url::display('/scripts/global_view.js?' . @filemtime(PUBLIC_PATH . '/scripts/global_view.js')));
|
||||
|
||||
$catDAO = new FreshRSS_CategoryDAO();
|
||||
$this->view->categories = $catDAO->listCategories();
|
||||
try {
|
||||
$this->updateContext();
|
||||
} catch (Minz_Exception $e) {
|
||||
Minz_Error::error(404);
|
||||
}
|
||||
|
||||
$this->view->categories = FreshRSS_Context::$categories;
|
||||
|
||||
Minz_View::prependTitle(_t('gen.title.global_view') . ' · ');
|
||||
}
|
||||
@@ -214,11 +151,25 @@ class FreshRSS_index_Controller extends Minz_ActionController {
|
||||
Minz_Error::error(403);
|
||||
}
|
||||
|
||||
try {
|
||||
$this->updateContext();
|
||||
} catch (Minz_Exception $e) {
|
||||
Minz_Error::error(404);
|
||||
}
|
||||
|
||||
// No layout for RSS output.
|
||||
$this->view->rss_title = FreshRSS_Context::$name . ' | ' . Minz_View::title();
|
||||
$this->view->_useLayout(false);
|
||||
header('Content-Type: application/rss+xml; charset=utf-8');
|
||||
}
|
||||
|
||||
/**
|
||||
* This action updates the Context object by using request parameters.
|
||||
*/
|
||||
private function updateContext() {
|
||||
FreshRSS_Context::_get(Minz_Request::param('get', 'a'));
|
||||
}
|
||||
|
||||
/**
|
||||
* This action displays the about page of FreshRSS.
|
||||
*/
|
||||
|
||||
@@ -6,6 +6,9 @@
|
||||
*/
|
||||
class FreshRSS_Context {
|
||||
public static $conf = null;
|
||||
public static $categories = array();
|
||||
|
||||
public static $name = '';
|
||||
|
||||
public static $total_unread = 0;
|
||||
public static $total_starred = array(
|
||||
@@ -21,6 +24,7 @@ class FreshRSS_Context {
|
||||
'feed' => false,
|
||||
'category' => false,
|
||||
);
|
||||
public static $get_unread = 0;
|
||||
public static $order = 'DESC';
|
||||
|
||||
public static function init() {
|
||||
@@ -37,8 +41,16 @@ class FreshRSS_Context {
|
||||
Minz_Session::_param('language', self::$conf->language);
|
||||
Minz_Translate::init();
|
||||
|
||||
$catDAO = new FreshRSS_CategoryDAO();
|
||||
$entryDAO = FreshRSS_Factory::createEntryDao();
|
||||
|
||||
// Get the current state.
|
||||
// self::$state = self::$conf->default_view;
|
||||
self::$categories = $catDAO->listCategories();
|
||||
|
||||
// Update number of read / unread variables.
|
||||
self::$total_starred = $entryDAO->countUnreadReadFavorites();
|
||||
self::$total_unread = FreshRSS_CategoryDAO::CountUnreads(self::$categories, 1);
|
||||
}
|
||||
|
||||
public static function isStateEnabled($state) {
|
||||
@@ -53,6 +65,62 @@ class FreshRSS_Context {
|
||||
}
|
||||
}
|
||||
|
||||
public static function _get($get) {
|
||||
$type = $get[0];
|
||||
$id = substr($get, 2);
|
||||
$nb_unread = 0;
|
||||
|
||||
switch($type) {
|
||||
case 'a':
|
||||
self::$current_get['all'] = true;
|
||||
self::$name = _t('your_rss_feeds');
|
||||
self::$get_unread = self::$total_unread;
|
||||
break;
|
||||
case 's':
|
||||
self::$current_get['starred'] = true;
|
||||
self::$name = _t('your_favorites');
|
||||
self::$get_unread = self::$total_starred['unread'];
|
||||
break;
|
||||
case 'f':
|
||||
self::$current_get['feed'] = $id;
|
||||
|
||||
$feed = FreshRSS_CategoryDAO::findFeed(self::$categories, $id);
|
||||
if ($feed === null) {
|
||||
$feedDAO = FreshRSS_Factory::createFeedDao();
|
||||
$feed = $feedDAO->searchById($id);
|
||||
|
||||
if (!$feed) {
|
||||
// TODO: raise an exception
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
self::$name = $feed->name();
|
||||
self::$get_unread = $feed->nbNotRead();
|
||||
break;
|
||||
case 'c':
|
||||
self::$current_get['category'] = $id;
|
||||
if (!isset(self::$categories[$id])) {
|
||||
$catDAO = new FreshRSS_CategoryDAO();
|
||||
$cat = $catDAO->searchById($id);
|
||||
|
||||
if (!$cat) {
|
||||
// TODO: raise an exception
|
||||
return false;
|
||||
}
|
||||
} else {
|
||||
$cat = self::$categories[$id];
|
||||
}
|
||||
|
||||
self::$name = $cat->name();
|
||||
self::$get_unread = $cat->nbNotRead();
|
||||
break;
|
||||
default:
|
||||
// TODO: raise an exception!
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
public static function currentGet() {
|
||||
if (self::$current_get['all']) {
|
||||
return 'a';
|
||||
|
||||
Reference in New Issue
Block a user