From 704000f08d738b3b40d31a600dff45239548c26d Mon Sep 17 00:00:00 2001
From: Tobias Speicher <rootcommander@gmail.com>
Date: Thu, 24 Mar 2022 15:34:26 +0100
Subject: [PATCH] Replace deprecated String.prototype.substr()

.substr() is deprecated so we replace it with .slice() which works similarily but isn't deprecated

Signed-off-by: Tobias Speicher <rootcommander@gmail.com>
---
 .eslintrc.js              |  4 ++++
 src/contactsController.js | 16 ++++++++--------
 src/utils/mapUtils.js     |  4 ++--
 3 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 6ee99ecd..f68954d9 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -9,6 +9,10 @@ module.exports = {
 		'@nextcloud'
 	],
 	rules: {
+		'no-restricted-properties': [
+			'error',
+			{ property: 'substr', message: 'Use .slice instead of .substr.' },
+		],
 		'jsdoc/require-jsdoc': 'off',
 		'jsdoc/tag-lines': 'off',
 		'vue/first-attribute-linebreak': 'off'
diff --git a/src/contactsController.js b/src/contactsController.js
index c12091a1..119e07fb 100644
--- a/src/contactsController.js
+++ b/src/contactsController.js
@@ -416,8 +416,8 @@ ContactsController.prototype = {
         for (var i = 0; i < contacts.length; i++) {
 
             var geo = [];
-            if (contacts[i].GEO.substr(0,4) === "geo:") {
-                geo = contacts[i].GEO.substr(4).split(",");
+            if (contacts[i].GEO.slice(0,4) === "geo:") {
+                geo = contacts[i].GEO.slice(4).split(",");
             } else {
                 geo = contacts[i].GEO.split(";");
             }
@@ -429,12 +429,12 @@ ContactsController.prototype = {
                 date = new Date();
             }
             if (isNaN(date)) {
-                var year = parseInt(contacts[i].REV.substr(0,4));
-                var month = parseInt(contacts[i].REV.substr(4,2))-1;
-                var day = parseInt(contacts[i].REV.substr(6,2))-1;
-                var hour = parseInt(contacts[i].REV.substr(9,2))-1;
-                var min = parseInt(contacts[i].REV.substr(11,2))-1;
-                var sec = parseInt(contacts[i].REV.substr(13,2))-1;
+                var year = parseInt(contacts[i].REV.slice(0,4));
+                var month = parseInt(contacts[i].REV.slice(4,6))-1;
+                var day = parseInt(contacts[i].REV.slice(6,8))-1;
+                var hour = parseInt(contacts[i].REV.slice(9,11))-1;
+                var min = parseInt(contacts[i].REV.slice(11,13))-1;
+                var sec = parseInt(contacts[i].REV.slice(13,15))-1;
                 date = new Date(year,month,day,hour,min,sec);
                 date = date.getTime();
             }
diff --git a/src/utils/mapUtils.js b/src/utils/mapUtils.js
index 5a2be9cc..fb77471c 100644
--- a/src/utils/mapUtils.js
+++ b/src/utils/mapUtils.js
@@ -60,9 +60,9 @@ export const getShouldMapUseImperial = () => {
 
 export const geoToLatLng = (geo) => {
 	let ll
-	const fourFirsts = geo.substr(0, 4)
+	const fourFirsts = geo.slice(0, 4)
 	if (fourFirsts === 'geo:') {
-		ll = geo.substr(4).split(',')
+		ll = geo.slice(4).split(',')
 	} else {
 		ll = geo.split(';')
 	}