CMS MADE SIMPLE FORGE

SEO Tools

 

[#7152] Fatal errors x 2( actual cmsms version 1.10.1 )

avatar
Created By: cleatus mcfarlan (cleatus)
Date Submitted: Thu Nov 10 18:03:55 -0500 2011

Assigned To: Henning Schaefer (hschaefer)
Version: 1.5
CMSMS Version: 1.10
Severity: Critical
Resolution: None
State: Open
Summary:
Fatal errors x 2( actual cmsms version 1.10.1 )
Detailed Description:
in admin: Fatal error: Call to a member function Name() on a non-object in
/home/username/public_html/modules/SEOTools/SEOTools.module.php on line 670

when loading home page of site: Fatal error: Cannot access protected property
Content::$mProperties in
/home/username/public_html/modules/SEOTools/action.default.php on line 169

site and modules were upgraded not clean install. working on another site with
1.10.1 and seotools 1.5. previous version was working fine. permissions are the
same as other working site. did not change non working site besides upgrading.
everything else on site is working fine. any help would be appreciated.



History

Comments
avatar
Date: 2011-11-16 03:58
Posted By: Anne Datema (adatema)

To confirm:
I also have problems with the installation of SEOtools-1.5 on a
to-version-10.0.1-upgraded cmsms-site.

Hopefully this problem can be resolved. Thanks for this module, it works great
for me!
      
avatar
Date: 2011-11-16 05:20
Posted By: David Watson (dathwa)

Hello.
Similar problems here too. Upgrading from 1.9.4.3 to 1.10.1.
The site now breaks on everything except the home page.
PHP Fatal error:  Call to a member function GetContent() on a non-object in
modules/SEOTools/action.default.php on line 20
PHP Fatal error:  Cannot access protected property Content::$mName in
modules/SEOTools/action.default.php on line 77

Everything else working fine.

A quick fix for this would be hugely appreciated.

Thanks.

      
avatar
Date: 2011-11-16 06:54
Posted By: Darren Kierman (kierweb)

I only got 1 error, which was about Content::$mName 

You can fix it temporary by disabling the line as follows:

e.g.:

//  $description = str_replace('{title}',$curcontent->mName,$description);
      
avatar
Date: 2011-12-12 02:00
Posted By: Prue Rowland (psy)

Alternatively, change action.default.php line 77 to:

  $description = str_replace('{title}',$curcontent->Name(),$description);


Change action.default.php line 169 and 170 to:

if (($curcontent->Properties()->GetPropertyValue('image') != -1) &&
($curcontent->Properties()->GetPropertyValue('image') != ""))  {
    	$image = $curcontent->Properties()->GetPropertyValue('image');



And I'm not sure if this is correct (at least I have the keywords and no
errors!) change $content to $this->curcontent:

From about line 670 in module.SEOTools.php 

	/* Generate keywords from page title, description and content */
$title_keywords = $this->get_keywords($this->curcontent['content_en'],
$this->getPreference('keyword_minlength','6'));
$description_keywords = $this->get_keywords($this->curcontent[$description_id],
$this->getPreference('keyword_minlength','6'));

	
//	$headline_keywords =
$this->get_keywords($this->get_headlines($content['content']),
$this->getPreference('keyword_minlength','6'));
$headline_keywords =
$this->get_keywords($this->get_headlines($this->curcontent['title']),
$this->getPreference('keyword_minlength','6'));
	
	
$content_keywords = $this->get_keywords($this->curcontent['content_en'],
$this->getPreference('keyword_minlength','6'));
	
Hope this fudge helps until we get an official bug fix.
Cheers
psy