From c2a6310699a700f2405d5536b8b4f794924c11d5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Aur=C3=A9lien=20Scoubeau?= <aurelien.scoubeau@gmail.com>
Date: Wed, 8 Aug 2018 16:47:19 +0200
Subject: [PATCH] All default props are factories, closes #13

---
 src/helpers/defaultHref.js      | 10 ----------
 src/helpers/description.js      |  7 +++++++
 src/helpers/href.js             | 16 ++++++++++++++++
 src/helpers/icon.js             |  7 +++++++
 src/helpers/title.js            |  6 ++++++
 src/providers/Facebook.vue      |  6 +++---
 src/providers/GooglePlus.vue    |  6 ++++--
 src/providers/Line.vue          |  6 ++++--
 src/providers/LinkedIn.vue      | 13 +++++++------
 src/providers/LiveJournal.vue   |  9 ++++++---
 src/providers/MoiMir.vue        | 17 +++++++++--------
 src/providers/Odnoklassniki.vue |  9 +++++----
 src/providers/Pinterest.vue     | 14 +++++++-------
 src/providers/Reddit.vue        |  9 ++++++---
 src/providers/Telegram.vue      |  6 +++---
 src/providers/Tumblr.vue        | 13 +++++++------
 src/providers/Twitter.vue       |  9 ++++++---
 src/providers/Viber.vue         |  6 ++++--
 src/providers/Vkontakte.vue     | 18 +++++++++---------
 src/providers/WhatsApp.vue      |  6 +++---
 20 files changed, 119 insertions(+), 74 deletions(-)
 delete mode 100644 src/helpers/defaultHref.js
 create mode 100644 src/helpers/description.js
 create mode 100644 src/helpers/href.js
 create mode 100644 src/helpers/icon.js
 create mode 100644 src/helpers/title.js

diff --git a/src/helpers/defaultHref.js b/src/helpers/defaultHref.js
deleted file mode 100644
index f904202..0000000
--- a/src/helpers/defaultHref.js
+++ /dev/null
@@ -1,10 +0,0 @@
-/**
- * default page url prop value
- * @return {String} document location href without hash
- */
-export default function defaultHref() {
-  return document.location.href.replace(
-    document.location.hash,
-    ''
-  );
-}
diff --git a/src/helpers/description.js b/src/helpers/description.js
new file mode 100644
index 0000000..1e6a70e
--- /dev/null
+++ b/src/helpers/description.js
@@ -0,0 +1,7 @@
+/**
+ * @return {String} description meta content
+ */
+ export function metaDescription() {
+  const meta = document.querySelector('meta[name="description"]')
+  return meta ? meta.content : ''
+}
diff --git a/src/helpers/href.js b/src/helpers/href.js
new file mode 100644
index 0000000..099f47c
--- /dev/null
+++ b/src/helpers/href.js
@@ -0,0 +1,16 @@
+/**
+ * @return {String} document location href without hash
+ */
+export function documentHrefWithoutHash() {
+  return document.location.href.replace(
+    document.location.hash,
+    ''
+  );
+}
+
+/**
+ * @return {String} document location
+ */
+export function documentHref() {
+  return document.location.href;
+}
diff --git a/src/helpers/icon.js b/src/helpers/icon.js
new file mode 100644
index 0000000..b5ea97a
--- /dev/null
+++ b/src/helpers/icon.js
@@ -0,0 +1,7 @@
+/**
+ * @return {String} apple-touch-icon link source
+ */
+ export function linkAppleTouchIcon() {
+  const link = document.querySelector('link[rel="apple-touch-icon"]')
+  return link ? link.src : ''
+}
diff --git a/src/helpers/title.js b/src/helpers/title.js
new file mode 100644
index 0000000..64169e5
--- /dev/null
+++ b/src/helpers/title.js
@@ -0,0 +1,6 @@
+/**
+ * @return {String} document title
+ */
+ export function documentTitle() {
+  return document.title;
+}
diff --git a/src/providers/Facebook.vue b/src/providers/Facebook.vue
index beb8347..ad2abe4 100644
--- a/src/providers/Facebook.vue
+++ b/src/providers/Facebook.vue
@@ -19,18 +19,18 @@
 </template>
 
 <script>
-  import defaultHref from '../helpers/defaultHref';
+  import {documentHrefWithoutHash} from '../helpers/href';
 
   export default {
     name: 'VueGoodshareFacebook',
     props: {
       page_url: {
         type: String,
-        default: defaultHref
+        default: documentHrefWithoutHash
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/GooglePlus.vue b/src/providers/GooglePlus.vue
index bdab89f..3a8032e 100644
--- a/src/providers/GooglePlus.vue
+++ b/src/providers/GooglePlus.vue
@@ -14,16 +14,18 @@
 </template>
 
 <script>
+  import {documentHref} from '../helpers/href';
+
   export default {
     name: 'VueGoodshareGooglePlus',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Line.vue b/src/providers/Line.vue
index 866ef20..3a98e90 100644
--- a/src/providers/Line.vue
+++ b/src/providers/Line.vue
@@ -14,16 +14,18 @@
 </template>
 
 <script>
+  import {documentHref} from '../helpers/href';
+
   export default {
     name: 'VueGoodshareLine',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/LinkedIn.vue b/src/providers/LinkedIn.vue
index 0bb33ef..a688e5a 100644
--- a/src/providers/LinkedIn.vue
+++ b/src/providers/LinkedIn.vue
@@ -21,27 +21,28 @@
 </template>
 
 <script>
-  // Variables
-  const description = document.querySelector('meta[name="description"]')
+  import {documentHref} from '../helpers/href';
+  import {documentTitle} from '../helpers/title';
+  import {metaDescription} from '../helpers/description';
   
   export default {
     name: 'VueGoodshareLinkedIn',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       page_title: {
         type: String,
-        default: document.title
+        default: documentTitle
       },
       page_description: {
         type: String,
-        default: (description) ? description.content : ''
+        default: metaDescription
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/LiveJournal.vue b/src/providers/LiveJournal.vue
index 4c3d7eb..081859f 100644
--- a/src/providers/LiveJournal.vue
+++ b/src/providers/LiveJournal.vue
@@ -15,20 +15,23 @@
 </template>
 
 <script>
+  import {documentHref} from '../helpers/href';
+  import {documentTitle} from '../helpers/title';
+
   export default {
     name: 'VueGoodshareLiveJournal',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       page_title: {
         type: String,
-        default: document.title
+        default: documentTitle
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/MoiMir.vue b/src/providers/MoiMir.vue
index 3988cbb..802b1d2 100644
--- a/src/providers/MoiMir.vue
+++ b/src/providers/MoiMir.vue
@@ -22,32 +22,33 @@
 </template>
 
 <script>
-  // Variables
-  const description = document.querySelector('meta[name="description"]')
-  const image = document.querySelector('link[rel="apple-touch-icon"]')
+  import {documentHref} from '../helpers/href';
+  import {documentTitle} from '../helpers/title';
+  import {metaDescription} from '../helpers/description';
+  import {linkAppleTouchIcon} from '../helpers/icon';
   
   export default {
     name: 'VueGoodshareMoiMir',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       page_title: {
         type: String,
-        default: document.title
+        default: documentTitle
       },
       page_description: {
         type: String,
-        default: (description) ? description.content : ''
+        default: metaDescription
       },
       page_image: {
         type: String,
-        default: (image) ? image.src : ''
+        default: linkAppleTouchIcon
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Odnoklassniki.vue b/src/providers/Odnoklassniki.vue
index b1ff441..0895e32 100644
--- a/src/providers/Odnoklassniki.vue
+++ b/src/providers/Odnoklassniki.vue
@@ -20,22 +20,23 @@
 </template>
 
 <script>
-  import defaultHref from '../helpers/defaultHref';
+  import {documentHrefWithoutHash} from '../helpers/href';
+  import {documentTitle} from '../helpers/title';
 
   export default {
     name: 'VueGoodshareOdnoklassniki',
     props: {
       page_url: {
         type: String,
-        default: defaultHref
+        default: documentHrefWithoutHash
       },
       page_title: {
         type: String,
-        default: document.title
+        default: documentTitle
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Pinterest.vue b/src/providers/Pinterest.vue
index 74bc6bf..08e911c 100644
--- a/src/providers/Pinterest.vue
+++ b/src/providers/Pinterest.vue
@@ -20,28 +20,28 @@
 </template>
 
 <script>
-  // Variables
-  const description = document.querySelector('meta[name="description"]')
-  const image = document.querySelector('link[rel="apple-touch-icon"]')
+  import {documentHref} from '../helpers/href';
+  import {metaDescription} from '../helpers/description';
+  import {linkAppleTouchIcon} from '../helpers/icon';
   
   export default {
     name: 'VueGoodsharePinterest',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       page_description: {
         type: String,
-        default: (description) ? description.content : ''
+        default: metaDescription
       },
       page_image: {
         type: String,
-        default: (image) ? image.src : ''
+        default: linkAppleTouchIcon
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Reddit.vue b/src/providers/Reddit.vue
index 2035935..7678637 100644
--- a/src/providers/Reddit.vue
+++ b/src/providers/Reddit.vue
@@ -20,20 +20,23 @@
 </template>
 
 <script>
+  import {documentHref} from '../helpers/href';
+  import {documentTitle} from '../helpers/title';
+
   export default {
     name: 'VueGoodshareReddit',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       page_title: {
         type: String,
-        default: document.title
+        default: documentTitle
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Telegram.vue b/src/providers/Telegram.vue
index 59eb8d2..cc41c06 100644
--- a/src/providers/Telegram.vue
+++ b/src/providers/Telegram.vue
@@ -14,18 +14,18 @@
 </template>
 
 <script>
-  import defaultHref from '../helpers/defaultHref';
+  import {documentHrefWithoutHash} from '../helpers/href';
 
   export default {
     name: 'VueGoodshareTelegram',
     props: {
       page_url: {
         type: String,
-        default: defaultHref
+        default: documentHrefWithoutHash
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Tumblr.vue b/src/providers/Tumblr.vue
index b093882..c901279 100644
--- a/src/providers/Tumblr.vue
+++ b/src/providers/Tumblr.vue
@@ -21,27 +21,28 @@
 </template>
 
 <script>
-  // Variables
-  const description = document.querySelector('meta[name="description"]')
+  import {documentHref} from '../helpers/href';
+  import {documentTitle} from '../helpers/title';
+  import {metaDescription} from '../helpers/description';
   
   export default {
     name: 'VueGoodshareTumblr',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       page_title: {
         type: String,
-        default: document.title
+        default: documentTitle
       },
       page_description: {
         type: String,
-        default: (description) ? description.content : ''
+        default: metaDescription
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Twitter.vue b/src/providers/Twitter.vue
index 74f8583..be3046b 100644
--- a/src/providers/Twitter.vue
+++ b/src/providers/Twitter.vue
@@ -15,20 +15,23 @@
 </template>
 
 <script>
+  import {documentHref} from '../helpers/href';
+  import {documentTitle} from '../helpers/title';
+
   export default {
     name: 'VueGoodshareTwitter',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       page_title: {
         type: String,
-        default: document.title
+        default: documentTitle
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Viber.vue b/src/providers/Viber.vue
index 8eb8eb0..c148ffa 100644
--- a/src/providers/Viber.vue
+++ b/src/providers/Viber.vue
@@ -14,16 +14,18 @@
 </template>
 
 <script>
+  import {documentHref} from '../helpers/href';
+
   export default {
     name: 'VueGoodshareviber',
     props: {
       page_url: {
         type: String,
-        default: document.location.href
+        default: documentHref
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/Vkontakte.vue b/src/providers/Vkontakte.vue
index 1ed1055..1e89c30 100644
--- a/src/providers/Vkontakte.vue
+++ b/src/providers/Vkontakte.vue
@@ -22,33 +22,33 @@
 </template>
 
 <script>
-  import defaultHref from '../helpers/defaultHref';
-  // Variables
-  const description = document.querySelector('meta[name="description"]')
-  const image = document.querySelector('link[rel="apple-touch-icon"]')
+  import {documentHref} from '../helpers/href';
+  import {documentTitle} from '../helpers/title';
+  import {metaDescription} from '../helpers/description';
+  import {linkAppleTouchIcon} from '../helpers/icon';
   
   export default {
     name: 'VueGoodshareVkontakte',
     props: {
       page_url: {
         type: String,
-        default: defaultHref
+        default: documentHref
       },
       page_title: {
         type: String,
-        default: document.title
+        default: documentTitle
       },
       page_description: {
         type: String,
-        default: (description) ? description.content : ''
+        default: metaDescription
       },
       page_image: {
         type: String,
-        default: (image) ? image.src : ''
+        default: linkAppleTouchIcon
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,
diff --git a/src/providers/WhatsApp.vue b/src/providers/WhatsApp.vue
index fc15755..d627eb2 100644
--- a/src/providers/WhatsApp.vue
+++ b/src/providers/WhatsApp.vue
@@ -14,18 +14,18 @@
 </template>
 
 <script>
-  import defaultHref from '../helpers/defaultHref';
+  import {documentHref} from '../helpers/href';
 
   export default {
     name: 'VueGoodshareWhatsApp',
     props: {
       page_url: {
         type: String,
-        default: defaultHref
+        default: documentHref
       },
       button_design: {
         type: String,
-        default: 'flat'
+        default: () => 'flat'
       },
       title_social: String,
       has_icon: Boolean,