RemoveBackpackuserModel::up() - Code Metrics - Inspection of "Added support for Backpack 4.1" - Laravel-Backpack/PermissionManager - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#228)

by Cristian

created 2020-03-31 12:33 UTC

RemoveBackpackuserModel::up() A

↳ Parent: RemoveBackpackuserModel

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	10
rs	9.9332
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Log;

class RemoveBackpackuserModel extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        // establish the table names
        $model_has_roles = config('permission.table_names.model_has_roles');
        $model_has_permissions = config('permission.table_names.model_has_permissions');

        // replace the BackpackUser model with User
        $this->replaceModels($model_has_roles);
        $this->replaceModels($model_has_permissions);
    }

    public function replaceModels($table_name)
    {
        Log::info('Replacing BackpackUser model in '.$table_name);

        // if you've ended up with duplicate entries (both for App\User and App\Models\BackpackUser)
        // we can just delete them
        $userEntries = DB::table($table_name)
            ->where('model_type', "App\User")
            ->get();

        foreach ($userEntries as $entry) {
            DB::table($table_name)
                ->where('role_id', $entry->role_id)
                ->where('model_type', 'App\Models\BackpackUser')
                ->where('model_id', $entry->model_id)
                ->delete();
        }

        // for the rest of them, we can just replace the BackpackUser model with User
        DB::table($table_name)
            ->where('model_type', "App\Models\BackpackUser")
            ->update([
                'model_type' => "App\User",
            ]);
    }
}


1			<?php
2
3			use Illuminate\Database\Migrations\Migration;
4			use Illuminate\Support\Facades\DB;
5			use Illuminate\Support\Facades\Log;
6
7			class RemoveBackpackuserModel extends Migration
8			{
9			/**
10			* Run the migrations.
11			*
12			* @return void
13			*/
14			public function up()
15			{
16			// establish the table names
17			$model_has_roles = config('permission.table_names.model_has_roles');
18			$model_has_permissions = config('permission.table_names.model_has_permissions');
19
20			// replace the BackpackUser model with User
21			$this->replaceModels($model_has_roles);
22			$this->replaceModels($model_has_permissions);
23			}
24
25			public function replaceModels($table_name)
26			{
27			Log::info('Replacing BackpackUser model in '.$table_name);
28
29			// if you've ended up with duplicate entries (both for App\User and App\Models\BackpackUser)
30			// we can just delete them
31			$userEntries = DB::table($table_name)
32			->where('model_type', "App\User")
33			->get();
34
35			foreach ($userEntries as $entry) {
36			DB::table($table_name)
37			->where('role_id', $entry->role_id)
38			->where('model_type', 'App\Models\BackpackUser')
39			->where('model_id', $entry->model_id)
40			->delete();
41			}
42
43			// for the rest of them, we can just replace the BackpackUser model with User
44			DB::table($table_name)
45			->where('model_type', "App\Models\BackpackUser")
46			->update([
47			'model_type' => "App\User",
48			]);
49			}
50			}
51

Laravel-Backpack / PermissionManager

Scrutinizer GitHub App not installed

Pull Request — master (#228)

RemoveBackpackuserModel::up() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like