Skip to content

Commit

Permalink
chore: fix code style
Browse files Browse the repository at this point in the history
  • Loading branch information
rahmanramsi authored and github-actions[bot] committed Dec 7, 2023
1 parent b79f61c commit 074bea9
Show file tree
Hide file tree
Showing 99 changed files with 709 additions and 737 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public function handle(Conference $conference): void

foreach ([
UserRole::Author->value,
'Co Author'
'Co Author',
] as $authorPosition) {
ParticipantPosition::firstOrCreate([
'name' => $authorPosition,
Expand Down
4 changes: 2 additions & 2 deletions app/Actions/SubmissionFiles/UploadSubmissionFileAction.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@ class UploadSubmissionFileAction
public function handle(Submission $submission, Media $file, string $category, SubmissionFileType $type)
{
return $submission->submissionFiles()->updateOrCreate([
'media_id' => $file->getKey()
'media_id' => $file->getKey(),
], [
'media_id' => $file->getKey(),
'submission_file_type_id' => $type->getKey(),
'category' => $category
'category' => $category,
]);
}
}
1 change: 0 additions & 1 deletion app/Actions/Submissions/RequestWithdrawalAction.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@

namespace App\Actions\Submissions;

use App\Models\Enums\SubmissionStatus;
use App\Models\Submission;
use Lorisleiva\Actions\Concerns\AsAction;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ public function handle(Submission $submission, User $user, Role $role)
'role_id' => $role->getKey(),
]);
DB::commit();

return $submissionParticipant;
} catch (\Throwable $th) {
DB::rollBack();
Expand Down
5 changes: 3 additions & 2 deletions app/Actions/User/CreateParticipantFromUserAction.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ public function handle(User $user)

$participant = Participant::where('email', $user->email)->first();

if (!$participant) {
if (! $participant) {
$participant = ParticipantCreateAction::run($userData);
}

Expand All @@ -35,13 +35,14 @@ public function handle(User $user)
UserRole::Author->value, UserRole::Reviewer->value, UserRole::Editor->value => true,
default => false,
};
if (!$shouldCreateParticipant) {
if (! $shouldCreateParticipant) {
continue;
}
$position = ParticipantPosition::where('name', $userRole)->first();
$participant->positions()->detach($position);
$participant->positions()->attach($position);
}

return $participant;
}
}
8 changes: 4 additions & 4 deletions app/Classes/NavigationItem.php
Original file line number Diff line number Diff line change
Expand Up @@ -57,23 +57,23 @@ protected function getUrlParserByType($type): Closure
{
return match ($type) {
'external-link' => fn ($navItem) => $navItem->data['url'] ?? '#',
'announcements' => fn (Conference $conference = null) => match ($conference?->status) {
'announcements' => fn (?Conference $conference = null) => match ($conference?->status) {
ConferenceStatus::Archived => route('livewirePageGroup.archive-conference.pages.announcement-list', ['conference' => $conference->path]),
default => route('livewirePageGroup.current-conference.pages.announcement-list')
},
'current-conference' => fn () => route('livewirePageGroup.current-conference.pages.home'),
'register' => fn () => route('livewirePageGroup.website.pages.register'),
'login' => fn () => route('livewirePageGroup.website.pages.login'),
'home' => fn (Conference $conference = null) => match ($conference?->status) {
'home' => fn (?Conference $conference = null) => match ($conference?->status) {
ConferenceStatus::Active => route('livewirePageGroup.current-conference.pages.home'),
ConferenceStatus::Archived => route('livewirePageGroup.archive-conference.pages.home', ['conference' => $conference->path]),
default => route('livewirePageGroup.website.pages.home'),
},
'about' => fn (Conference $conference = null) => match ($conference?->status) {
'about' => fn (?Conference $conference = null) => match ($conference?->status) {
ConferenceStatus::Archived => route('livewirePageGroup.archive-conference.pages.about', ['conference' => $conference->path]),
default => route('livewirePageGroup.current-conference.pages.about')
},
'contact' => fn (Conference $conference = null) => match ($conference?->status) {
'contact' => fn (?Conference $conference = null) => match ($conference?->status) {
ConferenceStatus::Archived => route('livewirePageGroup.archive-conference.pages.contact', ['conference' => $conference->path]),
default => route('livewirePageGroup.current-conference.pages.contact')
},
Expand Down
8 changes: 4 additions & 4 deletions app/Constants/SubmissionStatusRecommendation.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,14 @@ final class SubmissionStatusRecommendation

public const DECLINE = 'Decline';

public const REVISION_REQUIRED = "Revision Required";
public const REVISION_REQUIRED = 'Revision Required';

public static function list()
{
return [
static::ACCEPT => static::ACCEPT,
static::DECLINE => static::DECLINE,
static::REVISION_REQUIRED => static::REVISION_REQUIRED
self::ACCEPT => self::ACCEPT,
self::DECLINE => self::DECLINE,
self::REVISION_REQUIRED => self::REVISION_REQUIRED,
];
}
}
2 changes: 1 addition & 1 deletion app/Infolists/Components/BladeEntry.php
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

class BladeEntry extends Entry
{
public static function make(string $name, string $blade = null, array $viewData = []): static
public static function make(string $name, ?string $blade = null, array $viewData = []): static
{
$static = app(static::class, ['name' => $name]);
$static->configure();
Expand Down
2 changes: 1 addition & 1 deletion app/Infolists/Components/LivewireEntry.php
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@

class LivewireEntry extends Entry
{
public static function make(string $name, string $component = null, array $viewData = []): static
public static function make(string $name, ?string $component = null, array $viewData = []): static
{
$static = app(static::class, ['name' => $name]);
$static->configure();
Expand Down
2 changes: 1 addition & 1 deletion app/Mail/Templates/AcceptAbstractMail.php
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ public static function getDefaultDescription(): string
public static function getDefaultHtmlTemplate(): string
{

return <<<'HTML'
return <<<'HTML'
<p> This is an automated notification from the Leconfe System to inform you about a new submission.</p>
<p>
Submission Details:
Expand Down
4 changes: 1 addition & 3 deletions app/Mail/Templates/NewPaperUploadedMail.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@

namespace App\Mail\Templates;

use App\Models\MailTemplate;
use App\Models\Submission;
use App\Models\SubmissionFile;

/**
Expand Down Expand Up @@ -33,7 +31,7 @@ public static function getDefaultDescription(): string

public static function getDefaultHtmlTemplate(): string
{
return <<<HTML
return <<<'HTML'
<p>This is a automatic notification to let you know that "{{ uploader }}" has uploaded a new paper for the submission titled "{{ submissionTitle }}".</p>
HTML;
}
Expand Down
2 changes: 1 addition & 1 deletion app/Mail/Templates/NewRevisionUploadedMail.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public static function getDefaultDescription(): string

public static function getDefaultHtmlTemplate(): string
{
return <<<HTML
return <<<'HTML'
<p>This is a automatic notification to let you know that "{{ uploader }}" has uploaded a new revision for the submission titled "{{ submissionTitle }}".</p>
HTML;
}
Expand Down
2 changes: 1 addition & 1 deletion app/Mail/Templates/ReviewCompleteMail.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ public static function getDefaultDescription(): string

public static function getDefaultHtmlTemplate(): string
{
return <<<HTML
return <<<'HTML'
<p>This is a automatic notification to let you know that {{ reviewer }} has completed the review of the submission titled "{{ submissionTitle }}".</p>
HTML;
}
Expand Down
5 changes: 1 addition & 4 deletions app/Mail/Templates/ReviewerAcceptedInvitationMail.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,13 @@
namespace App\Mail\Templates;

use App\Models\Review;
use App\Models\User;

class ReviewerAcceptedInvitationMail extends TemplateMailable
{

public string $reviewer;

public string $submissionTitle;


public function __construct(Review $review)
{
$this->reviewer = $review->user->fullName;
Expand All @@ -31,7 +28,7 @@ public static function getDefaultDescription(): string

public static function getDefaultHtmlTemplate(): string
{
return <<<HTML
return <<<'HTML'
<p>This is a automatic notification to let you know that {{ reviewer }} has accepted the invitation to review the submission titled "{{ submissionTitle }}".</p>
HTML;
}
Expand Down
3 changes: 1 addition & 2 deletions app/Mail/Templates/ReviewerDeclinedInvitationMail.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ class ReviewerDeclinedInvitationMail extends TemplateMailable

public string $submissionTitle;


public function __construct(Review $review)
{
$this->reviewer = $review->user->fullName;
Expand All @@ -29,7 +28,7 @@ public static function getDefaultDescription(): string

public static function getDefaultHtmlTemplate(): string
{
return <<<HTML
return <<<'HTML'
<p>This is a automatic notification to let you know that {{ reviewer }} has declined the invitation to review the submission titled "{{ submissionTitle }}".</p>
HTML;
}
Expand Down
2 changes: 1 addition & 1 deletion app/Mail/Templates/SubmissionWithdrawnMail.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ public static function getDefaultDescription(): string

public static function getDefaultHtmlTemplate(): string
{
return <<<HTML
return <<<'HTML'
<p>This is a automatic notification to let you know that your submission titled "{{ title }}" has been withdrawn.</p>
HTML;
}
Expand Down
2 changes: 1 addition & 1 deletion app/Mail/Templates/SubmissionWithdrawnRequestMail.php
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ public static function getDefaultDescription(): string

public static function getDefaultHtmlTemplate(): string
{
return <<<HTML
return <<<'HTML'
<p>This is an automated notification. We wanted to inform you that your submission titled "{{ title }}" has been requested to be withdrawn.</p>
HTML;
}
Expand Down
2 changes: 2 additions & 0 deletions app/Mail/Templates/Traits/CanCustomizeTemplate.php
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,14 @@ abstract protected function resolveTemplateModel(): MailTemplateInterface;
public function contentUsing(?string $customizedContent): static
{
$this->customizedContent = $customizedContent;

return $this;
}

public function subjectUsing(?string $customizedSubject): static
{
$this->customizedSubject = $customizedSubject;

return $this;
}

Expand Down
2 changes: 1 addition & 1 deletion app/Models/Announcement.php
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public function getUrl()
};
}

public function registerMediaConversions(Media $media = null): void
public function registerMediaConversions(?Media $media = null): void
{
$this->addMediaConversion('small')
->keepOriginalImageFormat()
Expand Down
4 changes: 2 additions & 2 deletions app/Models/Concerns/HasTopics.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ public function topics(): MorphToMany
->morphToMany(Topic::class, 'topicable');
}

public function attachTopics(array|ArrayAccess|Topic $topics, string $type = null): static
public function attachTopics(array|ArrayAccess|Topic $topics, ?string $type = null): static
{
$topics = collect(Topic::findOrCreate($topics, $type));

Expand All @@ -33,7 +33,7 @@ public function attachTopics(array|ArrayAccess|Topic $topics, string $type = nul
return $this;
}

public function attachTopic(string|Topic $topic, string $type = null)
public function attachTopic(string|Topic $topic, ?string $type = null)
{
return $this->attachTopics([$topic], $type);
}
Expand Down
2 changes: 1 addition & 1 deletion app/Models/Conference.php
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ public function getFilamentAvatarUrl(): ?string
return $this->getFirstMediaUrl('logo', 'tenant');
}

public function registerMediaConversions(Media $media = null): void
public function registerMediaConversions(?Media $media = null): void
{
$this->addMediaConversion('tenant')
->keepOriginalImageFormat()
Expand Down
6 changes: 3 additions & 3 deletions app/Models/Enums/SubmissionStage.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@ enum SubmissionStage: string implements HasLabel

case Wizard = 'Wizard';
case CallforAbstract = 'Call for Abstract';
case PeerReview = "Peer Review";
case Editing = "Editing";
case Proceeding = "Proceeding";
case PeerReview = 'Peer Review';
case Editing = 'Editing';
case Proceeding = 'Proceeding';

public function getLabel(): ?string
{
Expand Down
4 changes: 2 additions & 2 deletions app/Models/Enums/SubmissionStatus.php
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@ enum SubmissionStatus: string implements HasLabel

case Incomplete = 'Incomplete';
case Queued = 'Queued';
case OnReview = "On Review";
case Editing = "Editing";
case OnReview = 'On Review';
case Editing = 'Editing';
case Published = 'Published';
case Declined = 'Declined';
case Scheduled = 'Scheduled';
Expand Down
11 changes: 5 additions & 6 deletions app/Models/Participant.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\MorphToMany;
use Illuminate\Notifications\Notifiable;
use Illuminate\Support\Facades\Hash;
use Illuminate\Support\Str;
use Kra8\Snowflake\HasShortflakePrimary;
use Plank\Metable\Metable;
Expand All @@ -22,9 +21,9 @@
use Spatie\MediaLibrary\InteractsWithMedia;
use Spatie\MediaLibrary\MediaCollections\Models\Media;

class Participant extends Model implements HasMedia, Sortable, HasAvatar
class Participant extends Model implements HasAvatar, HasMedia, Sortable
{
use HasFactory, HasShortflakePrimary, InteractsWithMedia, Metable, SortableTrait, Notifiable;
use HasFactory, HasShortflakePrimary, InteractsWithMedia, Metable, Notifiable, SortableTrait;

/**
* The table associated with the model.
Expand All @@ -51,7 +50,7 @@ class Participant extends Model implements HasMedia, Sortable, HasAvatar
protected function fullName(): Attribute
{
return Attribute::make(
get: fn () => Str::squish($this->given_name . ' ' . $this->family_name),
get: fn () => Str::squish($this->given_name.' '.$this->family_name),
);
}

Expand All @@ -60,7 +59,7 @@ protected static function newFactory(): Factory
return ParticipantFactory::new();
}

public function registerMediaConversions(Media $media = null): void
public function registerMediaConversions(?Media $media = null): void
{
$this->addMediaConversion('avatar')
->keepOriginalImageFormat()
Expand Down Expand Up @@ -103,6 +102,6 @@ public function getFilamentAvatarUrl(): ?string
->map(fn (string $segment): string => filled($segment) ? mb_substr($segment, 0, 1) : '')
->join(' ');

return 'https://ui-avatars.com/api/?name=' . urlencode($name) . '&color=FFFFFF&background=111827&font-size=0.33';
return 'https://ui-avatars.com/api/?name='.urlencode($name).'&color=FFFFFF&background=111827&font-size=0.33';
}
}
10 changes: 5 additions & 5 deletions app/Models/Review.php
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,12 @@

class Review extends Model implements HasMedia
{
use HasFactory, InteractsWithMedia, HasShortflakePrimary;
use HasFactory, HasShortflakePrimary, InteractsWithMedia;

protected $casts = [
'date_assigned' => 'datetime',
'date_confirmed' => 'datetime',
'date_completed' => 'datetime'
'date_completed' => 'datetime',
];

protected $fillable = [
Expand All @@ -28,7 +28,7 @@ class Review extends Model implements HasMedia
'date_completed',
'quality',
'review_author_editor',
'review_editor'
'review_editor',
];

protected static function booted(): void
Expand All @@ -42,7 +42,7 @@ protected static function booted(): void

public function reviewSubmitted(): bool
{
return !is_null($this->recommendation) && !is_null($this->date_completed);
return ! is_null($this->recommendation) && ! is_null($this->date_completed);
}

public function assignedFiles()
Expand Down Expand Up @@ -77,7 +77,7 @@ public function user()

public function confirmed(): bool
{
return !$this->needConfirmation();
return ! $this->needConfirmation();
}

public function needConfirmation(): bool
Expand Down
Loading

0 comments on commit 074bea9

Please sign in to comment.