公告错误：Matery theme的公告为空时依旧显示公告栏

[ferreroxc]

描述bug
简单说明bug的现象、相关的错误提示、日志等

• 我的Notion数据库中没有公告=>没有type为Notice的数据行
• 根据/themes/matery/components/Announcement.js，当notice为空时，应当返回<></>，也就是不显示公告栏
  https://github.com/tangly1024/NotionNext/blob/2e7aa0b21e62cd2ae80a175f13ef7f29a191d0d4/themes/matery/components/Announcement.js#LL6C1-L10C4

const Announcement = ({ notice }) => {
  const { locale } = useGlobal()
  if (!notice) {
    return <></>
  }

• 但是页面依旧显示了公告栏

复现步骤
出现这个bug的操作步骤

• 创建一个不含type=Notice的Notion数据库
• 使用matery theme
• yarn dev，去localhost:3000查看网页
• 看到公告栏

期望的正常结果
希望按这个步骤，正常操作结果是什么

• 看不到公告栏

截图
相关的页面，应该的结果

环境

• 操作系统: [例如. iOS, Android, macOS, windows] Mac OS 13.4 (22F66)
• 浏览器 [例如. chrome, safari, firefox] chrome
• NotionNext版本 3.13.6

补充说明
与问题相关的其它说明

问题分析

notice是{}

const Announcement = ({ notice }) => {
  const { locale } = useGlobal()
  if (!notice) {
    return <></>
  }

在这段代码里

• notice为{}，是一个empty object
• => !notice为false（判断empty object需要用其他方法）
• => if不成立，return了公告栏的div

但如果

• notice为null
• => !notice为true
• => if成立，return空，公告栏不显示

为什么notice是{}而不是null

在lib/notion/getNotionData.js中的deepClone把null变成了{}

NotionNext/lib/notion/getNotionData.js

Line 28 in 6357aa8

const notionPageData = deepClone(await getNotionPageData({ pageId, from }))

NotionNext/lib/utils.js

Line 89 in 6357aa8

export function deepClone(obj) {

解决思路

1. 在/themes/matery/components/Announcement.js里面把if(!notice)改成判断object为空的方法
2. 在deepClone里面把newObj = Array.isArray(obj) ? [] : {}改成newObj = Array.isArray(obj) ? [] : null，但不知道这样会不会影响到其他代码逻辑

期待大家评论哪个解决方法更好，请 @tangly1024 把这个issue assign给我吧诶嘿

[tangly1024]

感谢 @ferreroxc 对此问题的认真讨论❤️

如您所言，两个方法都能解决问题。个人偏向第二种，没有公告就是null，而不返回一个{}对象。

代价是会影响代码逻辑的影响，这个影响也许不会很大，原因是绝大大部分的判断我都是用 if (!object) ，而非Object.keys(object).length等方法。