Skip to content

Commit aa324c4

Browse files
author
Chri$
committed
feat:LAR-8 refactor create article process with action and add features test
1 parent 03bfda4 commit aa324c4

File tree

5 files changed

+156
-35
lines changed

5 files changed

+156
-35
lines changed

‎app/Actions/Article/CreateArticleAction.php

+61
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,61 @@
1+
<?php
2+
3+
4+
declare(strict_types=1);
5+
6+
namespace App\Actions\Article;
7+
8+
use Carbon\Carbon;
9+
use DateTimeInterface;
10+
use App\Data\Article\CreateArticleData;
11+
use App\Gamify\Points\ArticleCreated;
12+
use App\Models\Article;
13+
use App\Notifications\PostArticleToTelegram;
14+
use Illuminate\Support\Facades\Auth;
15+
16+
final class CreateArticleAction
17+
{
18+
public function execute(CreateArticleData $articleData): Article
19+
{
20+
if ($articleData->published_at && !($articleData->published_at instanceof DateTimeInterface)) {
21+
$articleData->published_at = new Carbon(
22+
time: $articleData->published_at,
23+
tz: config('app.timezone')
24+
);
25+
}
26+
27+
/** @var Article $article */
28+
$article = Article::query()->create([
29+
'title' => $articleData->title,
30+
'slug' => $articleData->title,
31+
'body' => $articleData->body,
32+
'published_at' => $articleData->published_at,
33+
'submitted_at' => $articleData->submitted_at,
34+
'approved_at' => $articleData->approved_at,
35+
'show_toc' => $articleData->show_toc,
36+
'canonical_url' => $articleData->canonical_url,
37+
'user_id' => Auth::id(),
38+
]);
39+
40+
if (collect($article->associateTags)->isNotEmpty()) {
41+
$article->syncTags(tags: $article->associateTags);
42+
}
43+
44+
if ($article->file) {
45+
$article->addMedia($article->file->getRealPath())->toMediaCollection('media');
46+
}
47+
48+
if ($article->isAwaitingApproval()) {
49+
// Envoi de la notification sur le channel Telegram pour la validation de l'article.
50+
Auth::user()?->notify(new PostArticleToTelegram($article));
51+
session()->flash('status', __('Merci d\'avoir soumis votre article. Vous aurez des nouvelles que lorsque nous accepterons votre article.'));
52+
}
53+
54+
if (Auth::user()?->hasAnyRole(['admin', 'moderator'])) {
55+
givePoint(new ArticleCreated($article));
56+
}
57+
58+
59+
return $article;
60+
}
61+
}

‎app/Data/Article/CreateArticleData.php

+23
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace App\Data\Article;
6+
7+
use Carbon\Carbon;
8+
use Spatie\LaravelData\Data;
9+
10+
final class CreateArticleData extends Data
11+
{
12+
public function __construct(
13+
public string $title,
14+
public string $slug,
15+
public string $body,
16+
public Carbon $published_at,
17+
public Carbon | null $submitted_at,
18+
public Carbon | null $approved_at,
19+
public string $show_toc,
20+
public string $canonical_url,
21+
public array $associateTags = [],
22+
) {}
23+
}

‎app/Gamify/Points/ArticleCreated.php

+25
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace App\Gamify\Points;
6+
7+
use App\Models\Article;
8+
use App\Models\User;
9+
use QCod\Gamify\PointType;
10+
11+
final class ArticleCreated extends PointType
12+
{
13+
public int $points = 50;
14+
15+
public function __construct(Article $subject)
16+
{
17+
$this->subject = $subject;
18+
}
19+
20+
public function payee(): User
21+
{
22+
// @phpstan-ignore-next-line
23+
return $this->getSubject()->user;
24+
}
25+
}

‎app/Livewire/Articles/Create.php

+5-35
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,13 @@
44

55
namespace App\Livewire\Articles;
66

7-
use App\Events\ArticleWasSubmittedForApproval;
8-
use App\Gamify\Points\PostCreated;
9-
use App\Models\Article;
7+
use App\Actions\Article\CreateArticleAction;
8+
use App\Data\Article\CreateArticleData;
109
use App\Models\Tag;
1110
use App\Models\User;
1211
use App\Traits\WithArticleAttributes;
1312
use App\Traits\WithTagsAssociation;
14-
use Carbon\Carbon;
15-
use DateTimeInterface;
13+
1614
use Illuminate\Contracts\View\View;
1715
use Illuminate\Support\Facades\Auth;
1816
use Livewire\Component;
@@ -52,15 +50,7 @@ public function store(): void
5250
/** @var User $user */
5351
$user = Auth::user();
5452

55-
if ($this->published_at && ! ($this->published_at instanceof DateTimeInterface)) {
56-
$this->published_at = new Carbon(
57-
time: $this->published_at,
58-
tz: config('app.timezone')
59-
);
60-
}
61-
62-
/** @var Article $article */
63-
$article = Article::create([
53+
$article = app(CreateArticleAction::class)->execute(CreateArticleData::from([
6454
'title' => $this->title,
6555
'slug' => $this->slug,
6656
'body' => $this->body,
@@ -70,28 +60,8 @@ public function store(): void
7060
'show_toc' => $this->show_toc,
7161
'canonical_url' => $this->canonical_url,
7262
'user_id' => $user->id,
73-
]);
74-
75-
if (collect($this->associateTags)->isNotEmpty()) {
76-
$article->syncTags(tags: $this->associateTags);
77-
}
78-
79-
if ($this->file) {
80-
$article->addMedia($this->file->getRealPath())->toMediaCollection('media');
81-
}
82-
83-
if ($article->isAwaitingApproval()) {
84-
if (app()->environment('production')) {
85-
// Envoi de la notification sur le channel Telegram pour la validation de l'article.
86-
event(new ArticleWasSubmittedForApproval($article));
87-
}
88-
89-
session()->flash('status', __('Merci d\'avoir soumis votre article. Vous aurez des nouvelles que lorsque nous accepterons votre article.'));
90-
}
63+
]));
9164

92-
if ($user->hasAnyRole(['admin', 'moderator'])) {
93-
givePoint(new PostCreated($article));
94-
}
9565

9666
$user->hasRole('user') ?
9767
$this->redirectRoute('dashboard') :

‎tests/Feature/Actions/Article/createArticleActionTest.php

+42
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
<?php
2+
3+
4+
declare(strict_types=1);
5+
6+
use App\Actions\Article\CreateArticleAction;
7+
use App\Data\Article\CreateArticleData;
8+
use App\Models\Article;
9+
use App\Models\Tag;
10+
11+
12+
beforeEach(function (): void {
13+
$this->user = $this->login();
14+
$this->tagOne = Tag::factory()->create(['name' => 'Tag 1', 'concerns' => ['article']]);
15+
$this->tagTwo = Tag::factory()->create(['name' => 'Tag 2', 'concerns' => ['article', 'post']]);
16+
});
17+
18+
19+
describe(CreateArticleAction::class, function (): void {
20+
it('return the created article', function (): void {
21+
$articleDataWithoutTag = CreateArticleData::from([
22+
'title' => 'Article title',
23+
'slug' => 'Article slug',
24+
'published_at' => Now(),
25+
'submitted_at' => null,
26+
'approved_at' => null,
27+
'show_toc' => 'Article show_toc',
28+
'canonical_url' => 'Article canonical_url',
29+
'body' => 'Article body',
30+
'associateTags' => [],
31+
]);
32+
33+
$article = app(CreateArticleAction::class)->execute($articleDataWithoutTag);
34+
35+
expect($article)
36+
->toBeInstanceOf(Article::class)
37+
->and($article->tags)
38+
->toHaveCount(0)
39+
->and($article->user_id)
40+
->toBe($this->user->id);
41+
});
42+
})->group('Articles');

0 commit comments

Comments
 (0)