use Array.prototype.some() instead of ...map().includes()

Author: yiyione

rest_server/src/controllers/item_controller.js

@@ -16,9 +16,8 @@ async function checkReadPermission(userInfo, item, categories) {
   if (categories === undefined) {
     categories = await MarketplaceItem.getCategories(item);
   }
-  const categoriesArray = categories.map(category => category.name);
   if (
-    !categoriesArray.includes(OFFICIAL_EXAMPLE) &&
+    !categories.some(category => category.name === OFFICIAL_EXAMPLE) &&
     userInfo.username === item.author
   ) {
     return true;
@@ -43,9 +42,8 @@ async function checkWritePermission(tokenInfo, item, categories) {
   if (categories === undefined) {
     categories = await MarketplaceItem.getCategories(item);
   }
-  const categoriesArray = categories.map(category => category.name);
   if (
-    !categoriesArray.includes(OFFICIAL_EXAMPLE) &&
+    !categories.some(category => category.name === OFFICIAL_EXAMPLE) &&
     tokenInfo.username === item.author
   ) {
     return true;