Merge pull request #28 from bilfeldt/fixes/service-provider

Remove the spatie package skeleton and fix issues with component name…

Author: bilfeldt

CHANGELOG.md

@@ -19,3 +19,5 @@ to the new namespace syntax
 ```blade
 <x-flash::alert ... /> // new syntax
 ```
+
+The session and view keys are also removed from the configuration, leaving the configuration with using the hardcoded value `messages` for both.

composer.json

@@ -17,7 +17,6 @@
     ],
     "require": {
         "php": "^8.0",
-        "spatie/laravel-package-tools": "^1.4.3",
         "illuminate/contracts": "^8.51 || ^9.0"
     },
     "require-dev": {

config/flash-message.php

@@ -4,89 +4,74 @@
 
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Support\Facades\Blade;
+use Illuminate\Support\ServiceProvider;
 use Illuminate\Support\ViewErrorBag;
 use Illuminate\View\Factory;
 use Illuminate\View\View;
-use Spatie\LaravelPackageTools\Package;
-use Spatie\LaravelPackageTools\PackageServiceProvider;
 
-class FlashMessageServiceProvider extends PackageServiceProvider
+class FlashMessageServiceProvider extends ServiceProvider
 {
     public const VIEW_COMPONENT_NAMESPACE = 'flash';
 
-    public function configurePackage(Package $package): void
+    /**
+     * Bootstrap any package services.
+     *
+     * @return void
+     */
+    public function boot()
     {
-        /*
-         * This class is a Package Service Provider
-         *
-         * More info: https://github.com/spatie/laravel-package-tools
-         */
-        $package
-            ->name('laravel-flash-message')
-            ->hasConfigFile()
-            ->hasViews(); // required for the view component blade files to be registered
-            // The package does not allow for namespaces (`<x-namespace:component />`) but only prefixes (`<x-prefix-component />`
-            // so we register those manually using `componentNamespace()`
-            //->hasViewComponents(
-            //    self::VIEW_COMPONENT_NAMESPACE,
-            //    Messages::class,
-            //    Alert::class,
-            //    AlertError::class,
-            //    AlertInfo::class,
-            //    AlertMessage::class,
-            //    AlertSuccess::class,
-            //    AlertWarning::class
-            //);
-    }
+        $this->loadViewsFrom(__DIR__.'/../resources/views', self::VIEW_COMPONENT_NAMESPACE);
+
+        $this->publishes([
+            __DIR__.'/../resources/views' => resource_path('views/vendor/'.self::VIEW_COMPONENT_NAMESPACE),
+        ]);
 
-    public function packageBooted()
-    {
         Blade::componentNamespace('Bilfeldt\\LaravelFlashMessage\\View\\Components', self::VIEW_COMPONENT_NAMESPACE);
 
         // This is used when adding a message from a controller: view('posts-index')->withMessage(...)
         View::macro('withMessage', function (Message $message, string $bag = 'default'): View {
             /** @var ViewFlashMessageBag $viewFlashMessageBag */
-            $viewFlashMessageBag = \Illuminate\Support\Facades\View::shared(config('flash-message.view_share'), new ViewFlashMessageBag());
+            $viewFlashMessageBag = \Illuminate\Support\Facades\View::shared('messages', new ViewFlashMessageBag());
 
-            \Illuminate\Support\Facades\View::share(config('flash-message.view_share'), $viewFlashMessageBag->push($message, $bag));
+            \Illuminate\Support\Facades\View::share('messages', $viewFlashMessageBag->push($message, $bag));
 
             return $this;
         });
 
         // This is used when adding a message from the View Facade: \Illuminate\Support\Facades\View::withMessage(...)
         Factory::macro('withMessage', function (Message $message, string $bag = 'default'): Factory {
             /** @var ViewFlashMessageBag $viewFlashMessageBag */
-            $viewFlashMessageBag = \Illuminate\Support\Facades\View::shared(config('flash-message.view_share'), new ViewFlashMessageBag());
+            $viewFlashMessageBag = \Illuminate\Support\Facades\View::shared('messages', new ViewFlashMessageBag());
 
-            \Illuminate\Support\Facades\View::share(config('flash-message.view_share'), $viewFlashMessageBag->push($message, $bag));
+            \Illuminate\Support\Facades\View::share('messages', $viewFlashMessageBag->push($message, $bag));
 
             return $this;
         });
 
         // This is used when adding messages from a controller: view('posts-index')->withMessages(...)
         View::macro('withMessages', function (array $messages, string $bag = 'default'): View {
             /** @var ViewFlashMessageBag $viewFlashMessageBag */
-            $viewFlashMessageBag = \Illuminate\Support\Facades\View::shared(config('flash-message.view_share'), new ViewFlashMessageBag());
+            $viewFlashMessageBag = \Illuminate\Support\Facades\View::shared('messages', new ViewFlashMessageBag());
 
             /** @var \Bilfeldt\LaravelFlashMessage\Message $message */
             foreach ($messages as $message) {
                 $viewFlashMessageBag->push($message, $bag);
             }
-            \Illuminate\Support\Facades\View::share(config('flash-message.view_share'), $viewFlashMessageBag);
+            \Illuminate\Support\Facades\View::share('messages', $viewFlashMessageBag);
 
             return $this;
         });
 
         // This is used when adding messages from the View Facade: \Illuminate\Support\Facades\View::withMessages(...)
         Factory::macro('withMessages', function (array $messages, string $bag = 'default'): Factory {
             /** @var ViewFlashMessageBag $viewFlashMessageBag */
-            $viewFlashMessageBag = \Illuminate\Support\Facades\View::shared(config('flash-message.view_share'), new ViewFlashMessageBag());
+            $viewFlashMessageBag = \Illuminate\Support\Facades\View::shared('messages', new ViewFlashMessageBag());
 
             /** @var \Bilfeldt\LaravelFlashMessage\Message $message */
             foreach ($messages as $message) {
                 $viewFlashMessageBag->push($message, $bag);
             }
-            \Illuminate\Support\Facades\View::share(config('flash-message.view_share'), $viewFlashMessageBag);
+            \Illuminate\Support\Facades\View::share('messages', $viewFlashMessageBag);
 
             return $this;
         });

src/FlashMessageServiceProvider.php

@@ -43,7 +43,7 @@ public function handle($request, Closure $next)
         // without having to bind. An empty collection is set when there aren't any messages.
         $this->view->share(
             'messages',
-            ViewFlashMessageBag::make($request->session()->get(config('flash-message.session_flash')) ?: [])
+            ViewFlashMessageBag::make($request->session()->get('messages') ?: [])
         );
 
         // Putting the messages in the view for every view allows the developer to just

src/Http/Middleware/ShareMessagesFromSession.php

@@ -26,15 +26,15 @@ public function render()
     {
         switch ($this->level) {
             case Message::LEVEL_MESSAGE:
-                return view('flash-message::components.alert-message');
+                return view('flash::components.alert-message');
             case Message::LEVEL_INFO:
-                return view('flash-message::components.alert-info');
+                return view('flash::components.alert-info');
             case Message::LEVEL_SUCCESS:
-                return view('flash-message::components.alert-success');
+                return view('flash::components.alert-success');
             case Message::LEVEL_WARNING:
-                return view('flash-message::components.alert-warning');
+                return view('flash::components.alert-warning');
             case Message::LEVEL_ERROR:
-                return view('flash-message::components.alert-error');
+                return view('flash::components.alert-error');
             default:
                 throw new \InvalidArgumentException('Invalid alert level: '.$this->level);
         }

src/View/Components/Alert.php

@@ -6,6 +6,6 @@ class AlertError extends AbstractAlert
 {
     public function render()
     {
-        return view('flash-message::components.alert-error');
+        return view('flash::components.alert-error');
     }
 }

src/View/Components/AlertError.php

@@ -6,6 +6,6 @@ class AlertInfo extends AbstractAlert
 {
     public function render()
     {
-        return view('flash-message::components.alert-info');
+        return view('flash::components.alert-info');
     }
 }

src/View/Components/AlertInfo.php

@@ -6,6 +6,6 @@ class AlertMessage extends AbstractAlert
 {
     public function render()
     {
-        return view('flash-message::components.alert-message');
+        return view('flash::components.alert-message');
     }
 }

src/View/Components/AlertMessage.php

@@ -6,6 +6,6 @@ class AlertSuccess extends AbstractAlert
 {
     public function render()
     {
-        return view('flash-message::components.alert-success');
+        return view('flash::components.alert-success');
     }
 }