1
lunchbox
xoops 2.4.3 and article RC 2.0
  • 2010/1/2 16:49

  • lunchbox

  • Just popping in

  • Posts: 9

  • Since: 2002/5/31


After I upgraded my site to XOOPS 2.4.3 (from 2.3.3), I found that every time I tried to add a new category while using the article module, I got the following error message:

Fatal errorCall to a member function getVar() on a non-object in ..../modules/article/include/form.category.php on line 26


Besides, I couldn't add a topic either.

Fatal errorCall to a member function getVar() on a non-object in ..../modules/article/edit.topic.php on line 29


Could anyone tell me how to fix this? Thanks a lot!

2
voltan
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/2 18:37

  • voltan

  • Theme Designer

  • Posts: 724

  • Since: 2006/12/5


One of the possibilities :

article need this folder : Frameworks/textsanitizer . please chack your Frameworks folder for it ! you can use (( textsanitizer )) from XOOPS 2.3.3 pack

3
lunchbox
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/3 2:17

  • lunchbox

  • Just popping in

  • Posts: 9

  • Since: 2002/5/31


Voltan, thanks for your reply.

I've checked the Frameworks/textsanitizer folder, it's the same one in XOOPS 2.3.3.

I've also tried to reinstall article RC2 module on XOOPS 2.3.3 site, and the module was working perfectly.

I think maybe this is a compatibility issue..

thanks anyway.



4
voltan
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/3 8:05

  • voltan

  • Theme Designer

  • Posts: 724

  • Since: 2006/12/5


I use article 1 in XOOPS 2.4.x whit out problem !

5
lunchbox
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/4 14:57

  • lunchbox

  • Just popping in

  • Posts: 9

  • Since: 2002/5/31


I've tried to install article module version RC2 and version 1 on a newly installed XOOPS 2.4.3, none of them could allow me to add a category or add a topic...

I don't know where the problem is, maybe I should just use 2.3.3b instead.

Anyway, thanks for the help! voltan

6
chefry
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/8 0:27

  • chefry

  • Home away from home

  • Posts: 1005

  • Since: 2006/10/14


I'm having the same problem on a new install :(

7
ghia
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/8 14:00

  • ghia

  • Community Support Member

  • Posts: 4953

  • Since: 2008/7/3 1


It seems there is a difference in the handling by XOOPS object between 2.3.3 and 2.4.3.

The problem seems to be line 1070 in /kernel/object.php in function get.
Replace
if (!isset($id)) {
by
if (empty($id)) {


8
trabis
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/8 14:12

  • trabis

  • Core Developer

  • Posts: 2269

  • Since: 2006/9/1 1


Quote:

ghia wrote:
It seems there is a difference in the handling by XOOPS object between 2.3.3 and 2.4.3.

The problem seems to be line 1070 in /kernel/object.php in function get.
Replace
if (!isset($id)) {
by
if (empty($id)) {



If you do that then mypoints will stop working!
Well, basically the change was made to allow getting database fields with primary key = 0;

Ghia, do you think it is bad pratice to have rows with id=0?
It will not be a problem for me to revert that change but at the moment I don't see a reason why core should exclude 0 from being a valid row id.

9
ghia
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/8 14:21

  • ghia

  • Community Support Member

  • Posts: 4953

  • Since: 2008/7/3 1


Because under normal circumstances the id fields are auto-increment and have always 1 as first row, so, in practice the 0 as id field is never present.

I think MyPoints should adapt to the core and not vice versa.

10
trabis
Re: xoops 2.4.3 and article RC 2.0
  • 2010/1/8 19:01

  • trabis

  • Core Developer

  • Posts: 2269

  • Since: 2006/9/1 1


I took it has a bug, I did not change it just to work with mypoints!
It will be reverted.

Login

Who's Online

340 user(s) are online (218 user(s) are browsing Support Forums)


Members: 0


Guests: 340


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