ParseError: syntax error, unexpected

Hi hristos,

That you for reporting the bug. This is the line that has the error:

public function fetchView(string $type, int $id): StatisticsViewsResult|false {

The vertical pipe, |, in `StatisticsViewsResult|false` is the cause the error. This syntax is not supported in PHP 7. We will remove the `|false` in 8.x-2.17

Upgrading to PHP 8.1 or higher should also resolve the error.

Two of the tests are stilling failing under php 7.4 / mysql 5.7
https://www.drupal.org/pift-ci-job/2748210 →

I added PHP 7.4 testing to reduce the chances of bugs like this happening in the future.

Automatically closed - issue fixed for 2 weeks with no activity.

This issue was resolved. A new release, 8.x-2.17, will be made soon, May 31st.

This is an excellent opportunity to beta test the dev branch, and report any issues.

Contributors (5)

bluegeek9 → , roshni27 → , abhishek_gupta1 → , sarwan_verma → , SandeepSingh199 →

Changelog

Issues: 32 issues resolved.

Changes since 8.x-2.16 → :

Bug

• #3432630 → : Remove Not rendering HTML
• #3395217 → by bluegeek9 → : On a fresh install of a Drupal site with this module included, we get dependency errors
• #3400985 → by bluegeek9 → : Site break when placed visitors module
• #3413155 → by abhishek_gupta1 → , bluegeek9 → : Give Default value to all $agent keys
• #3383142 → by bluegeek9 → : ParseError: syntax error, unexpected
• #3380760 → : Warning: Undefined array key

Feature

• #3376256 → : Page Performance Metrics
• #3378568 → : Ajax Replace Report
• #3376234 → : Route reports
• #3378580 → : Browser Report(s)
• #1087268 → by bluegeek9 → : Exclude robots from the statistics
• #3376233 → by bluegeek9 → : View filter date range
• #3369318 → : Log OS and device infomation - BrowserCap replacement
• #3376235 → by bluegeek9 → : Drush command: Download MaxMind database
• #3250285 → by bluegeek9 → : Performance: Add db indexes

Task

• #3389685 → : Three different errors displayed
• #3376397 → by bluegeek9 → , roshni27 → : Remove block settings from visitors.config
• #3376392 → : Move Visitors Report menu
• #3444385 → : Module path
• #3443031 → by bluegeek9 → : Issues reported by PHPCS
• #3392006 → by bluegeek9 → , sarwan_verma → : watchdog_exception() deprecated
• #3423013 → : Deprecated function user_role_names()
• #3423001 → by bluegeek9 → : composer.json missing "repositories"
• #3397326 → by bluegeek9 → , SandeepSingh199 → : Issues reported by PHPStan
• #3401384 → : Remove Drupal 9 from GitLab CI
• #3393046 → by bluegeek9 → : Code Coverage
• #3377961 → : Replace visitors/hosts reports with views
• #3377962 → : Replace visitors/hits with Views
• #3377964 → : Replace /visitors/pages with Views
• #3377960 → by bluegeek9 → : Log visitor local time
• #3376241 → : Drush Command: Rebuild Geo Location
• #3377958 → : Tracking cookies