From 26a330f743c5d41ed2d6513aad20c8d2723b9510 Mon Sep 17 00:00:00 2001
From: Mateusz Konieczny <matkoniecz@gmail.com>
Date: Tue, 7 Feb 2023 10:31:43 +0100
Subject: [PATCH] likes typology display

---
 app/map_styles/polygon.xml                    | 41 +++++++++++++++++++
 app/src/api/config/aggregationsConfig.ts      |  6 +++
 app/src/api/config/dataFields.ts              |  5 +++
 app/src/frontend/building/data-container.tsx  |  1 +
 .../building/data-containers/community.tsx    |  7 ++++
 .../frontend/config/category-maps-config.ts   | 15 +++++++
 app/src/frontend/config/data-fields-config.ts |  6 ++-
 app/src/frontend/config/tileserver-config.ts  |  1 +
 app/src/tiles/dataDefinition.ts               |  8 ++++
 .../extract_data/export_attributes.sql        |  1 +
 migrations/037.is_domestic.down.sql           |  1 +
 migrations/037.is_domestic.up.sql             |  1 +
 12 files changed, 92 insertions(+), 1 deletion(-)

diff --git a/app/map_styles/polygon.xml b/app/map_styles/polygon.xml
index e0fd9c36..59649184 100644
--- a/app/map_styles/polygon.xml
+++ b/app/map_styles/polygon.xml
@@ -664,6 +664,47 @@
             <LineSymbolizer stroke="#888" stroke-width="3.0"/>
         </Rule>
     </Style>
+    <Style name="typology_likes">
+        <Rule>
+            <Filter>[likes] &gt;= 100</Filter>
+            <PolygonSymbolizer fill="#bd0026" />
+        </Rule>
+        <Rule>
+            <Filter>[likes] &gt;= 50 and [likes] &lt; 100</Filter>
+            <PolygonSymbolizer fill="#e31a1c" />
+        </Rule>
+        <Rule>
+            <Filter>[likes] &gt;= 20 and [likes] &lt; 50</Filter>
+            <PolygonSymbolizer fill="#fc4e2a" />
+        </Rule>
+        <Rule>
+            <Filter>[likes] &gt;= 10 and [likes] &lt; 20</Filter>
+            <PolygonSymbolizer fill="#fd8d3c" />
+        </Rule>
+        <Rule>
+            <Filter>[likes] &gt;= 3 and [likes] &lt; 10</Filter>
+            <PolygonSymbolizer fill="#feb24c" />
+        </Rule>
+        <Rule>
+            <Filter>[likes] = 2</Filter>
+            <PolygonSymbolizer fill="#fed976" />
+        </Rule>
+        <Rule>
+            <Filter>[likes] = 1</Filter>
+            <PolygonSymbolizer fill="#ffe8a9" />
+        </Rule>
+
+        <Rule>
+            <MaxScaleDenominator>17061</MaxScaleDenominator>
+            <MinScaleDenominator>4264</MinScaleDenominator>
+            <LineSymbolizer stroke="#888" stroke-width="1.0"/>
+        </Rule>
+        <Rule>
+            <MaxScaleDenominator>4264</MaxScaleDenominator>
+            <MinScaleDenominator>0</MinScaleDenominator>
+            <LineSymbolizer stroke="#888" stroke-width="3.0"/>
+        </Rule>
+    </Style>
     <Style name="community_local_significance_total">
         <Rule>
             <Filter>[community_local_significance_total] &gt;= 100</Filter>
diff --git a/app/src/api/config/aggregationsConfig.ts b/app/src/api/config/aggregationsConfig.ts
index 64f3b6e3..9b1ae622 100644
--- a/app/src/api/config/aggregationsConfig.ts
+++ b/app/src/api/config/aggregationsConfig.ts
@@ -19,6 +19,12 @@ export const aggregationsConfig: { [key in keyof typeof buildingUserAttributesCo
             aggregationMethod: 'countTrue'
         }
     ],
+    community_type_worth_keeping: [
+        {
+            aggregateFieldName: 'community_type_worth_keeping_total',
+            aggregationMethod: 'countTrue'
+        }
+    ],
     community_local_significance: [
         {
             aggregateFieldName: 'community_local_significance_total',
diff --git a/app/src/api/config/dataFields.ts b/app/src/api/config/dataFields.ts
index 674a5bf8..f4fd0f57 100644
--- a/app/src/api/config/dataFields.ts
+++ b/app/src/api/config/dataFields.ts
@@ -301,6 +301,11 @@ export const buildingAttributesConfig = valueType<DataFieldConfig>()({ /* eslint
         derivedEdit: true,
         verify: false
     },
+    community_type_worth_keeping_total: {
+        edit: false,
+        derivedEdit: true,
+        verify: false
+    },
     community_local_significance_total: {
         edit: false,
         derivedEdit: true,
diff --git a/app/src/frontend/building/data-container.tsx b/app/src/frontend/building/data-container.tsx
index 7a8dea17..712c3dcb 100644
--- a/app/src/frontend/building/data-container.tsx
+++ b/app/src/frontend/building/data-container.tsx
@@ -97,6 +97,7 @@ const withCopyEdit: (wc: React.ComponentType<CategoryViewProps>) => DataContaine
                                      'current_landuse_verified',
                                      'planning_list_grade',
                                      'likes_total',
+                                     'community_type_worth_keeping_total',
                                      'community_local_significance_total',
                                      'community_expected_planning_application_total'
                                     ]
diff --git a/app/src/frontend/building/data-containers/community.tsx b/app/src/frontend/building/data-containers/community.tsx
index 839c2ccc..5ba7da15 100644
--- a/app/src/frontend/building/data-containers/community.tsx
+++ b/app/src/frontend/building/data-containers/community.tsx
@@ -28,6 +28,10 @@ const CommunityView: React.FunctionComponent<CategoryViewProps> = (props) => {
         e.preventDefault();
         props.onMapColourScale('likes')
     }
+    const switchToTypologyMapStyle = (e) => {
+        e.preventDefault();
+        props.onMapColourScale('typology_likes')
+    }
     const switchToLocalSignificanceMapStyle = (e) => {
         e.preventDefault();
         props.onMapColourScale('community_local_significance_total')
@@ -117,6 +121,9 @@ const CommunityView: React.FunctionComponent<CategoryViewProps> = (props) => {
                     mode={props.mode}
                 />
             }
+            <button className={`map-switcher-inline ${props.mapColourScale == "likes" ? "enabled-state" : "disabled-state"} btn btn-outline btn-outline-dark ${darkLightTheme}`} onClick={switchToTypologyMapStyle}> 
+                {(props.mapColourScale == "likes")? 'Showing likes for typologies (not about specific buildings)' : 'Click to see likes for typologies (not about specific buildings) mapped'}
+            </button>
             
 
             <UserOpinionEntry
diff --git a/app/src/frontend/config/category-maps-config.ts b/app/src/frontend/config/category-maps-config.ts
index 2a16e905..10afb3ac 100644
--- a/app/src/frontend/config/category-maps-config.ts
+++ b/app/src/frontend/config/category-maps-config.ts
@@ -134,6 +134,21 @@ export const categoryMapsConfig: {[key in Category]: CategoryMapDefinition[]} =
                 ]
             }
         },
+        {
+            mapStyle: 'typology_likes',
+            legend: {
+                title: 'Liked typologies',
+                elements: [
+                    { color: '#bd0026', text: '👍👍👍👍 100+' },
+                    { color: '#e31a1c', text: '👍👍👍 50–99' },
+                    { color: '#fc4e2a', text: '👍👍 20–49' },
+                    { color: '#fd8d3c', text: '👍👍 10–19' },
+                    { color: '#feb24c', text: '👍 3–9' },
+                    { color: '#fed976', text: '👍 2' },
+                    { color: '#ffe8a9', text: '👍 1'}
+                ]
+            }
+        },
         {
             mapStyle: 'community_local_significance_total',
             legend: {
diff --git a/app/src/frontend/config/data-fields-config.ts b/app/src/frontend/config/data-fields-config.ts
index 68567076..d9435fe4 100644
--- a/app/src/frontend/config/data-fields-config.ts
+++ b/app/src/frontend/config/data-fields-config.ts
@@ -583,7 +583,11 @@ export const dataFields = { /* eslint-disable @typescript-eslint/camelcase */
         example: 100,
         tooltip: "People who like the building and think it contributes to the city.",
     },
-
+    community_type_worth_keeping_total: {
+        category: Category.Community,
+        title: "People who think this type of building is contributes to the city.",
+        example: 100,
+    },
     community_local_significance_total: {
         category: Category.Community,
         title: "People who think the building should be recorded as one of local significance",
diff --git a/app/src/frontend/config/tileserver-config.ts b/app/src/frontend/config/tileserver-config.ts
index fd57d954..fe747581 100644
--- a/app/src/frontend/config/tileserver-config.ts
+++ b/app/src/frontend/config/tileserver-config.ts
@@ -9,6 +9,7 @@ export type BuildingMapTileset = 'date_year' |
     'location' |
     'is_domestic' |
     'likes' |
+    'typology_likes' |
     'community_local_significance_total' |
     'community_expected_planning_application_total' |
     'community_in_public_ownership' |
diff --git a/app/src/tiles/dataDefinition.ts b/app/src/tiles/dataDefinition.ts
index d399a08d..66f2738c 100644
--- a/app/src/tiles/dataDefinition.ts
+++ b/app/src/tiles/dataDefinition.ts
@@ -127,6 +127,14 @@ const LAYER_QUERIES = {
             is_domestic <> 'yes'
             AND
             likes_total > 0`,
+    typology_likes: `
+        SELECT
+            geometry_id,
+            community_type_worth_keeping_total AS likes
+        FROM
+            buildings
+        WHERE
+            community_type_worth_keeping_total > 0`,
     community_local_significance_total: `
         SELECT
             geometry_id,
diff --git a/maintenance/extract_data/export_attributes.sql b/maintenance/extract_data/export_attributes.sql
index 78d087ac..4368838f 100644
--- a/maintenance/extract_data/export_attributes.sql
+++ b/maintenance/extract_data/export_attributes.sql
@@ -55,6 +55,7 @@ COPY (SELECT
     planning_local_list_url,
     planning_historic_area_assessment_url,
     is_domestic,
+    community_type_worth_keeping_total,
     likes_total
 FROM buildings)
 TO '/tmp/building_attributes.csv'
diff --git a/migrations/037.is_domestic.down.sql b/migrations/037.is_domestic.down.sql
index 68c44f64..b39cc375 100644
--- a/migrations/037.is_domestic.down.sql
+++ b/migrations/037.is_domestic.down.sql
@@ -1 +1,2 @@
 ALTER TABLE buildings DROP COLUMN IF EXISTS is_domestic;
+ALTER TABLE buildings DROP COLUMN IF EXISTS community_type_worth_keeping_total;
diff --git a/migrations/037.is_domestic.up.sql b/migrations/037.is_domestic.up.sql
index 33d556c3..3be40166 100644
--- a/migrations/037.is_domestic.up.sql
+++ b/migrations/037.is_domestic.up.sql
@@ -1 +1,2 @@
 ALTER TABLE buildings ADD COLUMN IF NOT EXISTS is_domestic text null;
+ALTER TABLE buildings ADD COLUMN IF NOT EXISTS community_type_worth_keeping_total integer DEFAULT 0;