Skip to content

Commit c81cccf

Browse files
committed
PHP CS Fixer修正
1 parent ba27f62 commit c81cccf

18 files changed

Lines changed: 91 additions & 96 deletions

Controller/MailMagazineController.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ public function index(Request $request, ?int $page_no = null): Response|array
179179
#[Template('@MailMagazine44/admin/template_select.twig')]
180180
public function select(Request $request, ?int $id = null): Response|array
181181
{
182-
/** @var MailMagazineTemplate $Template */
182+
/** @var MailMagazineTemplate|null $Template */
183183
$Template = null;
184184

185185
// テンプレート選択によるPOSTの場合はテンプレートからデータを取得する

Controller/MailMagazineHistoryController.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,9 +153,9 @@ public function condition(#[MapEntity(id: 'id')] MailMagazineSendHistory $mailMa
153153
/**
154154
* search_dataの配列を表示用に変換する.
155155
*
156-
* @param array $searchData
156+
* @param array<string, mixed> $searchData
157157
*
158-
* @return array
158+
* @return array<string, mixed>
159159
*/
160160
protected function searchDataToDisplayData(array $searchData): array
161161
{

Entity/AdminCustomerQueryCustomizer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ class AdminCustomerQueryCustomizer extends WhereCustomizer
2222
/**
2323
* {@inheritdoc}
2424
*
25-
* @param array $params
25+
* @param array<string, mixed> $params
2626
* @param $queryKey
2727
*
2828
* @return WhereClause[]

Entity/MailMagazineSendHistory.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ public function setStartDate(?\DateTimeInterface $startDate): self
250250
/**
251251
* Get start_date.
252252
*
253-
* @return \DateTime
253+
* @return \DateTimeInterface|null
254254
*/
255255
public function getStartDate(): ?\DateTimeInterface
256256
{
@@ -274,7 +274,7 @@ public function setEndDate(?\DateTimeInterface $endDate): self
274274
/**
275275
* Get end_date.
276276
*
277-
* @return \DateTime
277+
* @return \DateTimeInterface|null
278278
*/
279279
public function getEndDate(): ?\DateTimeInterface
280280
{
@@ -322,7 +322,7 @@ public function setCreateDate(?\DateTimeInterface $createDate): self
322322
/**
323323
* Get create_date.
324324
*
325-
* @return \DateTime
325+
* @return \DateTimeInterface|null
326326
*/
327327
public function getCreateDate(): ?\DateTimeInterface
328328
{
@@ -346,7 +346,7 @@ public function setUpdateDate(?\DateTimeInterface $updateDate): self
346346
/**
347347
* Get update_date.
348348
*
349-
* @return \DateTime
349+
* @return \DateTimeInterface|null
350350
*/
351351
public function getUpdateDate(): ?\DateTimeInterface
352352
{

Entity/MailMagazineTemplate.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ public function setCreateDate(?\DateTimeInterface $createDate): self
112112
/**
113113
* Get create_date.
114114
*
115-
* @return \DateTime
115+
* @return \DateTimeInterface|null
116116
*/
117117
public function getCreateDate(): ?\DateTimeInterface
118118
{
@@ -136,7 +136,7 @@ public function setUpdateDate(?\DateTimeInterface $updateDate): self
136136
/**
137137
* Get update_date.
138138
*
139-
* @return \DateTime
139+
* @return \DateTimeInterface|null
140140
*/
141141
public function getUpdateDate(): ?\DateTimeInterface
142142
{

Form/Extension/EntryMailMagazineTypeExtension.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ class EntryMailMagazineTypeExtension extends AbstractTypeExtension
2525
* {@inheritdoc}
2626
*
2727
* @param FormBuilderInterface $builder
28-
* @param array $options
28+
* @param array<string, mixed> $options
2929
*/
3030
public function buildForm(FormBuilderInterface $builder, array $options): void
3131
{

MailMagazineNav.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ class MailMagazineNav implements EccubeNav
2020
/**
2121
* {@inheritdoc}
2222
*
23-
* @return array
23+
* @return array<string, array<string, mixed>>
2424
*/
2525
public static function getNav(): array
2626
{

Repository/MailMagazineSendHistoryRepository.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public function __construct(
4242
}
4343

4444
/**
45-
* @param array $searchData
45+
* @param array<string, mixed> $searchData
4646
*
4747
* @return QueryBuilder
4848
*/

Repository/MailMagazineTemplateRepository.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313

1414
namespace Plugin\MailMagazine44\Repository;
1515

16-
use Doctrine\ORM\Query;
1716
use Doctrine\Persistence\ManagerRegistry;
1817
use Eccube\Repository\AbstractRepository;
1918
use Plugin\MailMagazine44\Entity\MailMagazineTemplate;
@@ -42,15 +41,15 @@ public function __construct(ManagerRegistry $registry, string $entityClass = Mai
4241
/**
4342
* find all.
4443
*
45-
* @return list<array<string, mixed>>
44+
* @return list<MailMagazineTemplate>
4645
*/
4746
public function findAll(): array
4847
{
4948
$query = $this
5049
->getEntityManager()
5150
->createQuery('SELECT m FROM Plugin\MailMagazine44\Entity\MailMagazineTemplate m ORDER BY m.id DESC');
52-
$result = $query
53-
->getResult(Query::HYDRATE_ARRAY);
51+
/** @var list<MailMagazineTemplate> $result */
52+
$result = $query->getResult();
5453

5554
return $result;
5655
}

Service/MailMagazineService.php

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -150,12 +150,10 @@ public function setMailMagazineDir(string $mailMagazineDir): self
150150
/**
151151
* メールを送信する.
152152
*
153-
* @param array $formData メルマガ情報
153+
* @param array{email: string, subject: string, body: string, htmlBody?: string|null} $formData メルマガ情報
154154
* email: 送信先メールアドレス
155155
* subject: 件名
156156
* body:本文
157-
*
158-
* @return int
159157
*/
160158
public function sendMail(array $formData): void
161159
{
@@ -180,7 +178,7 @@ public function sendMail(array $formData): void
180178
*
181179
* 配信履歴データ(MailMagazineSendHistory)の作成と、配信履歴ファイルを作成します。
182180
*
183-
* @param array $formData
181+
* @param array<string, mixed> $formData
184182
*
185183
* @return int 採番されたsend_id
186184
* エラー時はfalseを返す
@@ -275,8 +273,8 @@ public function createMailMagazineHistory(array $formData): ?int
275273
* none,bbb@example.com,bbb
276274
* none,ccc@example.com,ccc
277275
*
278-
* @param string|$fileHistory 履歴ファイル
279-
* @param string|$fileResult 結果ファイル
276+
* @param string $fileHistory 履歴ファイル
277+
* @param string $fileResult 結果ファイル
280278
*/
281279
private function mergeHistoryFile(string $fileHistory, string $fileResult): void
282280
{
@@ -349,12 +347,11 @@ public function markRetry(int $sendId): void
349347
* @param int $offset
350348
* @param int $max
351349
*
352-
* @return bool|MailMagazineSendHistory
350+
* @return MailMagazineSendHistory
353351
*/
354352
public function sendrMailMagazine(int $sendId, int $offset = 0, int $max = 100): MailMagazineSendHistory
355353
{
356354
// send_historyを取得する
357-
/** @var MailMagazineSendHistory $sendHistory */
358355
$sendHistory = $this->mailMagazineSendHistoryRepository->find($sendId);
359356

360357
if (is_null($sendHistory)) {
@@ -482,7 +479,7 @@ public function sendrMailMagazine(int $sendId, int $offset = 0, int $max = 100):
482479
/**
483480
* 送信完了報告メールを送信する.
484481
*
485-
* @return number
482+
* @return bool
486483
*/
487484
public function sendMailMagazineCompleateReportMail(): bool
488485
{
@@ -541,7 +538,7 @@ public function unlinkHistoryFiles(int $historyId): void
541538
/**
542539
* テストメール送信
543540
*
544-
* @param array|$mailData メールデータ
541+
* @param array{email: string, subject: string, body: string, htmlBody?: string|null, name: string} $mailData メールデータ
545542
*/
546543
public function sendTestMail(array $mailData): void
547544
{
@@ -550,8 +547,8 @@ public function sendTestMail(array $mailData): void
550547
}
551548

552549
/**
553-
* @param array|$mailData メールデータ
554-
* @param string|$name 名前
550+
* @param array{email?: string, subject?: string, body?: string, htmlBody?: string|null, name?: string} $mailData メールデータ
551+
* @param string $name 名前
555552
*/
556553
public function replaceMailVars(array &$mailData, string $name): void
557554
{

0 commit comments

Comments
 (0)