Revert "Apply fixes from StyleCI"

emmadesilva · emmadesilva · commit f79d3365147d · 2024-01-10T10:57:01.000+01:00
This reverts commit 9d7a02f.
diff --git a/packages/framework/src/Console/Commands/PublishHomepageCommand.php b/packages/framework/src/Console/Commands/PublishHomepageCommand.php
@@ -33,17 +33,17 @@ class PublishHomepageCommand extends Command
 
     /** @var array<string, array{name: string, description: string, group: string}> */
     protected array $options = [
-        'welcome' => [
+        'welcome'=> [
             'name' => 'Welcome',
             'description' => 'The default welcome page.',
             'group' => 'hyde-welcome-page',
         ],
-        'posts' => [
+        'posts'=> [
             'name' => 'Posts Feed',
             'description' => 'A feed of your latest posts. Perfect for a blog site!',
             'group' => 'hyde-posts-page',
         ],
-        'blank' => [
+        'blank'=>  [
             'name' => 'Blank Starter',
             'description' => 'A blank Blade template with just the base layout.',
             'group' => 'hyde-blank-page',
diff --git a/packages/framework/src/Markdown/Contracts/FrontMatter/BlogPostSchema.php b/packages/framework/src/Markdown/Contracts/FrontMatter/BlogPostSchema.php
@@ -13,11 +13,11 @@
 interface BlogPostSchema extends PageSchema
 {
     public const BLOG_POST_SCHEMA = [
-        'title' => 'string',
-        'description' => 'string',
-        'category' => 'string',
-        'date' => 'string',
-        'author' => ['string', AuthorSchema::AUTHOR_SCHEMA],
-        'image' => ['string', FeaturedImageSchema::FEATURED_IMAGE_SCHEMA],
+        'title'        => 'string',
+        'description'  => 'string',
+        'category'     => 'string',
+        'date'         => 'string',
+        'author'       => ['string', AuthorSchema::AUTHOR_SCHEMA],
+        'image'        => ['string', FeaturedImageSchema::FEATURED_IMAGE_SCHEMA],
     ];
 }
diff --git a/packages/framework/src/Markdown/Contracts/FrontMatter/PageSchema.php b/packages/framework/src/Markdown/Contracts/FrontMatter/PageSchema.php
@@ -12,8 +12,8 @@
 interface PageSchema extends FrontMatterSchema
 {
     public const PAGE_SCHEMA = [
-        'title' => 'string',
-        'canonicalUrl' => 'string', // While not present in the page data, it is supported as a front matter key for the accessor data source.
-        'navigation' => NavigationSchema::NAVIGATION_SCHEMA,
+        'title'         => 'string',
+        'canonicalUrl'  => 'string', // While not present in the page data, it is supported as a front matter key for the accessor data source.
+        'navigation'    => NavigationSchema::NAVIGATION_SCHEMA,
     ];
 }
diff --git a/packages/framework/src/Markdown/Contracts/FrontMatter/SubSchemas/AuthorSchema.php b/packages/framework/src/Markdown/Contracts/FrontMatter/SubSchemas/AuthorSchema.php
@@ -12,8 +12,8 @@
 interface AuthorSchema extends BlogPostSchema
 {
     public const AUTHOR_SCHEMA = [
-        'name' => 'string',
-        'username' => 'string',
-        'website' => 'string',
+        'name'      => 'string',
+        'username'  => 'string',
+        'website'   => 'string',
     ];
 }
diff --git a/packages/framework/src/Markdown/Contracts/FrontMatter/SubSchemas/FeaturedImageSchema.php b/packages/framework/src/Markdown/Contracts/FrontMatter/SubSchemas/FeaturedImageSchema.php
@@ -13,13 +13,13 @@
 interface FeaturedImageSchema extends BlogPostSchema
 {
     public const FEATURED_IMAGE_SCHEMA = [
-        'source' => 'string', // Name of a file in _media/ or a remote URL (required)
-        'altText' => 'string', // The alt text (important for accessibility) // todo: Support alt, description
-        'titleText' => 'string', // The title text (hover tooltip & metadata) // todo: Support title, caption
-        'licenseName' => 'string', // The name of the license (e.g. "CC BY 4.0")
-        'licenseUrl' => 'string', // The URL of the license (e.g. "https://creativecommons.org/licenses/by/4.0/")
-        'authorName' => 'string', // The name of the author/photographer of the image (e.g. "John Doe", Wikimedia Commons)
-        'authorUrl' => 'string', // The URL of the author/photographer of the image (e.g. "https://commons.wikimedia.org/wiki/User:John_Doe", Wikimedia Commons)
-        'copyright' => 'string', // The copyright text (e.g. "© 2023 John Doe")
+        'source'       => 'string', // Name of a file in _media/ or a remote URL (required)
+        'altText'      => 'string', // The alt text (important for accessibility) // todo: Support alt, description
+        'titleText'    => 'string', // The title text (hover tooltip & metadata) // todo: Support title, caption
+        'licenseName'  => 'string', // The name of the license (e.g. "CC BY 4.0")
+        'licenseUrl'   => 'string', // The URL of the license (e.g. "https://creativecommons.org/licenses/by/4.0/")
+        'authorName'   => 'string', // The name of the author/photographer of the image (e.g. "John Doe", Wikimedia Commons)
+        'authorUrl'    => 'string', // The URL of the author/photographer of the image (e.g. "https://commons.wikimedia.org/wiki/User:John_Doe", Wikimedia Commons)
+        'copyright'    => 'string', // The copyright text (e.g. "© 2023 John Doe")
     ];
 }
diff --git a/packages/framework/src/Markdown/Contracts/FrontMatter/SubSchemas/NavigationSchema.php b/packages/framework/src/Markdown/Contracts/FrontMatter/SubSchemas/NavigationSchema.php
@@ -13,9 +13,9 @@
 interface NavigationSchema extends PageSchema
 {
     public const NAVIGATION_SCHEMA = [
-        'label' => 'string', // The text to display
-        'priority' => 'int',  // Order is also supported
-        'hidden' => 'bool',  // Visible is also supported (but obviously invert the value)
-        'group' => 'string', // Category is also supported
+        'label'     => 'string', // The text to display
+        'priority'  => 'int',  // Order is also supported
+        'hidden'    => 'bool',  // Visible is also supported (but obviously invert the value)
+        'group'     => 'string', // Category is also supported
     ];
 }
diff --git a/packages/framework/src/Support/Filesystem/MediaFile.php b/packages/framework/src/Support/Filesystem/MediaFile.php
@@ -70,18 +70,18 @@ public function getMimeType(): string
         // See if we can find a mime type for the extension instead of
         // having to rely on a PHP extension and filesystem lookups.
         $lookup = [
-            'txt' => 'text/plain',
-            'md' => 'text/markdown',
+            'txt'  => 'text/plain',
+            'md'   => 'text/markdown',
             'html' => 'text/html',
-            'css' => 'text/css',
-            'svg' => 'image/svg+xml',
-            'png' => 'image/png',
-            'jpg' => 'image/jpeg',
+            'css'  => 'text/css',
+            'svg'  => 'image/svg+xml',
+            'png'  => 'image/png',
+            'jpg'  => 'image/jpeg',
             'jpeg' => 'image/jpeg',
-            'gif' => 'image/gif',
+            'gif'  => 'image/gif',
             'json' => 'application/json',
-            'js' => 'application/javascript',
-            'xml' => 'application/xml',
+            'js'   => 'application/javascript',
+            'xml'  => 'application/xml',
         ];
 
         if (isset($lookup[$extension])) {
diff --git a/packages/framework/src/Support/Models/RouteListItem.php b/packages/framework/src/Support/Models/RouteListItem.php
@@ -28,10 +28,10 @@ public function __construct(Route $route)
     public function toArray(): array
     {
         return [
-            'page_type' => $this->stylePageType($this->route->getPageClass()),
+            'page_type'   => $this->stylePageType($this->route->getPageClass()),
             'source_file' => $this->styleSourcePath($this->route->getSourcePath()),
             'output_file' => $this->styleOutputPath($this->route->getOutputPath()),
-            'route_key' => $this->styleRouteKey($this->route->getRouteKey()),
+            'route_key'   => $this->styleRouteKey($this->route->getRouteKey()),
         ];
     }
 
diff --git a/packages/framework/tests/Feature/MarkdownServiceTest.php b/packages/framework/tests/Feature/MarkdownServiceTest.php
@@ -93,7 +93,7 @@ public function test_raw_html_tags_are_stripped_by_default()
 
     public function test_raw_html_tags_are_not_stripped_when_explicitly_enabled()
     {
-        config(['markdown.allow_html' => true]);
+        config(['markdown.allow_html' =>true]);
         $markdown = '<p>foo</p><style>bar</style><script>hat</script>';
         $service = new MarkdownService($markdown);
         $html = $service->parse();
diff --git a/packages/framework/tests/Feature/Support/MediaFileTest.php b/packages/framework/tests/Feature/Support/MediaFileTest.php
@@ -75,8 +75,8 @@ public function test_to_array_returns_array_of_file_properties()
         $this->file('foo.txt', 'foo bar');
 
         $this->assertSame([
-            'name' => 'foo.txt',
-            'path' => 'foo.txt',
+            'name'     => 'foo.txt',
+            'path'     => 'foo.txt',
             'length' => 7,
             'mimeType' => 'text/plain',
         ], MediaFile::make('foo.txt')->toArray());
diff --git a/packages/framework/tests/Feature/Support/ProjectFileTest.php b/packages/framework/tests/Feature/Support/ProjectFileTest.php
@@ -74,8 +74,8 @@ public function test_to_array_returns_array_of_file_properties()
         $this->file('foo.txt', 'foo bar');
 
         $this->assertSame([
-            'name' => 'foo.txt',
-            'path' => 'foo.txt',
+            'name'     => 'foo.txt',
+            'path'     => 'foo.txt',
         ], ProjectFileTestClass::make('foo.txt')->toArray());
     }
 
diff --git a/packages/framework/tests/Feature/Support/SourceFileTest.php b/packages/framework/tests/Feature/Support/SourceFileTest.php
@@ -93,8 +93,8 @@ public function test_to_array_returns_array_of_file_properties()
         $this->file('foo.txt', 'foo bar');
 
         $this->assertSame([
-            'name' => 'foo.txt',
-            'path' => 'foo.txt',
+            'name'     => 'foo.txt',
+            'path'     => 'foo.txt',
             'pageClass' => HydePage::class,
         ], SourceFile::make('foo.txt')->toArray());
     }
diff --git a/packages/framework/tests/Unit/BreadcrumbsComponentTest.php b/packages/framework/tests/Unit/BreadcrumbsComponentTest.php
@@ -47,7 +47,7 @@ public function testCanRender()
         $view = Mockery::mock(\Illuminate\View\View::class);
         $mock = Mockery::mock(Factory::class);
         $mock->shouldReceive('make')->once()->with('hyde::components.breadcrumbs')->andReturn($view);
-        app()->singleton('view', fn () => $mock);
+        app()->singleton('view', fn () =>$mock);
         View::swap($mock);
 
         $this->assertSame($view, (new BreadcrumbsComponent())->render());
diff --git a/packages/framework/tests/Unit/Pages/BaseHydePageUnitTest.php b/packages/framework/tests/Unit/Pages/BaseHydePageUnitTest.php
@@ -37,8 +37,8 @@ protected function setUp(): void
             ]),
             'share' => null,
         ]));
-        app()->bind(\Illuminate\Contracts\View\Factory::class, fn () => $mock);
-        app()->bind('view', fn () => $mock);
+        app()->bind(\Illuminate\Contracts\View\Factory::class, fn () =>$mock);
+        app()->bind('view', fn () =>$mock);
 
         Render::swap(new RenderData());
     }
diff --git a/packages/framework/tests/Unit/SchemaContractsTest.php b/packages/framework/tests/Unit/SchemaContractsTest.php
@@ -45,42 +45,42 @@ class SchemaContractsTest extends UnitTestCase
     public function testSchemasAreNotAccidentallyChanged()
     {
         $this->assertSame([
-            'title' => 'string',
-            'canonicalUrl' => 'string',
-            'navigation' => NavigationSchema::NAVIGATION_SCHEMA,
+            'title'         => 'string',
+            'canonicalUrl'  => 'string',
+            'navigation'    => NavigationSchema::NAVIGATION_SCHEMA,
         ], PageSchema::PAGE_SCHEMA);
 
         $this->assertSame([
-            'label' => 'string',
-            'priority' => 'int',
-            'hidden' => 'bool',
-            'group' => 'string',
+            'label'     => 'string',
+            'priority'  => 'int',
+            'hidden'    => 'bool',
+            'group'     => 'string',
         ], NavigationSchema::NAVIGATION_SCHEMA);
 
         $this->assertSame([
-            'title' => 'string',
-            'description' => 'string',
-            'category' => 'string',
-            'date' => 'string',
-            'author' => ['string', AuthorSchema::AUTHOR_SCHEMA],
-            'image' => ['string', FeaturedImageSchema::FEATURED_IMAGE_SCHEMA],
+            'title'        => 'string',
+            'description'  => 'string',
+            'category'     => 'string',
+            'date'         => 'string',
+            'author'       => ['string', AuthorSchema::AUTHOR_SCHEMA],
+            'image'        => ['string', FeaturedImageSchema::FEATURED_IMAGE_SCHEMA],
         ], BlogPostSchema::BLOG_POST_SCHEMA);
 
         $this->assertSame([
-            'name' => 'string',
-            'username' => 'string',
-            'website' => 'string',
+            'name'      => 'string',
+            'username'  => 'string',
+            'website'   => 'string',
         ], AuthorSchema::AUTHOR_SCHEMA);
 
         $this->assertSame([
-            'source' => 'string',
-            'altText' => 'string',
-            'titleText' => 'string',
-            'licenseName' => 'string',
-            'licenseUrl' => 'string',
-            'authorName' => 'string',
-            'authorUrl' => 'string',
-            'copyright' => 'string',
+            'source'         => 'string',
+            'altText'        => 'string',
+            'titleText'      => 'string',
+            'licenseName'    => 'string',
+            'licenseUrl'     => 'string',
+            'authorName'     => 'string',
+            'authorUrl'      => 'string',
+            'copyright'      => 'string',
         ], FeaturedImageSchema::FEATURED_IMAGE_SCHEMA);
     }
 
diff --git a/packages/framework/tests/Unit/Views/FeaturedImageViewTest.php b/packages/framework/tests/Unit/Views/FeaturedImageViewTest.php
@@ -266,7 +266,7 @@ protected function stripWhitespace(string $string): string
         return str_replace([' ', "\r", "\n"], '', $string);
     }
 
-    protected function renderComponent(FeaturedImage|array $data = ['image.source' => 'foo']): string
+    protected function renderComponent(FeaturedImage|array $data = ['image.source'=>'foo']): string
     {
         $image = $data instanceof FeaturedImage ? $data : $this->make($data);
 
diff --git a/packages/publications/src/Views/Components/RelatedPublicationsComponent.php b/packages/publications/src/Views/Components/RelatedPublicationsComponent.php
@@ -105,9 +105,9 @@ protected function getAllRelatedPages(Collection $publicationPages, Collection $
             if ($matchedTagCount) {
                 $allRelatedPages->add(
                     collect([
-                        'count' => $matchedTagCount,
+                        'count'      => $matchedTagCount,
                         'identifier' => $publicationPage->identifier,
-                        'page' => $publicationPage,
+                        'page'       => $publicationPage,
                     ])
                 );
             }
diff --git a/packages/publications/tests/Feature/CreatesNewPublicationPageTest.php b/packages/publications/tests/Feature/CreatesNewPublicationPageTest.php
@@ -203,7 +203,7 @@ public function testItCreatesValidYaml()
             '__createdAt' => 1640995200,
             'title' => 'Hello World',
             'description' => "This is a description.\nIt can be multiple lines.\n",
-            'tags' => ['tag1', 'tag2', 'foo bar'],
+            'tags' =>  ['tag1', 'tag2', 'foo bar'],
         ], Yaml::parse(Str::between($contents, '---', '---')));
     }
 
diff --git a/packages/publications/tests/Feature/MakePublicationCommandTest.php b/packages/publications/tests/Feature/MakePublicationCommandTest.php
diff --git a/packages/publications/tests/Feature/PublicationListPageTest.php b/packages/publications/tests/Feature/PublicationListPageTest.php
diff --git a/packages/publications/tests/Feature/PublicationTypeTest.php b/packages/publications/tests/Feature/PublicationTypeTest.php
diff --git a/packages/realtime-compiler/src/Http/ExceptionHandler.php b/packages/realtime-compiler/src/Http/ExceptionHandler.php
diff --git a/packages/realtime-compiler/src/Http/HtmlResponse.php b/packages/realtime-compiler/src/Http/HtmlResponse.php
diff --git a/packages/realtime-compiler/src/Models/FileObject.php b/packages/realtime-compiler/src/Models/FileObject.php
diff --git a/packages/realtime-compiler/src/Routing/Router.php b/packages/realtime-compiler/src/Routing/Router.php

Original file line number	Diff line number	Diff line change
`@@ -12,8 +12,8 @@`
`12`	`12`	`interface PageSchema extends FrontMatterSchema`
`13`	`13`	`{`
`14`	`14`	`public const PAGE_SCHEMA = [`
`15`		`- 'title' => 'string',`
`16`		`- 'canonicalUrl' => 'string', // While not present in the page data, it is supported as a front matter key for the accessor data source.`
`17`		`- 'navigation' => NavigationSchema::NAVIGATION_SCHEMA,`
	`15`	`+ 'title' => 'string',`
	`16`	`+ 'canonicalUrl' => 'string', // While not present in the page data, it is supported as a front matter key for the accessor data source.`
	`17`	`+ 'navigation' => NavigationSchema::NAVIGATION_SCHEMA,`
`18`	`18`	`];`
`19`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,9 +13,9 @@`
`13`	`13`	`interface NavigationSchema extends PageSchema`
`14`	`14`	`{`
`15`	`15`	`public const NAVIGATION_SCHEMA = [`
`16`		`- 'label' => 'string', // The text to display`
`17`		`- 'priority' => 'int', // Order is also supported`
`18`		`- 'hidden' => 'bool', // Visible is also supported (but obviously invert the value)`
`19`		`- 'group' => 'string', // Category is also supported`
	`16`	`+ 'label' => 'string', // The text to display`
	`17`	`+ 'priority' => 'int', // Order is also supported`
	`18`	`+ 'hidden' => 'bool', // Visible is also supported (but obviously invert the value)`
	`19`	`+ 'group' => 'string', // Category is also supported`
`20`	`20`	`];`
`21`	`21`	`}`
Original file line number	Diff line number	Diff line change
`@@ -28,10 +28,10 @@ public function __construct(Route $route)`
`28`	`28`	`public function toArray(): array`
`29`	`29`	`{`
`30`	`30`	`return [`
`31`		`- 'page_type' => $this->stylePageType($this->route->getPageClass()),`
	`31`	`+ 'page_type' => $this->stylePageType($this->route->getPageClass()),`
`32`	`32`	`'source_file' => $this->styleSourcePath($this->route->getSourcePath()),`
`33`	`33`	`'output_file' => $this->styleOutputPath($this->route->getOutputPath()),`
`34`		`- 'route_key' => $this->styleRouteKey($this->route->getRouteKey()),`
	`34`	`+ 'route_key' => $this->styleRouteKey($this->route->getRouteKey()),`
`35`	`35`	`];`
`36`	`36`	`}`
`37`	`37`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ public function test_raw_html_tags_are_stripped_by_default()`
`93`	`93`
`94`	`94`	`public function test_raw_html_tags_are_not_stripped_when_explicitly_enabled()`
`95`	`95`	`{`
`96`		`- config(['markdown.allow_html' => true]);`
	`96`	`+ config(['markdown.allow_html' =>true]);`
`97`	`97`	`$markdown = '<p>foo</p><style>bar</style><script>hat</script>';`
`98`	`98`	`$service = new MarkdownService($markdown);`
`99`	`99`	`$html = $service->parse();`