Skip to content

Commit 4964eee

Browse files
authored
Merge pull request #32 from D3lph1/dev
Code refactoring.
2 parents ebeeaf2 + 957747f commit 4964eee

36 files changed

+620
-559
lines changed

app/Composers/ShopLayoutComposer.php

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,13 @@
22

33
namespace App\Composers;
44

5+
use App\Contracts\ComposerContract;
56
use App\DataTransferObjects\MonitoringPlayers;
67
use App\Models\Server;
78
use App\Repositories\NewsRepository;
89
use App\Services\Monitoring\MonitoringInterface;
9-
use Illuminate\View\View;
1010
use Illuminate\Http\Request;
11-
use App\Services\QueryManager;
12-
use App\Contracts\ComposerContract;
11+
use Illuminate\View\View;
1312

1413
/**
1514
* Class ShopLayoutComposer
@@ -41,11 +40,6 @@ class ShopLayoutComposer implements ComposerContract
4140
*/
4241
private $servers;
4342

44-
/**
45-
* @var QueryManager
46-
*/
47-
private $qm;
48-
4943
/**
5044
* @var NewsRepository
5145
*/
@@ -58,16 +52,14 @@ class ShopLayoutComposer implements ComposerContract
5852

5953
/**
6054
* @param Request $request
61-
* @param QueryManager $qm
6255
* @param NewsRepository $newsRepository
6356
* @param MonitoringInterface $monitoring
6457
*/
65-
public function __construct(Request $request, QueryManager $qm, NewsRepository $newsRepository, MonitoringInterface $monitoring)
58+
public function __construct(Request $request, NewsRepository $newsRepository, MonitoringInterface $monitoring)
6659
{
6760
$this->request = $request;
6861
$this->currentServer = $request->get('currentServer');
6962
$this->servers = $request->get('servers');
70-
$this->qm = $qm;
7163
$this->newsRepository = $newsRepository;
7264
$this->monitoring = $monitoring;
7365
}

app/Console/Commands/Payment/Complete.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
namespace App\Console\Commands\Payment;
44

5-
use App\Services\QueryManager;
5+
use App\Repositories\PaymentRepository;
66
use Illuminate\Console\Command;
77
use Symfony\Component\Console\Input\InputArgument;
88

@@ -30,18 +30,18 @@ class Complete extends Command
3030
protected $description = 'Complete payment and give products or money to user';
3131

3232
/**
33-
* @var QueryManager
33+
* @var PaymentRepository
3434
*/
35-
protected $qm;
35+
protected $paymentRepository;
3636

3737
/**
3838
* Create a new command instance.
3939
*
40-
* @param QueryManager $qm
40+
* @param PaymentRepository $paymentRepository
4141
*/
42-
public function __construct(QueryManager $qm)
42+
public function __construct(PaymentRepository $paymentRepository)
4343
{
44-
$this->qm = $qm;
44+
$this->paymentRepository = $paymentRepository;
4545
parent::__construct();
4646
}
4747

@@ -54,7 +54,7 @@ public function handle()
5454
{
5555
$id = (int)$this->argument('id');
5656
$distributor = \App::make('distributor');
57-
$payment = $this->qm->payment($id);
57+
$payment = $this->paymentRepository->find($id);
5858

5959
if (!$payment) {
6060
$this->error('Payment not found');
@@ -70,7 +70,7 @@ public function handle()
7070

7171
if (!$payment->products) {
7272
refill_user_balance((int)$payment->cost, $payment->user_id);
73-
$this->qm->completePayment($payment->id, 'Завершен администратором');
73+
$this->paymentRepository->complete($payment->id, 'Завершен администратором');
7474
$this->info('Payment successfully completed and user balance updated');
7575

7676
return 0;

app/Console/Commands/Server/Rcon.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
use D3lph1\MinecraftRconManager\Connector;
99
use D3lph1\MinecraftRconManager\Exceptions\ConnectSocketException;
1010
use Illuminate\Console\Command;
11+
use Illuminate\Http\Request;
1112

1213
/**
1314
* Class Rcon
@@ -60,8 +61,10 @@ public function __construct(ServerRepository $serverRepository, Connector $rconC
6061
*
6162
* @return mixed
6263
*/
63-
public function handle()
64+
public function handle(Request $request)
6465
{
66+
$this->info($request->method());
67+
6568
$servers = $this->serverRepository->all();
6669
$names = $servers->map(function ($item) {
6770
return $item->name;

app/Http/Controllers/Admin/Items/EditController.php

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
namespace App\Http\Controllers\Admin\Items;
44

55
use App\Http\Requests\Admin\SaveEditedItemRequest;
6+
use App\Repositories\ItemRepository;
67
use App\Services\AdminItems;
78
use Illuminate\Http\Request;
89
use App\Http\Controllers\Controller;
@@ -34,13 +35,16 @@ public function __construct(AdminItems $adminItems)
3435
/**
3536
* Render the edit item page.
3637
*
37-
* @param Request $request
38+
* @param Request $request
39+
* @param ItemRepository $itemRepository
3840
*
3941
* @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
4042
*/
41-
public function render(Request $request)
43+
public function render(Request $request, ItemRepository $itemRepository)
4244
{
43-
$item = $this->qm->item($request->route('item'), [
45+
$itemId = (int)$request->route('item');
46+
47+
$item = $itemRepository->find($itemId, [
4448
'id',
4549
'name',
4650
'type',

app/Http/Controllers/Admin/Items/ListController.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace App\Http\Controllers\Admin\Items;
44

5+
use App\Repositories\ItemRepository;
56
use Illuminate\Http\Request;
67
use App\Http\Controllers\Admin\ListParent;
78

@@ -17,17 +18,18 @@ class ListController extends ListParent
1718
/**
1819
* Render page with items list.
1920
*
20-
* @param Request $request
21+
* @param Request $request
22+
* @param ItemRepository $itemRepository
2123
*
2224
* @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
2325
*/
24-
public function render(Request $request)
26+
public function render(Request $request, ItemRepository $itemRepository)
2527
{
2628
$orderBy = $this->checkOrderBy($request->get('orderBy'));
2729
$orderType = $this->checkOrderType($request->get('orderType'));
2830
$filter = $request->get('filter');
2931

30-
$items = $this->qm->items([
32+
$items = $itemRepository->forAdmin([
3133
'id',
3234
'name',
3335
'type',

app/Http/Controllers/Admin/Products/AddController.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
use App\Exceptions\ItemNotFoundException;
66
use App\Http\Requests\Admin\SaveAddedProductRequest;
77
use App\Repositories\ItemRepository;
8+
use App\Repositories\ServerRepository;
89
use App\Services\AdminProducts;
910
use Carbon\Carbon;
1011
use Illuminate\Http\Request;
@@ -42,15 +43,15 @@ public function __construct(AdminProducts $adminProducts)
4243
*
4344
* @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
4445
*/
45-
public function render(Request $request, ItemRepository $itemRepository)
46+
public function render(Request $request, ItemRepository $itemRepository, ServerRepository $serverRepository)
4647
{
4748
$items = $itemRepository->all([
4849
'id',
4950
'name',
5051
'type'
5152
]);
5253

53-
$categories = $this->qm->allCategoriesWithServers([
54+
$categories = $serverRepository->allWithCategories([
5455
'categories.name as category',
5556
'categories.id as category_id',
5657
'servers.name as server',

app/Http/Controllers/Admin/Products/EditController.php

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
namespace App\Http\Controllers\Admin\Products;
44

55
use App\Repositories\ItemRepository;
6+
use App\Repositories\ProductRepository;
7+
use App\Repositories\ServerRepository;
68
use App\Services\AdminProducts;
79
use Illuminate\Http\Request;
810
use App\Http\Controllers\Controller;
@@ -34,14 +36,16 @@ public function __construct(AdminProducts $adminProducts)
3436
/**
3537
* Render the edit given product page.
3638
*
37-
* @param Request $request
38-
* @param ItemRepository $itemRepository
39+
* @param Request $request
40+
* @param ItemRepository $itemRepository
41+
* @param ProductRepository $productRepository
42+
* @param ServerRepository $serverRepository
3943
*
4044
* @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
4145
*/
42-
public function render(Request $request, ItemRepository $itemRepository)
46+
public function render(Request $request, ItemRepository $itemRepository, ProductRepository $productRepository, ServerRepository $serverRepository)
4347
{
44-
$product = $this->qm->productForAdmin($request->route('product'), [
48+
$product = $productRepository->forEditProducts($request->route('product'), [
4549
'products.id',
4650
'products.price',
4751
'products.stack',
@@ -62,7 +66,7 @@ public function render(Request $request, ItemRepository $itemRepository)
6266
'type'
6367
]);
6468

65-
$categories = $this->qm->allCategoriesWithServers([
69+
$categories = $serverRepository->allWithCategories([
6670
'categories.name as category',
6771
'categories.id as category_id',
6872
'servers.name as server',

app/Http/Controllers/Admin/Products/ListController.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace App\Http\Controllers\Admin\Products;
44

5+
use App\Repositories\ProductRepository;
56
use Illuminate\Http\Request;
67
use App\Http\Controllers\Admin\ListParent;
78

@@ -17,17 +18,18 @@ class ListController extends ListParent
1718
/**
1819
* Render the page with products list.
1920
*
20-
* @param Request $request
21+
* @param Request $request
22+
* @param ProductRepository $productRepository
2123
*
2224
* @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
2325
*/
24-
public function render(Request $request)
26+
public function render(Request $request, ProductRepository $productRepository)
2527
{
2628
$orderBy = $this->checkOrderBy($request->get('orderBy'));
2729
$orderType = $this->checkOrderType($request->get('orderType'));
2830
$filter = $request->get('filter');
2931

30-
$products = $this->qm->productsForAdmin([
32+
$products = $productRepository->forAdmin([
3133
'products.id',
3234
'products.price',
3335
'products.item_id',

app/Http/Controllers/Admin/Servers/EditController.php

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
use App\Exceptions\Server\AttemptToDeleteTheLastCategoryException;
66
use App\Exceptions\Server\AttemptToDeleteTheLastServerException;
77
use App\Http\Requests\Admin\SaveEditedServerRequest;
8+
use App\Repositories\ServerRepository;
89
use Illuminate\Http\Request;
910

1011
/**
@@ -17,13 +18,14 @@
1718
class EditController extends BaseController
1819
{
1920
/**
20-
* Render edit server page
21+
* Render edit server page.
2122
*
22-
* @param Request $request
23+
* @param Request $request
24+
* @param ServerRepository $serverRepository
2325
*
2426
* @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
2527
*/
26-
public function render(Request $request)
28+
public function render(Request $request, ServerRepository $serverRepository)
2729
{
2830
$server = null;
2931
foreach ($request->get('servers') as $s) {
@@ -37,7 +39,7 @@ public function render(Request $request)
3739
\App::abort(404);
3840
}
3941

40-
$categories = $this->qm->serverCategories($server->first()->id, [
42+
$categories = $serverRepository->categories($server->first()->id, [
4143
'id',
4244
'name'
4345
]);

app/Http/Controllers/Admin/Statistic/PaymentsController.php

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
namespace App\Http\Controllers\Admin\Statistic;
44

55
use App\Repositories\PaymentRepository;
6+
use App\Repositories\ProductRepository;
67
use App\Repositories\UserRepository;
78
use Illuminate\Http\Request;
89
use App\Http\Controllers\Controller;
@@ -58,20 +59,22 @@ public function render(Request $request, PaymentRepository $pr, UserRepository $
5859
/**
5960
* Get detail information about concrete payment.
6061
*
61-
* @param Request $request
62+
* @param Request $request
63+
* @param PaymentRepository $paymentRepository
64+
* @param ProductRepository $productRepository
6265
*
6366
* @return \Illuminate\Http\JsonResponse
6467
*/
65-
public function info(Request $request)
68+
public function info(Request $request, PaymentRepository $paymentRepository, ProductRepository $productRepository)
6669
{
67-
$payment = $this->qm->payment((int)$request->route('payment'), ['products']);
70+
$payment = $paymentRepository->find((int)$request->route('payment'), ['products']);
6871

6972
if (!$payment) {
7073
return json_response('payment not found');
7174
}
7275
$unserialized = unserialize($payment->products);
7376
$ids = array_keys($unserialized);
74-
$products = $this->qm->product($ids, [
77+
$products = $productRepository->getWithItems($ids, [
7578
'products.id',
7679
'items.name',
7780
'items.image'

0 commit comments

Comments
 (0)