1
goffy
newbb on xoops.org
  • 2019/6/24 9:40

  • goffy

  • Just can't stay away

  • Posts: 543

  • Since: 2010/12/27


Hi

which version of newbb is used on xoops.org?

with latest version from https://github.com/XoopsModules25x/newbb I get an error when I want to have the list with recent topics (like it is possible on xoops.org)
Quote:
Error: Cannot use string offset as an array in Datei /modules/newbb/class/TopicRenderer.php Zeile 745


I use:
Xoops 2.5.10
Latest Version of NewBB from github

2
Mamba
Re: newbb on xoops.org
  • 2019/6/24 11:10

  • Mamba

  • Moderator

  • Posts: 11409

  • Since: 2004/4/23


NewBB 5.0 and XOOPS 2.5.10
Support XOOPS => DONATE
Use 2.5.11 | Docs | Modules | Bugs

3
goffy
Re: newbb on xoops.org
  • 2019/6/24 12:12

  • goffy

  • Just can't stay away

  • Posts: 543

  • Since: 2010/12/27


so we use the same :-| but I get an error

I tried with php 7.2 and 7.3

any idea?

4
zyspec
Re: newbb on xoops.org
  • 2019/6/24 16:18

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


@goffy,

It looks to me like there's a minor bug in the logic. $status is defined as a string variable, but later it is accessed as an array.

The code in ./class/TopicRenderer.php (line 730) is:
$status '';


Should be changed to:
$status = [];

5
goffy
Re: newbb on xoops.org
  • 2019/6/24 18:43

  • goffy

  • Just can't stay away

  • Posts: 543

  • Since: 2010/12/27


hi zyspec

I have already seen this. if I change as suggested I get Quote:
Fatal error: Maximum execution time of 120 seconds exceeded in C:\Homepages\wamp\www\myxoops\html\class\database\mysqldatabase.php on line 277


I try to find out the problem

6
zyspec
Re: newbb on xoops.org
  • 2019/6/26 16:58

  • zyspec

  • Module Developer

  • Posts: 1095

  • Since: 2004/9/21


Ok - I'd recommend you turn on debug and look at the MySQL queries in debug to see if that leads you in the right direction.

The other thing to try is to look a the value of $types in the Smarty template debug information.

Hopefully one of these will lead you in the right direction. I don't use newbb on any of my sites (including my debug site) right now so I found the bug above by code inspection. There's probably something else that assumes $status is a string... looks like allowing "multiple" types was added somewhere along the way and by irmtfan after initial development.

Login

Who's Online

216 user(s) are online (103 user(s) are browsing Support Forums)


Members: 0


Guests: 216


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