From 91eb5aeef8e7dbb520210c46d52b947c97a43e75 Mon Sep 17 00:00:00 2001
From: Mike Simpson <mike.simpson@ncl.ac.uk>
Date: Tue, 27 Jun 2023 19:47:40 +0100
Subject: [PATCH] Add alternative building footprint links (Location)

---
 app/src/api/config/dataFields.ts                |  4 ++++
 .../building/data-containers/location.tsx       | 17 +++++++++++------
 app/src/frontend/config/data-fields-config.ts   |  6 ++++++
 maintenance/extract_data/export_attributes.sql  |  1 +
 migrations/041.ui_revamp_tweaks.down.sql        |  3 ++-
 migrations/041.ui_revamp_tweaks.up.sql          |  3 ++-
 6 files changed, 26 insertions(+), 8 deletions(-)

diff --git a/app/src/api/config/dataFields.ts b/app/src/api/config/dataFields.ts
index 6688529d..f632688d 100644
--- a/app/src/api/config/dataFields.ts
+++ b/app/src/api/config/dataFields.ts
@@ -107,6 +107,10 @@ export const buildingAttributesConfig = valueType<DataFieldConfig>()({ /* eslint
         edit: true,
         verify: true
     },
+    location_alternative_footprint_links: {
+        edit: true,
+        verify: true
+    },
     date_year: {
         edit: true,
         verify: true,
diff --git a/app/src/frontend/building/data-containers/location.tsx b/app/src/frontend/building/data-containers/location.tsx
index 5ba69820..c31d3bbb 100644
--- a/app/src/frontend/building/data-containers/location.tsx
+++ b/app/src/frontend/building/data-containers/location.tsx
@@ -315,12 +315,17 @@ const LocationView: React.FunctionComponent<CategoryViewProps> = (props) => {
                     </>
                 }
                 <hr/>
-                <DataEntry
-                    title="Alternative open building footprint ID(s)"
-                    slug=""
-                    value=""
-                    mode='view'
-                    tooltip="Under Development.<br><br>Please enter links to any additional data sources for open building footprint IDs."
+                <MultiDataEntry
+                    title={dataFields.location_alternative_footprint_links.title}
+                    slug="location_alternative_footprint_links"
+                    value={props.building.location_alternative_footprint_links}
+                    mode={props.mode}
+                    copy={props.copy}
+                    onChange={props.onChange}
+                    tooltip={dataFields.location_alternative_footprint_links.tooltip}
+                    placeholder="https://..."
+                    editableEntries={true}
+                    isUrl={true}
                 />
             </DataEntryGroup>
         </Fragment>
diff --git a/app/src/frontend/config/data-fields-config.ts b/app/src/frontend/config/data-fields-config.ts
index 33bfced3..90c1ad5b 100644
--- a/app/src/frontend/config/data-fields-config.ts
+++ b/app/src/frontend/config/data-fields-config.ts
@@ -203,6 +203,12 @@ export const dataFields = { /* eslint-disable @typescript-eslint/camelcase */
         tooltip: "Ordnance Survey Topography Layer ID (TOID)",
         example: "",
     },
+    location_alternative_footprint_links: {
+        category: Category.Location,
+        title: "Alternative open building footprint links",
+        tooltip: "Links to alternative building footprint datasets (include the direct link to the footprint of this building where possible).",
+        example: ["", "", ""],
+    },
 
     /**
      * UPRNs is not part of the buildings table, but the string fields 
diff --git a/maintenance/extract_data/export_attributes.sql b/maintenance/extract_data/export_attributes.sql
index ce1e602b..4f5672bf 100644
--- a/maintenance/extract_data/export_attributes.sql
+++ b/maintenance/extract_data/export_attributes.sql
@@ -14,6 +14,7 @@ COPY (SELECT
     location_address_links,
     location_latitude,
     location_longitude,
+    location_alternative_footprint_links
     current_landuse_group,
     current_landuse_order,
     building_attachment_form,
diff --git a/migrations/041.ui_revamp_tweaks.down.sql b/migrations/041.ui_revamp_tweaks.down.sql
index 477f342e..dc30b17f 100644
--- a/migrations/041.ui_revamp_tweaks.down.sql
+++ b/migrations/041.ui_revamp_tweaks.down.sql
@@ -1 +1,2 @@
-ALTER TABLE buildings DROP COLUMN IF EXISTS location_name_link;
\ No newline at end of file
+ALTER TABLE buildings DROP COLUMN IF EXISTS location_name_link;
+ALTER TABLE buildings DROP COLUMN IF EXISTS location_alternative_footprint_links;
\ No newline at end of file
diff --git a/migrations/041.ui_revamp_tweaks.up.sql b/migrations/041.ui_revamp_tweaks.up.sql
index a2c7903a..bb1d12b7 100644
--- a/migrations/041.ui_revamp_tweaks.up.sql
+++ b/migrations/041.ui_revamp_tweaks.up.sql
@@ -1 +1,2 @@
-ALTER TABLE buildings ADD COLUMN IF NOT EXISTS location_name_link text;
\ No newline at end of file
+ALTER TABLE buildings ADD COLUMN IF NOT EXISTS location_name_link text;
+ALTER TABLE buildings ADD COLUMN IF NOT EXISTS location_alternative_footprint_links text[];
\ No newline at end of file