fix: Most active Places Sort by

src/entities/Place/model.ts

@@ -161,9 +161,7 @@ export default class PlaceModel extends Model<PlaceAttributes> {
     )
 
     const filterMostActivePlaces =
-      options.order_by === PlaceListOrderBy.MOST_ACTIVE &&
-      !!options.hotScenesPositions &&
-      options.hotScenesPositions.length > 0
+      !!options.hotScenesPositions && options.hotScenesPositions.length > 0
 
     const sql = SQL`
       ${conditional(

src/entities/Place/routes/getPlace.test.ts

@@ -5,8 +5,8 @@ import { Request } from "decentraland-gatsby/dist/entities/Route/wkc/request/Req
 import { hotSceneGenesisPlaza } from "../../../__data__/hotSceneGenesisPlaza"
 import { placeGenesisPlazaWithAggregatedAttributes } from "../../../__data__/placeGenesisPlazaWithAggregatedAttributes"
 import { sceneStatsGenesisPlaza } from "../../../__data__/sceneStatsGenesisPlaza"
-import DataTeam from "../../../api/DataTeam"
-import * as hotScenesModule from "../../../modules/hotScenes"
+import * as sceneStatsModule from "../../SceneStats/utils"
+import * as hotScenesModule from "../../RealmProvider/utils"
 import PlaceCategories from "../../PlaceCategories/model"
 import PlaceModel from "../model"
 import { getPlace } from "./getPlace"
@@ -15,11 +15,12 @@ const place_id = randomUUID()
 const findOne = jest.spyOn(PlaceModel, "namedQuery")
 const catalystHotScenes = jest.spyOn(hotScenesModule, "getHotScenes")
 const findPC = jest.spyOn(PlaceCategories, "namedQuery")
-const catalystSceneStats = jest.spyOn(DataTeam.get(), "getSceneStats")
+const catalystSceneStats = jest.spyOn(sceneStatsModule, "getSceneStats")
 
 afterEach(() => {
   findOne.mockReset()
   catalystHotScenes.mockReset()
+  catalystSceneStats.mockReset()
   findPC.mockReset()
 })
 test("should return 400 when UUID is incorrect", async () => {

src/entities/Place/routes/getPlace.ts

@@ -6,8 +6,8 @@ import Response from "decentraland-gatsby/dist/entities/Route/wkc/response/Respo
 import Router from "decentraland-gatsby/dist/entities/Route/wkc/routes/Router"
 import { bool } from "decentraland-gatsby/dist/entities/Schema/utils"
 
-import { getHotScenes } from "../../../modules/hotScenes"
-import { getSceneStats } from "../../../modules/sceneStats"
+import { getHotScenes } from "../../RealmProvider/utils"
+import { getSceneStats } from "../../SceneStats/utils"
 import PlaceModel from "../model"
 import { getPlaceParamsSchema } from "../schemas"
 import { AggregatePlaceAttributes, GetPlaceParams } from "../types"

src/entities/Place/routes/getPlaceList.test.ts

@@ -3,18 +3,19 @@ import { Request } from "decentraland-gatsby/dist/entities/Route/wkc/request/Req
 import { hotSceneGenesisPlaza } from "../../../__data__/hotSceneGenesisPlaza"
 import { placeGenesisPlazaWithAggregatedAttributes } from "../../../__data__/placeGenesisPlazaWithAggregatedAttributes"
 import { sceneStatsGenesisPlaza } from "../../../__data__/sceneStatsGenesisPlaza"
-import DataTeam from "../../../api/DataTeam"
-import * as hotScenesModule from "../../../modules/hotScenes"
+import * as sceneStatsModule from "../../SceneStats/utils"
+import * as hotScenesModule from "../../RealmProvider/utils"
 import PlaceModel from "../model"
 import { getPlaceList } from "./getPlaceList"
 
 const find = jest.spyOn(PlaceModel, "namedQuery")
 const catalystHotScenes = jest.spyOn(hotScenesModule, "getHotScenes")
-const catalystSceneStats = jest.spyOn(DataTeam.get(), "getSceneStats")
+const catalystSceneStats = jest.spyOn(sceneStatsModule, "getSceneStats")
 
 afterEach(() => {
   find.mockReset()
   catalystHotScenes.mockReset()
+  catalystSceneStats.mockReset()
 })
 
 test("should return a list of places with no query", async () => {

src/entities/Place/routes/getPlaceList.ts

@@ -8,8 +8,8 @@ import {
   oneOf,
 } from "decentraland-gatsby/dist/entities/Schema/utils"
 
-import { getHotScenes } from "../../../modules/hotScenes"
-import { getSceneStats } from "../../../modules/sceneStats"
+import { getHotScenes } from "../../RealmProvider/utils"
+import { getSceneStats } from "../../SceneStats/utils"
 import PlaceModel from "../model"
 import { getPlaceListQuerySchema } from "../schemas"
 import {

src/entities/Place/routes/getPlaceMostActiveList.ts

@@ -9,8 +9,8 @@ import {
 } from "decentraland-gatsby/dist/entities/Schema/utils"
 import { flat, sort } from "radash"
 
-import { getHotScenes } from "../../../modules/hotScenes"
-import { getSceneStats } from "../../../modules/sceneStats"
+import { getHotScenes } from "../../RealmProvider/utils"
+import { getSceneStats } from "../../SceneStats/utils"
 import PlaceModel from "../model"
 import { FindWithAggregatesOptions, PlaceListOrderBy } from "../types"
 import { placesWithUserCount, placesWithUserVisits } from "../utils"
@@ -60,7 +60,7 @@ export const getPlaceMostActiveList = Router.memo(
       only_highlighted: !!bool(query.only_highlighted),
       positions: query.positions,
       hotScenesPositions: hotScenesPositions,
-      order_by: PlaceListOrderBy.MOST_ACTIVE,
+      order_by: PlaceListOrderBy.UPDATED_AT,
       order: query.order,
       search: query.search,
       categories: query.categories,

src/entities/Place/routes/getPlaceUserVisitsList.ts

@@ -5,8 +5,8 @@ import Router from "decentraland-gatsby/dist/entities/Route/wkc/routes/Router"
 import { bool, numeric } from "decentraland-gatsby/dist/entities/Schema/utils"
 import { sort } from "radash"
 
-import { getHotScenes } from "../../../modules/hotScenes"
-import { getSceneStats } from "../../../modules/sceneStats"
+import { getHotScenes } from "../../RealmProvider/utils"
+import { getSceneStats } from "../../SceneStats/utils"
 import PlaceModel from "../model"
 import { FindWithAggregatesOptions, PlaceListOrderBy } from "../types"
 import { placesWithUserCount, placesWithUserVisits } from "../utils"

src/entities/RealmProvider/utils.ts

@@ -8,40 +8,50 @@ const DEFAULT_HOST_SCENE = [] as HotScene[]
 
 let memory = DEFAULT_HOST_SCENE
 
+// TODO: Remove urlLegacy reference once the Legacy Explorer is not longer used
 export default class RealmProvider {
   static Url = env(
+    "ARCHIPELAGO_URL",
+    "https://archipelago-ea-stats.decentraland.org/"
+  )
+  static UrlLegacy = env(
     "REALM_PROVIDER_URL",
     "https://realm-provider.decentraland.org/"
   )
   static Cache = new Map<string, RealmProvider>()
 
   private url: string
+  private urlLegacy: string
 
-  constructor(url: string) {
+  constructor(url: string, urlLegacy: string) {
     this.url = url
+    this.urlLegacy = urlLegacy
   }
 
   // Singleton instance based on URL
-  static from(url: string) {
+  static from(url: string, urlLegacy: string) {
     if (!this.Cache.has(url)) {
-      this.Cache.set(url, new RealmProvider(url))
+      this.Cache.set(url, new RealmProvider(url, urlLegacy))
     }
     return this.Cache.get(url)!
   }
 
   static get() {
-    return this.from(this.Url)
+    return this.from(this.Url, this.UrlLegacy)
   }
 
-  async getHotScenes(): Promise<HotScene[]> {
+  async getHotScenes(isLegeacy = false): Promise<HotScene[]> {
     const controller = new AbortController()
 
     const timeoutId = setTimeout(() => controller.abort(), Time.Second * 10)
 
     try {
-      const response = await fetch(`${this.url}hot-scenes`, {
-        signal: controller.signal as RequestInit["signal"],
-      })
+      const response = await fetch(
+        `${isLegeacy ? this.urlLegacy : this.url}hot-scenes`,
+        {
+          signal: controller.signal as RequestInit["signal"],
+        }
+      )
       if (!response.ok) {
         throw new Error(`Failed to fetch hot scenes: ${response.statusText}`)
       }
@@ -54,8 +64,35 @@ export default class RealmProvider {
 
 export const fetchHotScenesAndUpdateCache = async () => {
   try {
-    const response = await RealmProvider.get().getHotScenes()
-    memory = response
+    const [hotScenesLegacy, hotScenes] = await Promise.all([
+      RealmProvider.get().getHotScenes(true),
+      RealmProvider.get().getHotScenes(),
+    ])
+
+    // Create a Map to store scenes by their baseCoords
+    const scenesMap = new Map<string, HotScene>()
+
+    // Process both arrays
+    const processScene = (scene: HotScene, sceneMap: Map<string, HotScene>) => {
+      const key = `${scene.baseCoords[0]},${scene.baseCoords[1]}`
+      if (sceneMap.has(key)) {
+        // If scene exists, sum the users count
+        const existingScene = sceneMap.get(key)!
+        sceneMap.set(key, {
+          ...existingScene,
+          usersTotalCount:
+            existingScene.usersTotalCount + scene.usersTotalCount,
+        })
+      } else {
+        sceneMap.set(key, scene)
+      }
+    }
+
+    // Process both arrays
+    hotScenesLegacy.forEach((scene) => processScene(scene, scenesMap))
+    hotScenes.forEach((scene) => processScene(scene, scenesMap))
+
+    memory = Array.from(scenesMap.values())
   } catch (error) {
     memory = DEFAULT_HOST_SCENE
   }