remove option ALLOW_DUPLICATE_POSTS as confusing/useless

This commit is contained in:
Andrew Dolgov 2016-01-07 22:01:35 +03:00
parent 4f186b1f10
commit 5bdcb8fd78
2 changed files with 3 additions and 29 deletions

View File

@ -164,7 +164,7 @@ class Pref_Prefs extends Handler_Protected {
global $access_level_names;
$prefs_blacklist = array("STRIP_UNSAFE_TAGS", "REVERSE_HEADLINES",
$prefs_blacklist = array("ALLOW_DUPLICATE_POSTS", "STRIP_UNSAFE_TAGS", "REVERSE_HEADLINES",
"SORT_HEADLINES_BY_FEED_DATE", "DEFAULT_ARTICLE_LIMIT",
"FEEDS_SORT_BY_UNREAD");

View File

@ -750,27 +750,9 @@
db_query("UPDATE ttrss_entries SET date_updated = NOW()
WHERE id = '$base_entry_id'");
// if we allow duplicate posts, we have to continue to
// create the user entries for this feed (if needed)
if (get_pref("ALLOW_DUPLICATE_POSTS", $owner_uid, false)) {
$query = "SELECT int_id FROM ttrss_user_entries WHERE
ref_id = '$base_entry_id' AND owner_uid = '$owner_uid'
AND (feed_id = '$feed' OR feed_id IS NULL) LIMIT 1";
$result = db_query($query);
if (db_num_rows($result) == 0) {
_debug("allow duplicate posts is enabled and user record is not found, continuing.");
} else {
continue;
}
} else {
continue;
}
}
_debug("hash differs, applying plugin filters:", $debug_enabled);
foreach ($pluginhost->get_hooks(PluginHost::HOOK_ARTICLE_FILTER) as $plugin) {
@ -944,16 +926,8 @@
// check for user post link to main table
// do we allow duplicate posts with same GUID in different feeds?
if (get_pref("ALLOW_DUPLICATE_POSTS", $owner_uid, false)) {
$dupcheck_qpart = "AND (feed_id = '$feed' OR feed_id IS NULL)";
} else {
$dupcheck_qpart = "";
}
$query = "SELECT ref_id, int_id FROM ttrss_user_entries WHERE
ref_id = '$ref_id' AND owner_uid = '$owner_uid'
$dupcheck_qpart";
ref_id = '$ref_id' AND owner_uid = '$owner_uid'";
// if ($_REQUEST["xdebug"]) print "$query\n";