Fork me on GitHub
Subscribe 2

Ticket #970 (fixed bug)

Make FluxBB compatible with Laravel 4.1.26

  • Created: 2014-05-12 12:00:29
  • Reported by: XAOS-Eric
  • Assigned to: Franz
  • Milestone: 2.0-alpha2
  • Component: code
  • Priority: normal

I upgraded to Laravel 4.1.27 yesterday on my development laptop, and noticed that the new version of Laravel breaks our users implementation. I have done some testing, and based on, the documentation, this fixes the issues I had with laravel 4.1.27 comparability.

The following should be added to FluxBB\Models\User:

public function getRememberToken()
{
    return $this->remember_token;
}

public function setRememberToken($value)
{
    $this->remember_token = $value;
}

public function getRememberTokenName()
{
    return 'remember_token';
}

And the following should be added to FluxBB\Migrations\Install\Users:

$table->string('remember_token')->nullable();

I'm able to push the changes to github later tonight, unless this should be implemented differently?

Edit: if needed, here is the link to docs http://laravel.com/docs/upgrade#upgrade-4.1.26, and existing default User.php models provided by laravel need to be manually updated as well.

History

XAOS-Eric 2014-05-12 12:04:55

  • Description changed. (Diff)

Franz 2014-05-13 05:46:27

  • Milestone set to 2.0-alpha2.
  • Owner set to Franz.

Thanks for the heads-up. Good point.

XAOS-Eric 2014-05-14 13:44:35

I just submitted a pull request for this. Referenced this ticket in the pull request.

https://github.com/fluxbb/core/pull/24

Franz 2014-05-14 13:49:02

  • Status changed from open to fixed.

Thanks for the PR.