Revize 6c9cb54a

Re #8475 - Added code comments

static/js/create_certificate.js

@@ +1 @@
+
+    // VUE instance of certificate creation page
     var createCertificateApp = new Vue({
             el: "#create-certificate-content",
             data: {
-                authorities: [{id: 5, CN: "Test"}],
+                // available certificate authorities
+                authorities: [],
+                // data of the selected certificate authorities to be displayed in the form
                 selectedCAData: {
                     CN: "",
                     C:  "",
@@ -11 +15 @@
                     OU: "",
                     emailAddress: ""
                 },
+                // Data of the new certificate to be created received from the input fields
                 certificateData: {
                     subject: {
                         CN: "",
@@ -32 +37 @@
                     CA: null
                 }
             },
+            // actions to be performed when the page is loaded
+            // - initialize notBefore and notAfter with current date and current date + 1 month respectively
             mounted () {
-              this.certificateData.notBefore = new Date().toDateInputValue();
+              this.certificateData.notBefore = new Date().toDateInputValue(); // init notBefore to current date
               var endDate = new Date(new Date().getTime()+(30*24*60*60*1000));
-              this.certificateData.notAfter = endDate.toDateInputValue();
+              this.certificateData.notAfter = endDate.toDateInputValue(); // init notAfter to notBefore + 30 days
             },
             methods: {
+                // handle certificate creation request
                 onCreateCertificate: function () {
-
+                    // validate input data
+                    // - validate if subject CN is filled in
+                    if(this.certificateData.subject.CN === "") {
+                        alert("CN field must be filled in!"); // TODO highlight the field
+                        return;
+                    }
+                    // - validate if notAfter is > notBefore
+                    if(new Date(this.certificateData.notBefore) > new Date(this.certificateData.notAfter)) {
+                        alert("notBefore must chronologically lower than notAfter"); // TODO highlight the field
+                        return;
+                    }
+                    // send the request and handle response
                     axios.post(API_URL + "certificates", this.certificateData)
                         .then(function (response) {
+                            // on success return to index page
                             if(response.data["success"]) {
                                 alert("Certificate was successfully created.");
                                 window.location.href = "/static/index.html";
                             }
+                            // on error display server response message
                             else {
                                 alert(response.data["data"]);
                             }
@@ -55 +76 @@
                         });
                 }
             },
+            // data watches
             watch: {
+                // if the selected CA is changed, the Issuer input fileds must be filled in
                 'certificateData.CA': function (val, oldVal) {
+                    // self-signed certificate - all fields are empty
                     if (val === "null" || val == null) {
                         createCertificateApp.selectedCAData = {
                             CN: "",
@@ -68 +92 @@
                             emailAddress: ""
                         };
                     }
+                    // a CA is selected - get CA's details and display them
                     else {
                         axios.get(API_URL + "certificates/" + val + "/details")
                             .then(function (response) {
@@ -85 +110 @@
             }
         });
 
+    // Initialize available CA select values
     axios.get(API_URL+"certificates", {
             params: {
                 filtering: {