Merge pull request #110 from aymericdo/some_refacto

some refacto

src/db/bordeaux-address.model.ts

@@ -22,9 +22,10 @@ const schema = new Schema({
 })
 
 schema.index(
-  { nom_voie: 'text', numero: 'text' },
+  { nom_commune: 'text', nom_voie: 'text', numero: 'text' },
   {
     weights: {
+      nom_commune: 10,
       nom_voie: 10,
       numero: 1,
     },
@@ -33,7 +34,7 @@ schema.index(
 )
 
 schema.index(
-  { nom_voie: 1, numero: 1 },
+  { nom_commune: 1, nom_voie: 1, numero: 1 },
   {
     collation: {
       locale: 'fr',

src/db/est_ensemble-address.model.ts

@@ -29,9 +29,10 @@ const schema = new Schema({
 })
 
 schema.index(
-  { nom_voie: 'text', numero: 'text' },
+  { nom_commune: 'text', nom_voie: 'text', numero: 'text' },
   {
     weights: {
+      nom_commune: 10,
       nom_voie: 10,
       numero: 1,
     },
@@ -40,7 +41,7 @@ schema.index(
 )
 
 schema.index(
-  { nom_voie: 1, numero: 1 },
+  { nom_commune: 1, nom_voie: 1, numero: 1 },
   {
     collation: {
       locale: 'fr',

src/db/lille-address.model.ts

@@ -29,17 +29,18 @@ const schema = new Schema({
 })
 
 schema.index(
-  { nom_voie: 'text', numero: 'text' },
+  { nom_commune: 'text', nom_voie: 'text', numero: 'text' },
   {
     weights: {
+      nom_commune: 10,
       nom_voie: 10,
       numero: 1,
     },
     default_language: 'french',
   }
 )
 schema.index(
-  { nom_voie: 1, numero: 1 },
+  { nom_commune: 1, nom_voie: 1, numero: 1 },
   {
     collation: {
       locale: 'fr',

src/db/lyon-address.model.ts

@@ -30,17 +30,18 @@ const schema = new Schema({
 })
 
 schema.index(
-  { nom_voie: 'text', numero: 'text' },
+  { nom_commune: 'text', nom_voie: 'text', numero: 'text' },
   {
     weights: {
+      nom_commune: 10,
       nom_voie: 10,
       numero: 1,
     },
     default_language: 'french',
   }
 )
 schema.index(
-  { nom_voie: 1, numero: 1 },
+  { nom_commune: 1, nom_voie: 1, numero: 1 },
   {
     collation: {
       locale: 'fr',

src/db/montpellier-address.model.ts

@@ -29,9 +29,10 @@ const schema = new Schema({
 })
 
 schema.index(
-  { nom_voie: 'text', numero: 'text' },
+  { nom_commune: 'text', nom_voie: 'text', numero: 'text' },
   {
     weights: {
+      nom_commune: 10,
       nom_voie: 10,
       numero: 1,
     },
@@ -40,7 +41,7 @@ schema.index(
 )
 
 schema.index(
-  { nom_voie: 1, numero: 1 },
+  { nom_commune: 1, nom_voie: 1, numero: 1 },
   {
     collation: {
       locale: 'fr',

src/db/plaine_commune-address.model.ts

@@ -29,9 +29,10 @@ const schema = new Schema({
 })
 
 schema.index(
-  { nom_voie: 'text', numero: 'text' },
+  { nom_commune: 'text', nom_voie: 'text', numero: 'text' },
   {
     weights: {
+      nom_commune: 10,
       nom_voie: 10,
       numero: 1,
     },
@@ -40,7 +41,7 @@ schema.index(
 )
 
 schema.index(
-  { nom_voie: 1, numero: 1 },
+  { nom_commune: 1, nom_voie: 1, numero: 1 },
   {
     collation: {
       locale: 'fr',

src/db/rent.service.ts

@@ -64,7 +64,7 @@ export async function getChloroplethMapData(
     }
   }
 
-  return await Rent.find(filter, { isLegal: 1, district: 1 }).lean()
+  return await Rent.find(filter, { isLegal: 1, district: 1 })
 }
 
 export async function getPriceDiffData(
@@ -227,7 +227,7 @@ export async function getLegalPerClassicRenterData(
   return (await Rent.find(filter, {
     isLegal: 1,
     renter: 1,
-  }).lean()) as unknown as { isLegal: boolean; renter: string }[]
+  })) as unknown as { isLegal: boolean; renter: string }[]
 }
 
 export async function getLegalPerRenterData(
@@ -307,7 +307,7 @@ export async function getLegalPerWebsiteData(
   return (await Rent.find(filter, {
     isLegal: 1,
     website: 1,
-  }).lean()) as unknown as {
+  })) as unknown as {
     isLegal: boolean
     website: string
   }[]
@@ -427,18 +427,15 @@ export async function getRelevantAdsData(
     }
   ).lean()) as unknown as RelevantAdsData[]
 
-  return ads.map((ad) => {
+  return await Promise.all(ads.map(async (ad) => {
     let blurry = false
 
     if (!ad.longitude || !ad.latitude) {
       const mainCity = cityList[ad.city].mainCity
 
-      const polygon = new DistrictsList(mainCity as AvailableMainCities).currentPolygon(ad.district)
+      const feature = await new DistrictsList(mainCity as AvailableMainCities).currentFeature(ad.district)
 
-      const point = randomPositionInPolygon({
-        type: 'Feature',
-        geometry: polygon,
-      })
+      const point = randomPositionInPolygon(feature)
 
       ad.longitude = point[0]
       ad.latitude = point[1]
@@ -455,7 +452,7 @@ export async function getRelevantAdsData(
       blurry,
       exceeding,
     }
-  })
+  }))
 }
 
 export async function getRelevantAdsDataTotalCount(filterParam: {

src/interfaces/json-item-bordeaux.ts

@@ -1,3 +1,5 @@
+import { DISPLAY_ZONE_FIELD } from '@services/districts/districts-list'
+
 export interface BordeauxEncadrementItem {
   zone: number;
   maison: boolean;
@@ -15,6 +17,7 @@ export interface BordeauxDistrictItem {
     com_code: number
     zonage: string
     commune: string
+    [DISPLAY_ZONE_FIELD]: string
   }
   geometry: {
     type: 'Polygon'

src/services/address/address-factory.ts

@@ -0,0 +1,16 @@
+import { Ad } from '@interfaces/ad'
+import { AddressService, DefaultAddressService } from '@services/address/default-address-service'
+import { ParisAddressService } from '@services/address/paris-address-service'
+
+export class AddressServiceFactory {
+  getDiggerStrategy(city: string, postalCode: string, ad: Ad): AddressService {
+    switch (city) {
+      case 'paris': {
+        return new ParisAddressService(city, postalCode, ad)
+      }
+      default: {
+        return new DefaultAddressService(city, postalCode, ad)
+      }
+    }
+  }
+}