CMS MADE SIMPLE FORGE

Statistics

 

[#9309] Statistics uses 'old' way of getting page info

avatar
Created By: Prue Rowland (psy)
Date Submitted: Mon Jun 10 22:03:15 -0400 2013

Assigned To: Fernando Morgado (JoMorg)
Version: 1.1.3
CMSMS Version: None
Severity: Major
Resolution: Fixed
State: Closed
Summary:
Statistics uses 'old' way of getting page info
Detailed Description:
Versions 1.1.3 uses a deprecated method of accessing current page information,
rendering the module useless in current versions and filling the error.log with
PHP warnings.

To fix:

In Statistics.module.php replace every instance of:

$this->cms->variables['pageinfo']->content_id

with:

cms_utils::get_current_pageid()

Then replace every instance of:

$this->cms->variables['pageinfo']->content_alias

with:

cms_utils::get_current_alias()



History

Comments
avatar
Date: 2013-06-11 03:22
Posted By: Morten Poulsen (silmarillion)

I will fix, review the rest of the module, and make a release soon! Thank you!
      
avatar
Date: 2016-06-21 08:38
Posted By: Fernando Morgado (JoMorg)

Fixed, thanks.
      
Updates

Updated: 2016-06-21 08:38
resolution_id: 6 => 7
cmsms_version_id: 30005 => -1
assigned_to_id: 100 => 12532
state: Open => Closed

Updated: 2013-06-11 03:22
resolution_id: => 6