fix php 8.2 deprecation error, bump min php version to php7.4

Author: arrilot

.gitignore

@@ -2,4 +2,5 @@
 composer.phar
 composer.lock
 .DS_Store
+.phpunit.result.cache
 /.idea

.travis.yml

@@ -11,8 +11,6 @@
 
 Run ```composer require arrilot/laravel-widgets```
 
-Laravel >=5.5 uses Package Auto-Discovery, so you don't need to manually add the ServiceProvider and Facades
-
 ## Usage
 
 Let's consider that we want to make a list of recent news and reuse it in several views.
@@ -43,7 +41,7 @@ class RecentNews extends AbstractWidget
      *
      * @var array
      */
-    protected $config = [];
+    protected array $config = [];
 
     /**
      * Treat this method as a controller action.
@@ -92,7 +90,7 @@ This can be easily achieved by:
 class RecentNews extends AbstractWidget
 {
     ...
-    protected $config = [
+    protected array $config = [
         'count' => 5
     ];
     ...
@@ -112,7 +110,7 @@ Config array is available in every widget method so you can use it to configure
 class RecentNews extends AbstractWidget
 {
     ...
-    protected $config = [
+    protected array $config = [
         'count' => 5,
         'foo'   => 'bar'
     ];
@@ -129,7 +127,7 @@ In this case do the following:
 
 1) Do not add `protected $config = [...]` line to a child.
 
-2) Instead add defaults like this:
+2) Instead, add defaults like this:
 
 ```php
 public function __construct(array $config = [])
@@ -157,7 +155,7 @@ public function run($sortBy, $sortOrder) { }
 
 ## Namespaces
 
-By default the package tries to find your widget in the ```App\Widgets``` namespace.
+By default, the package tries to find your widget in the ```App\Widgets``` namespace.
 
 You can override this by publishing package config (```php artisan vendor:publish --provider="Arrilot\Widgets\ServiceProvider"```) and setting `default_namespace` property.
 
@@ -183,7 +181,7 @@ No problem, there are several ways to call those widgets:
 
 ## Asynchronous widgets
 
-In some situations it can be very beneficial to load widget content with AJAX.
+Sometimes it can be very beneficial to load widget content with AJAX.
 
 Fortunately, this can be achieved very easily!
 All you need to do is to change facade or blade directive - `Widget::` => `AsyncWidget::`, `@widget` => `@asyncWidget`
@@ -195,12 +193,12 @@ For example, if you pass something like user_id through widget params and turn e
 
 > Note: You can set `use_jquery_for_ajax_calls` to `true` in the config file to use it for ajax calls if you want to, but you need to manually add jquery to your page in this case.
 
-By default nothing is shown until ajax call is finished.
+By default, nothing is shown until ajax call is finished.
 
 This can be customized by adding a `placeholder()` method to the widget class.
 
 ```php
-public function placeholder()
+public function placeholder(): string
 {
     return 'Loading...';
 }
@@ -243,7 +241,7 @@ This container is defined by `AbstractWidget::container()` method and can be cus
  *
  * @return array
  */
-public function container()
+public function container(): array
 {
     return [
         'element'       => 'div',
@@ -292,7 +290,7 @@ class RecentNews extends AbstractWidget
      *
      * @var array
      */
-    public $cacheTags = ['news', 'frontend'];
+    public array $cacheTags = ['news', 'frontend'];
 }
 ```
 

README.md

@@ -11,16 +11,16 @@
     ],
     "homepage": "https://github.com/arrilot/laravel-widgets",
     "require": {
-        "php": ">=7.0",
-        "illuminate/support": ">=5.5",
-        "illuminate/contracts": ">=5.5",
-        "illuminate/view": ">=5.5",
-        "illuminate/container": ">=5.5",
-        "illuminate/console": ">=5.5",
-        "illuminate/cache": ">=5.5"
+        "php": ">=7.4",
+        "illuminate/support": ">=8",
+        "illuminate/contracts": ">=8",
+        "illuminate/view": ">=8",
+        "illuminate/container": ">=8",
+        "illuminate/console": ">=8",
+        "illuminate/cache": ">=8"
     },
     "require-dev": {
-        "phpunit/phpunit": "~6.0"
+        "phpunit/phpunit": "~8.0"
     },
     "autoload": {
         "psr-4": {

composer.json

@@ -1,15 +1,14 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit backupGlobals="false"
+<phpunit bootstrap="vendor/autoload.php"
+         backupGlobals="false"
          backupStaticAttributes="false"
-         bootstrap="vendor/autoload.php"
          colors="true"
+         verbose="true"
          convertErrorsToExceptions="true"
          convertNoticesToExceptions="true"
          convertWarningsToExceptions="true"
          processIsolation="false"
-         stopOnFailure="false"
-         syntaxCheck="false"
->
+         stopOnFailure="false">
     <testsuites>
         <testsuite name="Package Test Suite">
             <directory>tests</directory>

phpunit.xml

@@ -6,6 +6,13 @@
 
 class WidgetGroupCollection
 {
+    /**
+     * The application wrapper.
+     *
+     * @var ApplicationWrapperContract
+     */
+    protected ApplicationWrapperContract $app;
+
     /**
      * The array of widget groups.
      *

src/WidgetGroupCollection.php

@@ -13,7 +13,7 @@ class AsyncWidgetFactoryTest extends TestCase
      */
     protected $factory;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->factory = new AsyncWidgetFactory(new TestApplicationWrapper());
     }

tests/AsyncWidgetFactoryTest.php

@@ -7,7 +7,7 @@
 
 class TestCase extends BaseTestCase
 {
-    public function tearDown()
+    public function tearDown(): void
     {
         WidgetId::reset();
     }

tests/Support/TestCase.php

@@ -14,7 +14,7 @@ class WidgetFactoryTest extends TestCase
      */
     protected $factory;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->factory = new WidgetFactory(new TestApplicationWrapper());
     }

tests/WidgetFactoryTest.php

@@ -14,7 +14,7 @@ class WidgetGroupCollectionTest extends TestCase
      */
     protected $collection;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->collection = new WidgetGroupCollection(new TestApplicationWrapper());
     }