From c168bb5201b955ef7ab7a92c703a3b155647a2fa Mon Sep 17 00:00:00 2001 From: Elliot DeNolf Date: Sun, 13 Feb 2022 08:57:31 -0500 Subject: [PATCH] fix: removing onlyNameIfPublic hook, causes confusion in other hooks --- src/templates/js-blog/collections/Users.js | 11 ----------- src/templates/ts-blog/src/collections/Users.ts | 13 +------------ 2 files changed, 1 insertion(+), 23 deletions(-) diff --git a/src/templates/js-blog/collections/Users.js b/src/templates/js-blog/collections/Users.js index b66c201da4..cbb85230f3 100644 --- a/src/templates/js-blog/collections/Users.js +++ b/src/templates/js-blog/collections/Users.js @@ -1,11 +1,3 @@ -const onlyNameIfPublic = ({ req: { user }, doc }) => { - // Only return name if not logged in - if (!user) { - return { name: doc.name }; - } - return doc; -}; - const Users = { slug: 'users', auth: true, @@ -15,9 +7,6 @@ const Users = { access: { read: () => true, }, - hooks: { - beforeRead: [onlyNameIfPublic] - }, fields: [ // Email added by default { diff --git a/src/templates/ts-blog/src/collections/Users.ts b/src/templates/ts-blog/src/collections/Users.ts index ab110286d7..419974d410 100644 --- a/src/templates/ts-blog/src/collections/Users.ts +++ b/src/templates/ts-blog/src/collections/Users.ts @@ -1,12 +1,4 @@ -import { CollectionBeforeReadHook, CollectionConfig } from 'payload/types'; - -const onlyNameIfPublic: CollectionBeforeReadHook = ({ req: { user }, doc }) => { - // Only return name if not logged in - if (!user) { - return { name: doc.name }; - } - return doc; -}; +import { CollectionConfig } from 'payload/types'; const Users: CollectionConfig = { slug: 'users', @@ -17,9 +9,6 @@ const Users: CollectionConfig = { access: { read: () => true, }, - hooks: { - beforeRead: [onlyNameIfPublic] - }, fields: [ // Email added by default {