Prepare Release 1.3.0

bryannielsen · github-actions[bot] · commit 368509c18da9 · 2024-03-18T15:40:41.000Z
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 ## [Unreleased]
 
+## [1.3.0] - 2024-03-18
+
 ### Added
 
 - Ability to pass a 'tagdata' parameter to the ExpressionEngine Template library
@@ -216,7 +218,9 @@
 
 - Initial Beta Release
 
-[Unreleased]: https://github.com/ExpressionEngine/Coilpack/compare/1.2.2...HEAD
+[Unreleased]: https://github.com/ExpressionEngine/Coilpack/compare/1.3.0...HEAD
+
+[1.3.0]: https://github.com/ExpressionEngine/Coilpack/compare/1.2.2...1.3.0
 
 [1.2.2]: https://github.com/ExpressionEngine/Coilpack/compare/1.2.1...1.2.2
 
diff --git a/src/Auth/CoilpackUserProvider.php b/src/Auth/CoilpackUserProvider.php
@@ -10,7 +10,6 @@ class CoilpackUserProvider extends EloquentUserProvider
     /**
      * Retrieve a user by the given credentials.
      *
-     * @param  array  $credentials
      * @return \Illuminate\Contracts\Auth\Authenticatable|null
      */
     public function retrieveByCredentials(array $credentials)
@@ -29,8 +28,6 @@ public function retrieveByCredentials(array $credentials)
     /**
      * Validate a user against the given credentials.
      *
-     * @param  \Illuminate\Contracts\Auth\Authenticatable  $user
-     * @param  array  $credentials
      * @return bool
      */
     public function validateCredentials(Authenticatable $user, array $credentials)
@@ -78,7 +75,6 @@ public function retrieveByToken($identifier, $token)
     /**
      * Update the "remember me" token for the given user in storage.
      *
-     * @param  \Illuminate\Contracts\Auth\Authenticatable  $user
      * @param  string  $token
      * @return void
      */
diff --git a/src/Auth/SessionGuard.php b/src/Auth/SessionGuard.php
@@ -47,7 +47,6 @@ public function user()
         return $this->user;
     }
 
-
     /**
      * Remove the user data from the session and cookies.
      *
diff --git a/src/CacheDriver.php b/src/CacheDriver.php
@@ -18,10 +18,10 @@ public function __construct(CacheRepository $repository)
      * Look for a value in the cache. If it exists, return the data
      * if not, return FALSE
      *
-     * @param  string  $key 	Key name
-     * @param  const  $scope	Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
-     *		local or global scoping of the cache item
-     * @return	mixed	value matching $key or FALSE on failure
+     * @param  string  $key  Key name
+     * @param  const  $scope  Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
+     *                        local or global scoping of the cache item
+     * @return mixed value matching $key or FALSE on failure
      */
     public function get($key, $scope = Cache::LOCAL_SCOPE)
     {
@@ -39,12 +39,12 @@ public function get($key, $scope = Cache::LOCAL_SCOPE)
     /**
      * Save value to cache
      *
-     * @param  string  $key		Key name
-     * @param  mixed  $data		Data to store
-     * @param  int  $ttl = 60	Cache TTL (in seconds)
-     * @param  const  $scope	Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
-     *		local or global scoping of the cache item
-     * @return	bool	TRUE on success, FALSE on failure
+     * @param  string  $key  Key name
+     * @param  mixed  $data  Data to store
+     * @param  int  $ttl  = 60	Cache TTL (in seconds)
+     * @param  const  $scope  Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
+     *                        local or global scoping of the cache item
+     * @return bool TRUE on success, FALSE on failure
      */
     public function save($key, $data, $ttl = 60, $scope = Cache::LOCAL_SCOPE)
     {
@@ -67,10 +67,10 @@ public function save($key, $data, $ttl = 60, $scope = Cache::LOCAL_SCOPE)
      *
      * ee()->cache->delete('/namespace_name/');
      *
-     * @param  string  $key	Key name
-     * @param  const  $scope	Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
-     *		local or global scoping of the cache item
-     * @return	bool	TRUE on success, FALSE on failure
+     * @param  string  $key  Key name
+     * @param  const  $scope  Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
+     *                        local or global scoping of the cache item
+     * @return bool TRUE on success, FALSE on failure
      */
     public function delete($key, $scope = Cache::LOCAL_SCOPE)
     {
@@ -84,9 +84,9 @@ public function delete($key, $scope = Cache::LOCAL_SCOPE)
     /**
      * Clean the cache
      *
-     * @param  const  $scope	Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
-     *		local or global scoping of the cache item
-     * @return	bool	TRUE on success, FALSE on failure
+     * @param  const  $scope  Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
+     *                        local or global scoping of the cache item
+     * @return bool TRUE on success, FALSE on failure
      */
     public function clean($scope = Cache::LOCAL_SCOPE)
     {
@@ -100,7 +100,7 @@ public function clean($scope = Cache::LOCAL_SCOPE)
     /**
      * Cache Info
      *
-     * @return	mixed	array containing cache info on success OR FALSE on failure
+     * @return mixed array containing cache info on success OR FALSE on failure
      */
     public function cache_info()
     {
@@ -110,10 +110,10 @@ public function cache_info()
     /**
      * Get Cache Metadata
      *
-     * @param  string  $key	Key to get cache metadata on
-     * @param  const  $scope	Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
-     *		local or global scoping of the cache item
-     * @return	mixed	cache item metadata
+     * @param  string  $key  Key to get cache metadata on
+     * @param  const  $scope  Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE for
+     *                        local or global scoping of the cache item
+     * @return mixed cache item metadata
      */
     public function get_metadata($key, $scope = Cache::LOCAL_SCOPE)
     {
@@ -173,10 +173,10 @@ protected function unwrapData($data)
      *
      * For the file driver, namespaces will be actual folders
      *
-     * @param  string  $key	Key name
-     * @param  const  $scope	Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE
-     *		 for local or global scoping of the cache item
-     * @return	string	Key prefixed with namespace
+     * @param  string  $key  Key name
+     * @param  const  $scope  Cache::LOCAL_SCOPE or Cache::GLOBAL_SCOPE
+     *                        for local or global scoping of the cache item
+     * @return string Key prefixed with namespace
      */
     protected function _namespaced_key($key, $scope = Cache::LOCAL_SCOPE)
     {
diff --git a/src/CacheManager.php b/src/CacheManager.php
@@ -70,7 +70,7 @@ protected function accessRestrictedProperty($object, $property)
     /**
      * Returns the name of the adapter currently in use
      *
-     * @return	string	Name of adapter
+     * @return string Name of adapter
      */
     public function get_adapter()
     {
@@ -82,7 +82,7 @@ public function get_adapter()
      * Configuration screen, and also optionally an error message if the driver
      * selected cannot be used
      *
-     * @return	string	HTML dropdown and optional error message
+     * @return string HTML dropdown and optional error message
      */
     public function admin_setting()
     {
diff --git a/src/CoilpackServiceProvider.php b/src/CoilpackServiceProvider.php
@@ -2,7 +2,6 @@
 
 namespace Expressionengine\Coilpack;
 
-use Expressionengine\Coilpack\Auth\CoilpackUserProvider;
 use Expressionengine\Coilpack\Api\Graph\Support\FieldtypeRegistrar;
 use Illuminate\Support\Facades\App;
 use Illuminate\Support\Facades\Event;
diff --git a/src/Extensions.php b/src/Extensions.php
@@ -24,7 +24,7 @@ public function __construct($data = [])
      *
      * @param	string	Name of the  extension hook
      * @param	mixed
-     * @return	mixed
+     * @return mixed
      */
     public function call($which)
     {
@@ -64,7 +64,7 @@ public function call($which)
      * Check If Hook Has Activated Extension
      *
      * @param	string	Name of the  extension hook
-     * @return	bool
+     * @return bool
      */
     public function active_hook($which)
     {
diff --git a/src/Fieldtypes/Date.php b/src/Fieldtypes/Date.php
@@ -7,7 +7,7 @@
 
 class Date extends Generic
 {
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         return [
             new Parameter([
diff --git a/src/Fieldtypes/Duration.php b/src/Fieldtypes/Duration.php
@@ -7,7 +7,7 @@
 
 class Duration extends Generic
 {
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         return [
             new Parameter([
diff --git a/src/Fieldtypes/Fieldtype.php b/src/Fieldtypes/Fieldtype.php
@@ -36,7 +36,7 @@ public function withSettings($settings)
     /**
      * A list of supported parameters for the fieldtype
      */
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         return [];
     }
diff --git a/src/Fieldtypes/Grid.php b/src/Fieldtypes/Grid.php
@@ -55,7 +55,7 @@ public function generateGraphType(ChannelField $field)
         ]);
     }
 
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         $parameters = $this->presenter->defineParameters();
         $parameters[] = new Parameter([
diff --git a/src/Fieldtypes/Member.php b/src/Fieldtypes/Member.php
@@ -25,7 +25,7 @@ public function apply(FieldContent $content, $parameters = [])
         return FieldtypeOutput::for($this)->value($data);
     }
 
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         return $this->presenter->defineParameters();
     }
diff --git a/src/Fieldtypes/Number.php b/src/Fieldtypes/Number.php
@@ -7,7 +7,7 @@
 
 class Number extends Generic
 {
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         return [
             new Parameter([
diff --git a/src/Fieldtypes/OptionFieldtype.php b/src/Fieldtypes/OptionFieldtype.php
@@ -70,7 +70,7 @@ public function apply(FieldContent $content, array $parameters = [])
             ]);
     }
 
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         return [
             new Parameter([
diff --git a/src/Fieldtypes/Relationship.php b/src/Fieldtypes/Relationship.php
@@ -25,7 +25,7 @@ public function apply(FieldContent $content, $parameters = [])
         return FieldtypeOutput::for($this)->value($data);
     }
 
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         return $this->presenter->defineParameters();
     }
diff --git a/src/Fieldtypes/Rte.php b/src/Fieldtypes/Rte.php
@@ -7,7 +7,7 @@
 
 class Rte extends Generic
 {
-    public function parametersForField(Field $field = null): array
+    public function parametersForField(?Field $field = null): array
     {
         return [
             new Parameter([
diff --git a/src/Models/Content/ContentModel.php b/src/Models/Content/ContentModel.php
@@ -32,8 +32,8 @@ abstract class ContentModel extends VariableColumnModel
     /**
      * A link back to the owning Structure object.
      *
-     * @return	Structure	A link back to the Structure object that defines
-     *						this Content's structure.
+     * @return Structure A link back to the Structure object that defines
+     *                   this Content's structure.
      */
     abstract public function getStructure();
 
@@ -150,7 +150,7 @@ public function getCustomFieldNames()
     /**
      * Get the layout for this content.
      */
-    public function getDisplay(LayoutInterface $layout = null)
+    public function getDisplay(?LayoutInterface $layout = null)
     {
         $this->usesCustomFields();
 
@@ -172,7 +172,7 @@ function ($field) {
      * Safely updates any properties that might exist,
      * passing them through the getters along the way.
      *
-     * @param  array  $data Data to update
+     * @param  array  $data  Data to update
      * @return $this
      */
     public function set(array $data = [])
@@ -414,7 +414,7 @@ protected function addFacade($id, $info, $name_prefix = '')
      * tables. If the list of changed properties is not supplied we will get
      * the list of dirty properties.
      *
-     * @param  array  $changed An associative array of class properties that have changed
+     * @param  array  $changed  An associative array of class properties that have changed
      */
     public function saveFieldData($changed = null)
     {
diff --git a/src/Models/Content/Display/LayoutDisplay.php b/src/Models/Content/Display/LayoutDisplay.php
@@ -17,7 +17,7 @@ class LayoutDisplay
     /**
      * Adds a LayoutTab to the display
      *
-     * @param  LayoutTab  $tab The LayoutTab to add
+     * @param  LayoutTab  $tab  The LayoutTab to add
      * @return void
      */
     public function addTab(LayoutTab $tab)
@@ -28,7 +28,7 @@ public function addTab(LayoutTab $tab)
     /**
      * Sets the tabs (in bulk) to the display
      *
-     * @param  array  $tab An array of LayoutTabs
+     * @param  array  $tab  An array of LayoutTabs
      * @return void
      */
     public function setTabs(array $tabs)
@@ -41,7 +41,7 @@ public function setTabs(array $tabs)
     /**
      * Fetches a LayoutTab based on its tab id.
      *
-     * @param  mixed  $tab_id The id of the tab
+     * @param  mixed  $tab_id  The id of the tab
      * @return LayoutTab The requested LayoutTab
      *
      * @throws InvalidArgumentException When no tab with the given id is available
diff --git a/src/Models/Content/FieldModel.php b/src/Models/Content/FieldModel.php
diff --git a/src/Models/File/Column/LocalPath.php b/src/Models/File/Column/LocalPath.php
diff --git a/src/Models/Member/Member.php b/src/Models/Member/Member.php
diff --git a/src/Models/Site/Site.php b/src/Models/Site/Site.php
diff --git a/src/Routing/TemplateRoute.php b/src/Routing/TemplateRoute.php
diff --git a/src/Session/DatabaseSessionHandler.php b/src/Session/DatabaseSessionHandler.php
diff --git a/src/View/FormTag.php b/src/View/FormTag.php
diff --git a/src/View/Tags/Structure/Nav.php b/src/View/Tags/Structure/Nav.php
diff --git a/src/View/TemplateStub.php b/src/View/TemplateStub.php

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,6 @@ public function user()`
`47`	`47`	`return $this->user;`
`48`	`48`	`}`
`49`	`49`
`50`		`-`
`51`	`50`	`/**`
`52`	`51`	`* Remove the user data from the session and cookies.`
`53`	`52`	`*`
Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ protected function accessRestrictedProperty($object, $property)`
`70`	`70`	`/**`
`71`	`71`	`* Returns the name of the adapter currently in use`
`72`	`72`	`*`
`73`		`- * @return string Name of adapter`
	`73`	`+ * @return string Name of adapter`
`74`	`74`	`*/`
`75`	`75`	`public function get_adapter()`
`76`	`76`	`{`
`@@ -82,7 +82,7 @@ public function get_adapter()`
`82`	`82`	`* Configuration screen, and also optionally an error message if the driver`
`83`	`83`	`* selected cannot be used`
`84`	`84`	`*`
`85`		`- * @return string HTML dropdown and optional error message`
	`85`	`+ * @return string HTML dropdown and optional error message`
`86`	`86`	`*/`
`87`	`87`	`public function admin_setting()`
`88`	`88`	`{`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ public function __construct($data = [])`
`24`	`24`	`*`
`25`	`25`	`* @param string Name of the extension hook`
`26`	`26`	`* @param mixed`
`27`		`- * @return mixed`
	`27`	`+ * @return mixed`
`28`	`28`	`*/`
`29`	`29`	`public function call($which)`
`30`	`30`	`{`
`@@ -64,7 +64,7 @@ public function call($which)`
`64`	`64`	`* Check If Hook Has Activated Extension`
`65`	`65`	`*`
`66`	`66`	`* @param string Name of the extension hook`
`67`		`- * @return bool`
	`67`	`+ * @return bool`
`68`	`68`	`*/`
`69`	`69`	`public function active_hook($which)`
`70`	`70`	`{`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`
`8`	`8`	`class Date extends Generic`
`9`	`9`	`{`
`10`		`- public function parametersForField(Field $field = null): array`
	`10`	`+ public function parametersForField(?Field $field = null): array`
`11`	`11`	`{`
`12`	`12`	`return [`
`13`	`13`	`new Parameter([`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`
`8`	`8`	`class Duration extends Generic`
`9`	`9`	`{`
`10`		`- public function parametersForField(Field $field = null): array`
	`10`	`+ public function parametersForField(?Field $field = null): array`
`11`	`11`	`{`
`12`	`12`	`return [`
`13`	`13`	`new Parameter([`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ public function withSettings($settings)`
`36`	`36`	`/**`
`37`	`37`	`* A list of supported parameters for the fieldtype`
`38`	`38`	`*/`
`39`		`- public function parametersForField(Field $field = null): array`
	`39`	`+ public function parametersForField(?Field $field = null): array`
`40`	`40`	`{`
`41`	`41`	`return [];`
`42`	`42`	`}`