Jump to:

1016 Posts in 567 Topics by 309 members

Forum Module

SilverStripe Forums » Forum Module » Forum module upgrade failed

Discuss the Forum Module.

Moderators: martimiz, Sean, biapar, Willr, Ingo, swaiba, simon_w

Page: 1
Go to End
Author Topic: 463 Views
  • ApocSyn
    Avatar
    Community Member
    3 Posts

    Forum module upgrade failed Link to this post

    I am on version 2.4.5 of of Silverstripe and recently tried to upgrade from the .3 forums to the master using the upgrade method located here:

    https://github.com/silverstripe/silverstripe-forum/blob/ec4828380b24d91b6cdf8d730db3975ce3dc515e/docs/en/Upgrading.md

    I did the following: Deleted old forums, uploaded new, ran /dev/build, then ran the upgrade script.

    After the upgrade I also installed the master tagfield.

    And then I ran into the problems. The front page initially came with a console warning regarding an outdated method used on line 706 of /forum/code/ForumHolder.php located which is this:

    $this->getNewPostsAvailable(null, null, $forumID, $threadID, &$data);

    Also all Forum pages will no longer save and publish correctly.

    This is a new site so I could carpet bomb it and start over. But has anyone else ran into these problems?

  • ApocSyn
    Avatar
    Community Member
    3 Posts

    Re: Forum module upgrade failed Link to this post

    Here is the exact error message received:

    Warning: Call-time pass-by-reference has been deprecated in /home/.../....com/forum/code/ForumHolder.php on line 706

    This is with the latest Master forum and the latest stable Silverstripe release.

    Any ideas?

  • Willr
    Avatar
    Forum Moderator
    5464 Posts

    Re: Forum module upgrade failed Link to this post

    Note this is just a warning, not an error message.

    this->getNewPostsAvailable(null, null, $forumID, $threadID, &$data); Should be $this->getNewPostsAvailable(null, null, $forumID, $threadID, $data); (without the &).

  • ApocSyn
    Avatar
    Community Member
    3 Posts

    Re: Forum module upgrade failed Link to this post

    Thanks Will!

    I ended up blowing the test site up and reinstalling. Getting a similar error but now I know how to fix it

    463 Views
Page: 1
Go to Top

Want to know more about the company that brought you SilverStripe? Then check out SilverStripe.com

Comments on this website? Please give feedback.