vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #10373 from scmmishra/tracking-fix-referrer

Browse source 
fix: referrer length too long
This commit is contained in:
mergify[bot] 2020-05-18 10:57:32 +00:00 • committed by GitHub
commit 5f0b68cddd
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 19 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
frappe/website/doctype/web_page_view/web_page_view.py
View file

@ -12,9 +12,15 @@ class WebPageView(Document):
@frappe.whitelist(allow_guest=True)
def make_view_log(path, referrer=None, browser=None, version=None, url=None, user_tz=None):
if not is_tracking_enabled():
return
request_dict = frappe.request.__dict__
user_agent = request_dict.get('environ', {}).get('HTTP_USER_AGENT')
if referrer:
referrer = referrer.split('?')[0]
is_unique = True
if referrer.startswith(url):
is_unique = False
@ -22,18 +28,20 @@ def make_view_log(path, referrer=None, browser=None, version=None, url=None, use
if path != "/" and path.startswith('/'):
path = path[1:]
if is_tracking_enabled():
view = frappe.new_doc("Web Page View")
view.path = path
view.referrer = referrer
view.browser = browser
view.browser_version = version
view.time_zone = user_tz
view.user_agent = user_agent
view.is_unique = is_unique
view.insert(ignore_permissions=True)
view = frappe.new_doc("Web Page View")
view.path = path
view.referrer = referrer
view.browser = browser
view.browser_version = version
view.time_zone = user_tz
view.user_agent = user_agent
view.is_unique = is_unique
return
try:
view.insert(ignore_permissions=True)
except Exception:
if frappe.message_log:
frappe.message_log.pop()
@frappe.whitelist()
def get_page_view_count(path):