Баг: удаление всех закладок

[Akela 9]

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

Для примера, при выводе 10 новостей на страницу и наличии 15 новостей в закладках, при заходе на страницу http://site/favorites/page/3/ сработает данный баг и ВСЕ закладки удалятся.

НЕ повторяйте данный эскперимент у себя на сайте, за последствия вы сами отвечаете .

Описание :

Если в навигации по закладкам перейти на несуществующую страницу, то все закладки удаляются, благодаря следующему коду в show.short.php :

	} elseif( ! $news_found and $do == 'favorites' ) {


		if ( $member_id['favorites'] ) $db->query( "UPDATE " . USERPREFIX . "_users SET favorites='' WHERE user_id = '{$member_id['user_id']}'" );


		msgbox( $lang['all_info'], $lang['fav_notfound'] );

	}

Временное решение : заменить вышеописанный код на

	} elseif( ! $news_found and $do == 'favorites' ) {


		if ( $member_id['favorites'] and ! $count_all ) $db->query( "UPDATE " . USERPREFIX . "_users SET favorites='' WHERE user_id = '{$member_id['user_id']}'" );


		msgbox( $lang['all_info'], $lang['fav_notfound'] );

	}

будет выводить сообщение "Вы ничего не вносили в свои закладки", но хотя бы сами закладки не обнулит.

Надеюсь, что разработчик найдет более правильное решение.

Изменено 21 мая 2010 пользователем Akela

[WWW.ZEOS.IN 1 161]

[celsoft 6 072]

Akela,

Принято, будет исправлено в новой версии скрипта. Это кстати правильное исправление, я только подправлю в будущем текст сообщения, если зашел на несуществующую страницу.

[Akela 9]

Akela,

Принято, будет исправлено в новой версии скрипта. Это кстати правильное исправление, я только подправлю в будущем текст сообщения, если зашел на несуществующую страницу.

Кстати, похожая, но не такая страшная, ситуация во всех местах, где есть постарничный вывод при заходе на несуществующую страницу. Может быть стоит добавить предварительную проверку, что $cstart меньше $count_all ?