Implements deleting notes and switching between public and private.

app/Events/WorkOrderNoteDeleted.php

@@ -0,0 +1,40 @@
+<?php
+
+namespace App\Events;
+
+use Illuminate\Broadcasting\Channel;
+use Illuminate\Broadcasting\InteractsWithSockets;
+use Illuminate\Broadcasting\PresenceChannel;
+use Illuminate\Broadcasting\PrivateChannel;
+use Illuminate\Contracts\Broadcasting\ShouldBroadcast;
+use Illuminate\Foundation\Events\Dispatchable;
+use Illuminate\Queue\SerializesModels;
+use App\WorkOrderNote;
+
+class WorkOrderNoteDeleted implements ShouldBroadcast
+{
+    use Dispatchable, InteractsWithSockets, SerializesModels;
+
+    public $note;
+
+    /**
+     * Create a new event instance.
+     *
+     * @return void
+     */
+    public function __construct($note)
+    {
+        $array = json_decode($note, true);
+        $this->note = $array;
+    }
+
+    /**
+     * Get the channels the event should broadcast on.
+     *
+     * @return \Illuminate\Broadcasting\Channel|array
+     */
+    public function broadcastOn()
+    {
+        return new Channel('wonotes.'.$this->note['woid']);
+    }
+}

app/Http/Controllers/Api/WorkOrderNotesController.php

@@ -57,6 +57,7 @@ class WorkOrderNotesController extends Controller
     public function update(Request $request, WorkOrderNote $workOrderNote)
     {
         $workOrderNote->thenote = $request->input('thenote');
+        $workOrderNote->notetype = $request->input('notetype');
         $workOrderNote->save();
         event(new \App\Events\WorkOrderNoteUpdated($workOrderNote));
         return response()->json($workOrderNote, 200);
@@ -70,6 +71,9 @@ class WorkOrderNotesController extends Controller
      */
     public function destroy(WorkOrderNote $workOrderNote)
     {
-        //
+        $deletedNote = $workOrderNote->toJson();
+        $workOrderNote->delete();
+        event(new \App\Events\WorkOrderNoteDeleted($deletedNote));
+        return response()->json("", 204);
     }
 }

public/js/app.js

@@ -2597,6 +2597,74 @@ __webpack_require__.r(__webpack_exports__);
 
 /***/ }),
 
+/***/ "./node_modules/babel-loader/lib/index.js?!./node_modules/vue-loader/lib/index.js?!./resources/js/components/NoteDeleteModal.vue?vue&type=script&lang=js&":
+/*!**************************************************************************************************************************************************************************!*\
+  !*** ./node_modules/babel-loader/lib??ref--4-0!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/components/NoteDeleteModal.vue?vue&type=script&lang=js& ***!
+  \**************************************************************************************************************************************************************************/
+/*! exports provided: default */
+/***/ (function(module, __webpack_exports__, __webpack_require__) {
+
+"use strict";
+__webpack_require__.r(__webpack_exports__);
+/* harmony import */ var _Modal_vue__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! ./Modal.vue */ "./resources/js/components/Modal.vue");
+/* harmony import */ var _CircleSpinner_vue__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! ./CircleSpinner.vue */ "./resources/js/components/CircleSpinner.vue");
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+
+
+/* harmony default export */ __webpack_exports__["default"] = ({
+  components: {
+    Modal: _Modal_vue__WEBPACK_IMPORTED_MODULE_0__["default"],
+    CircleSpinner: _CircleSpinner_vue__WEBPACK_IMPORTED_MODULE_1__["default"]
+  },
+  props: {
+    modalId: {
+      type: String,
+      require: true
+    },
+    note: {
+      type: Object,
+      required: true
+    }
+  },
+  data: function data() {
+    return {
+      noteDeleting: false
+    };
+  },
+  methods: {
+    deleteNote: function deleteNote(note) {
+      var _this = this;
+
+      this.noteDeleting = true;
+      axios["delete"]("/api/workorders/notes/".concat(this.note.noteid)).then(function (response) {
+        _this.noteDeleting = false;
+        $("#".concat(_this.modalId)).modal('hide');
+      })["catch"](function (error) {
+        _this.noteDeleting = false;
+        console.log(error);
+      });
+    }
+  }
+});
+
+/***/ }),
+
 /***/ "./node_modules/babel-loader/lib/index.js?!./node_modules/vue-loader/lib/index.js?!./resources/js/components/NoteFormModal.vue?vue&type=script&lang=js&":
 /*!************************************************************************************************************************************************************************!*\
   !*** ./node_modules/babel-loader/lib??ref--4-0!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/components/NoteFormModal.vue?vue&type=script&lang=js& ***!
@@ -2628,6 +2696,29 @@ __webpack_require__.r(__webpack_exports__);
 //
 //
 //
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
+//
 
 
 /* harmony default export */ __webpack_exports__["default"] = ({
@@ -2644,14 +2735,19 @@ __webpack_require__.r(__webpack_exports__);
       type: String,
       require: true
     },
-    noteType: {
-      type: Number
-    },
     noteUser: {
       type: String
     },
     woid: {
       type: Number
+    },
+    // If true, user is intending to switch the note
+    // from public to private or vice/versa. Modal
+    // Will be displayed differently, but the update
+    // logic is only different for one line of code.
+    changeType: {
+      type: Boolean,
+      "default": false
     }
   },
   data: function data() {
@@ -2661,11 +2757,16 @@ __webpack_require__.r(__webpack_exports__);
     };
   },
   methods: {
-    updateNote: function updateNote(note) {
+    updateNote: function updateNote() {
       var _this = this;
 
       this.noteSaving = true;
-      axios.put('/api/workorders/notes/' + note.noteid, note).then(function (response) {
+
+      if (this.changeType) {
+        this.note.notetype = this.note.notetype === 1 ? 0 : 1;
+      }
+
+      axios.put('/api/workorders/notes/' + this.note.noteid, this.note).then(function (response) {
         _this.noteSaving = false;
 
         _this.hideModal();
@@ -2689,8 +2790,11 @@ __webpack_require__.r(__webpack_exports__);
 "use strict";
 __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var _mixins_dateMixin__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! ../mixins/dateMixin */ "./resources/js/mixins/dateMixin.js");
-/* harmony import */ var _components_NoteFormModal_vue__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! ../components/NoteFormModal.vue */ "./resources/js/components/NoteFormModal.vue");
-/* harmony import */ var _components_CircleSpinner_vue__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(/*! ../components/CircleSpinner.vue */ "./resources/js/components/CircleSpinner.vue");
+/* harmony import */ var _NoteFormModal_vue__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! ./NoteFormModal.vue */ "./resources/js/components/NoteFormModal.vue");
+/* harmony import */ var _CircleSpinner_vue__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(/*! ./CircleSpinner.vue */ "./resources/js/components/CircleSpinner.vue");
+/* harmony import */ var _NoteDeleteModal_vue__WEBPACK_IMPORTED_MODULE_3__ = __webpack_require__(/*! ./NoteDeleteModal.vue */ "./resources/js/components/NoteDeleteModal.vue");
+//
+//
 //
 //
 //
@@ -2735,10 +2839,12 @@ __webpack_require__.r(__webpack_exports__);
 
 
 
+
 /* harmony default export */ __webpack_exports__["default"] = ({
   components: {
-    NoteFormModal: _components_NoteFormModal_vue__WEBPACK_IMPORTED_MODULE_1__["default"],
-    CircleSpinner: _components_CircleSpinner_vue__WEBPACK_IMPORTED_MODULE_2__["default"]
+    NoteFormModal: _NoteFormModal_vue__WEBPACK_IMPORTED_MODULE_1__["default"],
+    CircleSpinner: _CircleSpinner_vue__WEBPACK_IMPORTED_MODULE_2__["default"],
+    NoteDeleteModal: _NoteDeleteModal_vue__WEBPACK_IMPORTED_MODULE_3__["default"]
   },
   mixins: [_mixins_dateMixin__WEBPACK_IMPORTED_MODULE_0__["default"]],
   props: {
@@ -3551,17 +3657,24 @@ __webpack_require__.r(__webpack_exports__);
     }).listen('WorkOrderNoteUpdated', function (e) {
       var index = _this.workOrderNotes.findIndex(function (note) {
         return note.noteid === e.note.noteid;
-      }); // Note has to be edited this way, or else Vue cannot
-      // see it to recompute the computed values.
+      }); // Sometimes when switching or deleting items from the end of a list,
+      // They go away before the modal can be hidden, this will hide the
+      // modal in case the component can't hide it itself. There may be 
+      // a better way to do this.
 
 
+      $("#note".concat(e.note.noteid, "switchModal")).modal('hide'); // Note has to be edited this way, or else Vue cannot
+      // see it to recompute the computed values.
+
       _this.workOrderNotes.splice(index, 1, e.note);
     }).listen('WorkOrderNoteDeleted', function (e) {
-      var index = _this.notes.findIndex(function (note) {
-        return note.noteid === e.noteid;
+      var index = _this.workOrderNotes.findIndex(function (note) {
+        return note.noteid === e.note['noteid'];
       });
 
-      _this.notes.splice(index, 1);
+      $("#note".concat(e.note['noteid'], "deleteModal")).modal('hide');
+
+      _this.workOrderNotes.splice(index, 1);
     });
   }
 });
@@ -51492,10 +51605,10 @@ render._withStripped = true
 
 /***/ }),
 
-/***/ "./node_modules/vue-loader/lib/loaders/templateLoader.js?!./node_modules/vue-loader/lib/index.js?!./resources/js/components/NoteFormModal.vue?vue&type=template&id=1391a09c&":
-/*!****************************************************************************************************************************************************************************************************************!*\
-  !*** ./node_modules/vue-loader/lib/loaders/templateLoader.js??vue-loader-options!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/components/NoteFormModal.vue?vue&type=template&id=1391a09c& ***!
-  \****************************************************************************************************************************************************************************************************************/
+/***/ "./node_modules/vue-loader/lib/loaders/templateLoader.js?!./node_modules/vue-loader/lib/index.js?!./resources/js/components/NoteDeleteModal.vue?vue&type=template&id=49f798d5&":
+/*!******************************************************************************************************************************************************************************************************************!*\
+  !*** ./node_modules/vue-loader/lib/loaders/templateLoader.js??vue-loader-options!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/components/NoteDeleteModal.vue?vue&type=template&id=49f798d5& ***!
+  \******************************************************************************************************************************************************************************************************************/
 /*! exports provided: render, staticRenderFns */
 /***/ (function(module, __webpack_exports__, __webpack_require__) {
 
@@ -51525,38 +51638,11 @@ var render = function() {
           attrs: { slot: "header", id: _vm.modalId + "Label" },
           slot: "header"
         },
-        [_vm._v("\n        Edit Note\n    ")]
+        [_vm._v("\n        Delete Note\n    ")]
       ),
       _vm._v(" "),
       _c("div", { attrs: { slot: "body" }, slot: "body" }, [
-        _c("div", { staticClass: "form-group" }, [
-          _c("label", { attrs: { for: "content" } }, [_vm._v("Content")]),
-          _vm._v(" "),
-          _c("textarea", {
-            directives: [
-              {
-                name: "model",
-                rawName: "v-model",
-                value: _vm.note.thenote,
-                expression: "note.thenote"
-              }
-            ],
-            staticClass: "form-control",
-            attrs: {
-              name: "content" + _vm.note.noteid,
-              id: "content" + _vm.note.noteid
-            },
-            domProps: { value: _vm.note.thenote },
-            on: {
-              input: function($event) {
-                if ($event.target.composing) {
-                  return
-                }
-                _vm.$set(_vm.note, "thenote", $event.target.value)
-              }
-            }
-          })
-        ])
+        _vm._v("\n        Are you sure?\n    ")
       ]),
       _vm._v(" "),
       _c("div", { attrs: { slot: "footer" }, slot: "footer" }, [
@@ -51572,17 +51658,17 @@ var render = function() {
         _c(
           "button",
           {
-            staticClass: "btn btn-primary",
+            staticClass: "btn btn-danger",
             attrs: { type: "button" },
             on: {
               click: function($event) {
-                return _vm.updateNote(_vm.note)
+                return _vm.deleteNote(_vm.note)
               }
             }
           },
           [
-            !_vm.noteSaving
-              ? _c("div", [_vm._v("Save")])
+            !_vm.noteDeleting
+              ? _c("div", [_vm._v("Confirm")])
               : _c("circle-spinner", { attrs: { size: 2 } })
           ],
           1
@@ -51596,6 +51682,187 @@ render._withStripped = true
 
 
 
+/***/ }),
+
+/***/ "./node_modules/vue-loader/lib/loaders/templateLoader.js?!./node_modules/vue-loader/lib/index.js?!./resources/js/components/NoteFormModal.vue?vue&type=template&id=1391a09c&":
+/*!****************************************************************************************************************************************************************************************************************!*\
+  !*** ./node_modules/vue-loader/lib/loaders/templateLoader.js??vue-loader-options!./node_modules/vue-loader/lib??vue-loader-options!./resources/js/components/NoteFormModal.vue?vue&type=template&id=1391a09c& ***!
+  \****************************************************************************************************************************************************************************************************************/
+/*! exports provided: render, staticRenderFns */
+/***/ (function(module, __webpack_exports__, __webpack_require__) {
+
+"use strict";
+__webpack_require__.r(__webpack_exports__);
+/* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "render", function() { return render; });
+/* harmony export (binding) */ __webpack_require__.d(__webpack_exports__, "staticRenderFns", function() { return staticRenderFns; });
+var render = function() {
+  var _vm = this
+  var _h = _vm.$createElement
+  var _c = _vm._self._c || _h
+  return _c(
+    "div",
+    [
+      _vm.changeType
+        ? _c(
+            "modal",
+            {
+              attrs: {
+                id: _vm.modalId,
+                tabindex: "-1",
+                role: "dialog",
+                "aria-labelledby": _vm.modalId + "Label"
+              }
+            },
+            [
+              _vm.populateWith.notetype === 0
+                ? _c(
+                    "h5",
+                    {
+                      staticClass: "modal-title",
+                      attrs: { slot: "header", id: _vm.modalId + "Label" },
+                      slot: "header"
+                    },
+                    [_vm._v("\n            Change to Private Note\n        ")]
+                  )
+                : _c(
+                    "h5",
+                    {
+                      staticClass: "modal-title",
+                      attrs: { slot: "header", id: _vm.modalId + "Label" },
+                      slot: "header"
+                    },
+                    [_vm._v("\n            Change to Public Note\n        ")]
+                  ),
+              _vm._v(" "),
+              _c("div", { attrs: { slot: "body" }, slot: "body" }, [
+                _vm._v("\n            Are you sure?\n        ")
+              ]),
+              _vm._v(" "),
+              _c("div", { attrs: { slot: "footer" }, slot: "footer" }, [
+                _c(
+                  "button",
+                  {
+                    staticClass: "btn btn-secondary",
+                    attrs: { type: "button", "data-dismiss": "modal" }
+                  },
+                  [_vm._v("Close")]
+                ),
+                _vm._v(" "),
+                _c(
+                  "button",
+                  {
+                    staticClass: "btn btn-primary",
+                    attrs: { type: "button" },
+                    on: {
+                      click: function($event) {
+                        return _vm.updateNote()
+                      }
+                    }
+                  },
+                  [
+                    !_vm.noteSaving
+                      ? _c("div", [_vm._v("Confirm")])
+                      : _c("circle-spinner", { attrs: { size: 2 } })
+                  ],
+                  1
+                )
+              ])
+            ]
+          )
+        : _c(
+            "modal",
+            {
+              attrs: {
+                id: _vm.modalId,
+                tabindex: "-1",
+                role: "dialog",
+                "aria-labelledby": _vm.modalId + "Label"
+              }
+            },
+            [
+              _c(
+                "h5",
+                {
+                  staticClass: "modal-title",
+                  attrs: { slot: "header", id: _vm.modalId + "Label" },
+                  slot: "header"
+                },
+                [_vm._v("\n            Edit Note\n        ")]
+              ),
+              _vm._v(" "),
+              _c("div", { attrs: { slot: "body" }, slot: "body" }, [
+                _c("div", { staticClass: "form-group" }, [
+                  _c("label", { attrs: { for: "content" } }, [
+                    _vm._v("Content")
+                  ]),
+                  _vm._v(" "),
+                  _c("textarea", {
+                    directives: [
+                      {
+                        name: "model",
+                        rawName: "v-model",
+                        value: _vm.note.thenote,
+                        expression: "note.thenote"
+                      }
+                    ],
+                    staticClass: "form-control",
+                    attrs: {
+                      name: "content" + _vm.note.noteid,
+                      id: "content" + _vm.note.noteid
+                    },
+                    domProps: { value: _vm.note.thenote },
+                    on: {
+                      input: function($event) {
+                        if ($event.target.composing) {
+                          return
+                        }
+                        _vm.$set(_vm.note, "thenote", $event.target.value)
+                      }
+                    }
+                  })
+                ])
+              ]),
+              _vm._v(" "),
+              _c("div", { attrs: { slot: "footer" }, slot: "footer" }, [
+                _c(
+                  "button",
+                  {
+                    staticClass: "btn btn-secondary",
+                    attrs: { type: "button", "data-dismiss": "modal" }
+                  },
+                  [_vm._v("Close")]
+                ),
+                _vm._v(" "),
+                _c(
+                  "button",
+                  {
+                    staticClass: "btn btn-primary",
+                    attrs: { type: "button" },
+                    on: {
+                      click: function($event) {
+                        return _vm.updateNote()
+                      }
+                    }
+                  },
+                  [
+                    !_vm.noteSaving
+                      ? _c("div", [_vm._v("Save")])
+                      : _c("circle-spinner", { attrs: { size: 2 } })
+                  ],
+                  1
+                )
+              ])
+            ]
+          )
+    ],
+    1
+  )
+}
+var staticRenderFns = []
+render._withStripped = true
+
+
+
 /***/ }),
 
 /***/ "./node_modules/vue-loader/lib/loaders/templateLoader.js?!./node_modules/vue-loader/lib/index.js?!./resources/js/components/Notes.vue?vue&type=template&id=3d897ee6&":
@@ -51711,7 +51978,7 @@ var render = function() {
       _vm._l(this.notes, function(note, index) {
         return _c(
           "li",
-          { key: index, staticClass: "row no-gutters mb-2" },
+          { key: note.noteid, staticClass: "row no-gutters mb-2" },
           [
             _c("note-form-modal", {
               attrs: {
@@ -51720,6 +51987,21 @@ var render = function() {
               }
             }),
             _vm._v(" "),
+            _c("note-delete-modal", {
+              attrs: {
+                "modal-id": "note" + note.noteid + "deleteModal",
+                note: note
+              }
+            }),
+            _vm._v(" "),
+            _c("note-form-modal", {
+              attrs: {
+                "modal-id": "note" + note.noteid + "switchModal",
+                "populate-with": note,
+                "change-type": ""
+              }
+            }),
+            _vm._v(" "),
             _c(
               "div",
               {
@@ -51757,9 +52039,31 @@ var render = function() {
                             [_c("i", { staticClass: "fas fa-fw fa-edit" })]
                           ),
                           _vm._v(" "),
-                          _vm._m(0, true),
+                          _c(
+                            "button",
+                            {
+                              staticClass: "btn btn-sm btn-danger m-1",
+                              attrs: {
+                                "data-toggle": "modal",
+                                "data-target":
+                                  "#note" + note.noteid + "deleteModal"
+                              }
+                            },
+                            [_c("i", { staticClass: "fas fa-fw fa-trash-alt" })]
+                          ),
                           _vm._v(" "),
-                          _vm._m(1, true)
+                          _c(
+                            "button",
+                            {
+                              staticClass: "btn btn-sm btn-primary m-1",
+                              attrs: {
+                                "data-toggle": "modal",
+                                "data-target":
+                                  "#note" + note.noteid + "switchModal"
+                              }
+                            },
+                            [_c("i", { staticClass: "fa fa-fw fa-random" })]
+                          )
                         ]
                       : _vm._e()
                   ],
@@ -51787,24 +52091,7 @@ var render = function() {
     2
   )
 }
-var staticRenderFns = [
-  function() {
-    var _vm = this
-    var _h = _vm.$createElement
-    var _c = _vm._self._c || _h
-    return _c("button", { staticClass: "btn btn-sm btn-danger m-1" }, [
-      _c("i", { staticClass: "fas fa-fw fa-trash-alt" })
-    ])
-  },
-  function() {
-    var _vm = this
-    var _h = _vm.$createElement
-    var _c = _vm._self._c || _h
-    return _c("button", { staticClass: "btn btn-sm btn-primary m-1" }, [
-      _c("i", { staticClass: "fa fa-fw fa-random" })
-    ])
-  }
-]
+var staticRenderFns = []
 render._withStripped = true
 
 
@@ -68730,6 +69017,75 @@ __webpack_require__.r(__webpack_exports__);
 
 
 
+/***/ }),
+
+/***/ "./resources/js/components/NoteDeleteModal.vue":
+/*!*****************************************************!*\
+  !*** ./resources/js/components/NoteDeleteModal.vue ***!
+  \*****************************************************/
+/*! exports provided: default */
+/***/ (function(module, __webpack_exports__, __webpack_require__) {
+
+"use strict";
+__webpack_require__.r(__webpack_exports__);
+/* harmony import */ var _NoteDeleteModal_vue_vue_type_template_id_49f798d5___WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! ./NoteDeleteModal.vue?vue&type=template&id=49f798d5& */ "./resources/js/components/NoteDeleteModal.vue?vue&type=template&id=49f798d5&");
+/* harmony import */ var _NoteDeleteModal_vue_vue_type_script_lang_js___WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! ./NoteDeleteModal.vue?vue&type=script&lang=js& */ "./resources/js/components/NoteDeleteModal.vue?vue&type=script&lang=js&");
+/* empty/unused harmony star reexport *//* harmony import */ var _node_modules_vue_loader_lib_runtime_componentNormalizer_js__WEBPACK_IMPORTED_MODULE_2__ = __webpack_require__(/*! ../../../node_modules/vue-loader/lib/runtime/componentNormalizer.js */ "./node_modules/vue-loader/lib/runtime/componentNormalizer.js");
+
+
+
+
+
+/* normalize component */
+
+var component = Object(_node_modules_vue_loader_lib_runtime_componentNormalizer_js__WEBPACK_IMPORTED_MODULE_2__["default"])(
+  _NoteDeleteModal_vue_vue_type_script_lang_js___WEBPACK_IMPORTED_MODULE_1__["default"],
+  _NoteDeleteModal_vue_vue_type_template_id_49f798d5___WEBPACK_IMPORTED_MODULE_0__["render"],
+  _NoteDeleteModal_vue_vue_type_template_id_49f798d5___WEBPACK_IMPORTED_MODULE_0__["staticRenderFns"],
+  false,
+  null,
+  null,
+  null
+  
+)
+
+/* hot reload */
+if (false) { var api; }
+component.options.__file = "resources/js/components/NoteDeleteModal.vue"
+/* harmony default export */ __webpack_exports__["default"] = (component.exports);
+
+/***/ }),
+
+/***/ "./resources/js/components/NoteDeleteModal.vue?vue&type=script&lang=js&":
+/*!******************************************************************************!*\
+  !*** ./resources/js/components/NoteDeleteModal.vue?vue&type=script&lang=js& ***!
+  \******************************************************************************/
+/*! exports provided: default */
+/***/ (function(module, __webpack_exports__, __webpack_require__) {
+
+"use strict";
+__webpack_require__.r(__webpack_exports__);
+/* harmony import */ var _node_modules_babel_loader_lib_index_js_ref_4_0_node_modules_vue_loader_lib_index_js_vue_loader_options_NoteDeleteModal_vue_vue_type_script_lang_js___WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! -!../../../node_modules/babel-loader/lib??ref--4-0!../../../node_modules/vue-loader/lib??vue-loader-options!./NoteDeleteModal.vue?vue&type=script&lang=js& */ "./node_modules/babel-loader/lib/index.js?!./node_modules/vue-loader/lib/index.js?!./resources/js/components/NoteDeleteModal.vue?vue&type=script&lang=js&");
+/* empty/unused harmony star reexport */ /* harmony default export */ __webpack_exports__["default"] = (_node_modules_babel_loader_lib_index_js_ref_4_0_node_modules_vue_loader_lib_index_js_vue_loader_options_NoteDeleteModal_vue_vue_type_script_lang_js___WEBPACK_IMPORTED_MODULE_0__["default"]); 
+
+/***/ }),
+
+/***/ "./resources/js/components/NoteDeleteModal.vue?vue&type=template&id=49f798d5&":
+/*!************************************************************************************!*\
+  !*** ./resources/js/components/NoteDeleteModal.vue?vue&type=template&id=49f798d5& ***!
+  \************************************************************************************/
+/*! exports provided: render, staticRenderFns */
+/***/ (function(module, __webpack_exports__, __webpack_require__) {
+
+"use strict";
+__webpack_require__.r(__webpack_exports__);
+/* harmony import */ var _node_modules_vue_loader_lib_loaders_templateLoader_js_vue_loader_options_node_modules_vue_loader_lib_index_js_vue_loader_options_NoteDeleteModal_vue_vue_type_template_id_49f798d5___WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! -!../../../node_modules/vue-loader/lib/loaders/templateLoader.js??vue-loader-options!../../../node_modules/vue-loader/lib??vue-loader-options!./NoteDeleteModal.vue?vue&type=template&id=49f798d5& */ "./node_modules/vue-loader/lib/loaders/templateLoader.js?!./node_modules/vue-loader/lib/index.js?!./resources/js/components/NoteDeleteModal.vue?vue&type=template&id=49f798d5&");
+/* harmony reexport (safe) */ __webpack_require__.d(__webpack_exports__, "render", function() { return _node_modules_vue_loader_lib_loaders_templateLoader_js_vue_loader_options_node_modules_vue_loader_lib_index_js_vue_loader_options_NoteDeleteModal_vue_vue_type_template_id_49f798d5___WEBPACK_IMPORTED_MODULE_0__["render"]; });
+
+/* harmony reexport (safe) */ __webpack_require__.d(__webpack_exports__, "staticRenderFns", function() { return _node_modules_vue_loader_lib_loaders_templateLoader_js_vue_loader_options_node_modules_vue_loader_lib_index_js_vue_loader_options_NoteDeleteModal_vue_vue_type_template_id_49f798d5___WEBPACK_IMPORTED_MODULE_0__["staticRenderFns"]; });
+
+
+
 /***/ }),
 
 /***/ "./resources/js/components/NoteFormModal.vue":

resources/js/components/NoteDeleteModal.vue

@@ -0,0 +1,54 @@
+<template>
+     <modal :id="modalId" tabindex="-1" role="dialog" :aria-labelledby="modalId+'Label'">
+        <h5 slot="header" class="modal-title" :id="modalId+'Label'">
+            Delete Note
+        </h5>
+        <div slot="body">
+            Are you sure?
+        </div>
+        <div slot="footer">
+            <button type="button" class="btn btn-secondary" data-dismiss="modal">Close</button>
+            <button type="button" class="btn btn-danger" @click="deleteNote(note)">
+                <div v-if="!noteDeleting">Confirm</div>
+                <circle-spinner v-else :size="2"></circle-spinner>
+            </button>
+        </div>
+    </modal>
+</template>
+<script>
+import Modal from './Modal.vue'
+import CircleSpinner from './CircleSpinner.vue'
+export default {
+    components: {
+        Modal,
+        CircleSpinner,
+    },
+    props: {
+        modalId: {
+            type: String,
+            require: true,
+        },
+        note: {
+            type: Object,
+            required: true,
+        },
+    },
+    data () {
+        return {
+            noteDeleting: false
+        }
+    },
+    methods: {
+        deleteNote: function(note) {
+            this.noteDeleting = true
+            axios.delete(`/api/workorders/notes/${this.note.noteid}`).then( response => {
+                this.noteDeleting = false
+                $(`#${this.modalId}`).modal('hide')
+            }).catch( error => {
+                this.noteDeleting = false
+                console.log(error)
+            })
+        }
+    }
+}
+</script>

resources/js/components/NoteFormModal.vue

@@ -1,22 +1,45 @@
 <template>
-    <modal :id="modalId" tabindex="-1" role="dialog" :aria-labelledby="modalId+'Label'">
-        <h5 slot="header" class="modal-title" :id="modalId+'Label'">
-            Edit Note
-        </h5>
-        <div slot="body">
-            <div class="form-group">
-                <label for="content">Content</label>
-                <textarea :name="'content'+note.noteid" :id="'content'+note.noteid" class="form-control" v-model="note.thenote"></textarea>
+    <div>
+        <modal v-if="changeType" :id="modalId" tabindex="-1" role="dialog" :aria-labelledby="modalId+'Label'">
+            <!-- We want the original notetype, if we used note.notetype the text would change as soon as we hit the Confirm button.
+            This way the change in text occurs when we receive the updated info from the backend, so we can make sure there are no
+            errors preventing the data from actually changing first -->
+            <h5 v-if="populateWith.notetype === 0 " slot="header" class="modal-title" :id="modalId+'Label'">
+                Change to Private Note
+            </h5>
+            <h5 v-else slot="header" class="modal-title" :id="modalId+'Label'">
+                Change to Public Note
+            </h5>
+            <div slot="body">
+                Are you sure?
             </div>
-        </div>
-        <div slot="footer">
-            <button type="button" class="btn btn-secondary" data-dismiss="modal">Close</button>
-            <button type="button" class="btn btn-primary" @click="updateNote(note)">
-                <div v-if="!noteSaving">Save</div>
-                <circle-spinner v-else :size="2"></circle-spinner>
-            </button>
-        </div>
-    </modal>
+            <div slot="footer">
+                <button type="button" class="btn btn-secondary" data-dismiss="modal">Close</button>
+                <button type="button" class="btn btn-primary" @click="updateNote()">
+                    <div v-if="!noteSaving">Confirm</div>
+                    <circle-spinner v-else :size="2"></circle-spinner>
+                </button>
+            </div>
+        </modal>
+        <modal v-else :id="modalId" tabindex="-1" role="dialog" :aria-labelledby="modalId+'Label'">
+            <h5 slot="header" class="modal-title" :id="modalId+'Label'">
+                Edit Note
+            </h5>
+            <div slot="body">
+                <div class="form-group">
+                    <label for="content">Content</label>
+                    <textarea :name="'content'+note.noteid" :id="'content'+note.noteid" class="form-control" v-model="note.thenote"></textarea>
+                </div>
+            </div>
+            <div slot="footer">
+                <button type="button" class="btn btn-secondary" data-dismiss="modal">Close</button>
+                <button type="button" class="btn btn-primary" @click="updateNote()">
+                    <div v-if="!noteSaving">Save</div>
+                    <circle-spinner v-else :size="2"></circle-spinner>
+                </button>
+            </div>
+        </modal>
+    </div>
 </template>
 <script>
 import Modal from '../components/Modal.vue'
@@ -35,26 +58,34 @@ export default {
             type: String,
             require: true
         },
-        noteType: {
-            type: Number
-        },
         noteUser: {
             type: String
         },
         woid: {
             type: Number
+        },
+        // If true, user is intending to switch the note
+        // from public to private or vice/versa. Modal
+        // Will be displayed differently, but the update
+        // logic is only different for one line of code.
+        changeType: {
+            type: Boolean,
+            default: false,
         }
     },
     data () {
         return {
             note: JSON.parse(JSON.stringify(this.populateWith)),
-            noteSaving: false
+            noteSaving: false,
         }
     },
     methods: {
-        updateNote (note) {
+        updateNote () {
             this.noteSaving = true
-            axios.put('/api/workorders/notes/' + note.noteid, note)
+            if (this.changeType) {
+                this.note.notetype = (this.note.notetype === 1 ? 0 : 1)
+            }
+            axios.put('/api/workorders/notes/' + this.note.noteid, this.note)
                 .then((response) => {
                     this.noteSaving = false
                     this.hideModal()

resources/js/components/Notes.vue

@@ -16,16 +16,18 @@
                     </div>
                 </div>
             </div>
-        <li class="row no-gutters mb-2" v-bind:key="index" v-for="(note, index) in this.notes">
+        <li class="row no-gutters mb-2" v-bind:key="note.noteid" v-for="(note, index) in this.notes">
             <note-form-modal :modal-id="'note'+note.noteid+'editModal'" :populate-with="note"></note-form-modal>
+            <note-delete-modal :modal-id="`note${note.noteid}deleteModal`" :note="note"></note-delete-modal>
+            <note-form-modal :modal-id="`note${note.noteid}switchModal`" :populate-with="note" change-type></note-form-modal>
             <div class="col-md-1 d-flex flex-column mx-md-3" :class="noteOrders[index]">
                 <div class="text-center p-0 m-0">{{note.noteuser}}</div>
                 <div class="text-muted text-small text-center p-0 m-0">{{getHRDate(note.notetime)}}</div>
                 <div class="btn-group justify-content-center p-0 m-0">
                     <template v-if="authusername === note.noteuser || authusername === 'admin'">
                         <button class="btn btn-sm btn-primary m-1" data-toggle="modal" :data-target="'#note'+note.noteid+'editModal'"><i class="fas fa-fw fa-edit"></i></button>
-                        <button class="btn btn-sm btn-danger m-1"><i class="fas fa-fw fa-trash-alt"></i></button>
-                        <button class="btn btn-sm btn-primary m-1"><i class="fa fa-fw fa-random"></i></button>
+                        <button class="btn btn-sm btn-danger m-1" data-toggle="modal" :data-target="`#note${note.noteid}deleteModal`"><i class="fas fa-fw fa-trash-alt"></i></button>
+                        <button class="btn btn-sm btn-primary m-1" data-toggle="modal" :data-target="`#note${note.noteid}switchModal`"><i class="fa fa-fw fa-random"></i></button>
                     </template>
                 </div>
             </div>
@@ -41,12 +43,14 @@
 </template>
 <script>
 import dateMixin from '../mixins/dateMixin'
-import NoteFormModal from '../components/NoteFormModal.vue'
-import CircleSpinner from '../components/CircleSpinner.vue'
+import NoteFormModal from './NoteFormModal.vue'
+import CircleSpinner from './CircleSpinner.vue'
+import NoteDeleteModal from './NoteDeleteModal.vue'
 export default {
     components: {
         NoteFormModal,
         CircleSpinner,
+        NoteDeleteModal
     },
     mixins:[dateMixin],
     props: {

resources/js/views/WorkOrder.vue

@@ -229,15 +229,21 @@ export default {
                 let index = this.workOrderNotes.findIndex((note) => {
                     return note.noteid === e.note.noteid
                 })
+                // Sometimes when switching or deleting items from the end of a list,
+                // They go away before the modal can be hidden, this will hide the
+                // modal in case the component can't hide it itself. There may be 
+                // a better way to do this.
+                $(`#note${e.note.noteid}switchModal`).modal('hide')
                 // Note has to be edited this way, or else Vue cannot
                 // see it to recompute the computed values.
                 this.workOrderNotes.splice(index, 1, e.note)
             })
             .listen('WorkOrderNoteDeleted', (e) => {
-                let index = this.notes.findIndex((note) => {
-                    return note.noteid === e.noteid
+                let index = this.workOrderNotes.findIndex((note) => {
+                    return note.noteid === e.note['noteid']
                 })
-                this.notes.splice(index, 1)
+                $(`#note${e.note['noteid']}deleteModal`).modal('hide')
+                this.workOrderNotes.splice(index, 1)
             })
     },
 }

routes/api.php

@@ -30,6 +30,7 @@ Route::middleware('auth:api')->group( function() {
 
     Route::post('/workorders/notes', 'Api\WorkOrderNotesController@store');
     Route::put('/workorders/notes/{workOrderNote}', 'Api\WorkOrderNotesController@update');
+    Route::delete('/workorders/notes/{workOrderNote}', 'Api\WorkOrderNotesController@destroy');
 
     Route::get('/assets/{asset}/credentials', 'Api\AssetsController@credentials');
     Route::put('/assets/{asset}', 'Api\AssetsController@update');