81100 refactor

packages/app/src/components/SearchPage.jsx

@@ -194,7 +194,7 @@ class SearchPage extends React.Component {
     else {
       selectedPagesIdList.add(pageId);
     }
-    switch (this.state.selectedPages.size) {
+    switch (selectedPagesIdList.size) {
       case 0:
         return this.setState({ selectAllCheckboxType: CheckboxType.NONE_CHECKED });
       case this.state.searchedPages.length:
@@ -207,7 +207,7 @@ class SearchPage extends React.Component {
   toggleAllCheckBox = (nextSelectAllCheckboxType) => {
     const { selectedPagesIdList, searchedPages } = this.state;
     if (nextSelectAllCheckboxType === CheckboxType.NONE_CHECKED) {
-      this.state.selectedPages.clear();
+      selectedPagesIdList.clear();
     }
     else {
       searchedPages.forEach((page) => {

packages/app/src/components/SearchPage/SearchResultList.tsx

@@ -16,7 +16,7 @@ type Props = {
   pages: ISearchedPage[],
   selectedPagesIdList: Set<string>
   onClickSearchResultItem?: (pageId: string) => void,
-  onClickCheckbox?: (page: string) => void,
+  onClickCheckbox?: (pageId: string) => void,
   searchResultCount?: number,
   activePage?: number,
   pagingLimit?: number,

packages/app/src/components/SearchPage/SearchResultListItem.tsx

@@ -70,7 +70,7 @@ type Props = {
   page: ISearchedPage,
   isSelected: boolean,
   isChecked: boolean,
-  onClickCheckbox?: (page: string) => void,
+  onClickCheckbox?: (pageId: string) => void,
   onClickSearchResultItem?: (pageId: string) => void,
 }