Account created on 24 April 2011, over 13 years ago
#

Merge Requests

More

Recent comments

🇺🇸United States bluegeek9

Hi @armz23,

That block was removed in 8.x-2.20.

That was showing the total page views. We are now tracking each visitor uniquely and showing unique visits.

There is not an effective way to revert to old versions of Visitors.

📌 Remove the Visitor block Fixed

https://www.drupal.org/project/visitors/releases/8.x-2.20

🇺🇸United States bluegeek9

Changelog

Issues: 4 issues resolved.

Changes since 8.x-2.23 :

Bug

Task

🇺🇸United States bluegeek9

I am not sure what you are requesting.

The visitor block was removed.

You can update the views to use total view instead of unique visitors.

🇺🇸United States bluegeek9

bluegeek9 made their first commit to this issue’s fork.

🇺🇸United States bluegeek9

After reviewing your Drupal.org profile, we have to pass on your offer. You are not credited with enough fixes to be made a co-maintainer.

🇺🇸United States bluegeek9

bluegeek9 made their first commit to this issue’s fork.

🇺🇸United States bluegeek9

Webforms are not nodes. This project only works with nodes.

🇺🇸United States bluegeek9

bluegeek9 made their first commit to this issue’s fork.

🇺🇸United States bluegeek9

Can you show which issues you and your team have worked on?

🇺🇸United States bluegeek9

bluegeek9 made their first commit to this issue’s fork.

🇺🇸United States bluegeek9

Hi @sourav_paul,

I addressed the bug in a different issue to clarify the change in the release notes.

📌 UNK: config_browser_name too long Active

8.x-2.24 will be released after 📌 Remove JavaScriptSnippet class Active is fixed.

🇺🇸United States bluegeek9

bluegeek9 made their first commit to this issue’s fork.

🇺🇸United States bluegeek9

Visitors does allow for the statistics to be copied. It does not use the migrate sub-system.

The feature was added in a recent version of Vistiors, so that will not be of much help if you are on Drupal8.

The Statistics module has a migrate plugin. You could migrate the view count to Drupal11. Then use Visitors to copy the view count. That might be your best bet if you need a solution today.

Since there is an existing plugin, it would not be too hard to adopt it for Visitors.

🇺🇸United States bluegeek9

bluegeek9 created an issue.

🇺🇸United States bluegeek9

bluegeek9 created an issue.

🇺🇸United States bluegeek9

bluegeek9 created an issue.

| CRM | Group
🇺🇸United States bluegeek9
🇺🇸United States bluegeek9

The issue is already fixed in the dev release.

🇺🇸United States bluegeek9

I am not able to reproduce the error. Which version of Vistitors are you using?

Maybe replace the active visitors view, with the one in config/optional ?

🇺🇸United States bluegeek9

Hi Wes,

It looks like the browser language is not included/available.

With a resolution of 800x600, I think it is a wearable, or embedded device.

I created another issue to work on the error.
📌 Browser languare Null Active

{
  "7": ["route","entity.node.canonical"],
  "8":["path","/node/1"],
  "9":["viewed","node:1"],
  "10":["server","a87f84d82bc8"]}
  &_id=3b3e8f5e050b8447
  &_idn=1
  &_refts=0
  &action_name=Custom software solutions | MyCompany Hosting LLC
  &cookie=1
  &h=14
  &idsite=1
  &m=12
  &pf_dm1=908
  &pf_dm2=0
  &pf_net=0
  &pf_onl=1
  &pf_srv=2
  &pf_tfr=0
  &pv_id=2SPdOc
  &r=679717
  &rec=1
  &res=800x600
  &s=31
  &send_image=0
  &uadata={"fullVersionList":[],"mobile":false,"model":"","platform":"","platformVersion":""}
  &uid=0
  &url=https://myserverhost.com/
🇺🇸United States bluegeek9

How often does the error occur?

Is this error on a local/development machine, or production?

🇺🇸United States bluegeek9

I don't think the error is caused by visitor_localtime. It allows NULL.

Maybe it is the language?

🇺🇸United States bluegeek9

@kongkx,

Which report has the bad link? I am having trouble reproducing the error.

🇺🇸United States bluegeek9

@sourabhsisodia_,

I think you did not use composer to install visitors. You need to use composer.

🇺🇸United States bluegeek9

bluegeek9 made their first commit to this issue’s fork.

🇺🇸United States bluegeek9

Thank you Wes!

I plan to make a new release. Would you verify the bug is resolved?

🇺🇸United States bluegeek9

I made a MR for just the cache issue.

🇺🇸United States bluegeek9

In Visitors 3.0.x, the geoip location service is/will be provided by MaxMind module.

🇺🇸United States bluegeek9

bluegeek9 changed the visibility of the branch 8.x-2.x to hidden.

🇺🇸United States bluegeek9

bluegeek9 changed the visibility of the branch 3484155-progress-bar to hidden.

🇺🇸United States bluegeek9

I am confident it is the visitors_title. It is limited to 255 characters. I also recently stumbled upoun this error.

🇺🇸United States bluegeek9

In 3.0.x, page views are stored in the visitors_log table. I want to track search metrics: keyword, page number, and search rank.

If the columns are added for search, maybe they could be used for entity type, bundle, entity id when viewing a canonical route.

🇺🇸United States bluegeek9

Hi wes.congruityworks.com,

It sounds like you did not uninstall the module before going to and reverting from 3.0.x. That branch should not be used.

🇺🇸United States bluegeek9

The change is in the Dev branch if you want to use it now.

A new release, 8.x-2.22, will be made before the end of the year if you want to wait.

🇺🇸United States bluegeek9

Hi Wes,

Thank you for reporting the error.
Did you recently install Visitors, or did this happen after a recent update?

🇺🇸United States bluegeek9

I left these four issues. All the changes in the pull request are whitespace changes.

FILE: /var/www/html/web/modules/custom/charts/modules/charts_chartjs/src/Settings/Chartjs/ChartjsScales.php
------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------------------------------------------
 32 | ERROR | Public method name "ChartjsScales::getXAxes" is not in lowerCamel format (Drupal.NamingConventions.ValidFunctionName.ScopeNotCamelCaps)
 42 | ERROR | Public method name "ChartjsScales::setXAxes" is not in lowerCamel format (Drupal.NamingConventions.ValidFunctionName.ScopeNotCamelCaps)
 52 | ERROR | Public method name "ChartjsScales::getYAxes" is not in lowerCamel format (Drupal.NamingConventions.ValidFunctionName.ScopeNotCamelCaps)
 62 | ERROR | Public method name "ChartjsScales::setYAxes" is not in lowerCamel format (Drupal.NamingConventions.ValidFunctionName.ScopeNotCamelCaps)
------------------------------------------------------------------------------------------------------------------------------------------------------

🇺🇸United States bluegeek9

bluegeek9 made their first commit to this issue’s fork.

Production build 0.71.5 2024