From 739bcad08a901ce0a49174d01ae5987502c0cef7 Mon Sep 17 00:00:00 2001 From: Maciej Ziarkowski Date: Thu, 7 Nov 2019 09:13:30 +0100 Subject: [PATCH] Lint semicolons --- app/src/api/api.ts | 24 +++++++-------- app/src/api/controllers/buildingController.ts | 2 +- app/src/api/services/building.ts | 4 +-- app/src/api/services/dataExtract.ts | 4 +-- app/src/api/services/user.ts | 10 +++---- .../frontend/building/building-not-found.tsx | 2 +- app/src/frontend/building/building-view.tsx | 30 +++++++++---------- app/src/frontend/building/categories.tsx | 6 ++-- .../frontend/building/container-header.tsx | 2 +- .../data-components/checkbox-data-entry.tsx | 2 +- .../data-components/data-entry-group.tsx | 2 +- .../building/data-components/data-entry.tsx | 4 +-- .../building/data-components/data-title.tsx | 8 ++--- .../data-components/like-data-entry.tsx | 2 +- .../data-components/multi-data-entry.tsx | 4 +-- .../data-components/numeric-data-entry.tsx | 2 +- .../data-components/select-data-entry.tsx | 2 +- .../data-components/textbox-data-entry.tsx | 2 +- .../data-components/uprns-data-entry.tsx | 2 +- .../data-components/year-data-entry.tsx | 4 +-- app/src/frontend/building/data-container.tsx | 22 +++++++------- .../frontend/building/data-containers/age.tsx | 2 +- .../building/data-containers/community.tsx | 2 +- .../building/data-containers/construction.tsx | 2 +- .../building/data-containers/like.tsx | 2 +- .../building/data-containers/location.tsx | 2 +- .../building/data-containers/planning.tsx | 4 +-- .../building/data-containers/size.tsx | 2 +- .../building/data-containers/streetscape.tsx | 2 +- .../data-containers/sustainability.tsx | 2 +- .../building/data-containers/team.tsx | 2 +- .../building/data-containers/type.tsx | 2 +- .../frontend/building/data-containers/use.tsx | 2 +- .../edit-history/building-edit-summary.tsx | 4 +-- .../building/edit-history/edit-history.tsx | 2 +- app/src/frontend/building/multi-edit.tsx | 12 ++++---- .../components/confirmation-modal.tsx | 16 +++++----- app/src/frontend/components/icons.tsx | 8 ++--- app/src/frontend/components/logo.tsx | 2 +- app/src/frontend/helpers.ts | 20 ++++++------- app/src/frontend/map-app.tsx | 16 +++++----- app/src/frontend/map/map.tsx | 6 ++-- app/src/frontend/map/search-box.tsx | 26 ++++++++-------- app/src/frontend/pages/about.tsx | 2 +- .../frontend/pages/contributor-agreement.tsx | 2 +- app/src/frontend/pages/privacy-policy.tsx | 2 +- app/src/frontend/user/forgotten-password.tsx | 2 +- app/src/frontend/user/login.tsx | 16 +++++----- app/src/frontend/user/my-account.tsx | 6 ++-- app/src/frontend/user/password-reset.tsx | 2 +- app/src/frontend/user/signup.tsx | 10 +++---- app/src/parse.ts | 2 +- app/src/server.tsx | 4 +-- app/src/tiles/tileCache.ts | 4 +-- app/src/tiles/tileserver.ts | 2 +- app/src/tiles/types.ts | 2 +- app/src/tiles/util.ts | 2 +- app/tslint.json | 3 +- 58 files changed, 170 insertions(+), 169 deletions(-) diff --git a/app/src/api/api.ts b/app/src/api/api.ts index 628c9165..fd1c1426 100644 --- a/app/src/api/api.ts +++ b/app/src/api/api.ts @@ -28,7 +28,7 @@ server.post('/login', function (req, res) { res.send(user); }).catch(function (error) { res.send(error); - }) + }); }); // POST user logout @@ -45,7 +45,7 @@ server.post('/logout', function (req, res) { server.post('/api/key', function (req, res) { if (!req.session.user_id) { res.send({ error: 'Must be logged in' }); - return + return; } getNewUserAPIKey(req.session.user_id).then(function (apiKey) { @@ -53,7 +53,7 @@ server.post('/api/key', function (req, res) { }).catch(function (error) { res.send(error); }); -}) +}); // GET search server.get('/search', function (req, res) { @@ -61,20 +61,20 @@ server.get('/search', function (req, res) { if (!searchTerm) { res.send({ error: 'Please provide a search term' - }) - return + }); + return; } queryLocation(searchTerm).then((results) => { if (typeof (results) === 'undefined') { res.send({ error: 'Database error' - }) - return + }); + return; } res.send({ results: results.map(item => { // map from DB results to GeoJSON Feature objects - const geom = JSON.parse(item.st_asgeojson) + const geom = JSON.parse(item.st_asgeojson); return { type: 'Feature', attributes: { @@ -82,13 +82,13 @@ server.get('/search', function (req, res) { zoom: item.zoom || 9 }, geometry: geom - } + }; }) - }) + }); }).catch(function (error) { res.send(error); }); -}) +}); server.use((err, req, res, next) => { if (res.headersSent) { @@ -103,7 +103,7 @@ server.use((err, req, res, next) => { server.use((req, res) => { res.status(404).json({ error: 'Resource not found'}); -}) +}); export default server; diff --git a/app/src/api/controllers/buildingController.ts b/app/src/api/controllers/buildingController.ts index 6b5e31b8..269ea7b1 100644 --- a/app/src/api/controllers/buildingController.ts +++ b/app/src/api/controllers/buildingController.ts @@ -107,7 +107,7 @@ const getBuildingLikeById = asyncController(async (req: express.Request, res: ex // any value returned means like res.send({ like: like }); } catch(error) { - res.send({ error: 'Database error' }) + res.send({ error: 'Database error' }); } }); diff --git a/app/src/api/services/building.ts b/app/src/api/services/building.ts index 388514e1..9cc611d8 100644 --- a/app/src/api/services/building.ts +++ b/app/src/api/services/building.ts @@ -277,7 +277,7 @@ async function updateBuildingData( console.log(update); const patches = compare(oldBuilding, update); - console.log('Patching', buildingId, patches) + console.log('Patching', buildingId, patches); const [forward, reverse] = patches; if (Object.keys(forward).length === 0) { throw 'No change provided'; @@ -337,7 +337,7 @@ function privateQueryBuildingBBOX(buildingId: number){ } async function expireBuildingTileCache(buildingId: number) { - const bbox = await privateQueryBuildingBBOX(buildingId) + const bbox = await privateQueryBuildingBBOX(buildingId); const buildingBbox: BoundingBox = [bbox.xmax, bbox.ymax, bbox.xmin, bbox.ymin]; tileCache.removeAllAtBbox(buildingBbox); } diff --git a/app/src/api/services/dataExtract.ts b/app/src/api/services/dataExtract.ts index 4f43f1cb..063e9a7b 100644 --- a/app/src/api/services/dataExtract.ts +++ b/app/src/api/services/dataExtract.ts @@ -50,12 +50,12 @@ function getDataExtractFromRow(er: DataExtractRow): DataExtract { extract_id: er.extract_id, extracted_on: er.extracted_on, download_path: getDownloadLinkForExtract(er) - } + }; } function getDownloadLinkForExtract(extract: DataExtractRow): string { const file_name = path.basename(extract.extract_path); - return `/downloads/${file_name}` // TODO: potentially move base path to env var + return `/downloads/${file_name}`; // TODO: potentially move base path to env var } export { diff --git a/app/src/api/services/user.ts b/app/src/api/services/user.ts index bce3ec14..c6cbc74e 100644 --- a/app/src/api/services/user.ts +++ b/app/src/api/services/user.ts @@ -71,9 +71,9 @@ async function authUser(username: string, password: string) { ); if (user && user.auth_ok) { - return { user_id: user.user_id } + return { user_id: user.user_id }; } else { - return { error: 'Username or password not recognised' } + return { error: 'Username or password not recognised' }; } } catch(err) { if (err instanceof errors.QueryResultError) { @@ -99,7 +99,7 @@ async function getUserById(id: string) { ] ); } catch(error) { - console.error('Error:', error) + console.error('Error:', error); return undefined; } } @@ -137,7 +137,7 @@ async function getNewUserAPIKey(id: string) { ] ); } catch(error) { - console.error('Error:', error) + console.error('Error:', error); return { error: 'Failed to generate new API key.' }; } } @@ -156,7 +156,7 @@ async function authAPIUser(key: string) { ] ); } catch(error) { - console.error('Error:', error) + console.error('Error:', error); return undefined; } } diff --git a/app/src/frontend/building/building-not-found.tsx b/app/src/frontend/building/building-not-found.tsx index 07c7c2af..35a2f3f1 100644 --- a/app/src/frontend/building/building-not-found.tsx +++ b/app/src/frontend/building/building-not-found.tsx @@ -5,7 +5,7 @@ import InfoBox from '../components/info-box'; interface BuildingNotFoundProps { - mode: string + mode: string; } const BuildingNotFound: React.FunctionComponent = (props) => ( diff --git a/app/src/frontend/building/building-view.tsx b/app/src/frontend/building/building-view.tsx index 9f4279c7..d21f7115 100644 --- a/app/src/frontend/building/building-view.tsx +++ b/app/src/frontend/building/building-view.tsx @@ -23,7 +23,7 @@ interface BuildingViewProps { building?: Building; building_like?: boolean; user?: any; - selectBuilding: (building: Building) => void + selectBuilding: (building: Building) => void; } /** @@ -39,7 +39,7 @@ const BuildingView: React.FunctionComponent = (props) => { title="Location" help="https://pages.colouring.london/location" intro="Where are the buildings? Address, location and cross-references." - /> + />; case 'use': return = (props) => { title="Land Use" intro="How are buildings used, and how does use change over time? Coming soon…" help="https://pages.colouring.london/use" - /> + />; case 'type': return = (props) => { title="Type" intro="How were buildings previously used?" help="https://www.pages.colouring.london/buildingtypology" - /> + />; case 'age': return + />; case 'size': return + />; case 'construction': return = (props) => { intro="How are buildings built? Coming soon…" help="https://pages.colouring.london/construction" inactive={true} - /> + />; case 'team': return = (props) => { intro="Who built the buildings? Coming soon…" help="https://pages.colouring.london/team" inactive={true} - /> + />; case 'sustainability': return = (props) => { intro="Are buildings energy efficient?" help="https://pages.colouring.london/sustainability" inactive={false} - /> + />; case 'streetscape': return = (props) => { intro="What's the building's context? Coming soon…" help="https://pages.colouring.london/streetscape" inactive={true} - /> + />; case 'community': return = (props) => { intro="How does this building work for the local community?" help="https://pages.colouring.london/community" inactive={true} - /> + />; case 'planning': return + />; case 'like': return + />; default: - return + return ; } -} +}; export default BuildingView; diff --git a/app/src/frontend/building/categories.tsx b/app/src/frontend/building/categories.tsx index 54d07541..f03205c4 100644 --- a/app/src/frontend/building/categories.tsx +++ b/app/src/frontend/building/categories.tsx @@ -1,7 +1,7 @@ import React from 'react'; import { NavLink } from 'react-router-dom'; -import './categories.css' +import './categories.css'; interface CategoriesProps { mode: 'view' | 'edit' | 'multi-edit'; @@ -119,7 +119,7 @@ const Categories: React.FC = (props) => ( building_id={props.building_id} /> -) +); interface CategoryProps { mode: 'view' | 'edit' | 'multi-edit'; @@ -152,6 +152,6 @@ const Category: React.FC = (props) => { ); -} +}; export default Categories; diff --git a/app/src/frontend/building/container-header.tsx b/app/src/frontend/building/container-header.tsx index e40d9e4e..9513a91b 100644 --- a/app/src/frontend/building/container-header.tsx +++ b/app/src/frontend/building/container-header.tsx @@ -19,6 +19,6 @@ const ContainerHeader: React.FunctionComponent = (props) = {props.children} -) +); export default ContainerHeader; diff --git a/app/src/frontend/building/data-components/checkbox-data-entry.tsx b/app/src/frontend/building/data-components/checkbox-data-entry.tsx index ddbeedf4..37ceddb0 100644 --- a/app/src/frontend/building/data-components/checkbox-data-entry.tsx +++ b/app/src/frontend/building/data-components/checkbox-data-entry.tsx @@ -33,6 +33,6 @@ const CheckboxDataEntry: React.FunctionComponent = (prop ); -} +}; export default CheckboxDataEntry; diff --git a/app/src/frontend/building/data-components/data-entry-group.tsx b/app/src/frontend/building/data-components/data-entry-group.tsx index a0a18829..de4cd710 100644 --- a/app/src/frontend/building/data-components/data-entry-group.tsx +++ b/app/src/frontend/building/data-components/data-entry-group.tsx @@ -30,7 +30,7 @@ const DataEntryGroup: React.FunctionComponent = (props) => ); -} +}; const CollapseIcon: React.FunctionComponent<{collapsed: boolean}> = (props) => ( diff --git a/app/src/frontend/building/data-components/data-entry.tsx b/app/src/frontend/building/data-components/data-entry.tsx index ebcfcdc4..963173b4 100644 --- a/app/src/frontend/building/data-components/data-entry.tsx +++ b/app/src/frontend/building/data-components/data-entry.tsx @@ -18,7 +18,7 @@ interface DataEntryProps extends BaseDataEntryProps { value?: string; maxLength?: number; placeholder?: string; - valueTransform?: (string) => string + valueTransform?: (string) => string; } const DataEntry: React.FunctionComponent = (props) => { @@ -48,7 +48,7 @@ const DataEntry: React.FunctionComponent = (props) => { /> ); -} +}; export default DataEntry; export { diff --git a/app/src/frontend/building/data-components/data-title.tsx b/app/src/frontend/building/data-components/data-title.tsx index fcffd790..5831048d 100644 --- a/app/src/frontend/building/data-components/data-title.tsx +++ b/app/src/frontend/building/data-components/data-title.tsx @@ -15,8 +15,8 @@ const DataTitle: React.FunctionComponent = (props) => { { props.title } { props.tooltip? : null } - ) -} + ); +}; interface DataTitleCopyableProps { @@ -48,7 +48,7 @@ const DataTitleCopyable: React.FunctionComponent = (prop ); -} +}; export default DataTitle; -export { DataTitleCopyable } +export { DataTitleCopyable }; diff --git a/app/src/frontend/building/data-components/like-data-entry.tsx b/app/src/frontend/building/data-components/like-data-entry.tsx index 086dfb7e..e034f2b3 100644 --- a/app/src/frontend/building/data-components/like-data-entry.tsx +++ b/app/src/frontend/building/data-components/like-data-entry.tsx @@ -46,6 +46,6 @@ const LikeDataEntry: React.FunctionComponent = (props) => { ); -} +}; export default LikeDataEntry; diff --git a/app/src/frontend/building/data-components/multi-data-entry.tsx b/app/src/frontend/building/data-components/multi-data-entry.tsx index 0fbe608b..5efd1a91 100644 --- a/app/src/frontend/building/data-components/multi-data-entry.tsx +++ b/app/src/frontend/building/data-components/multi-data-entry.tsx @@ -69,7 +69,7 @@ class MultiDataEntry extends Component { key={index} className="form-control"> {item} - + ; }) } @@ -98,7 +98,7 @@ class MultiDataEntry extends Component { onClick={this.add} disabled={props.mode === 'view'} className="btn btn-outline-dark">+ - + ; } } diff --git a/app/src/frontend/building/data-components/numeric-data-entry.tsx b/app/src/frontend/building/data-components/numeric-data-entry.tsx index 4867994f..3c1a21fe 100644 --- a/app/src/frontend/building/data-components/numeric-data-entry.tsx +++ b/app/src/frontend/building/data-components/numeric-data-entry.tsx @@ -42,6 +42,6 @@ const NumericDataEntry: React.FunctionComponent = (props) /> ); -} +}; export default NumericDataEntry; diff --git a/app/src/frontend/building/data-components/select-data-entry.tsx b/app/src/frontend/building/data-components/select-data-entry.tsx index 859c3a4a..7ed9a70c 100644 --- a/app/src/frontend/building/data-components/select-data-entry.tsx +++ b/app/src/frontend/building/data-components/select-data-entry.tsx @@ -41,6 +41,6 @@ const SelectDataEntry: React.FunctionComponent = (props) = ); -} +}; export default SelectDataEntry; diff --git a/app/src/frontend/building/data-components/textbox-data-entry.tsx b/app/src/frontend/building/data-components/textbox-data-entry.tsx index 3b5cca28..b1ba958a 100644 --- a/app/src/frontend/building/data-components/textbox-data-entry.tsx +++ b/app/src/frontend/building/data-components/textbox-data-entry.tsx @@ -39,6 +39,6 @@ const TextboxDataEntry: React.FunctionComponent = (props) > ); -} +}; export default TextboxDataEntry; diff --git a/app/src/frontend/building/data-components/uprns-data-entry.tsx b/app/src/frontend/building/data-components/uprns-data-entry.tsx index 44c414f9..f15de0d4 100644 --- a/app/src/frontend/building/data-components/uprns-data-entry.tsx +++ b/app/src/frontend/building/data-components/uprns-data-entry.tsx @@ -53,7 +53,7 @@ const UPRNsDataEntry: React.FC = (props) => { } - ) + ); }; export default UPRNsDataEntry; diff --git a/app/src/frontend/building/data-components/year-data-entry.tsx b/app/src/frontend/building/data-components/year-data-entry.tsx index 17f60836..1d35f8ee 100644 --- a/app/src/frontend/building/data-components/year-data-entry.tsx +++ b/app/src/frontend/building/data-components/year-data-entry.tsx @@ -23,7 +23,7 @@ class YearDataEntry extends Component { lower: props.lower, decade: Math.floor(props.year / 10) * 10, century: Math.floor(props.year / 100) * 100 - } + }; } // TODO add dropdown for decade, century // TODO roll in first/last year estimate @@ -62,7 +62,7 @@ class YearDataEntry extends Component { tooltip={dataFields.date_lower.tooltip} /> - ) + ); } } diff --git a/app/src/frontend/building/data-container.tsx b/app/src/frontend/building/data-container.tsx index 03edb95a..13b2cc27 100644 --- a/app/src/frontend/building/data-container.tsx +++ b/app/src/frontend/building/data-container.tsx @@ -23,7 +23,7 @@ interface DataContainerProps { mode: 'view' | 'edit'; building?: Building; building_like?: boolean; - selectBuilding: (building: Building) => void + selectBuilding: (building: Building) => void; } interface DataContainerState { @@ -89,7 +89,7 @@ const withCopyEdit = (WrappedComponent: React.ComponentType) toggleCopying() { this.setState({ copying: !this.state.copying - }) + }); } /** @@ -106,7 +106,7 @@ const withCopyEdit = (WrappedComponent: React.ComponentType) } this.setState({ keys_to_copy: keys - }) + }); } isEdited() { @@ -173,7 +173,7 @@ const withCopyEdit = (WrappedComponent: React.ComponentType) const data = await res.json(); if (data.error) { - this.setState({error: data.error}) + this.setState({error: data.error}); } else { this.props.selectBuilding(data); this.updateBuildingState('likes_total', data.likes_total); @@ -199,7 +199,7 @@ const withCopyEdit = (WrappedComponent: React.ComponentType) const data = await res.json(); if (data.error) { - this.setState({error: data.error}) + this.setState({error: data.error}); } else { this.props.selectBuilding(data); } @@ -210,14 +210,14 @@ const withCopyEdit = (WrappedComponent: React.ComponentType) render() { if (this.props.mode === 'edit' && !this.props.user){ - return + return ; } const currentBuilding = this.getEditedBuilding(); - const values_to_copy = {} + const values_to_copy = {}; for (const key of Object.keys(this.state.keys_to_copy)) { - values_to_copy[key] = currentBuilding[key] + values_to_copy[key] = currentBuilding[key]; } const data_string = JSON.stringify(values_to_copy); const copy: CopyProps = { @@ -225,7 +225,7 @@ const withCopyEdit = (WrappedComponent: React.ComponentType) toggleCopying: this.toggleCopying, toggleCopyAttribute: this.toggleCopyAttribute, copyingKey: (key: string) => this.state.keys_to_copy[key] - } + }; const headerBackLink = `/${this.props.mode}/categories${this.props.building != undefined ? `/${this.props.building.building_id}` : ''}`; const edited = this.isEdited(); @@ -347,7 +347,7 @@ const withCopyEdit = (WrappedComponent: React.ComponentType) ); } - } -} + }; +}; export default withCopyEdit; diff --git a/app/src/frontend/building/data-containers/age.tsx b/app/src/frontend/building/data-containers/age.tsx index cd872db4..1f1728b6 100644 --- a/app/src/frontend/building/data-containers/age.tsx +++ b/app/src/frontend/building/data-containers/age.tsx @@ -75,7 +75,7 @@ const AgeView: React.FunctionComponent = (props) => ( placeholder="https://..." /> -) +); const AgeContainer = withCopyEdit(AgeView); export default AgeContainer; diff --git a/app/src/frontend/building/data-containers/community.tsx b/app/src/frontend/building/data-containers/community.tsx index 4c232239..ddb6b670 100644 --- a/app/src/frontend/building/data-containers/community.tsx +++ b/app/src/frontend/building/data-containers/community.tsx @@ -28,7 +28,7 @@ const CommunityView: React.FunctionComponent = (props) => ( } -) +); const CommunityContainer = withCopyEdit(CommunityView); export default CommunityContainer; diff --git a/app/src/frontend/building/data-containers/construction.tsx b/app/src/frontend/building/data-containers/construction.tsx index d053f018..d72d0499 100644 --- a/app/src/frontend/building/data-containers/construction.tsx +++ b/app/src/frontend/building/data-containers/construction.tsx @@ -49,7 +49,7 @@ const ConstructionView = (props) => ( } -) +); const ConstructionContainer = withCopyEdit(ConstructionView); export default ConstructionContainer; diff --git a/app/src/frontend/building/data-containers/like.tsx b/app/src/frontend/building/data-containers/like.tsx index bd0922bc..7ee9de61 100644 --- a/app/src/frontend/building/data-containers/like.tsx +++ b/app/src/frontend/building/data-containers/like.tsx @@ -17,7 +17,7 @@ const LikeView: React.FunctionComponent = (props) => ( onLike={props.onLike} /> -) +); const LikeContainer = withCopyEdit(LikeView); export default LikeContainer; diff --git a/app/src/frontend/building/data-containers/location.tsx b/app/src/frontend/building/data-containers/location.tsx index e3e26f35..5e437b1f 100644 --- a/app/src/frontend/building/data-containers/location.tsx +++ b/app/src/frontend/building/data-containers/location.tsx @@ -114,7 +114,7 @@ const LocationView: React.FunctionComponent = (props) => ( onChange={props.onChange} /> -) +); const LocationContainer = withCopyEdit(LocationView); export default LocationContainer; diff --git a/app/src/frontend/building/data-containers/planning.tsx b/app/src/frontend/building/data-containers/planning.tsx index 38b346ea..366ac1eb 100644 --- a/app/src/frontend/building/data-containers/planning.tsx +++ b/app/src/frontend/building/data-containers/planning.tsx @@ -203,7 +203,7 @@ const PlanningView: React.FunctionComponent = (props) => ( /> -) +); const PlanningContainer = withCopyEdit(PlanningView); -export default PlanningContainer +export default PlanningContainer; diff --git a/app/src/frontend/building/data-containers/size.tsx b/app/src/frontend/building/data-containers/size.tsx index ff22e062..7ae4c2cf 100644 --- a/app/src/frontend/building/data-containers/size.tsx +++ b/app/src/frontend/building/data-containers/size.tsx @@ -147,7 +147,7 @@ const SizeView: React.FunctionComponent = (props) => ( ]} /> -) +); const SizeContainer = withCopyEdit(SizeView); export default SizeContainer; diff --git a/app/src/frontend/building/data-containers/streetscape.tsx b/app/src/frontend/building/data-containers/streetscape.tsx index c1584faa..fe163f09 100644 --- a/app/src/frontend/building/data-containers/streetscape.tsx +++ b/app/src/frontend/building/data-containers/streetscape.tsx @@ -19,7 +19,7 @@ const StreetscapeView: React.FunctionComponent = (props) => (
  • Building shading
    • -) +); const StreetscapeContainer = withCopyEdit(StreetscapeView); export default StreetscapeContainer; diff --git a/app/src/frontend/building/data-containers/sustainability.tsx b/app/src/frontend/building/data-containers/sustainability.tsx index 8ab41bd3..b44129a7 100644 --- a/app/src/frontend/building/data-containers/sustainability.tsx +++ b/app/src/frontend/building/data-containers/sustainability.tsx @@ -78,7 +78,7 @@ const SustainabilityView: React.FunctionComponent = (props) = /> ); - } + }; const SustainabilityContainer = withCopyEdit(SustainabilityView); export default SustainabilityContainer; diff --git a/app/src/frontend/building/data-containers/team.tsx b/app/src/frontend/building/data-containers/team.tsx index b2fb12eb..4e794353 100644 --- a/app/src/frontend/building/data-containers/team.tsx +++ b/app/src/frontend/building/data-containers/team.tsx @@ -25,7 +25,7 @@ const TeamView: React.FunctionComponent = (props) => ( } -) +); const TeamContainer = withCopyEdit(TeamView); export default TeamContainer; diff --git a/app/src/frontend/building/data-containers/type.tsx b/app/src/frontend/building/data-containers/type.tsx index 356b1640..0608834a 100644 --- a/app/src/frontend/building/data-containers/type.tsx +++ b/app/src/frontend/building/data-containers/type.tsx @@ -55,7 +55,7 @@ const TypeView: React.FunctionComponent = (props) => { /> ); - } + }; const TypeContainer = withCopyEdit(TypeView); export default TypeContainer; diff --git a/app/src/frontend/building/data-containers/use.tsx b/app/src/frontend/building/data-containers/use.tsx index 1e98413d..aeec0e17 100644 --- a/app/src/frontend/building/data-containers/use.tsx +++ b/app/src/frontend/building/data-containers/use.tsx @@ -32,7 +32,7 @@ const UseView: React.FunctionComponent = (props) => ( } -) +); const UseContainer = withCopyEdit(UseView); export default UseContainer; diff --git a/app/src/frontend/building/edit-history/building-edit-summary.tsx b/app/src/frontend/building/edit-history/building-edit-summary.tsx index fe5dacf5..b67ffb3f 100644 --- a/app/src/frontend/building/edit-history/building-edit-summary.tsx +++ b/app/src/frontend/building/edit-history/building-edit-summary.tsx @@ -9,7 +9,7 @@ import { CategoryEditSummary } from './category-edit-summary'; import './building-edit-summary.css'; interface BuildingEditSummaryProps { - historyEntry: EditHistoryEntry + historyEntry: EditHistoryEntry; } function formatDate(dt: Date) { @@ -47,7 +47,7 @@ const BuildingEditSummary: React.FunctionComponent = p } ); -} +}; export { BuildingEditSummary diff --git a/app/src/frontend/building/edit-history/edit-history.tsx b/app/src/frontend/building/edit-history/edit-history.tsx index ae6194ff..29dadcfb 100644 --- a/app/src/frontend/building/edit-history/edit-history.tsx +++ b/app/src/frontend/building/edit-history/edit-history.tsx @@ -41,7 +41,7 @@ const EditHistory: React.FunctionComponent = (props) => { ); -} +}; export { EditHistory diff --git a/app/src/frontend/building/multi-edit.tsx b/app/src/frontend/building/multi-edit.tsx index e3435080..6197b53c 100644 --- a/app/src/frontend/building/multi-edit.tsx +++ b/app/src/frontend/building/multi-edit.tsx @@ -20,7 +20,7 @@ interface MultiEditProps extends RouteComponentProps { const MultiEdit: React.FC = (props) => { if (!props.user){ - return + return ; } const cat = props.match.params.cat; if (cat === 'like') { @@ -46,14 +46,14 @@ const MultiEdit: React.FC = (props) => { let data: object; if (cat === 'like'){ - data = { like: true } + data = { like: true }; } else { try { // TODO: verify what happens if data is string[] data = JSON.parse(q.data as string); } catch (error) { - console.error(error, q) - data = {} + console.error(error, q); + data = {}; } } @@ -80,7 +80,7 @@ const MultiEdit: React.FC = (props) => { disabled={true} value={data[key]} /> - ) + ); })) } @@ -91,6 +91,6 @@ const MultiEdit: React.FC = (props) => { ); -} +}; export default MultiEdit; diff --git a/app/src/frontend/components/confirmation-modal.tsx b/app/src/frontend/components/confirmation-modal.tsx index 00067956..3545ac78 100644 --- a/app/src/frontend/components/confirmation-modal.tsx +++ b/app/src/frontend/components/confirmation-modal.tsx @@ -3,14 +3,14 @@ import React from 'react'; import './confirmation-modal.css'; interface ConfirmationModalProps { - show: boolean, - title: string, - description: string, - confirmButtonText?: string, - confirmButtonClass?: string, - cancelButtonClass?: string, - onConfirm: () => void, - onCancel: () => void + show: boolean; + title: string; + description: string; + confirmButtonText?: string; + confirmButtonClass?: string; + cancelButtonClass?: string; + onConfirm: () => void; + onCancel: () => void; } const ConfirmationModal: React.FunctionComponent = ({ diff --git a/app/src/frontend/components/icons.tsx b/app/src/frontend/components/icons.tsx index faf037fe..8f49b932 100644 --- a/app/src/frontend/components/icons.tsx +++ b/app/src/frontend/components/icons.tsx @@ -1,11 +1,11 @@ /** * Mini-library of icons */ -import { library } from '@fortawesome/fontawesome-svg-core' +import { library } from '@fortawesome/fontawesome-svg-core'; import { faAngleLeft, faCaretDown, faCaretRight, faCaretUp, faCheck, faCheckDouble, - faEye, faInfoCircle, faPaintBrush, faQuestionCircle, faSearch, faTimes } from '@fortawesome/free-solid-svg-icons' -import { FontAwesomeIcon } from '@fortawesome/react-fontawesome' -import React from 'react' + faEye, faInfoCircle, faPaintBrush, faQuestionCircle, faSearch, faTimes } from '@fortawesome/free-solid-svg-icons'; +import { FontAwesomeIcon } from '@fortawesome/react-fontawesome'; +import React from 'react'; library.add( faQuestionCircle, diff --git a/app/src/frontend/components/logo.tsx b/app/src/frontend/components/logo.tsx index aa932966..29a1bf13 100644 --- a/app/src/frontend/components/logo.tsx +++ b/app/src/frontend/components/logo.tsx @@ -45,6 +45,6 @@ const LogoGrid: React.FunctionComponent = () => (
    -) +); export { Logo }; diff --git a/app/src/frontend/helpers.ts b/app/src/frontend/helpers.ts index 3567d5a5..9648c708 100644 --- a/app/src/frontend/helpers.ts +++ b/app/src/frontend/helpers.ts @@ -1,27 +1,27 @@ import urlapi from 'url'; function sanitiseURL(string){ - let url_ + let url_; // http or https if (!(string.substring(0, 7) === 'http://' || string.substring(0, 8) === 'https://')){ - return null + return null; } try { - url_ = document.createElement('a') - url_.href = string + url_ = document.createElement('a'); + url_.href = string; } catch (error) { try { - url_ = urlapi.parse(string) + url_ = urlapi.parse(string); } catch (error) { - return null + return null; } } // required (www.example.com) if (!url_.hostname || url_.hostname === '' || url_.hostname === 'localhost'){ - return null + return null; } // optional (/some/path) @@ -33,7 +33,7 @@ function sanitiseURL(string){ // optional (#anchor) // url_.hash; - return `${url_.protocol}//${url_.hostname}${url_.pathname || ''}${url_.search || ''}${url_.hash || ''}` + return `${url_.protocol}//${url_.hostname}${url_.pathname || ''}${url_.search || ''}${url_.hash || ''}`; } /** @@ -63,8 +63,8 @@ function parseDate(isoUtcDate: string): Date { } function compareObjects(objA: object, objB: object): [object, object] { - const reverse = {} - const forward = {} + const reverse = {}; + const forward = {}; for (const [key, value] of Object.entries(objB)) { if (objA[key] !== value) { reverse[key] = objA[key]; diff --git a/app/src/frontend/map-app.tsx b/app/src/frontend/map-app.tsx index 90e131cc..71a61db4 100644 --- a/app/src/frontend/map-app.tsx +++ b/app/src/frontend/map-app.tsx @@ -85,7 +85,7 @@ class MapApp extends React.Component { revisionId = +revisionId; // bump revision id, only ever increasing if (revisionId > this.state.revision_id) { - this.setState({ revision_id: revisionId }) + this.setState({ revision_id: revisionId }); } } @@ -117,7 +117,7 @@ class MapApp extends React.Component { this.setState({ building: building }); } }).catch((err) => { - console.error(err) + console.error(err); this.setState({ building: building }); }); @@ -138,7 +138,7 @@ class MapApp extends React.Component { this.props.history.push(`/${mode}/${category}/${building.building_id}`); } }).catch((err) => { - console.error(err) + console.error(err); this.setState({ building_like: false }); }); } @@ -157,14 +157,14 @@ class MapApp extends React.Component { const q = parse(window.location.search); if (cat === 'like') { - this.likeBuilding(building.building_id) + this.likeBuilding(building.building_id); } else { try { // TODO: verify what happens if data is string[] const data = JSON.parse(q.data as string); - this.updateBuilding(building.building_id, data) + this.updateBuilding(building.building_id, data); } catch (error) { - console.error(error, q) + console.error(error, q); } } } @@ -181,7 +181,7 @@ class MapApp extends React.Component { res => res.json() ).then(function (res) { if (res.error) { - console.error({ error: res.error }) + console.error({ error: res.error }); } else { this.increaseRevision(res.revision_id); } @@ -202,7 +202,7 @@ class MapApp extends React.Component { res => res.json() ).then(res => { if (res.error) { - console.error({ error: res.error }) + console.error({ error: res.error }); } else { this.increaseRevision(res.revision_id); } diff --git a/app/src/frontend/map/map.tsx b/app/src/frontend/map/map.tsx index 293816c5..064fb5fe 100644 --- a/app/src/frontend/map/map.tsx +++ b/app/src/frontend/map/map.tsx @@ -9,8 +9,8 @@ import Legend from './legend'; import SearchBox from './search-box'; import ThemeSwitcher from './theme-switcher'; -import '../../../node_modules/leaflet/dist/leaflet.css' -import './map.css' +import '../../../node_modules/leaflet/dist/leaflet.css'; +import './map.css'; const OS_API_KEY = 'NVUxtY5r8eA6eIfwrPTAGKrAAsoeI9E9'; @@ -84,7 +84,7 @@ class ColouringMap extends Component { } }.bind(this)).catch( (err) => console.error(err) - ) + ); } themeSwitch(e) { diff --git a/app/src/frontend/map/search-box.tsx b/app/src/frontend/map/search-box.tsx index ed81f802..da622bba 100644 --- a/app/src/frontend/map/search-box.tsx +++ b/app/src/frontend/map/search-box.tsx @@ -11,12 +11,12 @@ interface SearchResult { label: string; zoom: number; }; - geometry: Point + geometry: Point; } interface SearchBoxProps { - onLocate: (lat: number, lng: number, zoom: number) => void + onLocate: (lat: number, lng: number, zoom: number) => void; } interface SearchBoxState { @@ -41,7 +41,7 @@ class SearchBox extends Component { collapsedSearch: true, //is this a small screen device? if not we will disable collapse option smallScreen: false - } + }; this.handleChange = this.handleChange.bind(this); this.search = this.search.bind(this); this.handleKeyPress = this.handleKeyPress.bind(this); @@ -94,7 +94,7 @@ class SearchBox extends Component { e.preventDefault(); this.setState({ fetching: true - }) + }); fetch( '/api/search?q='+this.state.q @@ -105,23 +105,23 @@ class SearchBox extends Component { this.setState({ results: data.results, fetching: false - }) + }); } else { console.error(data); this.setState({ results: [], fetching: false - }) + }); } }).catch((err) => { - console.error(err) + console.error(err); this.setState({ results: [], fetching: false - }) - }) + }); + }); } componentDidMount() { @@ -149,7 +149,7 @@ class SearchBox extends Component {
    - ) + ); } const resultsList = this.state.results.length? @@ -160,7 +160,7 @@ class SearchBox extends Component { const lng = result.geometry.coordinates[0]; const lat = result.geometry.coordinates[1]; const zoom = result.attributes.zoom; - const href = `?lng=${lng}&lat=${lat}&zoom=${zoom}` + const href = `?lng=${lng}&lat=${lat}&zoom=${zoom}`; return (
  • { href={href} >{`${label.substring(0, 4)} ${label.substring(4, 7)}`}
    • - ) + ); }) } @@ -197,7 +197,7 @@ class SearchBox extends Component { { resultsList } - ) + ); } } diff --git a/app/src/frontend/pages/about.tsx b/app/src/frontend/pages/about.tsx index f5b546c5..9b923d58 100644 --- a/app/src/frontend/pages/about.tsx +++ b/app/src/frontend/pages/about.tsx @@ -141,7 +141,7 @@ const AboutPage = () => ( onSubmit={function() {window.open( 'https://tinyletter.com/colouringlondon', 'popupwindow', - 'scrollbars=yes,width=800,height=600'); return true}}> + 'scrollbars=yes,width=800,height=600'); return true;}}>

    Keep in touch

    diff --git a/app/src/frontend/pages/contributor-agreement.tsx b/app/src/frontend/pages/contributor-agreement.tsx index f498f0a3..1b3c35f9 100644 --- a/app/src/frontend/pages/contributor-agreement.tsx +++ b/app/src/frontend/pages/contributor-agreement.tsx @@ -38,6 +38,6 @@ const ContributorAgreementPage : React.SFC = () => ( -) +); export default ContributorAgreementPage; diff --git a/app/src/frontend/pages/privacy-policy.tsx b/app/src/frontend/pages/privacy-policy.tsx index 34a8974f..1744a6e3 100644 --- a/app/src/frontend/pages/privacy-policy.tsx +++ b/app/src/frontend/pages/privacy-policy.tsx @@ -116,6 +116,6 @@ const PrivacyPolicyPage: React.SFC = () => ( -) +); export default PrivacyPolicyPage; diff --git a/app/src/frontend/user/forgotten-password.tsx b/app/src/frontend/user/forgotten-password.tsx index 344d4493..678132e4 100644 --- a/app/src/frontend/user/forgotten-password.tsx +++ b/app/src/frontend/user/forgotten-password.tsx @@ -80,6 +80,6 @@ export default class ForgottenPassword extends React.Component<{}, ForgottenPass - ) + ); } } diff --git a/app/src/frontend/user/login.tsx b/app/src/frontend/user/login.tsx index 038f6188..ca36b5ce 100644 --- a/app/src/frontend/user/login.tsx +++ b/app/src/frontend/user/login.tsx @@ -7,7 +7,7 @@ import SupporterLogos from '../components/supporter-logos'; import { User } from '../models/user'; interface LoginProps { - user: User, + user: User; login: (user: User) => void; } @@ -37,7 +37,7 @@ class Login extends Component { handleSubmit(event) { event.preventDefault(); - this.setState({error: undefined}) + this.setState({error: undefined}); fetch('/api/login', { method: 'POST', @@ -50,7 +50,7 @@ class Login extends Component { res => res.json() ).then(function(res){ if (res.error) { - this.setState({error: res.error}) + this.setState({error: res.error}); } else { fetch('/api/users/me', { credentials: 'same-origin' @@ -58,13 +58,13 @@ class Login extends Component { (res) => res.json() ).then(user => { if (user.error) { - this.setState({error: user.error}) + this.setState({error: user.error}); } else { - this.props.login(user) + this.props.login(user); } }).catch( (err) => this.setState({error: err}) - ) + ); } }.bind(this)).catch( (err) => this.setState({error: err}) @@ -73,7 +73,7 @@ class Login extends Component { render() { if (this.props.user && !this.props.user.error) { - return + return ; } return (

    @@ -130,7 +130,7 @@ class Login extends Component {
    - ) + ); } } diff --git a/app/src/frontend/user/my-account.tsx b/app/src/frontend/user/my-account.tsx index 5948ba99..dc8f6630 100644 --- a/app/src/frontend/user/my-account.tsx +++ b/app/src/frontend/user/my-account.tsx @@ -39,7 +39,7 @@ class MyAccountPage extends Component { res => res.json() ).then(function(res){ if (res.error) { - this.setState({error: res.error}) + this.setState({error: res.error}); } else { this.props.logout(); } @@ -59,7 +59,7 @@ class MyAccountPage extends Component { res => res.json() ).then(function(res){ if (res.error) { - this.setState({error: res.error}) + this.setState({error: res.error}); } else { this.props.updateUser(res); } @@ -170,7 +170,7 @@ class MyAccountPage extends Component { } else { return ( - ) + ); } } } diff --git a/app/src/frontend/user/password-reset.tsx b/app/src/frontend/user/password-reset.tsx index c23b4a11..6a06ade1 100644 --- a/app/src/frontend/user/password-reset.tsx +++ b/app/src/frontend/user/password-reset.tsx @@ -117,6 +117,6 @@ export default class PasswordReset extends React.Component - ) + ); } } diff --git a/app/src/frontend/user/signup.tsx b/app/src/frontend/user/signup.tsx index 64c1d0c9..cf422163 100644 --- a/app/src/frontend/user/signup.tsx +++ b/app/src/frontend/user/signup.tsx @@ -50,7 +50,7 @@ class SignUp extends Component { handleSubmit(event) { event.preventDefault(); - this.setState({error: undefined}) + this.setState({error: undefined}); fetch('/api/users', { method: 'POST', @@ -63,7 +63,7 @@ class SignUp extends Component { res => res.json() ).then(function(res){ if (res.error) { - this.setState({error: res.error}) + this.setState({error: res.error}); } else { fetch('/api/users/me', { credentials: 'same-origin' @@ -73,7 +73,7 @@ class SignUp extends Component { (user) => this.props.login(user) ).catch( (err) => this.setState({error: err}) - ) + ); } }.bind(this)).catch( (err) => this.setState({error: err}) @@ -82,7 +82,7 @@ class SignUp extends Component { render() { if (this.props.user) { - return + return ; } return (
    @@ -175,7 +175,7 @@ class SignUp extends Component {
    - ) + ); } } diff --git a/app/src/parse.ts b/app/src/parse.ts index a8c680c7..e85c2643 100644 --- a/app/src/parse.ts +++ b/app/src/parse.ts @@ -27,7 +27,7 @@ function parseBuildingURL(url) { const matches = re.exec(url); if (matches && matches.length >= 2) { - return strictParseInt(matches[1]) + return strictParseInt(matches[1]); } return undefined; } diff --git a/app/src/server.tsx b/app/src/server.tsx index 489133dc..8ac21919 100644 --- a/app/src/server.tsx +++ b/app/src/server.tsx @@ -37,9 +37,9 @@ const sess: any = { // TODO: remove any }; if (server.get('env') === 'production') { // trust first proxy - server.set('trust proxy', 1) + server.set('trust proxy', 1); // serve secure cookies - sess.cookie.secure = true + sess.cookie.secure = true; } server.use(session(sess)); diff --git a/app/src/tiles/tileCache.ts b/app/src/tiles/tileCache.ts index 6be7ce41..8b97eded 100644 --- a/app/src/tiles/tileCache.ts +++ b/app/src/tiles/tileCache.ts @@ -18,7 +18,7 @@ // and then use stdlib `import fs from 'fs';` import { Image } from 'mapnik'; import fs from 'node-fs'; -import { promisify } from 'util' +import { promisify } from 'util'; import { BoundingBox, TileParams } from './types'; import { formatParams, getXYZ } from './util'; @@ -113,7 +113,7 @@ class TileCache { if(!this.shouldBulkClearTileset(tileset)) continue; for (let z = this.cacheDomain.minZoom; z <= this.cacheDomain.maxZoom; z++) { - let tileBounds = getXYZ(bbox, z) + let tileBounds = getXYZ(bbox, z); for (let x = tileBounds.minX; x <= tileBounds.maxX; x++) { for (let y = tileBounds.minY; y <= tileBounds.maxY; y++) { for (const scale of this.cacheDomain.scales) { diff --git a/app/src/tiles/tileserver.ts b/app/src/tiles/tileserver.ts index 5463297b..07c99a58 100644 --- a/app/src/tiles/tileserver.ts +++ b/app/src/tiles/tileserver.ts @@ -31,7 +31,7 @@ const handleTileRequest = asyncController(async function (req: express.Request, }); // tiles router -const router = express.Router() +const router = express.Router(); router.get('/:tileset/:z/:x/:y(\\d+):scale(@\\dx)?.png', handleTileRequest); diff --git a/app/src/tiles/types.ts b/app/src/tiles/types.ts index eef82868..3d7ae7e1 100644 --- a/app/src/tiles/types.ts +++ b/app/src/tiles/types.ts @@ -44,7 +44,7 @@ type Tile = Image | Sharp; type RendererFunction = (tileParams: TileParams, dataParams: any) => Promise; interface TileRenderer { - getTile: RendererFunction + getTile: RendererFunction; } export { diff --git a/app/src/tiles/util.ts b/app/src/tiles/util.ts index 1612bb42..7afdfdb8 100644 --- a/app/src/tiles/util.ts +++ b/app/src/tiles/util.ts @@ -13,7 +13,7 @@ function getBbox(z, x, y) { } function getXYZ(bbox, z) { - return mercator.xyz(bbox, z, false, '900913') + return mercator.xyz(bbox, z, false, '900913'); } function formatParams({ tileset, z, x, y, scale }: TileParams): string { diff --git a/app/tslint.json b/app/tslint.json index f28fa159..d6d58ca2 100644 --- a/app/tslint.json +++ b/app/tslint.json @@ -13,6 +13,7 @@ { "name": "libraries", "match": ".*", "order": 10 } ] } - ] + ], + "semicolon": true } } \ No newline at end of file