plausible/analytics
1
1
Fork 0
mirror of https://github.com/plausible/analytics.git synced 2024-12-24 10:02:10 +03:00
Code Issues Packages Projects Releases Wiki Activity

Filter for pageviews when no other filter is specified in base.ex

Browse Source
This commit is contained in:
Uku Taht 2021-08-18 12:09:40 +03:00
parent 669866a16b
commit 2f98da8f3a
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
lib/plausible/stats/base.ex
View File

@ -56,9 +56,17 @@ defmodule Plausible.Stats.Base do
q =
case query.filters["event:name"] do
{:is, name} -> from(e in q, where: e.name == ^name)
{:member, list} -> from(e in q, where: e.name in ^list)
nil -> q
{:is, name} ->
from(e in q, where: e.name == ^name)
{:member, list} ->
from(e in q, where: e.name in ^list)
# Logically, this isn't necessary and might even cause some confusion (request with no filters will still filter for event:name==pageviews).
nil ->
from(e in q, where: e.name == "pageview")
# When removing this, however, sampling in clickhouse goes a bit weird (dashboard with 3m pageviews will start getting sampled which is weird)
end
q =