1
indyone
PHP Notices in class/smarty/plugins/resource.db.php
  • 2004/1/10 10:46

  • indyone

  • Just popping in

  • Posts: 2

  • Since: 2004/1/10


I'm using the XOOPS 2.0.5.1 and i got this notices (i have E_ALL in my php.ini) when i load the homepage:
WarningUndefined indexdefault_theme in /home/httpd/html/portal.ee.teiath.gr/class/smarty/plugins/resource.db.php on line 71

WarningUndefined indexdefault_theme in /home/httpd/html/portal.ee.teiath.gr/class/smarty/plugins/resource.db.php on line 74

WarningUndefined variablefile_path in /home/httpd/html/portal.ee.teiath.gr/class/smarty/plugins/resource.db.php on line 96

I don't know if this notices appear in v2.0.5.2.
Also the third notice is a bug...
To fix 'em you must correct this lines:

In class/smarty/plugins/resource.db.php on line 71:
It says:
if ($GLOBALS['xoopsConfig']['default_theme'] != 'default') {

Change to:
if (isset($GLOBALS['xoopsConfig']['default_theme']) && $GLOBALS['xoopsConfig']['default_theme'] != 'default') {


In class/smarty/plugins/resource.db.php on line 96
It says:
if ($file_path != "" && file_exists($filepath)) {

Change to:
if ($filepath != "" && file_exists($filepath)) {

2
Mithrandir
Re: PHP Notices in class/smarty/plugins/resource.db.php

This has been corrected in 2.0.5.2 as far as I can see from my copy of it.

3
Herko
Re: PHP Notices in class/smarty/plugins/resource.db.php
  • 2004/1/10 13:33

  • Herko

  • XOOPS is my life!

  • Posts: 4238

  • Since: 2002/2/4 1


Quote:

Mithrandir wrote:
This has been corrected in 2.0.5.2 as far as I can see from my copy of it.


No, the 2.0.5.2 patch only fixes a Cross-site-scripting (CSS) vulnerability in the mylinks module.

Herko

4
Mithrandir
Re: PHP Notices in class/smarty/plugins/resource.db.php

Well, my local copy of XOOPS 2.0.5.2 does not have the mentioned errors but instead the mentioned correct lines.

5
Herko
Re: PHP Notices in class/smarty/plugins/resource.db.php
  • 2004/1/10 14:07

  • Herko

  • XOOPS is my life!

  • Posts: 4238

  • Since: 2002/2/4 1


Mithrandir: are you using the CVS version? If so, the files may have been fixed after 2.0.5.2. Otherwise the files may have been fixed previously, but accidently not included in one of the patches. If your install of XOOPS 2.0.5.2 was a clean one, then you have all the latest updated files, while an upgraded site may miss them.

Herko

6
skalpa
Re: PHP Notices in class/smarty/plugins/resource.db.php
  • 2004/1/10 14:26

  • skalpa

  • Quite a regular

  • Posts: 300

  • Since: 2003/4/16


It's not in 2.0.5.2 as the behavior of the feature it's supposed to enable (the reading of the templates from the themes folder) may change soon, and has never worked until now.
However the errors here are almost "typos" so those have effectively been corrected in the CVS repository .

PS: line 71, the correct fix was: you should replace default_theme with theme_set.

Skalpa.>

7
indyone
Re: PHP Notices in class/smarty/plugins/resource.db.php
  • 2004/1/10 15:20

  • indyone

  • Just popping in

  • Posts: 2

  • Since: 2004/1/10


After rtying to patch XOOPS i noticed that:
- The patch 2.0.5.1 to 2.0.5.2 does not correct this problem
- The XOOPS 2.0.5.2 Package has fixed these notices

So, i downloaded the full package of XOOPS 2.0.5.2...
I suppose that you should include all changes from 2.0.5.1 to 2.0.5.2 patch...

8
Herko
Re: PHP Notices in class/smarty/plugins/resource.db.php
  • 2004/1/10 16:13

  • Herko

  • XOOPS is my life!

  • Posts: 4238

  • Since: 2002/2/4 1


Quote:

indyone wrote:
I suppose that you should include all changes from 2.0.5.1 to 2.0.5.2 patch...


I suppose you're right. The 2.0.5.1->2.0.5.2 patch was a security patch only tho. As Kazu announced in the release post, we'll be releasing 2.0.6 soon, which has many more bugfixes (also ones from before 2.0.5.2), so it was announced.

Herko

9
Mithrandir
Re: PHP Notices in class/smarty/plugins/resource.db.php

I have the CVS version

Login

Who's Online

311 user(s) are online (211 user(s) are browsing Support Forums)


Members: 0


Guests: 311


more...

Donat-O-Meter

Stats
Goal: $100.00
Due Date: Nov 30
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $100.00
Make donations with PayPal!

Latest GitHub Commits