fix: Load random page (#3229)

* fix: Load random page

* fix: refactor loadRandomPage method

* fix: notEnoughItemsToAskMore condition

Author: cyaiox

src/components/ItemEditorPage/LeftPanel/Items/Items.tsx

@@ -278,7 +278,7 @@ export default class Items extends React.PureComponent<Props, State> {
     const { isLoading, totalItems, reviewedItems } = this.props
     const { currentTab } = this.state
     let headerInnerContent
-    const notEnoughItemsToAskMore = !!totalItems && (totalItems < LEFT_PANEL_PAGE_SIZE || reviewedItems.length === totalItems)
+    const notEnoughItemsToAskMore = !totalItems || totalItems < LEFT_PANEL_PAGE_SIZE || reviewedItems.length === totalItems
 
     switch (currentTab) {
       case ItemPanelTabs.TO_REVIEW:

src/components/ItemEditorPage/LeftPanel/LeftPanel.tsx

@@ -151,23 +151,21 @@ export default class LeftPanel extends React.PureComponent<Props, State> {
   loadRandomPage = (currentItems: Item[]) => {
     const { pages } = this.state
     const { totalItems, totalCollections, onSetReviewedItems } = this.props
+
     const totalResources = this.isCollectionTabActive() ? totalCollections : totalItems
     const totalPages = Math.ceil(totalResources! / LEFT_PANEL_PAGE_SIZE)
-    if (pages.length !== totalPages) {
-      let randomPage: number | undefined
-      while (randomPage === undefined) {
-        randomPage = this.getRandomPage(1, totalPages)
-        if (pages.includes(randomPage)) {
-          randomPage = undefined
-        }
-      }
-      onSetReviewedItems(currentItems)
-      if (randomPage !== undefined) {
-        this.setState(prevState => ({ pages: [...prevState.pages, randomPage as number] }), this.fetchResource)
+
+    if (totalPages > pages.length) {
+      const availablePages = [...Array(totalPages).keys()].map(i => i + 1).filter(page => !pages.includes(page))
+
+      if (availablePages.length > 0) {
+        const randomPage = availablePages[Math.floor(Math.random() * availablePages.length)]
+
+        this.setState(prevState => ({ pages: [...prevState.pages, randomPage] }), this.fetchResource)
       }
-    } else {
-      onSetReviewedItems(currentItems)
     }
+
+    onSetReviewedItems(currentItems)
   }
 
   handleTabChange = (tab: ItemEditorTabs) => {