Merge pull request #86 from lode/upgrade-php-features

Author: lode

.github/workflows/lint.yml

@@ -1,6 +1,7 @@
 name: lint
 on:
   pull_request:
+    types: [opened, synchronize, reopened, ready_for_review]
     paths:
       - "**.php"
 

.github/workflows/test.yml

@@ -1,6 +1,7 @@
 name: test
 on:
   pull_request:
+    types: [opened, synchronize, reopened, ready_for_review]
     paths:
       - "**.php"
 

README.md

@@ -32,7 +32,7 @@ If you used v1 of this library, see [UPGRADE_1_TO_2.md](/UPGRADE_1_TO_2.md) on h
 ```php
 use alsvanzelf\jsonapi\ResourceDocument;
 
-$document = new ResourceDocument($type='user', $id=42);
+$document = new ResourceDocument(type: 'user', id: 42);
 $document->add('name', 'Zaphod Beeblebrox');
 $document->add('heads', 2);
 $document->sendResponse();

examples/collection.php

@@ -16,7 +16,7 @@
 $collection = [];
 
 foreach ($users as $user) {
-	$resource = ResourceObject::fromObject($user, $type='user', $user->id);
+	$resource = ResourceObject::fromObject($user, type: 'user', id: $user->id);
 
 	if ($user->id == 42) {
 		$ship = new ResourceObject('ship', 5);

examples/collection_canonical.php

@@ -54,7 +54,7 @@
 }
 
 $document->setSelfLink('http://example.com/articles');
-$document->setPaginationLinks($previous=null, $next='http://example.com/articles?page[offset]=2', $first=null, $last='http://example.com/articles?page[offset]=10');
+$document->setPaginationLinks(previousHref: null, nextHref: 'http://example.com/articles?page[offset]=2', firstHref: null, lastHref: 'http://example.com/articles?page[offset]=10');
 $document->unsetJsonapiObject();
 
 /**

examples/cursor_pagination_profile.php

@@ -25,8 +25,8 @@
 $document = CollectionDocument::fromResources($user1, $user2, $user42);
 $document->applyProfile($profile);
 
-$profile->setCount($document, $exactTotal=3, $bestGuessTotal=10);
-$profile->setLinksFirstPage($document, $currentUrl='/users?sort=42&page[size]=10', $lastCursor='zaphod');
+$profile->setCount($document, exactTotal: 3, bestGuessTotal: 10);
+$profile->setLinksFirstPage($document, baseOrCurrentUrl: '/users?sort=42&page[size]=10', lastCursor: 'zaphod');
 
 /**
  * get the json

examples/errors_all_options.php

@@ -11,37 +11,37 @@
  * setting all options
  */
 
-$errorHumanApi = new ErrorObject($genericCode='Invalid input', $genericTitle='Too much options', $specificDetails='Please, choose a bit less. Consult your ...', $specificAboutLink='https://www.example.com/explanation.html', $genericTypeLink='https://www.example.com/documentation.html');
+$errorHumanApi = new ErrorObject(genericCode: 'Invalid input', genericTitle: 'Too much options', specificDetails: 'Please, choose a bit less. Consult your ...', specificAboutLink: 'https://www.example.com/explanation.html', genericTypeLink: 'https://www.example.com/documentation.html');
 
 $errorSpecApi = new ErrorObject();
 
 // mark the cause of the error
-$errorSpecApi->blameJsonPointer($pointer='/data/attributes/title');
-$errorSpecApi->blameQueryParameter($parameter='filter');
-$errorSpecApi->blameHeader($headerName='X-Foo');
+$errorSpecApi->blameJsonPointer(pointer: '/data/attributes/title');
+$errorSpecApi->blameQueryParameter(parameter: 'filter');
+$errorSpecApi->blameHeader(headerName: 'X-Foo');
 
 // an identifier useful for helpdesk purposes
-$errorSpecApi->setUniqueIdentifier($id=42);
+$errorSpecApi->setUniqueIdentifier(id: 42);
 
 // add meta data as you would on a normal json response
-$errorSpecApi->addMeta($key='foo', $value='bar');
+$errorSpecApi->addMeta(key: 'foo', value: 'bar');
 
 // or as object
 $metaObject = new \stdClass();
 $metaObject->property = 'value';
-$errorSpecApi->addMeta($key='object', $metaObject);
+$errorSpecApi->addMeta(key: 'object', value: $metaObject);
 
 // the http status code
 // @note it is better to set this on the jsonapi\errors object ..
 //       .. as only a single one can be consumed by the browser
-$errorSpecApi->setHttpStatusCode($httpStatusCode=404);
+$errorSpecApi->setHttpStatusCode(httpStatusCode: 404);
 
 // if not set during construction, set them here
-$errorSpecApi->setApplicationCode($genericCode='Invalid input');
-$errorSpecApi->setHumanTitle($genericTitle='Too much options');
-$errorSpecApi->setHumanDetails($specificDetails='Please, choose a bit less. Consult your ...');
-$errorSpecApi->setAboutLink($specificAboutLink='https://www.example.com/explanation.html', ['foo'=>'bar']);
-$errorSpecApi->setTypeLink($genericTypeLink='https://www.example.com/documentation.html', ['foo'=>'bar']);
+$errorSpecApi->setApplicationCode(genericCode: 'Invalid input');
+$errorSpecApi->setHumanTitle(genericTitle: 'Too much options');
+$errorSpecApi->setHumanDetails(specificDetails: 'Please, choose a bit less. Consult your ...');
+$errorSpecApi->setAboutLink(href: 'https://www.example.com/explanation.html', meta: ['foo'=>'bar']);
+$errorSpecApi->setTypeLink(href: 'https://www.example.com/documentation.html', meta: ['foo'=>'bar']);
 
 /**
  * prepare multiple error objects for the errors response
@@ -68,7 +68,7 @@
 $document->addErrorObject($errorSpecApi);
 $document->addErrorObject($anotherError);
 $document->addException($someException);
-$document->add($genericCode='Authentication error', $genericTitle='Not logged in');
+$document->add(genericCode: 'Authentication error', genericTitle: 'Not logged in');
 $document->addLink('redirect', '/login', ['label'=>'Log in']);
 
 $document->setHttpStatusCode(400);

examples/extension.php

@@ -30,7 +30,7 @@
  * get the json
  */
 
-$contentType = Converter::prepareContentType(ContentTypeEnum::Official, [$extension], []);
+$contentType = Converter::prepareContentType(ContentTypeEnum::Official, extensions: [$extension], profiles: []);
 echo '<code>Content-Type: '.$contentType.'</code>'.PHP_EOL;
 
 $options = [

examples/output.php

@@ -16,7 +16,7 @@
 
 echo '<h2>Get the array</h2>';
 echo '<pre style="font-size: large;">$document->toArray();</pre>';
-echo '<pre>'.var_export($document->toArray(), true).'</pre>';
+echo '<pre>'.var_export($document->toArray(), return: true).'</pre>';
 
 /**
  * get the json
@@ -25,16 +25,16 @@
 $options = ['prettyPrint' => true];
 echo '<h2>Get the json</h2>';
 echo '<pre style="font-size: large;">$document->toJson();</pre>';
-echo '<pre>'.var_export($document->toJson($options), true).'</pre>';
+echo '<pre>'.var_export($document->toJson($options), return: true).'</pre>';
 
 /**
  * use own json_encode
  */
 
 $options = ['prettyPrint' => true];
 echo '<h2>Use own <code>json_encode()</code></h2>';
-echo '<pre style="font-size: large;">json_encode($document, JSON_PRETTY_PRINT);</pre>';
-echo '<pre>'.var_export(json_encode($document, JSON_PRETTY_PRINT), true).'</pre>';
+echo '<pre style="font-size: large;">json_encode($document, flags: JSON_PRETTY_PRINT);</pre>';
+echo '<pre>'.var_export(json_encode($document, flags: JSON_PRETTY_PRINT), return: true).'</pre>';
 
 /**
  * get custom json (for a non-spec array)
@@ -51,7 +51,7 @@
 echo '$options = [\'array\' => $customArray];'.PHP_EOL;
 echo '$document->toJson($options);'.PHP_EOL;
 echo '</pre>';
-echo '<pre>'.var_export($document->toJson($options), true).'</pre>';
+echo '<pre>'.var_export($document->toJson($options), return: true).'</pre>';
 
 /**
  * get jsonp with callback
@@ -63,7 +63,7 @@
 echo '$options = [\'jsonpCallback\' => \'callback\'];'.PHP_EOL;
 echo '$document->toJson($options);'.PHP_EOL;
 echo '</pre>';
-echo '<pre>'.var_export($document->toJson($options), true).'</pre>';
+echo '<pre>'.var_export($document->toJson($options), return: true).'</pre>';
 
 /**
  * send json response

examples/profile.php

@@ -32,7 +32,7 @@
  * get the json
  */
 
-$contentType = Converter::prepareContentType(ContentTypeEnum::Official, [], [$profile]);
+$contentType = Converter::prepareContentType(ContentTypeEnum::Official, extensions: [], profiles: [$profile]);
 echo '<code>Content-Type: '.$contentType.'</code>'.PHP_EOL;
 
 $options = [