Add final to classes

driesvints · driesvints · commit bb6b55db8e73 · 2018-01-26T17:24:07.000+01:00
diff --git a/app/Jobs/BanUser.php b/app/Jobs/BanUser.php
@@ -5,7 +5,7 @@
 use App\User;
 use Carbon\Carbon;
 
-class BanUser
+final class BanUser
 {
     /**
      * @var \App\User
diff --git a/app/Jobs/ConfirmUser.php b/app/Jobs/ConfirmUser.php
@@ -4,7 +4,7 @@
 
 use App\User;
 
-class ConfirmUser
+final class ConfirmUser
 {
     /**
      * @var \App\User
diff --git a/app/Jobs/CreateReply.php b/app/Jobs/CreateReply.php
@@ -7,7 +7,7 @@
 use App\Models\ReplyAble;
 use App\Http\Requests\CreateReplyRequest;
 
-class CreateReply
+final class CreateReply
 {
     /**
      * @var string
diff --git a/app/Jobs/CreateThread.php b/app/Jobs/CreateThread.php
@@ -6,7 +6,7 @@
 use App\Models\Thread;
 use App\Http\Requests\ThreadRequest;
 
-class CreateThread
+final class CreateThread
 {
     /**
      * @var string
diff --git a/app/Jobs/DeleteReply.php b/app/Jobs/DeleteReply.php
@@ -4,7 +4,7 @@
 
 use App\Models\Reply;
 
-class DeleteReply
+final class DeleteReply
 {
     /**
      * @var \App\Models\Reply
diff --git a/app/Jobs/DeleteThread.php b/app/Jobs/DeleteThread.php
@@ -4,7 +4,7 @@
 
 use App\Models\Thread;
 
-class DeleteThread
+final class DeleteThread
 {
     /**
      * @var \App\Models\Thread
diff --git a/app/Jobs/DeleteUser.php b/app/Jobs/DeleteUser.php
@@ -4,7 +4,7 @@
 
 use App\User;
 
-class DeleteUser
+final class DeleteUser
 {
     /**
      * @var \App\User
diff --git a/app/Jobs/MarkThreadSolution.php b/app/Jobs/MarkThreadSolution.php
@@ -5,7 +5,7 @@
 use App\Models\Reply;
 use App\Models\Thread;
 
-class MarkThreadSolution
+final class MarkThreadSolution
 {
     /**
      * @var \App\Models\Thread
diff --git a/app/Jobs/RegisterUser.php b/app/Jobs/RegisterUser.php
@@ -7,7 +7,7 @@
 use App\Http\Requests\RegisterRequest;
 use Illuminate\Database\Eloquent\ModelNotFoundException;
 
-class RegisterUser
+final class RegisterUser
 {
     /**
      * @var string
diff --git a/app/Jobs/SendEmailConfirmation.php b/app/Jobs/SendEmailConfirmation.php
@@ -7,7 +7,7 @@
 use Illuminate\Contracts\Mail\Mailer;
 use Illuminate\Queue\SerializesModels;
 
-class SendEmailConfirmation
+final class SendEmailConfirmation
 {
     use SerializesModels;
 
diff --git a/app/Jobs/UnbanUser.php b/app/Jobs/UnbanUser.php
@@ -4,7 +4,7 @@
 
 use App\User;
 
-class UnbanUser
+final class UnbanUser
 {
     /**
      * @var \App\User
diff --git a/app/Jobs/UnmarkThreadSolution.php b/app/Jobs/UnmarkThreadSolution.php
@@ -4,7 +4,7 @@
 
 use App\Models\Thread;
 
-class UnmarkThreadSolution
+final class UnmarkThreadSolution
 {
     /**
      * @var \App\User
diff --git a/app/Jobs/UpdatePassword.php b/app/Jobs/UpdatePassword.php
@@ -5,7 +5,7 @@
 use App\User;
 use Illuminate\Contracts\Hashing\Hasher;
 
-class UpdatePassword
+final class UpdatePassword
 {
     /**
      * @var \App\User
diff --git a/app/Jobs/UpdateProfile.php b/app/Jobs/UpdateProfile.php
@@ -5,7 +5,7 @@
 use App\User;
 use App\Http\Requests\UpdateProfileRequest;
 
-class UpdateProfile
+final class UpdateProfile
 {
     /**
      * @var \App\User
diff --git a/app/Jobs/UpdateReply.php b/app/Jobs/UpdateReply.php
@@ -4,7 +4,7 @@
 
 use App\Models\Reply;
 
-class UpdateReply
+final class UpdateReply
 {
     /**
      * @var \App\Models\Reply
diff --git a/app/Jobs/UpdateThread.php b/app/Jobs/UpdateThread.php
@@ -5,7 +5,7 @@
 use App\Models\Thread;
 use App\Http\Requests\ThreadRequest;
 
-class UpdateThread
+final class UpdateThread
 {
     /**
      * @var \App\Models\Thread
diff --git a/app/Mail/EmailConfirmation.php b/app/Mail/EmailConfirmation.php
@@ -6,7 +6,7 @@
 use Illuminate\Mail\Mailable;
 use Illuminate\Queue\SerializesModels;
 
-class EmailConfirmation extends Mailable
+final class EmailConfirmation extends Mailable
 {
     use SerializesModels;
 
diff --git a/app/Models/Reply.php b/app/Models/Reply.php
@@ -8,7 +8,7 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\MorphTo;
 
-class Reply extends Model
+final class Reply extends Model
 {
     use HasAuthor, HasTimestamps, ModelHelpers;
 
diff --git a/app/Models/Tag.php b/app/Models/Tag.php
@@ -6,7 +6,7 @@
 use App\Helpers\ModelHelpers;
 use Illuminate\Database\Eloquent\Model;
 
-class Tag extends Model
+final class Tag extends Model
 {
     use HasSlug, ModelHelpers;
 
diff --git a/app/Models/Thread.php b/app/Models/Thread.php
@@ -16,7 +16,7 @@
 use App\Exceptions\CouldNotMarkReplyAsSolution;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 
-class Thread extends Model implements ReplyAble
+final class Thread extends Model implements ReplyAble
 {
     use HasAuthor, HasSlug, HasTimestamps, ModelHelpers, ReceivesReplies, HasTags;
 
diff --git a/app/Queries/SearchThreads.php b/app/Queries/SearchThreads.php
@@ -5,7 +5,7 @@
 use App\Models\Thread;
 use Illuminate\Contracts\Pagination\Paginator;
 
-class SearchThreads
+final class SearchThreads
 {
     /**
      * @return \App\Models\Thread[]
diff --git a/app/Queries/SearchUsers.php b/app/Queries/SearchUsers.php
@@ -5,7 +5,7 @@
 use App\User;
 use Illuminate\Contracts\Pagination\Paginator;
 
-class SearchUsers
+final class SearchUsers
 {
     /**
      * @return \App\User[]
diff --git a/app/Social/GithubUser.php b/app/Social/GithubUser.php
@@ -5,7 +5,7 @@
 use Carbon\Carbon;
 use Illuminate\Contracts\Support\Arrayable;
 
-class GithubUser implements Arrayable
+final class GithubUser implements Arrayable
 {
     /**
      * @var array
diff --git a/app/Spam/AkismetSpamDetector.php b/app/Spam/AkismetSpamDetector.php
@@ -5,7 +5,7 @@
 use App\User;
 use TijsVerkoyen\Akismet\Akismet;
 
-class AkismetSpamDetector implements SpamDetector
+final class AkismetSpamDetector implements SpamDetector
 {
     /**
      * @var \TijsVerkoyen\Akismet\Akismet
diff --git a/app/Spam/DummySpamDetector.php b/app/Spam/DummySpamDetector.php
@@ -4,7 +4,7 @@
 
 use App\User;
 
-class DummySpamDetector implements SpamDetector
+final class DummySpamDetector implements SpamDetector
 {
     /**
      * @var bool
diff --git a/app/Spam/SpamFilter.php b/app/Spam/SpamFilter.php
@@ -4,7 +4,7 @@
 
 use App\User;
 
-class SpamFilter implements SpamDetector
+final class SpamFilter implements SpamDetector
 {
     /**
      * @var \App\Spam\SpamDetector[]
diff --git a/app/User.php b/app/User.php
@@ -10,7 +10,7 @@
 use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Foundation\Auth\User as Authenticatable;
 
-class User extends Authenticatable
+final class User extends Authenticatable
 {
     use HasTimestamps, ModelHelpers, Notifiable;
 
diff --git a/app/Validation/DoesNotContainUrlRule.php b/app/Validation/DoesNotContainUrlRule.php
@@ -4,7 +4,7 @@
 
 use Illuminate\Validation\Factory as Validator;
 
-class DoesNotContainUrlRule
+final class DoesNotContainUrlRule
 {
     const NAME = 'not_contain_url';
 
diff --git a/app/Validation/PasscheckRule.php b/app/Validation/PasscheckRule.php
@@ -5,7 +5,7 @@
 use Auth;
 use Hash;
 
-class PasscheckRule
+final class PasscheckRule
 {
     const NAME = 'passcheck';
 
diff --git a/app/Validation/SpamRule.php b/app/Validation/SpamRule.php
@@ -5,7 +5,7 @@
 use App\Spam\SpamDetector;
 use Illuminate\Contracts\Auth\Guard;
 
-class SpamRule
+final class SpamRule
 {
     const NAME = 'spam';
 

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@`
`5`	`5`	`use App\User;`
`6`	`6`	`use Carbon\Carbon;`
`7`	`7`
`8`		`-class BanUser`
	`8`	`+final class BanUser`
`9`	`9`	`{`
`10`	`10`	`/**`
`11`	`11`	`* @var \App\User`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use App\User;`
`6`	`6`
`7`		`-class ConfirmUser`
	`7`	`+final class ConfirmUser`
`8`	`8`	`{`
`9`	`9`	`/**`
`10`	`10`	`* @var \App\User`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`use App\Models\ReplyAble;`
`8`	`8`	`use App\Http\Requests\CreateReplyRequest;`
`9`	`9`
`10`		`-class CreateReply`
	`10`	`+final class CreateReply`
`11`	`11`	`{`
`12`	`12`	`/**`
`13`	`13`	`* @var string`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@`
`6`	`6`	`use App\Models\Thread;`
`7`	`7`	`use App\Http\Requests\ThreadRequest;`
`8`	`8`
`9`		`-class CreateThread`
	`9`	`+final class CreateThread`
`10`	`10`	`{`
`11`	`11`	`/**`
`12`	`12`	`* @var string`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use App\Models\Reply;`
`6`	`6`
`7`		`-class DeleteReply`
	`7`	`+final class DeleteReply`
`8`	`8`	`{`
`9`	`9`	`/**`
`10`	`10`	`* @var \App\Models\Reply`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use App\Models\Thread;`
`6`	`6`
`7`		`-class DeleteThread`
	`7`	`+final class DeleteThread`
`8`	`8`	`{`
`9`	`9`	`/**`
`10`	`10`	`* @var \App\Models\Thread`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`
`5`	`5`	`use App\User;`
`6`	`6`
`7`		`-class DeleteUser`
	`7`	`+final class DeleteUser`
`8`	`8`	`{`
`9`	`9`	`/**`
`10`	`10`	`* @var \App\User`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`use App\Http\Requests\RegisterRequest;`
`8`	`8`	`use Illuminate\Database\Eloquent\ModelNotFoundException;`
`9`	`9`
`10`		`-class RegisterUser`
	`10`	`+final class RegisterUser`
`11`	`11`	`{`
`12`	`12`	`/**`
`13`	`13`	`* @var string`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`use Illuminate\Contracts\Mail\Mailer;`
`8`	`8`	`use Illuminate\Queue\SerializesModels;`
`9`	`9`
`10`		`-class SendEmailConfirmation`
	`10`	`+final class SendEmailConfirmation`
`11`	`11`	`{`
`12`	`12`	`use SerializesModels;`
`13`	`13`