Ticket #429 (closed defect: fixed)

Opened 8 months ago

Last modified 5 weeks ago

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

Reported by: iSa Owned by: PavelVinogradov
Priority: major Milestone: sams-2.0
Component: web Version: 2.0.0-b1
Keywords: Cc:

Description

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

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.

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

Attachments

fixed_sum.patch.tar.bz2 Download (442 bytes) - added by iSa 8 months ago.

Change History

Changed 8 months ago by iSa

  Changed 6 months ago by PavelVinogradov

  • status changed from new to closed
  • version changed from devel to 2.0.0-b1
  • resolution set to fixed

Thanks. Fixed in [798]

follow-up: ↓ 3   Changed 5 weeks ago by hangover <root@…>

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

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

in reply to: ↑ 2   Changed 5 weeks ago by PavelVinogradov

Replying to hangover <root@…>:

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

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

follow-up: ↓ 7   Changed 5 weeks ago by 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++;
 		}
        }

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

  Changed 5 weeks ago by PavelVinogradov

  • status changed from closed to reopened
  • resolution fixed deleted

  Changed 5 weeks ago by PavelVinogradov

  • owner changed from DmitryChemerik to PavelVinogradov
  • status changed from reopened to new

in reply to: ↑ 4   Changed 5 weeks ago by PavelVinogradov

Replying to hangover <root@…>:

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

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

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

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

  Changed 5 weeks ago by PavelVinogradov

  • status changed from new to closed
  • resolution set to fixed

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

Add/Change #429 (В отчете "Статитстика по посещаемым сайтам" неверно расчитывается суммарный трафик)

Author


E-mail address and user name can be saved in the Preferences.


Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.