do some warning housekeeping

This commit is contained in:
Andrew Dolgov 2010-11-05 17:16:29 +03:00
parent 3b5ecd1779
commit c3fddd057f
3 changed files with 23 additions and 17 deletions

View File

@ -282,10 +282,10 @@
$subop = db_escape_string($_REQUEST["subop"]); $subop = db_escape_string($_REQUEST["subop"]);
$view_mode = db_escape_string($_REQUEST["view_mode"]); $view_mode = db_escape_string($_REQUEST["view_mode"]);
$limit = (int) get_pref($link, "DEFAULT_ARTICLE_LIMIT"); $limit = (int) get_pref($link, "DEFAULT_ARTICLE_LIMIT");
$cat_view = db_escape_string($_REQUEST["cat"]); @$cat_view = db_escape_string($_REQUEST["cat"]);
$next_unread_feed = db_escape_string($_REQUEST["nuf"]); @$next_unread_feed = db_escape_string($_REQUEST["nuf"]);
$offset = db_escape_string($_REQUEST["skip"]); @$offset = db_escape_string($_REQUEST["skip"]);
$vgroup_last_feed = db_escape_string($_REQUEST["vgrlf"]); @$vgroup_last_feed = db_escape_string($_REQUEST["vgrlf"]);
$csync = $_REQUEST["csync"]; $csync = $_REQUEST["csync"];
$order_by = db_escape_string($_REQUEST["order_by"]); $order_by = db_escape_string($_REQUEST["order_by"]);

View File

@ -11,10 +11,11 @@
$pref_name = db_escape_string($pref_name); $pref_name = db_escape_string($pref_name);
$prefs_cache = true; $prefs_cache = true;
$profile = false;
if (!$user_id) { if (!$user_id) {
$user_id = $_SESSION["uid"]; $user_id = $_SESSION["uid"];
$profile = $_SESSION["profile"]; @$profile = $_SESSION["profile"];
} else { } else {
$user_id = sprintf("%d", $user_id); $user_id = sprintf("%d", $user_id);
$prefs_cache = false; $prefs_cache = false;
@ -29,7 +30,7 @@
if (get_schema_version($link) < 63) $profile_qpart = ""; if (get_schema_version($link) < 63) $profile_qpart = "";
if ($prefs_cache && !defined('DISABLE_SESSIONS') && !SINGLE_USER_MODE) { if ($prefs_cache && !defined('DISABLE_SESSIONS') && !SINGLE_USER_MODE) {
if ($_SESSION["prefs_cache"] && $_SESSION["prefs_cache"][$pref_name]) { if ($_SESSION["prefs_cache"] && @$_SESSION["prefs_cache"][$pref_name]) {
$tuple = $_SESSION["prefs_cache"][$pref_name]; $tuple = $_SESSION["prefs_cache"][$pref_name];
return convert_pref_type($tuple["value"], $tuple["type"]); return convert_pref_type($tuple["value"], $tuple["type"]);
} }
@ -84,7 +85,7 @@
if (!$user_id) { if (!$user_id) {
$user_id = $_SESSION["uid"]; $user_id = $_SESSION["uid"];
$profile = $_SESSION["profile"]; @$profile = $_SESSION["profile"];
} else { } else {
$user_id = sprintf("%d", $user_id); $user_id = sprintf("%d", $user_id);
$prefs_cache = false; $prefs_cache = false;

View File

@ -2066,6 +2066,7 @@
} }
function get_user_theme_path($link) { function get_user_theme_path($link) {
$theme_path = '';
if (get_schema_version($link) >= 63 && $_SESSION["uid"]) { if (get_schema_version($link) >= 63 && $_SESSION["uid"]) {
$theme_name = get_pref($link, "_THEME_ID"); $theme_name = get_pref($link, "_THEME_ID");
@ -2677,6 +2678,7 @@
$feeds_qpart = "ttrss_user_entries.feed_id = ttrss_feeds.id AND"; $feeds_qpart = "ttrss_user_entries.feed_id = ttrss_feeds.id AND";
} else { } else {
$from_qpart = "ttrss_user_entries,ttrss_entries"; $from_qpart = "ttrss_user_entries,ttrss_entries";
$feeds_qpart = '';
} }
$query = "SELECT count(int_id) AS unread $query = "SELECT count(int_id) AS unread
@ -3302,6 +3304,8 @@
if (!$owner_uid) $owner_uid = $_SESSION["uid"]; if (!$owner_uid) $owner_uid = $_SESSION["uid"];
$ext_tables_part = "";
if ($search) { if ($search) {
$search_query_part = getSearchSql($search, $match_on); $search_query_part = getSearchSql($search, $match_on);
@ -3547,7 +3551,6 @@
FROM FROM
$from_qpart $from_qpart
WHERE WHERE
$group_limit_part
$feed_check_qpart $feed_check_qpart
ttrss_user_entries.ref_id = ttrss_entries.id AND ttrss_user_entries.ref_id = ttrss_entries.id AND
ttrss_user_entries.owner_uid = '$owner_uid' AND ttrss_user_entries.owner_uid = '$owner_uid' AND
@ -4148,9 +4151,7 @@
print "<div id=\"subtoolbar_ftitle\">"; print "<div id=\"subtoolbar_ftitle\">";
if ($feed_site_url) { if ($feed_site_url) {
if (!$bottom) { $target = "target=\"_blank\"";
$target = "target=\"_blank\"";
}
print "<a title=\"".__("Visit the website")."\"$target href=\"$feed_site_url\">". print "<a title=\"".__("Visit the website")."\"$target href=\"$feed_site_url\">".
truncate_string($feed_title,30)."</a>"; truncate_string($feed_title,30)."</a>";
} else { } else {
@ -4179,6 +4180,8 @@
if ($search) { if ($search) {
$search_q = "&q=$search&m=$match_on&smode=$search_mode"; $search_q = "&q=$search&m=$match_on&smode=$search_mode";
} else {
$search_q = "";
} }
$rss_link = "backend.php?op=rss&id=$feed_id&is_cat=$is_cat&view-mode=$view_mode$search_q"; $rss_link = "backend.php?op=rss&id=$feed_id&is_cat=$is_cat&view-mode=$view_mode$search_q";
@ -5004,8 +5007,9 @@
catchup_feed($link, $subop_split[1], false); catchup_feed($link, $subop_split[1], false);
} }
// FIXME: might break tag display?
if ($feed_id > 0) { if ($feed > 0) {
$result = db_query($link, $result = db_query($link,
"SELECT id FROM ttrss_feeds WHERE id = '$feed' LIMIT 1"); "SELECT id FROM ttrss_feeds WHERE id = '$feed' LIMIT 1");
@ -5040,14 +5044,14 @@
/// START ///////////////////////////////////////////////////////////////////////////////// /// START /////////////////////////////////////////////////////////////////////////////////
$search = db_escape_string($_REQUEST["query"]); @$search = db_escape_string($_REQUEST["query"]);
if ($search) { if ($search) {
$disable_cache = true; $disable_cache = true;
} }
$search_mode = db_escape_string($_REQUEST["search_mode"]); @$search_mode = db_escape_string($_REQUEST["search_mode"]);
$match_on = db_escape_string($_REQUEST["match_on"]); @$match_on = db_escape_string($_REQUEST["match_on"]);
if (!$match_on) { if (!$match_on) {
$match_on = "both"; $match_on = "both";
@ -5295,7 +5299,7 @@
# $line["feed_title"]."</a> # $line["feed_title"]."</a>
if (!get_pref($link, 'VFEED_GROUP_BY_FEED')) { if (!get_pref($link, 'VFEED_GROUP_BY_FEED')) {
if ($line["feed_title"]) { if (@$line["feed_title"]) {
print "<span class=\"hlFeed\"> print "<span class=\"hlFeed\">
(<a href=\"javascript:viewfeed($feed_id, '', false)\">". (<a href=\"javascript:viewfeed($feed_id, '', false)\">".
$line["feed_title"]."</a>) $line["feed_title"]."</a>)
@ -5314,7 +5318,7 @@
print "<td class='hlMarkedPic'>$score_pic</td>"; print "<td class='hlMarkedPic'>$score_pic</td>";
if ($line["feed_title"] && !get_pref($link, 'VFEED_GROUP_BY_FEED')) { if (@$line["feed_title"] && !get_pref($link, 'VFEED_GROUP_BY_FEED')) {
print "<td onclick=\"viewfeed($feed_id)\" class=\"hlFeedIcon\">$feed_icon_img</td>"; print "<td onclick=\"viewfeed($feed_id)\" class=\"hlFeedIcon\">$feed_icon_img</td>";
} }
@ -6373,6 +6377,7 @@
$label_id = $line["id"]; $label_id = $line["id"];
$label_caption = $line["caption"]; $label_caption = $line["caption"];
$id = $line["id"];
if ($feed_id < -10 && $feed_id == -11-$label_id) { if ($feed_id < -10 && $feed_id == -11-$label_id) {
print "<li id=\"LHDL-$id\" print "<li id=\"LHDL-$id\"