1
|
|
|
<?php |
2
|
|
|
|
3
|
|
|
namespace Acacha\Users\Http\Controllers; |
4
|
|
|
|
5
|
|
|
use Acacha\Users\Events\UserMigrationStatusUpdated; |
6
|
|
|
use Acacha\Users\Http\Requests\UsersMigrationRequest; |
7
|
|
|
|
8
|
|
|
/** |
9
|
|
|
* Class UsersMigrationController. |
10
|
|
|
* |
11
|
|
|
* @package App\Http\Controllers |
12
|
|
|
*/ |
13
|
|
|
class UsersMigrationController extends Controller |
14
|
|
|
{ |
15
|
|
|
/** |
16
|
|
|
* Show users migration. |
17
|
|
|
* |
18
|
|
|
* @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View |
19
|
|
|
*/ |
20
|
|
|
public function index() |
21
|
|
|
{ |
22
|
|
|
$this->authorize('migrate-users'); |
23
|
|
|
$data = []; |
24
|
|
|
return view('acacha_users::users-migration',$data); |
25
|
|
|
} |
26
|
|
|
|
27
|
|
|
/** |
28
|
|
|
* Get total number of users in source migration database. |
29
|
|
|
*/ |
30
|
|
|
public function totalNumberOfUsers() |
31
|
|
|
{ |
32
|
|
|
$this->authorize('migrate-users'); |
33
|
|
|
return [ 'data' => 68 ]; |
34
|
|
|
} |
35
|
|
|
|
36
|
|
|
/** |
37
|
|
|
* Migrate multiple users. |
38
|
|
|
* |
39
|
|
|
*/ |
40
|
|
|
public function migrate(UsersMigrationRequest $request) |
41
|
|
|
{ |
42
|
|
|
$usersToMigrate = $this->usersToMigrateByRequest($request); |
43
|
|
|
dd($usersToMigrate); |
44
|
|
|
$progress= 0; |
45
|
|
|
foreach ($usersToMigrate as $user ) { |
|
|
|
|
46
|
|
|
event(new UserMigrationStatusUpdated($user, $progress)); |
47
|
|
|
sleep(1); |
48
|
|
|
dump ('User ' . $user . ' migrated!'); |
49
|
|
|
$progress = $progress + 50; |
50
|
|
|
} |
51
|
|
|
event(new UserMigrationStatusUpdated('Finish!', $progress)); |
52
|
|
|
} |
53
|
|
|
|
54
|
|
|
|
55
|
|
|
/** |
56
|
|
|
* Get users to migrate. |
57
|
|
|
* |
58
|
|
|
* @param $request |
59
|
|
|
*/ |
60
|
|
|
protected function usersToMigrateByRequest($request) |
61
|
|
|
{ |
62
|
|
|
if ($users = $request->has('users')) return $this->usersToMigrate($users, $request); |
63
|
|
|
if ($filter = $request->has('filter')) return $this->usersToMigrateByFilter($filter, $request); |
64
|
|
|
return $this->allUsers($request); |
65
|
|
|
} |
66
|
|
|
|
67
|
|
|
/** |
68
|
|
|
* Users to migrate by user id list. |
69
|
|
|
* |
70
|
|
|
* @param $users |
71
|
|
|
*/ |
72
|
|
|
protected function usersToMigrate($users, $request) |
|
|
|
|
73
|
|
|
{ |
74
|
|
|
|
75
|
|
|
} |
76
|
|
|
|
77
|
|
|
/** |
78
|
|
|
* Users to migrate by filter. |
79
|
|
|
* |
80
|
|
|
* @param $filter |
81
|
|
|
*/ |
82
|
|
|
protected function usersToMigrateByFilter($filter, $request) |
|
|
|
|
83
|
|
|
{ |
84
|
|
|
|
85
|
|
|
} |
86
|
|
|
|
87
|
|
|
/** |
88
|
|
|
* Get all valid users. |
89
|
|
|
* |
90
|
|
|
* @param $request |
91
|
|
|
* @return \Illuminate\Database\Eloquent\Collection|static[] |
92
|
|
|
*/ |
93
|
|
|
protected function allUsers($request) |
|
|
|
|
94
|
|
|
{ |
95
|
|
|
return \Scool\EbreEscoolModel\User::all(); |
96
|
|
|
} |
97
|
|
|
} |
98
|
|
|
|
There are different options of fixing this problem.
If you want to be on the safe side, you can add an additional type-check:
If you are sure that the expression is traversable, you might want to add a doc comment cast to improve IDE auto-completion and static analysis:
Mark the issue as a false-positive: Just hover the remove button, in the top-right corner of this issue for more options.