From 8a749c6acf930614a119b277170bcf7c1c1db9f9 Mon Sep 17 00:00:00 2001 From: Dan Brown Date: Sun, 18 Sep 2022 01:25:20 +0100 Subject: [PATCH] Added and ran PHPCS --- app/Actions/Webhook.php | 4 +- app/Actions/WebhookTrackedEvent.php | 4 +- app/Auth/Access/LdapService.php | 2 +- app/Auth/Access/Oidc/OidcService.php | 4 -- app/Config/dompdf.php | 1 + app/Config/snappy.php | 1 + app/Entities/Tools/PageEditActivity.php | 2 +- .../Auth/ForgotPasswordController.php | 1 - app/Http/Controllers/Auth/LoginController.php | 11 ++-- .../Controllers/Auth/RegisterController.php | 1 - .../Auth/ResetPasswordController.php | 1 - app/Http/Controllers/BookController.php | 2 +- app/Http/Controllers/BookSortController.php | 2 +- .../Controllers/PageRevisionController.php | 2 +- app/Uploads/ImageService.php | 2 +- composer.json | 1 + composer.lock | 58 ++++++++++++++++++- phpcs.xml | 35 +++++++++++ routes/api.php | 11 ++-- tests/Actions/AuditLogTest.php | 2 - tests/Uploads/AttachmentTest.php | 2 +- tests/UrlTest.php | 1 + 22 files changed, 120 insertions(+), 30 deletions(-) create mode 100644 phpcs.xml diff --git a/app/Actions/Webhook.php b/app/Actions/Webhook.php index 72a67ad92..3ed5bb56f 100644 --- a/app/Actions/Webhook.php +++ b/app/Actions/Webhook.php @@ -22,10 +22,10 @@ use Illuminate\Database\Eloquent\Relations\HasMany; */ class Webhook extends Model implements Loggable { - protected $fillable = ['name', 'endpoint', 'timeout']; - use HasFactory; + protected $fillable = ['name', 'endpoint', 'timeout']; + protected $casts = [ 'last_called_at' => 'datetime', 'last_errored_at' => 'datetime', diff --git a/app/Actions/WebhookTrackedEvent.php b/app/Actions/WebhookTrackedEvent.php index 6289581a2..28b3a366d 100644 --- a/app/Actions/WebhookTrackedEvent.php +++ b/app/Actions/WebhookTrackedEvent.php @@ -12,7 +12,7 @@ use Illuminate\Database\Eloquent\Model; */ class WebhookTrackedEvent extends Model { - protected $fillable = ['event']; - use HasFactory; + + protected $fillable = ['event']; } diff --git a/app/Auth/Access/LdapService.php b/app/Auth/Access/LdapService.php index 2540fe2d8..359eeca2f 100644 --- a/app/Auth/Access/LdapService.php +++ b/app/Auth/Access/LdapService.php @@ -105,7 +105,7 @@ class LdapService 'name' => $this->getUserResponseProperty($user, $displayNameAttr, $userCn), 'dn' => $user['dn'], 'email' => $this->getUserResponseProperty($user, $emailAttr, null), - 'avatar'=> $thumbnailAttr ? $this->getUserResponseProperty($user, $thumbnailAttr, null) : null, + 'avatar' => $thumbnailAttr ? $this->getUserResponseProperty($user, $thumbnailAttr, null) : null, ]; if ($this->config['dump_user_details']) { diff --git a/app/Auth/Access/Oidc/OidcService.php b/app/Auth/Access/Oidc/OidcService.php index 4f5a3e1ac..c9c3cc511 100644 --- a/app/Auth/Access/Oidc/OidcService.php +++ b/app/Auth/Access/Oidc/OidcService.php @@ -2,7 +2,6 @@ namespace BookStack\Auth\Access\Oidc; -use function auth; use BookStack\Auth\Access\GroupSyncService; use BookStack\Auth\Access\LoginService; use BookStack\Auth\Access\RegistrationService; @@ -10,14 +9,11 @@ use BookStack\Auth\User; use BookStack\Exceptions\JsonDebugException; use BookStack\Exceptions\StoppedAuthenticationException; use BookStack\Exceptions\UserRegistrationException; -use function config; use Illuminate\Support\Arr; use Illuminate\Support\Facades\Cache; use League\OAuth2\Client\OptionProvider\HttpBasicAuthOptionProvider; use League\OAuth2\Client\Provider\Exception\IdentityProviderException; use Psr\Http\Client\ClientInterface as HttpClient; -use function trans; -use function url; /** * Class OpenIdConnectService diff --git a/app/Config/dompdf.php b/app/Config/dompdf.php index 16a527b3c..09dd91bcc 100644 --- a/app/Config/dompdf.php +++ b/app/Config/dompdf.php @@ -7,6 +7,7 @@ * Configuration should be altered via the `.env` file or environment variables. * Do not edit this file unless you're happy to maintain any changes yourself. */ + $dompdfPaperSizeMap = [ 'a4' => 'a4', 'letter' => 'letter', diff --git a/app/Config/snappy.php b/app/Config/snappy.php index 8ab10a39c..a87ce805f 100644 --- a/app/Config/snappy.php +++ b/app/Config/snappy.php @@ -7,6 +7,7 @@ * Configuration should be altered via the `.env` file or environment variables. * Do not edit this file unless you're happy to maintain any changes yourself. */ + $snappyPaperSizeMap = [ 'a4' => 'A4', 'letter' => 'Letter', diff --git a/app/Entities/Tools/PageEditActivity.php b/app/Entities/Tools/PageEditActivity.php index 2672de941..646b200f1 100644 --- a/app/Entities/Tools/PageEditActivity.php +++ b/app/Entities/Tools/PageEditActivity.php @@ -42,7 +42,7 @@ class PageEditActivity $userMessage = trans('entities.pages_draft_edit_active.start_b', ['userName' => $firstDraft->createdBy->name ?? '']); } - $timeMessage = trans('entities.pages_draft_edit_active.time_b', ['minCount'=> 60]); + $timeMessage = trans('entities.pages_draft_edit_active.time_b', ['minCount' => 60]); return trans('entities.pages_draft_edit_active.message', ['start' => $userMessage, 'time' => $timeMessage]); } diff --git a/app/Http/Controllers/Auth/ForgotPasswordController.php b/app/Http/Controllers/Auth/ForgotPasswordController.php index 5e73b232c..b345fad1c 100644 --- a/app/Http/Controllers/Auth/ForgotPasswordController.php +++ b/app/Http/Controllers/Auth/ForgotPasswordController.php @@ -20,7 +20,6 @@ class ForgotPasswordController extends Controller | your application to your users. Feel free to explore this trait. | */ - use SendsPasswordResetEmails; /** diff --git a/app/Http/Controllers/Auth/LoginController.php b/app/Http/Controllers/Auth/LoginController.php index f1a1a8bd6..1d6a36c5b 100644 --- a/app/Http/Controllers/Auth/LoginController.php +++ b/app/Http/Controllers/Auth/LoginController.php @@ -24,8 +24,9 @@ class LoginController extends Controller | to conveniently provide its functionality to your applications. | */ - - use AuthenticatesUsers { logout as traitLogout; } + use AuthenticatesUsers { + logout as traitLogout; + } /** * Redirection paths. @@ -112,8 +113,10 @@ class LoginController extends Controller // If the class is using the ThrottlesLogins trait, we can automatically throttle // the login attempts for this application. We'll key this by the username and // the IP address of the client making these requests into this application. - if (method_exists($this, 'hasTooManyLoginAttempts') && - $this->hasTooManyLoginAttempts($request)) { + if ( + method_exists($this, 'hasTooManyLoginAttempts') && + $this->hasTooManyLoginAttempts($request) + ) { $this->fireLockoutEvent($request); Activity::logFailedLogin($username); diff --git a/app/Http/Controllers/Auth/RegisterController.php b/app/Http/Controllers/Auth/RegisterController.php index b0aec1177..15ee78d50 100644 --- a/app/Http/Controllers/Auth/RegisterController.php +++ b/app/Http/Controllers/Auth/RegisterController.php @@ -27,7 +27,6 @@ class RegisterController extends Controller | provide this functionality without requiring any additional code. | */ - use RegistersUsers; protected SocialAuthService $socialAuthService; diff --git a/app/Http/Controllers/Auth/ResetPasswordController.php b/app/Http/Controllers/Auth/ResetPasswordController.php index a31529b11..9df010736 100644 --- a/app/Http/Controllers/Auth/ResetPasswordController.php +++ b/app/Http/Controllers/Auth/ResetPasswordController.php @@ -20,7 +20,6 @@ class ResetPasswordController extends Controller | explore this trait and override any methods you wish to tweak. | */ - use ResetsPasswords; protected $redirectTo = '/'; diff --git a/app/Http/Controllers/BookController.php b/app/Http/Controllers/BookController.php index a041267bb..cc2f6f534 100644 --- a/app/Http/Controllers/BookController.php +++ b/app/Http/Controllers/BookController.php @@ -147,7 +147,7 @@ class BookController extends Controller { $book = $this->bookRepo->getBySlug($slug); $this->checkOwnablePermission('book-update', $book); - $this->setPageTitle(trans('entities.books_edit_named', ['bookName'=>$book->getShortName()])); + $this->setPageTitle(trans('entities.books_edit_named', ['bookName' => $book->getShortName()])); return view('books.edit', ['book' => $book, 'current' => $book]); } diff --git a/app/Http/Controllers/BookSortController.php b/app/Http/Controllers/BookSortController.php index 8aac2b769..5d0577a74 100644 --- a/app/Http/Controllers/BookSortController.php +++ b/app/Http/Controllers/BookSortController.php @@ -28,7 +28,7 @@ class BookSortController extends Controller $bookChildren = (new BookContents($book))->getTree(false); - $this->setPageTitle(trans('entities.books_sort_named', ['bookName'=>$book->getShortName()])); + $this->setPageTitle(trans('entities.books_sort_named', ['bookName' => $book->getShortName()])); return view('books.sort', ['book' => $book, 'current' => $book, 'bookChildren' => $bookChildren]); } diff --git a/app/Http/Controllers/PageRevisionController.php b/app/Http/Controllers/PageRevisionController.php index c4d5fbc7b..89775a602 100644 --- a/app/Http/Controllers/PageRevisionController.php +++ b/app/Http/Controllers/PageRevisionController.php @@ -91,7 +91,7 @@ class PageRevisionController extends Controller // TODO - Refactor PageContent so we don't need to juggle this $page->html = $revision->html; $page->html = (new PageContent($page))->render(); - $this->setPageTitle(trans('entities.pages_revision_named', ['pageName'=>$page->getShortName()])); + $this->setPageTitle(trans('entities.pages_revision_named', ['pageName' => $page->getShortName()])); return view('pages.revision', [ 'page' => $page, diff --git a/app/Uploads/ImageService.php b/app/Uploads/ImageService.php index 7b8af3b84..0199c207e 100644 --- a/app/Uploads/ImageService.php +++ b/app/Uploads/ImageService.php @@ -315,7 +315,7 @@ class ImageService { try { $thumb = $this->imageTool->make($imageData); - } catch (ErrorException|NotSupportedException $e) { + } catch (ErrorException | NotSupportedException $e) { throw new ImageUploadException(trans('errors.cannot_create_thumbs')); } diff --git a/composer.json b/composer.json index 6d4bb734d..615723be9 100644 --- a/composer.json +++ b/composer.json @@ -50,6 +50,7 @@ "nunomaduro/collision": "^5.10", "nunomaduro/larastan": "^1.0", "phpunit/phpunit": "^9.5", + "squizlabs/php_codesniffer": "^3.7", "ssddanbrown/asserthtml": "^1.0" }, "autoload": { diff --git a/composer.lock b/composer.lock index 6f9ed25cc..b807fd577 100644 --- a/composer.lock +++ b/composer.lock @@ -4,7 +4,7 @@ "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies", "This file is @generated automatically" ], - "content-hash": "4ab21f732b2380ed1c3dd1a4eca2ef1a", + "content-hash": "1d3bd88b99d07b5410ee4b245bece28e", "packages": [ { "name": "aws/aws-crt-php", @@ -10645,6 +10645,62 @@ }, "time": "2022-07-20T18:31:45+00:00" }, + { + "name": "squizlabs/php_codesniffer", + "version": "3.7.1", + "source": { + "type": "git", + "url": "https://github.com/squizlabs/PHP_CodeSniffer.git", + "reference": "1359e176e9307e906dc3d890bcc9603ff6d90619" + }, + "dist": { + "type": "zip", + "url": "https://api.github.com/repos/squizlabs/PHP_CodeSniffer/zipball/1359e176e9307e906dc3d890bcc9603ff6d90619", + "reference": "1359e176e9307e906dc3d890bcc9603ff6d90619", + "shasum": "" + }, + "require": { + "ext-simplexml": "*", + "ext-tokenizer": "*", + "ext-xmlwriter": "*", + "php": ">=5.4.0" + }, + "require-dev": { + "phpunit/phpunit": "^4.0 || ^5.0 || ^6.0 || ^7.0" + }, + "bin": [ + "bin/phpcs", + "bin/phpcbf" + ], + "type": "library", + "extra": { + "branch-alias": { + "dev-master": "3.x-dev" + } + }, + "notification-url": "https://packagist.org/downloads/", + "license": [ + "BSD-3-Clause" + ], + "authors": [ + { + "name": "Greg Sherwood", + "role": "lead" + } + ], + "description": "PHP_CodeSniffer tokenizes PHP, JavaScript and CSS files and detects violations of a defined set of coding standards.", + "homepage": "https://github.com/squizlabs/PHP_CodeSniffer", + "keywords": [ + "phpcs", + "standards" + ], + "support": { + "issues": "https://github.com/squizlabs/PHP_CodeSniffer/issues", + "source": "https://github.com/squizlabs/PHP_CodeSniffer", + "wiki": "https://github.com/squizlabs/PHP_CodeSniffer/wiki" + }, + "time": "2022-06-18T07:21:10+00:00" + }, { "name": "ssddanbrown/asserthtml", "version": "v1.0.1", diff --git a/phpcs.xml b/phpcs.xml new file mode 100644 index 000000000..8d4c6b702 --- /dev/null +++ b/phpcs.xml @@ -0,0 +1,35 @@ + + + The coding standard for BookStack + + app + bootstrap/app.php + database + public/index.php + routes + tests + + + + + + + + + + ./tests/* + + + + ./tests/* + + + + ./database/* + + + + ./app/Config/* + + + diff --git a/routes/api.php b/routes/api.php index 20e167d70..d350fd86b 100644 --- a/routes/api.php +++ b/routes/api.php @@ -1,5 +1,11 @@ $fileName, - 'uploaded_to'=> $page->id, + 'uploaded_to' => $page->id, 'extension' => 'txt', 'order' => 1, 'created_by' => $admin->id, diff --git a/tests/UrlTest.php b/tests/UrlTest.php index 13f9d9e0c..dd278c240 100644 --- a/tests/UrlTest.php +++ b/tests/UrlTest.php @@ -3,6 +3,7 @@ namespace Tests; use BookStack\Http\Request; + use function request; use function url;