valadanchik / newmegchild Goto Github PK

You need to check payment AMOUNT , because idram have issue . user can change amount in idram front end and that value will be chosen for charge. here You check only is set but you need to check value too.

You can also use
$request->has('VARIABLE_NAME')
Instead of
isset($request['VARIABLE_NAME'])
You can have
### Laravel Event/Listeners
after check statuses you can fire event and listener will work and do updates in db

You can use findOrFail instead of
$order = Order::where('order_payment_id', $request->order)->first();
if (!$order)
abort(404);

Need to move user name and password to config file like .env or new config file

Do it necessary create 2 objects of 1 class here ,maybe you can reuse one object in 2 jobs