ソースを参照

Corrected bug with the default "No Category"

The SQL optimisation patch had introduced a bug with "No Category", now
solved
Alexandre Alapetite 12 年 前
コミット
7627970862
3 ファイル変更5 行追加1 行削除
  1. 1 0
      app/controllers/feedController.php
  2. 1 1
      app/models/Category.php
  3. 3 0
      app/models/Feed.php

+ 1 - 0
app/controllers/feedController.php

@@ -17,6 +17,7 @@ class feedController extends ActionController {
 		if (Request::isPost ()) {
 			$url = Request::param ('url_rss');
 			$cat = Request::param ('category');
+			if (empty($cat)) $cat = '000000';
 			$user = Request::param ('username');
 			$pass = Request::param ('password');
 			$params = array ();

+ 1 - 1
app/models/Category.php

@@ -179,7 +179,7 @@ class CategoryDAO extends Model_pdo {
 		if ($prePopulateFeeds) {
 			$sql = 'SELECT c.id as c_id, c.name as c_name, c.color as c_color, count(e.id) as nbNotRead, f.* '
 			     . 'FROM  ' . $this->prefix . 'category c '
-			     . 'INNER JOIN  ' . $this->prefix . 'feed f ON f.category = c.id '
+			     . 'LEFT OUTER JOIN ' . $this->prefix . 'feed f ON f.category = c.id '
 			     . 'LEFT OUTER JOIN  ' . $this->prefix . 'entry e ON e.id_feed = f.id AND e.is_read = 0 '
 			     . 'GROUP BY f.id '
 			     . 'ORDER BY c.name, f.name';

+ 3 - 0
app/models/Feed.php

@@ -502,6 +502,9 @@ class HelperFeed {
 		}
 
 		foreach ($listDAO as $key => $dao) {
+			if (empty ($dao['url'])) {
+				continue;
+			}
 			if (isset ($dao['id'])) {
 				$key = $dao['id'];
 			}