Рейтинг.

[MSK 287]

1. Заметил, что при установке рейтинге к новости нет проверки на существование новости, которой выдается оценка.

Т.е. можно запросто забить таблицу лога рейтинга несуществующими news_id.

2. При удалении новостей, в том числе по крону, остаются записи логов рейтинга новости.

Первое относится и к голосованиям в новостях.

[celsoft 5 986]

MSK,

Вы сейчас вообще о чем? Стандартный рейтинг новостей в DLE, находится в таблице самой новости, какая проверка может быть при этом? Рейтинг невозмно назначить несуществующей новости, т.к. сам рейтинг находится в таблице этой новости.

[MSK 287]

Я говорю об /ajax/rating.php

об этом

if( $is_logged ) $where = "member = '{$member_id['name']}'";

else $where = "ip ='{$_IP}'";


$row = $db->super_query( "SELECT news_id FROM " . PREFIX . "_logs where news_id ='$news_id' AND {$where}" );


if( ! $row['news_id'] AND count( explode( ".", $_IP ) ) == 4 ) {


	$db->query( "UPDATE " . PREFIX . "_post set rating=rating+'$go_rate', vote_num=vote_num+1 where id ='$news_id'" );


	if( $is_logged ) $user_name = $member_id['name'];

	else $user_name = "noname";


	$db->query( "INSERT INTO " . PREFIX . "_logs (news_id, ip, member) values ('$news_id', '$_IP', '$user_name')" );

	clear_cache( 'news_' );

}

авторизованный пользователь может записать в logs оценку несуществующей новости.

[celsoft 5 986]

MSK,

это не рейтинг новости, а логи голосований, чтобы не принимать повторные голоса, и подменив данные человек лишь запретит сам себе голосование и не более того, на рейтинг новости это никак не скажется, и в сам рейтинг никакой записи быть не может

А логи очичаются в админпанели скрипта в специальном разделе, в мастере оптимизации.

[MSK 287]

Я не говорю, что это влияет на рейтинг.

Я написал:

Т.е. можно запросто забить таблицу лога рейтинга несуществующими news_id.

Т.е. Вы считаете это нормальным?

Чистка логов голосования в админке позволит повторно принимать голоса.

[MSK 287]

Следует считать, что исправлений не будет?

[celsoft 5 986]

MSK,

будет

[MSK 287]

Тогда предусмотрите, плиз, возможность чистки логов от несуществующих/удаленных news_id.

[prikindel 255]

Тогда предусмотрите, плиз, возможность чистки логов от несуществующих/удаленных news_id.

разумная мысль

[MSK 287]

Я имел ввиду чистку логов записанных до внесения правок в код.