From ae341afe9701e870569b28cf147caa260289413b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20Molakvo=C3=A6=20=28skjnldsv=29?=
 <skjnldsv@protonmail.com>
Date: Fri, 31 Aug 2018 09:35:44 +0200
Subject: Delete property
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 src/components/ContactDetails/ContactDetailsProperty.vue | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'src')

diff --git a/src/components/ContactDetails/ContactDetailsProperty.vue b/src/components/ContactDetails/ContactDetailsProperty.vue
index 4260f447..fa547d40 100644
--- a/src/components/ContactDetails/ContactDetailsProperty.vue
+++ b/src/components/ContactDetails/ContactDetailsProperty.vue
@@ -217,8 +217,11 @@ export default {
 	},
 
 	methods: {
+		/**
+		 * Delete this property
+		 */
 		deleteProp() {
-			alert('deleted')
+			this.contact.vCard.removeProperty(this.property)
 		}
 	}
 
-- 
cgit v1.2.3


From 33d002b2d98fa4267d53b189bc3a4b957e8bbe38 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20Molakvo=C3=A6=20=28skjnldsv=29?=
 <skjnldsv@protonmail.com>
Date: Fri, 31 Aug 2018 11:55:57 +0200
Subject: New property component
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 src/components/ContactDetails.vue                  | 12 ++--
 .../ContactDetails/ContactDetailsAddNewProp.vue    | 68 ++++++++++++++++++++++
 2 files changed, 76 insertions(+), 4 deletions(-)
 create mode 100644 src/components/ContactDetails/ContactDetailsAddNewProp.vue

(limited to 'src')

diff --git a/src/components/ContactDetails.vue b/src/components/ContactDetails.vue
index cae7cd9c..1d975c14 100644
--- a/src/components/ContactDetails.vue
+++ b/src/components/ContactDetails.vue
@@ -82,13 +82,15 @@
 			<section class="contact-details">
 
 				<!-- properties iteration -->
-				<contact-details-property v-for="(property, index) in sortedProperties" :key="index" :index="index"
+				<contact-property v-for="(property, index) in sortedProperties" :key="index" :index="index"
 					:sorted-properties="sortedProperties" :property="property" :contact="contact"
 					@updatedcontact="updateContact" />
 
 				<!-- addressbook change select -->
 				<property-select :prop-model="addressbookModel" :value.sync="addressbook"
 					:options="addressbooksOptions" class="property--addressbooks" />
+				
+				<add-new-prop :contact="contact" />
 			</section>
 		</template>
 	</div>
@@ -104,7 +106,8 @@ import Contact from '../models/contact'
 import rfcProps from '../models/rfcProps.js'
 
 import PopoverMenu from './core/popoverMenu'
-import ContactDetailsProperty from './ContactDetails/ContactDetailsProperty'
+import ContactProperty from './ContactDetails/ContactDetailsProperty'
+import AddNewProp from './ContactDetails/ContactDetailsAddNewProp'
 import PropertySelect from './Properties/PropertySelect'
 import PropertyGroups from './Properties/PropertyGroups'
 
@@ -115,9 +118,10 @@ export default {
 
 	components: {
 		PopoverMenu,
-		ContactDetailsProperty,
+		ContactProperty,
 		PropertySelect,
-		PropertyGroups
+		PropertyGroups,
+		AddNewProp
 	},
 
 	directives: {
diff --git a/src/components/ContactDetails/ContactDetailsAddNewProp.vue b/src/components/ContactDetails/ContactDetailsAddNewProp.vue
new file mode 100644
index 00000000..86eb4b77
--- /dev/null
+++ b/src/components/ContactDetails/ContactDetailsAddNewProp.vue
@@ -0,0 +1,68 @@
+<!--
+  - @copyright Copyright (c) 2018 John Molakvoæ <skjnldsv@protonmail.com>
+  -
+  - @author John Molakvoæ <skjnldsv@protonmail.com>
+  -
+  - @license GNU AGPL version 3 or any later version
+  -
+  - This program is free software: you can redistribute it and/or modify
+  - it under the terms of the GNU Affero General Public License as
+  - published by the Free Software Foundation, either version 3 of the
+  - License, or (at your option) any later version.
+  -
+  - This program is distributed in the hope that it will be useful,
+  - but WITHOUT ANY WARRANTY; without even the implied warranty of
+  - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+  - GNU Affero General Public License for more details.
+  -
+  - You should have received a copy of the GNU Affero General Public License
+  - along with this program. If not, see <http://www.gnu.org/licenses/>.
+  -
+  -->
+
+<template>
+	<div class="grid-span-1 property">
+		<div class="property__row">
+
+			<div class="property__label">{{ t('contacts', 'Add new property') }}</div>
+
+			<!-- type selector -->
+			<multiselect :options="availableProperties" :placeholder="t('contacts', 'Choose property type')"
+				class="multiselect-vue property__value"	@input="addProp" />
+		</div>
+	</div>
+</template>
+
+<script>
+import rfcProps from '../../models/rfcProps.js'
+import Contact from '../../models/contact'
+
+import Multiselect from 'vue-multiselect'
+
+export default {
+	name: 'ContactDetailsAddNewProp',
+
+	components: {
+		Multiselect
+	},
+
+	props: {
+		contact: {
+			type: Contact,
+			default: null
+		}
+	},
+
+	computed: {
+		availableProperties() {
+			return Object.keys(rfcProps.properties)
+		}
+	},
+
+	methods: {
+		addProp(name) {
+			console.log(name);
+		}
+	}
+}
+</script>
-- 
cgit v1.2.3


From 972c17ca34f287590b17a039e84b9d5766ad465a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20Molakvo=C3=A6=20=28skjnldsv=29?=
 <skjnldsv@protonmail.com>
Date: Mon, 3 Sep 2018 13:26:59 +0200
Subject: 2.1 vcard, new component layout and methods
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 src/components/ContactDetails.vue                  |  2 +-
 .../ContactDetails/ContactDetailsAddNewProp.vue    | 28 ++++++++++++++++------
 src/store/FakeName.vcf                             | 10 ++++++++
 3 files changed, 32 insertions(+), 8 deletions(-)

(limited to 'src')

diff --git a/src/components/ContactDetails.vue b/src/components/ContactDetails.vue
index 1d975c14..fdaf521a 100644
--- a/src/components/ContactDetails.vue
+++ b/src/components/ContactDetails.vue
@@ -89,7 +89,7 @@
 				<!-- addressbook change select -->
 				<property-select :prop-model="addressbookModel" :value.sync="addressbook"
 					:options="addressbooksOptions" class="property--addressbooks" />
-				
+
 				<add-new-prop :contact="contact" />
 			</section>
 		</template>
diff --git a/src/components/ContactDetails/ContactDetailsAddNewProp.vue b/src/components/ContactDetails/ContactDetailsAddNewProp.vue
index 86eb4b77..1efcfb43 100644
--- a/src/components/ContactDetails/ContactDetailsAddNewProp.vue
+++ b/src/components/ContactDetails/ContactDetailsAddNewProp.vue
@@ -21,14 +21,17 @@
   -->
 
 <template>
-	<div class="grid-span-1 property">
-		<div class="property__row">
+	<div class="grid-span-2 property">
+
+		<!-- title -->
+		<property-title :icon="'icon-add'" :readable-name="t('contacts', 'Add new property')" />
 
+		<div class="property__row">
 			<div class="property__label">{{ t('contacts', 'Add new property') }}</div>
 
 			<!-- type selector -->
-			<multiselect :options="availableProperties" :placeholder="t('contacts', 'Choose property type')"
-				class="multiselect-vue property__value"	@input="addProp" />
+			<multiselect :options="availableProperties" :placeholder="t('contacts', 'Choose property type')" class="multiselect-vue property__value"
+				track-by="id" label="name" @input="addProp" />
 		</div>
 	</div>
 </template>
@@ -36,6 +39,7 @@
 <script>
 import rfcProps from '../../models/rfcProps.js'
 import Contact from '../../models/contact'
+import propertyTitle from '../Properties/PropertyTitle'
 
 import Multiselect from 'vue-multiselect'
 
@@ -43,6 +47,7 @@ export default {
 	name: 'ContactDetailsAddNewProp',
 
 	components: {
+		propertyTitle,
 		Multiselect
 	},
 
@@ -55,13 +60,22 @@ export default {
 
 	computed: {
 		availableProperties() {
-			return Object.keys(rfcProps.properties)
+			return Object.keys(rfcProps.properties).map(key => {
+				return {
+					id: key,
+					name: rfcProps.properties[key].readableName
+				}
+			})
 		}
 	},
 
 	methods: {
-		addProp(name) {
-			console.log(name);
+		addProp({ id }) {
+			let defaultData = rfcProps.properties[id].defaultValue
+			let property = this.contact.vCard.addPropertyWithValue(id, defaultData ? defaultData.value : '')
+			if (defaultData && defaultData.type) {
+				property.setParameter('type', defaultData.type)
+			}
 		}
 	}
 }
diff --git a/src/store/FakeName.vcf b/src/store/FakeName.vcf
index cae71a7b..c21255de 100644
--- a/src/store/FakeName.vcf
+++ b/src/store/FakeName.vcf
@@ -9,6 +9,16 @@ REV:2017-07-27T05:56:33Z
 UID:5acf667e-1cbf-48a8-87fe-546ee31a0b23
 END:VCARD
 BEGIN:VCARD
+VERSION:2.1
+FN:Jean Dupont
+N:Dupont;Jean
+ADR;WORK;PREF;QUOTED-PRINTABLE:;Bruxelles 1200=Belgique;6A Rue Th. Decuyper
+LABEL;QUOTED-PRINTABLE;WORK;PREF:Rue Th. Decuyper 6A=Bruxelles 1200=Belgique
+TEL;CELL:+1234 56789
+EMAIL;INTERNET:jean.dupont@example.com
+UID:dfs541fds15
+END:VCARD
+BEGIN:VCARD
 VERSION:3.0
 N:Cunningham;Liam;;Mr.;
 FN:Liam Cunningham
-- 
cgit v1.2.3


From cf0a8ff7d4cfbaa11db443816c228d164fae61f8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20Molakvo=C3=A6=20=28skjnldsv=29?=
 <skjnldsv@protonmail.com>
Date: Mon, 3 Sep 2018 13:29:06 +0200
Subject: ParceVcf fix on failed vCard
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 src/services/parseVcf.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/services/parseVcf.js b/src/services/parseVcf.js
index e74b083f..65d5f95c 100644
--- a/src/services/parseVcf.js
+++ b/src/services/parseVcf.js
@@ -35,17 +35,19 @@ export default function parseVcf(data = '', addressbook) {
 
 	importState.total = vCards.length
 
-	return vCards.map(vCard => {
+	// Not using map because we want to only push valid contacts
+	// map force to return at least undefined
+	return vCards.reduce((contacts, vCard) => {
 		try {
 			// console.log(vCards.indexOf(vCard))
 			let contact = new Contact(vCard, addressbook)
 			importState.accepted++
-			return contact
+			contacts.push(contact)
 		} catch (e) {
 			// Parse error! Do not stop here...
 			importState.denied++
-			// eslint-disable-next-line
 			console.error(e)
 		}
-	})
+		return contacts
+	}, [])
 }
-- 
cgit v1.2.3


From d2f266e3b508399ced055970d0345747318bb665 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20Molakvo=C3=A6=20=28skjnldsv=29?=
 <skjnldsv@protonmail.com>
Date: Mon, 3 Sep 2018 17:32:33 +0200
Subject: Fix groups parsing and fix add new prop layout
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 .../ContactDetails/ContactDetailsAddNewProp.vue    |  2 +-
 src/models/contact.js                              |  6 ++---
 src/store/FakeName.vcf                             | 14 ++++++++++-
 src/store/groups.js                                | 28 ++++++++++++----------
 4 files changed, 33 insertions(+), 17 deletions(-)

(limited to 'src')

diff --git a/src/components/ContactDetails/ContactDetailsAddNewProp.vue b/src/components/ContactDetails/ContactDetailsAddNewProp.vue
index 1efcfb43..5d87ceab 100644
--- a/src/components/ContactDetails/ContactDetailsAddNewProp.vue
+++ b/src/components/ContactDetails/ContactDetailsAddNewProp.vue
@@ -27,7 +27,7 @@
 		<property-title :icon="'icon-add'" :readable-name="t('contacts', 'Add new property')" />
 
 		<div class="property__row">
-			<div class="property__label">{{ t('contacts', 'Add new property') }}</div>
+			<div class="property__label" />
 
 			<!-- type selector -->
 			<multiselect :options="availableProperties" :placeholder="t('contacts', 'Choose property type')" class="multiselect-vue property__value"
diff --git a/src/models/contact.js b/src/models/contact.js
index e5e9e83e..438b1f22 100644
--- a/src/models/contact.js
+++ b/src/models/contact.js
@@ -120,9 +120,9 @@ export default class Contact {
 	 * @memberof Contact
 	 */
 	get groups() {
-		let prop = this.vCard.getFirstProperty('categories')
-		if (prop) {
-			return this.vCard.getFirstProperty('categories').getValues()
+		let groupsProp = this.vCard.getFirstProperty('categories')
+		if (groupsProp) {
+			return groupsProp.getValues()
 		}
 		return []
 	}
diff --git a/src/store/FakeName.vcf b/src/store/FakeName.vcf
index c21255de..df963b77 100644
--- a/src/store/FakeName.vcf
+++ b/src/store/FakeName.vcf
@@ -8,6 +8,7 @@ MEMBER:urn:uuid:cd314b39-b71a-41cf-b4eb-5cb61dd6fa24
 REV:2017-07-27T05:56:33Z
 UID:5acf667e-1cbf-48a8-87fe-546ee31a0b23
 END:VCARD
+
 BEGIN:VCARD
 VERSION:2.1
 FN:Jean Dupont
@@ -18,11 +19,12 @@ TEL;CELL:+1234 56789
 EMAIL;INTERNET:jean.dupont@example.com
 UID:dfs541fds15
 END:VCARD
+
 BEGIN:VCARD
 VERSION:3.0
 N:Cunningham;Liam;;Mr.;
 FN:Liam Cunningham
-NICKNAME:Begivaing81
+NICKNAME;TYPE=work:Begivaing81
 BDAY;VALUE=text:5/26/1981
 GENDER:male
 ORG:Central Hardware;
@@ -32,6 +34,16 @@ CATEGORIES:Tech,University
 TEL;TYPE=VOICE,HOME;VALUE=text:06-22957835
 ADR;TYPE=HOME:;;Beilen;DR;9413 BA;Netherlands
 END:VCARD
+
+BEGIN:VCARD
+VERSION:4.0
+FN:Simon Perreault
+N:Perreault;Simon;;;ing. jr,M.Sc.
+BDAY:--0203
+ANNIVERSARY:20090808T1430-0500
+GENDER:M
+END:VCARD
+
 BEGIN:VCARD
 VERSION:3.0
 N:Kamiński;Przemysł;;Mr.;
diff --git a/src/store/groups.js b/src/store/groups.js
index 9c3061be..e1ed9721 100644
--- a/src/store/groups.js
+++ b/src/store/groups.js
@@ -35,18 +35,22 @@ const mutations = {
 	appendGroupsFromContacts(state, contacts) {
 		// init groups list
 		let groups = Object.values(contacts)
-			.map(contact => contact.groups.map(group => {
-				// extend group to model
-				return {
-					name: group,
-					contacts: []
-				}
-			})[0])
-		// ensure we only have one group of each
-		state.groups = state.groups.concat(groups)
-			.filter(function(group, index, self) {
-				return group && self.findIndex(search => search && search.name === group.name) === index
-			})
+			// iterate on every contacts
+			.reduce((groups, contact) => {
+				// transform group names into Object
+				contact.groups.map(groupName => {
+					// overriding existing groups: remove duplicates
+					groups[groupName] = {
+						name: groupName,
+						contacts: []
+					}
+				})
+				return groups
+			}, {})
+
+		// store in state
+		state.groups = Object.values(groups)
+
 		// append keys to groups
 		Object.values(contacts)
 			.forEach(contact => {
-- 
cgit v1.2.3