В отчете "Статитстика по посещаемым сайтам" неверно расчитывается суммарный трафик

[iSa]

Суммарный трафик в отчете "Статитстика по посещаемым сайтам" расчитывается по формуле:

r_size + r_hit = r_sum, что является неверным, так как r_size - суммарное кол-во байт, полученное пользователем, а r_hit - кол-во байт из r_size, взятые из кеша. Таким образом правильная формула должна выглядеть так: r_size - r_hit = r_sum, при realtraffic=real или r_size = r_sum, при realtraffic=full.

В аттаче патч для исправления.

[PavelVinogradov]

Thanks. Fixed in [798]

[hangover <root@…>]

В версии из транка приходится в usersbuttom_3_site.php возвращать вышеуказанные изменения взад, иначе неправильно формируется статистика в общем списке пользователей.

В настройках трафик "Реальный (Полученный прокси-сервером)".

[PavelVinogradov]

Replying to hangover <root@…>:

В версии из транка приходится в usersbuttom_3_site.php возвращать вышеуказанные изменения взад, иначе неправильно формируется статистика в общем списке пользователей. В настройках трафик "Реальный (Полученный прокси-сервером)".

А можно подробнее? Я посмотрел еще раз на исправление - оно корректное. Другой вопрос - что в некоторых других отчетах у нас возможно некорректно учитывается реальный/кешированный трафик.

[hangover <root@…>]

В версии из вчерашнего транка формирую отчёт "Статистика по посещаемым сайтам" по одному пользователю - всё правильно: "Размер"+"Из кэша"="Суммарный трафик"

Перехожу в общий список "Пользователи", выбираю период и формирую тот же самый отчёт - неправильно: значение в столбце "Размер" равно значению "Суммарный трафик".

Вношу изменения в usersbuttom_3_site.php:

--- usersbuttom_3_site.php.old	2010-07-30 12:24:24.000000000 +0700
+++ usersbuttom_3_site.php	2010-07-30 12:26:40.000000000 +0700
@@ -203,10 +203,10 @@
 			$URL["norm_url"][$count]=str_replace("/","",$row['url_domain']);
 			$URL["url_size"][$count]=$row['url_size'];
 			$URL["hit_size"][$count]=$row['hit_size'];
-			if($SAMSConf->realtraffic=="real")
-				$URL["sum_size"][$count]=$row['url_size']-$row['hit_size'];
-			else
-				$URL["sum_size"][$count]=$row['url_size'];
+//			if($SAMSConf->realtraffic=="real")
+				$URL["sum_size"][$count]=$row['url_size']+$row['hit_size'];
+//			else
+//				$URL["sum_size"][$count]=$row['url_size'];
 			$count++;
 		}
        }

Теперь отчёт отображается корректно: "Размер"+"Из кэша"="Суммарный трафик"

[PavelVinogradov]

Replying to hangover <root@…>:

В версии из вчерашнего транка формирую отчёт "Статистика по посещаемым сайтам" по одному пользователю - всё правильно: "Размер"+"Из кэша"="Суммарный трафик" Перехожу в общий список "Пользователи", выбираю период и формирую тот же самый отчёт - неправильно: значение в столбце "Размер" равно значению "Суммарный трафик". Теперь отчёт отображается корректно: "Размер"+"Из кэша"="Суммарный трафик"

Причина этого поведения описана в баге #497. На самом деле переменная $SAMSConf->realtraffic всегда пустая, т.к. не инициализируется в samsclass.php.

Поэтому текущая реализация отчета для пользователей правильная, а вот для конкретного пользователя - некорретно реализовано.

Тикет оставляю открытым чтобы исправить отчет для конкретного пользователя. Также необходимо исправить #497 чтобы все стало работать корретно.

[PavelVinogradov]

Fixed in [840], require fix #497 to work correctly.