From 2720b6596241874df330373645952f6a53ebeb5c Mon Sep 17 00:00:00 2001
From: Baptiste Lemoine <contact@cipherbliss.com>
Date: Thu, 16 Jan 2020 11:33:13 +0100
Subject: [PATCH] :zap: check for uniqueness of the slug, slug the title by
 default

---
 src/app/config/PollConfig.ts                  |  4 +-
 .../visibility/visibility.component.html      |  5 +-
 .../pages/visibility/visibility.component.ts  | 24 +------
 src/app/services/config.service.ts            | 64 ++++++++++++++++++-
 src/assets/scss/atoms/_forms.scss             |  2 +-
 5 files changed, 71 insertions(+), 28 deletions(-)

diff --git a/src/app/config/PollConfig.ts b/src/app/config/PollConfig.ts
index 6a620997..102a55e5 100644
--- a/src/app/config/PollConfig.ts
+++ b/src/app/config/PollConfig.ts
@@ -59,12 +59,14 @@ export class PollConfig {
     // access
     visibility = 'link_only'; // visible to anyone with the link:
     voteChoices = 'only_yes'; // possible answers to a vote choice: only "yes", "yes, maybe, no"
+    creationDate = new Date();
     expirationDate = ''; // expiracy date
     pollId = null; // id of the current poll when created. data given by the backend api
     selectedPoll = null; // current poll selected with createPoll or getPoll of ConfigService
     passwordAccess = 0;
     password = '';
-    customUrl = '';
+    customUrl = ''; // custom slug in the url, must be unique
+    customUrlIsUnique = null; // given by the backend
     urlPublic = environment.baseApiHref + '/default-url';
     urlAdmin = environment.baseApiHref + '/default-url/admin/d65es45fd45sdf45sd345f312sdf31sgfd345';
     canModifyAnswers = 1;// everybody, self, nobody (= just admin)
diff --git a/src/app/pages/visibility/visibility.component.html b/src/app/pages/visibility/visibility.component.html
index 89c9844c..314f3a9c 100644
--- a/src/app/pages/visibility/visibility.component.html
+++ b/src/app/pages/visibility/visibility.component.html
@@ -107,11 +107,14 @@
            class="input-lg"
            name="url"
            id="url"
-           [value]="baseUrl+'/'+config.customUrl">
+           [(ngModel)]="config.customUrl">
 
     <sub class="instructions">
         {{"visibility.access_instructions"|translate}}
     </sub>
+    <div class="preview-url">
+        {{baseUrl + '/' + config.customUrl}}
+    </div>
     <br>
     <label for="passwordAccess">
         {{"visibility.access_want"|translate}}
diff --git a/src/app/pages/visibility/visibility.component.ts b/src/app/pages/visibility/visibility.component.ts
index 75898128..323c4e5f 100644
--- a/src/app/pages/visibility/visibility.component.ts
+++ b/src/app/pages/visibility/visibility.component.ts
@@ -17,29 +17,9 @@ export class VisibilityComponent extends BaseComponent implements OnInit {
     }
 
     ngOnInit() {
-        this.config.customUrl = this.makeUuid();
-        this.config.expirationDate = (this.addDaysToDate(this.config.expiracyDateDefaultInDays, new Date())).toISOString().substring(0, 10);
+        this.config.customUrl = this.config.makeSlug();
+        this.config.expirationDate = (this.config.addDaysToDate(this.config.expiracyDateDefaultInDays, new Date())).toISOString().substring(0, 10);
     }
 
-    /**
-     * generate unique id to have a default url for future poll
-     */
-    makeUuid() {
-        return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, function (c) {
-            var r = Math.random() * 16 | 0, v = c == 'x' ? r : (r & 0x3 | 0x8);
-            return v.toString(16);
-        });
-    }
-
-    /**
-     * add some days to a date, to compute intervals
-     * @param days
-     * @param date
-     */
-    addDaysToDate(days: number, date: Date) {
-        date = new Date(date.valueOf());
-        date.setDate(date.getDate() + days);
-        return date;
-    };
 
 }
diff --git a/src/app/services/config.service.ts b/src/app/services/config.service.ts
index 3a889a90..9a008f05 100644
--- a/src/app/services/config.service.ts
+++ b/src/app/services/config.service.ts
@@ -30,6 +30,48 @@ export class ConfigService extends PollConfig {
         this.messageService.clear();
     }
 
+    /**
+     * generate unique id to have a default url for future poll
+     */
+    makeUuid() {
+        return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, function (c) {
+            var r = Math.random() * 16 | 0, v = c == 'x' ? r : (r & 0x3 | 0x8);
+            return v.toString(16);
+        });
+    }
+
+    makeSlug(str?: string) {
+        if (!str) {
+            str = this.creationDate.getFullYear() + '_' + (this.creationDate.getMonth() + 1) + '_' + this.creationDate.getDate() + '_' + this.myName + '_' + this.title;
+        }
+        str = str.replace(/^\s+|\s+$/g, ''); // trim
+        str = str.toLowerCase();
+
+        // remove accents, swap ñ for n, etc
+        var from = "àáäâèéëêìíïîòóöôùúüûñç·/_,:;";
+        var to = "aaaaeeeeiiiioooouuuunc------";
+        for (var i = 0, l = from.length; i < l; i++) {
+            str = str.replace(new RegExp(from.charAt(i), 'g'), to.charAt(i));
+        }
+
+        str = str.replace(/[^a-z0-9 -]/g, '') // remove invalid chars
+            .replace(/\s+/g, '-') // collapse whitespace and replace by -
+            .replace(/-+/g, '-'); // collapse dashes
+
+        return str;
+    }
+
+    /**
+     * add some days to a date, to compute intervals
+     * @param days
+     * @param date
+     */
+    addDaysToDate(days: number, date: Date) {
+        date = new Date(date.valueOf());
+        date.setDate(date.getDate() + days);
+        return date;
+    };
+
     /** ==================================
      *
      *  poll public calls to get non authenticated info
@@ -86,6 +128,22 @@ export class ConfigService extends PollConfig {
         return requestOptions;
     }
 
+    checkIfSlugIsUniqueInDatabase(slug: string) {
+        this.customUrlIsUnique = null;
+
+        this.loading = true;
+        this.http.get(`${this.baseHref}/check-slug-is-uniq`,
+            this.makeHeaders({slug: this.customUrl}),
+        )
+            .subscribe((res: any) => {
+
+                    this.customUrlIsUnique = res.data.isUnique;
+                    this.loading = false;
+                },
+                (e) => this.handleError(e))
+        ;
+    }
+
     /**
      * search in localstorage, fallback asking the backend to send an email to the owner if it exists
      * @param email
@@ -105,9 +163,9 @@ export class ConfigService extends PollConfig {
         )
             .subscribe(res => {
                     //     message: 'Trouvé! Allez voir votre boite email',
-                    this.myPolls = res;
-                    console.log('res', res)
-                    this.loading = false;
+                this.myPolls = res;
+                console.log('res', res);
+                this.loading = false;
                     this.messageService.add({
                         severity: 'success',
                         summary: 'Service Message',
diff --git a/src/assets/scss/atoms/_forms.scss b/src/assets/scss/atoms/_forms.scss
index 741bff52..eafe30d7 100644
--- a/src/assets/scss/atoms/_forms.scss
+++ b/src/assets/scss/atoms/_forms.scss
@@ -51,7 +51,7 @@ option {
 
     background-color: transparent;
     background-image: url("./assets/img/fleche_bas.svg");
-    padding-right: 1.5rem;
+    padding-right: 2.5rem;
 
     background-repeat: no-repeat;
     background-size: 9px 8px;