I'm having trouble updating the github_oauth_token and github_oauth_refresh_token fields in my User model that have the encrypted cast. All other fields update fine and the casting is working as expected however the fields will not save to the database.
User model
namespace App\Models;
use App\Mail\EmailVerification;
use Illuminate\Support\Facades\Crypt;
use Illuminate\Notifications\Notifiable;
use Illuminate\Support\Facades\Validator;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Support\Str;
use Illuminate\Support\Facades\Mail;
class User extends Authenticatable
{
use HasFactory, Notifiable;
/**
* The attributes that are mass assignable.
*
* #var array
*/
protected $fillable = [
'first_name',
'last_name',
'email',
'email_verification_token',
'password',
'github_oauth_token',
'github_oauth_refresh_token',
];
/**
* The attributes that should be hidden for arrays.
*
* #var array
*/
protected $hidden = [
'password',
'remember_token',
];
/**
* The attributes that should be cast to native types.
*
* #var array
*/
protected $casts = [
'email_verified_at' => 'datetime',
'github_oauth_token' => 'encrypted',
'github_oauth_refresh_token' => 'encrypted',
];
/**
* Set the user's github oauth token.
*
* #param string $value
* #return void
*/
public function setGithubOauthTokenAttribute($value)
{
$this->github_oauth_token = Crypt::encryptString($value);
}
/**
* Set the user's github oauth refresh token.
*
* #param string $value
* #return void
*/
public function setGithubOauthRefreshTokenAttribute($value)
{
$this->github_oauth_refresh_token = Crypt::encryptString($value);
}
}
User migration
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;
class CreateUsersTable extends Migration
{
/**
* Run the migrations.
*
* #return void
*/
public function up()
{
Schema::create('users', function (Blueprint $table) {
$table->id();
$table->string('first_name', 45);
$table->string('last_name', 45);
$table->string('email')->unique();
$table->string('email_verification_token', 30)->nullable();
$table->timestamp('email_verified_at')->nullable();
$table->string('password');
$table->string('github_oauth_token')->nullable();
$table->string('github_oauth_refresh_token')->nullable();
$table->rememberToken();
$table->timestamps();
});
}
/**
* Reverse the migrations.
*
* #return void
*/
public function down()
{
Schema::dropIfExists('users');
}
}
Controller method
public function handleGitHubCallback()
{
$gitHubUser = Socialite::driver('github')->user();
$user = Auth::user();
$user->github_oauth_token = $gitHubUser->token; // does not update
$user->github_oauth_refresh_token = $gitHubUser->refreshToken; // does not update
$user->first_name = 'Johnathan'; // updates fine
$user->save();
return redirect()->route('space.list.get')->with('success', Lang::get('messages.success_github_linked'));
}
The issue is in User Model.You are casting both fields in $cast property
protected $casts = [
'email_verified_at' => 'datetime',
'github_oauth_token' => 'encrypted',
'github_oauth_refresh_token' => 'encrypted',
];
So no need to add Mutators so remove both below Mutators.
public function setGithubOauthTokenAttribute($value)
{
$this->github_oauth_token = Crypt::encryptString($value);
}
public function setGithubOauthRefreshTokenAttribute($value)
{
$this->github_oauth_refresh_token = Crypt::encryptString($value);
}
Also if you want to use mutators then you have an error it should b like below
public function setGithubOauthTokenAttribute($value)
{
$this->attributes['github_oauth_token ']= Crypt::encryptString($value);
}
public function setGithubOauthRefreshTokenAttribute($value)
{
$this->attributes['github_oauth_refresh_token ']= Crypt::encryptString($value);
}
Related
I'm currently working on a laravel application where a user can like/dislike a content.
The migrations for the respective tables look as follows, where user_content_interaction is the relation between users and contents:
<?php
declare(strict_types=1);
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;
class CreateUserTable extends Migration
{
/**
* Run the migrations.
*
* #return void
*/
public function up()
{
Schema::create('user', function (Blueprint $table) {
$table->id();
$table->string('name');
$table->string('email')->unique();
$table->timestamp('email_verified_at')->nullable();
$table->string('password');
$table->rememberToken();
$table->timestamps();
});
}
/**
* Reverse the migrations.
*
* #return void
*/
public function down()
{
Schema::dropIfExists('user');
}
}
<?php
declare(strict_types=1);
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;
class CreateContentTable extends Migration
{
public const URL_LEN = 2083;
public const STR_LEN = 256;
/**
* Run the migrations.
*
* #return void
*/
public function up()
{
Schema::create('content', function (Blueprint $table) {
$table->id();
$table->string('title', self::STR_LEN);
$table->date('release_date');
$table->enum('content_type', ['movie', 'tv_show', 'short_film', 'mini_tv_show']);
$table->json('genre');
$table->json('tags');
$table->integer('runtime');
$table->longText('short_description');
$table->json('cast');
$table->json('directors');
$table->enum('age_restriction', [0, 6, 12, 16, 18]);
$table->string('poster_url', self::URL_LEN);
$table->string('youtube_trailer_url', self::URL_LEN);
$table->string('production_company', self::STR_LEN);
$table->integer('seasons');
$table->integer('average_episode_count');
$table->timestamps();
});
}
/**
* Reverse the migrations.
*
* #return void
*/
public function down()
{
Schema::dropIfExists('content');
}
}
<?php
declare(strict_types=1);
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;
class CreateUserContentInteractionTable extends Migration
{
/**
* Run the migrations.
*
* #return void
*/
public function up()
{
Schema::create('user_content_interaction', function (Blueprint $table) {
$table->foreignId('content_id')->constrained('content');
$table->foreignId('user_id')->constrained('user');
$table->primary([
'content_id',
'user_id'
]);
$table->enum('like_status', ['liked', 'disliked'])->nullable();
$table->integer('dislike_count')->default(0)->nullable(false);
$table->timestamps();
});
}
/**
* Reverse the migrations.
*
* #return void
*/
public function down()
{
Schema::dropIfExists('user_content_interaction');
}
}
The respective models look like this:
<?php
declare(strict_types=1);
namespace App\Models;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;
use Laravel\Sanctum\HasApiTokens;
class User extends Authenticatable
{
use HasApiTokens;
use HasFactory;
use Notifiable;
/**
* Name of the associated Table in the Database
*
* #var string
*/
protected $table = 'user';
/**
* The attributes that are mass assignable.
*
* #var array
*/
protected $fillable = [
'name',
'email',
'password',
];
/**
* The attributes that should be hidden for arrays.
*
* #var array
*/
protected $hidden = [
'password',
'remember_token',
];
/**
* The attributes that should be cast to native types.
*
* #var array
*/
protected $casts = [
'email_verified_at' => 'datetime',
];
public function contents(): \Illuminate\Database\Eloquent\Relations\HasMany
{
return $this->hasMany(Content::class);
}
}
<?php
declare(strict_types=1);
namespace App\Models;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
class Content extends Model
{
use HasFactory;
/**
* Name of the associated Table in the Database
*
* #var string
*/
protected $table = 'content';
/**
*
* The primary key of the table
*
* #var string
*/
protected $primaryKey = 'id';
/**
*
* The columns that are fillable
*
* #var array
*/
protected $fillable = [
'title',
'release_date',
'content_type',
'genre',
'tags',
'runtime',
'short_description',
'cast',
'directors',
'age_restriction',
'poster_url',
'youtube_trailer_url',
'production_company',
'seasons',
'average_episode_count',
'updated_at'
];
protected $casts = [
'varchar' => 'string',
'release_date' => 'string',
'content_type' => 'string',
'genre' => 'string',
'tags' => 'string',
'runtime' => 'int',
'short_description' => 'string',
'cast' => 'string',
'directors' => 'string',
'age_restriction' => 'string',
'seasons' => 'int',
'average_episode_count' => 'int',
];
}
<?php
declare(strict_types=1);
namespace App\Models;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Illuminate\Database\Eloquent\Relations\Pivot;
class UserContentInteraction extends Pivot
{
use HasFactory;
/**
* Name of the associated Table in the Database
*
* #var string
*/
protected $table = 'user_content_interaction';
protected $primaryKey = 'content_id';
protected $foreignKey = 'user_id';
public function user(): BelongsTo
{
return $this->belongsTo('User');
}
public function content(): BelongsTo
{
return $this->belongsTo('Content');
}
}
And I'm trying to seed the database on my dev env like this:
User::factory()->count(4)
->hasAttached(
Content::factory()->count(10),
[
'like_status' => Arr::random(['liked', 'disliked']),
'created_at' => Carbon::now()
],
)
->create();
Which results in the following error when calling the create() method of the user factory:
Call to undefined method Illuminate\Database\Eloquent\Relations\HasMany::attach()
I'm looking for any advice how to fix this problem, maybe I made a design mistake in the first place. Or there is an easier way to construct this relation. Though it would be cool if I could keep the custom name for the relation. I'm very unfamiliar with Eloquent ORM so when there is a much easier and lightweight way of doing this feel free to point that out.
EDIT:
I adjusted the relationship migration to look like this:
<?php
declare(strict_types=1);
use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;
class CreateUserContentInteractionTable extends Migration
{
/**
* Run the migrations.
*
* #return void
*/
public function up()
{
Schema::create('user_content_interactions', function (Blueprint $table) {
$table->id('user_content_interaction_id');
$table->foreignId('content_id')->constrained('content');
$table->foreignId('user_id')->constrained('user');
$table->enum('like_status', ['liked', 'disliked'])->nullable();
$table->integer('dislike_count')->default(0)->nullable(false);
$table->string('user_content_interaction_type');
$table->timestamps();
});
}
/**
* Reverse the migrations.
*
* #return void
*/
public function down()
{
Schema::dropIfExists('user_content_interaction');
}
}
fixed some errors and defined the relation in the model like this:
public function contents(): \Illuminate\Database\Eloquent\Relations\MorphToMany
{
return $this->morphToMany(Content::class, 'user_content_interaction');
}
Now I'm facing the problem that when seeding the ids from User and Content aren't taken into the relationship as foreign ids. The error message looks like this:
SQLSTATE[HY000]: General error: 1364 Field 'user_id' doesn't have a default value (SQL: insert into `user_content_interactions` (`content_id`, `created_at`, `like_status`, `user_content_interaction_id`, `user_content_interaction_type`) values (1, 2022-10-26 16:51:14, disliked, 1, App\Models\User))
I think I have some mistake in the seeder. I feel like I have to do something like this in the seeder:
User::factory()->count(4)
->hasAttached(
Content::factory()->count(10),
[
'user_id' => currentUser->id,
'content_id' => currentContent->id,
'like_status' => Arr::random(['liked', 'disliked']),
'created_at' => Carbon::now()
],
)
->create();
I am using solcialite to login Laravel via Gihub, but there is problem to login. I get a error: Symfony \ Component \ Debug \ Exception \ FatalThrowableError (E_RECOVERABLE_ERROR)
Type error: Argument 1 passed to Illuminate\Auth\SessionGuard::login() must implement interface Illuminate\Contracts\Auth\Authenticatable, null given, called in /var/www/html/testio/vendor/laravel/framework/src/Illuminate/Auth/AuthManager.php on line 292
My LoginController:
<?php
namespace App\Http\Controllers\Auth;
use App\Http\Controllers\Controller;
use Illuminate\Foundation\Auth\AuthenticatesUsers;
use Socialite;
use App\User;
use Auth;
class LoginController extends Controller
{
use AuthenticatesUsers;
protected $redirectTo = '/home';
public function __construct()
{
$this->middleware('guest')->except('logout');
}
/**
* Redirect the user to the GitHub authentication page.
*
* #return \Illuminate\Http\Response
*/
public function redirectToProvider()
{
return Socialite::driver('github')->redirect();
}
/**
* Obtain the user information from GitHub.
*
* #return \Illuminate\Http\Response
*/
public function handleProviderCallback()
{
$github_user = Socialite::driver('github')->user();
$user = $this->userFindorCreate($github_user);
Auth::login($user, true);
return redirect('/home');
// $user->token;
}
public function userFindorCreate($github_user){
$user = User::where('provider_id', $github_user->id)->first();
if(!$user){
$user = new User;
$user->name = $github_user->getName();
$user->email = $github_user->getEmail();
$user->provider_id = $github_user->getid();
$user->save();
}
}
}
My User.php model:
<?php
namespace App;
use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Foundation\Auth\User as AuthUser;
class User extends Authenticatable
{
use Notifiable;
/**
* The attributes that are mass assignable.
*
* #var array
*/
protected $fillable = [
'email',
];
/**
* The attributes that should be hidden for arrays.
*
* #var array
*/
protected $hidden = [
'password', 'remember_token',
];
}
create_users_table :
use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;
class CreateUsersTable extends Migration
{
/**
* Run the migrations.
*
* #return void
*/
public function up()
{
Schema::create('users', function (Blueprint $table) {
$table->increments('id');
$table->string('name')->nullable();
$table->string('email')->unique();
$table->string('password')->nullable();
$table->string('provider_id');
$table->rememberToken();
$table->timestamps();
});
}
Damn it what I am doing wrong?
I missed return $users; on last function:
public function userFindorCreate($github_user){
$user = User::where('provider_id', $github_user->id)->first();
if(!$user){
$user = new User;
$user->name = 'jojo';
$user->email = $github_user->getEmail();
$user->provider_id = $github_user->getid();
$user->save();
}
return $user;
}
I have created a trait that creates a table in the database if it doesn't exist.
the table is created perfectly in the DB but I get an error. The error is throw by Illuminate\Database\Eloquent\Concerns\HasAttributes.php
TRAIT:
namespace App\Traits;
use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;
trait Translatable{
//CREATE LANGUAGES TABLE IF DOES NOT EXIST
public function createTranslationTable()
{
//ADD LANGUAGES TABLE IF DOES NOT EXIST
if(Schema::hasTable('languages') === false){
Schema::create('languages', function (Blueprint $table) {
$table->increments('id');
$table->string('language');
$table->string('slug');
});
}
//CREATE TRANSLATION TABLE IF DOES NOT EXIST
if(Schema::hasTable('translations') === false){
Schema::create('translations', function (Blueprint $table) {
$table->increments('id');
$table->string('model');
$table->string('language_id');
$table->foreign('language_id')->references('id')->on('languages')->onDelete('cascade');
$table->string('field_name');
$table->string('field_translation');
});
}
}
public function translateMe()
{
$this->createTranslationTable;
}
}
ERROR:
App\User::createTranslationTable must return a relationship instance.
I am using a trait because I want it to be able to use it with any model like so for eaxample:
class User extends Authenticatable
{
use Notifiable;
use Translatable;
/**
* Call Translatable.
*
* #return void
*/
public function __construct()
{
$this->translateMe();
}
/**
* The attributes that are mass assignable.
*
* #var array
*/
protected $fillable = [
'name', 'email', 'password',
];
/**
* The attributes that should be hidden for arrays.
*
* #var array
*/
protected $hidden = [
'password', 'remember_token',
];
/**
* The attributes that are translatable.
*
* #var array
*/
protected $translatableItems = [
'name'
];
}
I found the solution and it was one simple one.
In the Translatable trait I was calling createTranslationTable method as a property instead then a method. so just adding the two parentesis fixed it.
BEFORE:
public function translateMe()
{
$this->createTranslationTable;
}
NOW:
public function translateMe()
{
$this->createTranslationTable();
}
No idea why I'm getting this error...
Call to a member function getRememberToken() on null (View: /home/vagrant/temptools/resources/views/layouts/main.blade.php) (View: /home/vagrant/temptools/resources/views/layouts/main.blade.php)
I have an Auth::check() on that blade page
I'm using https://github.com/invisnik/laravel-steam-auth
Routes:
Route::get('login', 'AuthController#redirectToSteam')->name('login');
Route::get('login/handle', 'AuthController#handle')->name('login.handle');
Route::post('logout', 'Auth\LoginController#logout')->name('logout');
AuthController:
namespace App\Http\Controllers;
use Invisnik\LaravelSteamAuth\SteamAuth;
use App\User;
use Auth;
class AuthController extends Controller
{
/**
* The SteamAuth instance.
*
* #var SteamAuth
*/
protected $steam;
/**
* The redirect URL.
*
* #var string
*/
protected $redirectURL = '/';
/**
* AuthController constructor.
*
* #param SteamAuth $steam
*/
public function __construct(SteamAuth $steam)
{
$this->steam = $steam;
}
/**
* Redirect the user to the authentication page
*
* #return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
*/
public function redirectToSteam()
{
return $this->steam->redirect();
}
/**
* Get user info and log in
*
* #return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector
*/
public function handle()
{
if ($this->steam->validate()) {
$info = $this->steam->getUserInfo();
if (!is_null($info)) {
$user = $this->findOrNewUser($info);
Auth::login($user, true);
return redirect($this->redirectURL); // redirect to site
}
}
return $this->redirectToSteam();
}
/**
* Getting user by info or created if not exists
*
* #param $info
* #return User
*/
protected function findOrNewUser($info)
{
$user = User::where('id', $info->steamID64)->first();
if (!is_null($user)) {
return $user;
}
return User::create([
'name' => $info->personaname,
'avatar' => $info->avatarfull,
'id' => $info->steamID64
]);
}
}
app/User:
namespace App;
use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;
class User extends Authenticatable
{
use Notifiable;
protected $fillable = [
'id', 'name', 'avatar',
];
protected $hidden = [
'remember_token',
];
}
create_users_table migration:
use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;
class CreateUsersTable extends Migration
{
/**
* Run the migrations.
*
* #return void
*/
public function up()
{
Schema::create('users', function (Blueprint $table) {
$table->bigInteger('id')->unsigned();
$table->string('name');
$table->string('avatar');
$table->rememberToken();
$table->timestamps();
$table->primary('id');
});
}
/**
* Reverse the migrations.
*
* #return void
*/
public function down()
{
Schema::dropIfExists('users');
}
}
Have deleted the password recovery migration.
I also get the error if I hit the logout path:
Call to a member function getRememberToken() on null
========
Looking into errors, the error seems to be in laravel/framework/sec/Illuminate/Auth/EloquentUserProvider.php line 67
public function retrieveByToken($identifier, $token)
{
$model = $this->createModel();
$model = $model->where($model->getAuthIdentifierName(), $identifier)->first();
$rememberToken = $model->getRememberToken();
return $model && $rememberToken && hash_equals($rememberToken, $token) ? $model : null;
}
Still no idea how to fix it though
Looks like this package uses Laravel Auth... How do you set unique ID for your users? Generally you should auto increment them... Don't forget that Laravel's auth is reliant on many conventions... Look at AuthenticatesUsers trait for a peek... but one of the fields required is email...
trait AuthenticatesUsers
{
/**
* Get the login username to be used by the controller.
*
* #return string
*/
public function username()
{
return 'email';
}
}
Take a look around the Auth structure, look at the default 'web' guard (look at auth.php in the config directory as a starting point... ).
public function up()
{
Schema::create('users', function (Blueprint $table) {
$table->increments('id');
$table->string('name');
$table->string('email')->unique(); //Unique ID for login in laravel? The AuthenticatesUser trait uses email as username...
$table->string('password');
$table->string('avatar');
$table->rememberToken();
$table->timestamps();
});
}
Hope this helps...
Go into this function parent:
public function retrieveByToken($identifier, $token){
$model = $this->createModel();
$model = $model->where($model->getAuthIdentifierName(), $identifier)->first();
$rememberToken = $model->getRememberToken();
return $model && $rememberToken && hash_equals($rememberToken, $token) ? $model : null;
}
dump the model and $identifier,
create a model row with that identifier and it will fixed!
I'm trying to install this https://github.com/invisnik/laravel-steam-auth (with some of my own custom fields)
It works fine and all except for that it doesn't create the users (it creates the table) when I log in and I don't know why.
AuthController.php
namespace App\Http\Controllers;
use Invisnik\LaravelSteamAuth\SteamAuth;
use App\User;
use Auth;
class AuthController extends Controller
{
/**
* #var SteamAuth
*/
private $steam;
public function __construct(SteamAuth $steam)
{
$this->steam = $steam;
}
public function login()
{
if ($this->steam->validate()) {
$info = $this->steam->getUserInfo();
if (!is_null($info)) {
$user = User::where('steamid', $info->steamID64)->first();
if (is_null($user)) {
$user = User::create([
'username' => $info->personaname,
'avatar' => $info->avatarfull,
'steamid' => $info->steamID64
]);
}
Auth::login($user, true);
return redirect('/'); // redirect to site
}
}
return $this->steam->redirect(); // redirect to Steam login page
}
}
create_users_table.php
use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;
class CreateUsersTable extends Migration
{
/**
* Run the migrations.
*
* #return void
*/
public function up()
{
Schema::create('users', function (Blueprint $table) {
$table->increments('id');
$table->string('nickname');
$table->string('avatar');
$table->string('steamid')->unique();
$table->string('name');
$table->string('rank');
$table->string('community_id');
$table->string('email')->unique();
$table->string('password', 60);
$table->rememberToken();
$table->timestamps();
});
}
/**
* Reverse the migrations.
*
* #return void
*/
public function down()
{
Schema::dropIfExists('users');
}
}
web.php
Route::get('login', 'AuthController#login')->name('login');
Route::post('logout', 'Auth\LoginController#logout')->name('logout');
steam-auth.php
return [
/*
* Redirect URL after login
*/
'redirect_url' => '/',
/*
* API Key (http://steamcommunity.com/dev/apikey)
*/
'api_key' => 'xxx',
/*
* Is using https ?
*/
'https' => false
];
User.php
namespace App;
use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;
class User extends Authenticatable
{
use Notifiable;
/**
* The attributes that are mass assignable.
*
* #var array
*/
protected $fillable = ['nickname', 'avatar', 'steamid', 'name', 'rank', 'community_id', 'email', 'password'];
/**
* The attributes that should be hidden for arrays.
*
* #var array
*/
protected $hidden = [
'password', 'remember_token',
];
}
Any ideas?
UPDATE:
$info seems to return null and I have no clue why.