Revert "fix(core): make all docs journal judgement reactive (#9290)"

This reverts commit 727f21d.

packages/frontend/core/src/components/hooks/use-block-suite-page-meta.ts

@@ -1,12 +1,12 @@
 import { DocsService } from '@affine/core/modules/doc';
-import { JournalService } from '@affine/core/modules/journal';
 import { WorkspaceService } from '@affine/core/modules/workspace';
 import type { DocCollection, DocMeta } from '@blocksuite/affine/store';
-import { LiveData, useLiveData, useService } from '@toeverything/infra';
+import { useService } from '@toeverything/infra';
 import { useCallback, useMemo } from 'react';
 
 import { useAsyncCallback } from './affine-async-hooks';
 import { useAllBlockSuiteDocMeta } from './use-all-block-suite-page-meta';
+import { useJournalInfoHelper } from './use-journal';
 
 /**
  * Get pageMetas excluding journal pages without updatedDate
@@ -15,26 +15,13 @@ import { useAllBlockSuiteDocMeta } from './use-all-block-suite-page-meta';
  */
 export function useBlockSuiteDocMeta(docCollection: DocCollection) {
   const pageMetas = useAllBlockSuiteDocMeta(docCollection);
-  const journalService = useService(JournalService);
-
-  const journalIds = useLiveData(
-    useMemo(
-      () =>
-        LiveData.computed(get =>
-          pageMetas
-            .map(meta => meta.id)
-            .filter(id => !!get(journalService.journalDate$(id)))
-        ),
-      [pageMetas, journalService]
-    )
-  );
-
+  const { isPageJournal } = useJournalInfoHelper();
   return useMemo(
     () =>
       pageMetas.filter(
-        pageMeta => !journalIds.includes(pageMeta.id) || !!pageMeta.updatedDate
+        pageMeta => !isPageJournal(pageMeta.id) || !!pageMeta.updatedDate
       ),
-    [journalIds, pageMetas]
+    [isPageJournal, pageMetas]
   );
 }
 

tests/affine-local/e2e/all-page.spec.ts

@@ -1,3 +1,4 @@
+/* eslint-disable unicorn/prefer-dom-node-dataset */
 import { test } from '@affine-test/kit/playwright';
 import {
   changeFilter,
@@ -19,12 +20,7 @@ import {
   getBlockSuiteEditorTitle,
   waitForAllPagesLoad,
   waitForEditorLoad,
-  waitForEmptyEditor,
 } from '@affine-test/kit/utils/page-logic';
-import {
-  ensurePagePropertiesVisible,
-  togglePropertyListVisibility,
-} from '@affine-test/kit/utils/properties';
 import { clickSideBarAllPageButton } from '@affine-test/kit/utils/sidebar';
 import { expect } from '@playwright/test';
 
@@ -290,12 +286,10 @@ test('select a group of items by clicking "Select All" in group header', async (
 
   const selectedItemCount = await page
     .locator('[data-testid="page-list-group-header"]')
-    // oxlint-disable-next-line
     .getAttribute('data-group-selected-items-count');
 
   const selectedGroupItemTotalCount = await page
     .locator('[data-testid="page-list-group-header"]')
-    // oxlint-disable-next-line
     .getAttribute('data-group-items-count');
   expect(selectedItemCount).toBe(selectedGroupItemTotalCount);
 
@@ -463,29 +457,3 @@ test('create a tag and delete it', async ({ page }) => {
   const newCell = page.getByTestId('tag-list-item').getByText('test-tag');
   await expect(newCell).not.toBeVisible();
 });
-
-test('create a empty page and turn it into journal, should disappear in all docs', async ({
-  page,
-}) => {
-  await openHomePage(page);
-  await waitForEditorLoad(page);
-  await clickNewPageButton(page);
-  await clickSideBarAllPageButton(page);
-  const docItem = page
-    .locator('[data-testid="page-list-item-title-text"]')
-    .first();
-  const docTitle = await docItem.textContent();
-  await expect(docTitle).toBe('Untitled');
-  // open and turn it into journal
-  await docItem.click();
-  await waitForEmptyEditor(page);
-  await ensurePagePropertiesVisible(page);
-  await togglePropertyListVisibility(page);
-  const journalProperty = page.locator('[data-info-id="journal"]').first();
-  await journalProperty.locator('[data-property-value="true"]').click();
-  // back to all docs, the journal page should disappear
-  await clickSideBarAllPageButton(page);
-  await waitForAllPagesLoad(page);
-  const newDocTitle = await docItem.textContent();
-  await expect(newDocTitle).not.toBe(docTitle);
-});