Laravel по-русски

С БД то и так все понятно... Вот в шлюзе (stripe or braintree)создать случайно две подписки одному юзеру значит они дважды будут снимать регулярные платежи...

Дело в том, что braintree позволяет множество подписок для юзера. При том что каждая снимает регулярные платежи. А вот как мы проектируем наш сайт это один юзер и одна активная подписка одновременно. А если использовать запрос сначала к braintree и затем в случае успеха insert в нашу БД, то реально возникает race condition и может наплодиться подписок в braintree.  И проверка в самом начале процедуры записи в БД не помогает...

Ну пока лид сказал блокировать кнопку и игнорить... И потом еще подумаем

Что много раз жать на кнопку что одновременно нажать на нее в разных браузерах суть то одна повторные запросы..

А еще оказалось семафор для блокировки прямо встроен в php http://php.net/manual/ru/function.sem-get.php

Нашел такой пример:

$sem = sem_get(1234, 1);
if (sem_acquire($sem)) {
    //successful lock, go ahead
    sem_release($sem);
} else {
    //Something went wrong...
}

Я читал это. Но там не до конца конкретно. Да проверять надо на бекенде... Лочить возможность  добавки пока первый запрос не прошел, отсекая повторные. Собственно про то как это лучше сделать я и спрашиваю.

Я бы весь request в аргумент метода точно в таком случае выборке по фильтру бы не запихивал..

Понял. И от чего тут можно отнаследовать? сделать интерфейс и от него два сервиса? Или от класса User отнаследовать 2 класса, каждый из который подключает свой trait ?

В общем в итоге получилось вот что. В зависимости от условий создания Subscription объекта и связанного объекта в шлюзе, создаю все это в двух разных методах трейта класса User.

public function createSubscriptionByAdmin($request)
{
	
	$plan = Plan::findOrFail($request['plan_id']);
	
	if ($this->hasCard()) {
		if (empty($request['trial_ends_at'])) {
			$response = GatewaySubscription::create([
				'planId' => $plan->gateway_id,
				'paymentMethodToken' => $this->defaultCard->gateway_id,
				'options' => ['startImmediately' => true]
			]);
			
		} else {
			
			$response = GatewaySubscription::create([
				'planId' => $plan->gateway_id,
				'paymentMethodToken' => $this->defaultCard->gateway_id,
				'firstBillingDate' => $request['trial_ends_at'],
			]);
			
		}
		
		if (!$response->success) {
				throw new Exception('Gateway failed to create subscription: '.$response->message);
		}
		
		return $this->subscriptions()->create([
			'gateway_id' => $response->subscription->id,
			'card_id' => $this->defaultCard->id, 
			'current_period_end' => $response->subscription->billingPeriodEndDate,
			'plan_id' => $plan->id,
			'trial_ends_at' => $request['trial_ends_at'],
		]);
	} else {
		return $this->subscriptions()->create([
			'plan_id' => $plan->id,
			'trial_ends_at' => $request['trial_ends_at'],
		]);
	}
	
}


public function createSubscription($request)
{
	if (isset($request['payment_method_nonce'])) {
		if ($this->gateway_id) {
			throw new \LogicException('When user have payment method add subscription only with this method');
		} else {
			$this->createGatewayCustomerWithPaymentMethod($request['payment_method_nonce']);
		}
	} else {
		if (!$this->gateway_id) {
			throw new \LogicException('When user have not payment method add subscription only with add payment data');
		} 
	
	}
	
	$plan = Plan::findOrFail($request['plan_id']);
	

	
	if (empty($request['trial_ends_at'])) {
		$response = GatewaySubscription::create([
			'planId' => $plan->gateway_id,
			'paymentMethodToken' => $this->defaultCard->gateway_id,
			'options' => ['startImmediately' => true]
		]);
		
	} else {
		
		$response = GatewaySubscription::create([
			'planId' => $plan->gateway_id,
			'paymentMethodToken' => $this->defaultCard->gateway_id,
			'firstBillingDate' => $request['trial_ends_at'],
		
		]);
		
	}
	
	if (! $response->success) {
		throw new Exception('Gateway failed to create subscription: '.$response->message);
	}

	return $this->subscriptions()->create([
		'gateway_id'   => $response->subscription->id,
		'card_id' => $this->defaultCard->id, 
		'current_period_end' => $response->subscription->billingPeriodEndDate,
		'plan_id' => $plan->id,
		'trial_ends_at' => $request['trial_ends_at'],
	 ]);
	
}

Вызывается в контроллерах так:

Для админа

$user-> createSubscriptionByAdmin();

В Request при этом plan_id и trial_ends_at выбранные админом в форме.

Для юзера

$user-> createSubscription(array_merge(
	$request->all(), [
		"trial_ends_at" => config('services.subscription.trial_ends_in_days') ? Carbon::today()->addDays(config('services.subscription.trial_ends_in_days')) : null,
	]
));

Юзер выбирает plan_id, но не выбирает trial_ends_at он задается на этот случае в конфиге.

Отличия метода создания Subscription для юзера и админа, в том что юзер может и должен в момент создания подписки добавить карту, если ее еще нет. А админ конечно за юзера это сделать не может и если карты у юзера нет добавляет подписку только в БД, а не в шлюзе где карта нужна для автопродления...

Вот и думаю как это все зарефакторить... Какой можно паттерн применить.. Может какой нибудь фабричный метод... Или в класс подписки часть из этого тащить из трейта юзера.

Суть что бы лучше читалось и меньше услоовий и повтора кода. Конечно можно тупо вынести повторящийся код в приватные методы. Жду предложений...

Нет, посмотрел свой код мутатор мне не подходит, так как мне нужно менять формат даты не в момент создания объекта именно...

Мне нужно следующее:

1) Принять дату из формы или из конфига
2) Заслать через API дату. Причем там есть варианты и более чистый что ли это через TrialDuration и TrialDurationUnit, а не просто DateTime
3) Использовать дату при создании объекта Subscription

Очевидно что мутатор по крайней мере задачу 2 не решает. Можно конечно написать Хелпер который переводит дату из TimeStamp в TrialDuration и TrialDurationUnit. А перед этим то что в форме перевести в Timestamp

Спасибо так и сделаю.

А если я хочу из формы принимать дату. То преобразовать ее из того что прислала форма ну например html 5 поле с датой или js календарь итд в Carbon дату где лучше? Мне кажется модель не должна знать что там за дата в форм, значки в контроллере или в реквесте быть может?

Как красивее? В реквесте по любому дату ведь проверять

Конечно будет работать и так и сяк. Я скорее о том, что везде будет достаточно $user->subscription->isTrial() в контроллерах.. Так как везде есть юзер...

В общем я понял, что подход верный...

А я вообще сделал по простому

class AddSubscriptionController extends Controller {

	public function __construct() {
		$this->middleware('adminUserIsNotSubscriebed');
	}

namespace App\Http\Middleware\Admin;

use Closure;
use Illuminate\Support\Facades\Config;
use App\Classes\Models\User;
class UserIsNotSubscriebed
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
		$user = User::find($request->user_id);
        
		if ($user->subscribed()) {
            return redirect()->route('admin.subscription', $request->user_id);
        }

        return $next($request);
    }
}

Если юзер с данными ID имеет подписку то из данной формы посылаем и все.

Но да может с политиками как то и лучше

Цена может быть со скидкой по коду по сравнению с планом. Пока это не нужно, но в кешире заложено когда придется все равно нужно будет прикрутить.

Пусть я сейчас это уберу.. name на случай когда несколько подписок, у нас пока и в обозримом одна, опять же поле можно удалить

card_id в подписках также нужно и скорее и сейчас надо оставить. Тк у user может быть одна карта по умолчанию, но вот именно подписка может продлятся другой.

Можно все убрать. Но делается и немного на вырост...

Кроме того убирание всего этого особо и не упростит.

Главное то продолжительность trial и сам факт его наличия

От вот этого

              if (!$trialDuration || !$trialDurationUnit) {
			$response = GatewaySubscription::create([
				'planId' => $plan->gateway_id,
				'price' => (string) round($plan->cost, 2),
				'paymentMethodToken' => $this->user->defaultCard->gateway_id,
				'trialPeriod' => false,
			]);
			
		} else {
			
			$response = GatewaySubscription::create([
				'planId' => $plan->gateway_id,
				'price' => (string) round($plan->cost, 2),
				'paymentMethodToken' => $this->user->defaultCard->gateway_id,
				'trialPeriod' => true,
				'trialDurationUnit' => $trialDurationUnit,
				'trialDuration' => $trialDuration,
			]);
			
		}
		

Все равно не уйти. Тк подписку надо создать не только у нас в БД, но и в платежном шлюзе. И там у Api свои особенности то есть  вот это все им надо отослать 'trialPeriod', 'trialDurationUnit', 'trialDuration' ну или отсылать в случае если создаем подписку без триала.  А просто дату окончания триала в явном виде им отослать нельзя точно.

Те мне по любому нужен сначала запрос к шлюзу. А затем создание модели подписки.

В пакете на основе которого, я все это делал примерно такая логика и есть https://github.com/laravel/cashier-brai … uilder.php https://github.com/laravel/cashier-brai … llable.php

Другое дело, можно ли вообще запихнуть это именно в метод подписки, а не сервиса? Вот в чем вопрос, а даже не в формате массива с данными..

Если лучше сервисный слой оставить то я и оставлю, ну да уберу лишние поля конечно..