@secretstache/wordpress-gutenberg 0.3.12 → 0.3.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@secretstache/wordpress-gutenberg",
-  "version": "0.3.12",
+  "version": "0.3.14",
   "description": "",
   "author": "Secret Stache",
   "license": "GPL-2.0-or-later",

package/src/components/MediaControl.js

@@ -2,7 +2,7 @@ import { Button, Icon as WPIcon } from '@wordpress/components';
 import { MediaUpload, MediaUploadCheck } from '@wordpress/block-editor';
 import { page as pageIcon } from '@wordpress/icons';
 
-import { MEDIA_TYPES } from '../utils/index.js';
+import { MEDIA_TYPE } from '../utils/index.js';
 
 export const ImageRenderer = ({
     imageId,
@@ -115,12 +115,12 @@ export const MediaControl = ({
     mediaFileName = '',
     onSelect,
     onRemove,
-    type = MEDIA_TYPES.IMAGE,
+    type = MEDIA_TYPE.IMAGE,
     selectButtonLabel,
     removeButtonLabel,
     ...other
 }) => {
-    if (type === MEDIA_TYPES.IMAGE) {
+    if (type === MEDIA_TYPE.IMAGE) {
         return (
             <MediaUploadCheck>
                 <MediaUpload
@@ -143,7 +143,7 @@ export const MediaControl = ({
                 />
             </MediaUploadCheck>
         );
-    } else if (type === MEDIA_TYPES.VIDEO) {
+    } else if (type === MEDIA_TYPE.VIDEO) {
         return (
             <MediaUploadCheck>
                 <MediaUpload
@@ -164,7 +164,7 @@ export const MediaControl = ({
                 />
             </MediaUploadCheck>
         );
-    } else if (type === MEDIA_TYPES.ANIMATION) {
+    } else if (type === MEDIA_TYPE.ANIMATION) {
         return (
             <MediaUploadCheck>
                 <MediaUpload

package/src/components/MediaTypeControl.js

@@ -2,11 +2,11 @@ import { useMemo } from '@wordpress/element';
 import { SelectControl } from '@wordpress/components';
 
 import { MediaControl } from './MediaControl.js';
-import { MEDIA_TYPE_LABELS, MEDIA_TYPES } from '../utils/index.js';
+import { MEDIA_TYPE_LABEL, MEDIA_TYPE } from '../utils/index.js';
 
 export const MediaTypeControl = (props) => {
     const {
-        types = [ MEDIA_TYPES.IMAGE, MEDIA_TYPES.VIDEO ],
+        types = [ MEDIA_TYPE.IMAGE, MEDIA_TYPE.VIDEO ],
 
         selectedType,
         onTypeChange,
@@ -22,9 +22,9 @@ export const MediaTypeControl = (props) => {
     } = props;
 
     const typesOptions = useMemo(() => types
-        ?.filter((type) => MEDIA_TYPE_LABELS[type]) // Ensure it's an allowed type
+        ?.filter((type) => MEDIA_TYPE_LABEL[type]) // Ensure it's an allowed type
         ?.map((type) => ({
-            label: MEDIA_TYPE_LABELS[type],
+            label: MEDIA_TYPE_LABEL[type],
             value: type,
         }))
     , [ types ]);

package/src/hooks/useDataQuery.js

@@ -1,7 +1,9 @@
 import { useSelect } from '@wordpress/data';
-import { QUERY_TYPES } from '../utils';
+import { QUERY_TYPE } from '../utils';
 
-// TODO: numberOfPosts -1 is not a valid value, the API requires the per_page to be >= 1
+// TODO: 1. numberOfPosts -1 is not a valid value, the API requires the per_page to be >= 1
+// TODO: 2. make 'dependencies' the separate second argument
+// TODO: 3. consider supporting the string value for the getPostType as well as function
 export const useDataQuery = (props) => {
     const {
         getPostType,
@@ -25,9 +27,9 @@ export const useDataQuery = (props) => {
             ...extraQueryArgs,
         };
 
-        if (queryType === QUERY_TYPES.BY_CATEGORY && categoriesTaxonomy && curatedCategoriesIds?.length > 0) {
+        if (queryType === QUERY_TYPE.BY_CATEGORY && categoriesTaxonomy && curatedCategoriesIds?.length > 0) {
             queryArgs[categoriesTaxonomy] = curatedCategoriesIds.join(',');
-        } else if (queryType === QUERY_TYPES.CURATED && curatedPostsIds?.length > 0) {
+        } else if (queryType === QUERY_TYPE.CURATED && curatedPostsIds?.length > 0) {
             queryArgs['include'] = curatedPostsIds;
             queryArgs['orderby'] = 'include';
         }

package/src/utils/attributes.js

@@ -144,11 +144,9 @@ export const getBaseBackgroundAttributes = ({
     const backgroundMediaAttribute = {
         isIncludeBackgroundMedia: {
             type: 'boolean',
-            default: null,
         },
         backgroundMediaType: {
             type: 'string',
-            default: null,
         },
         ...getMediaAttribute(mediaAttributeName),
     };
@@ -156,18 +154,15 @@ export const getBaseBackgroundAttributes = ({
     const overlayAttribute = {
         isIncludeOverlay: {
             type: 'boolean',
-            default: null,
         },
         overlayColor: {
             type: 'object',
-            default: null,
         },
     };
 
     return {
         backgroundColor: {
             type: 'object',
-            default: null,
         },
 
         ...(hasBackgroundMedia ? backgroundMediaAttribute : {}),

package/src/utils/constants.js

@@ -1,17 +1,17 @@
-export const QUERY_TYPES = {
+export const QUERY_TYPE = {
     LATEST: 'latest',
     CURATED: 'curated',
     BY_CATEGORY: 'by_category'
 };
 
-export const MEDIA_TYPES = {
+export const MEDIA_TYPE = {
     IMAGE: 'image',
     VIDEO: 'video',
     ANIMATION: 'animation',
 };
 
-export const MEDIA_TYPE_LABELS = {
-    [MEDIA_TYPES.IMAGE]: 'Image',
-    [MEDIA_TYPES.VIDEO]: 'Video',
-    [MEDIA_TYPES.ANIMATION]: 'Animation',
+export const MEDIA_TYPE_LABEL = {
+    [MEDIA_TYPE.IMAGE]: 'Image',
+    [MEDIA_TYPE.VIDEO]: 'Video',
+    [MEDIA_TYPE.ANIMATION]: 'Animation',
 };