cleverogre/ogre-issuu
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Improve args parsing to remove unnecessary defaults.

Browse Source
This commit is contained in:
dcooperdalrymple
2024-09-12 17:30:42 -05:00
parent 19c01f60ad
commit df844495b1
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
includes/api/class-publications.php
View File

@@ -21,20 +21,29 @@ class Publications {
$this->issuu = $issuu;
}
private static function parse_args(array $args, array $defaults):array {
return array_filter(
wp_parse_args($args, $defaults),
function ($value, $key) use ($defaults) {
return $value !== $defaults[$key];
},
ARRAY_FILTER_USE_BOTH
);
}
public function getPublicationAssetsBySlug(string $slug, array $args = []):stdClass {
$args = wp_parse_args($args, [
$args = self::parse_args($args, [
'assetType' => '',
'size' => 10,
'page' => 1,
'documentPageNumber' => 0.0,
]);
if (!$args['documentPageNumber']) unset($args['documentPageNumber']);
return $this->issuu->getResponse(method: 'GET', endpoint: "publications/{$slug}/assets", queryParameters: $args);
}
public function getPublicationEmbedBySlug(string $slug, array $args = []):stdClass {
$args = wp_parse_args($args, [
$args = self::parse_args($args, [
'responsive' => true,
'width' => '100%',
'height' => '100%',