From 8c569d5669a024f6116f334ab427815f72d669b7 Mon Sep 17 00:00:00 2001
From: urielfcampos <urielfcampos95@gmail.com>
Date: Mon, 13 May 2019 13:53:19 -0300
Subject: [PATCH] revisoes realizadas

---
 components/searchInput.vue | 31 +++++++++++++++++++++++++------
 layouts/default.vue        |  4 +---
 middleware/course.js       |  2 --
 pages/activeStudents.vue   | 16 ++++++++++------
 pages/allStudents.vue      |  6 +++++-
 shared/debounce.js         | 27 +++++++++++++++++++++++++++
 6 files changed, 68 insertions(+), 18 deletions(-)
 create mode 100644 shared/debounce.js

diff --git a/components/searchInput.vue b/components/searchInput.vue
index f6d27756..6baa3674 100644
--- a/components/searchInput.vue
+++ b/components/searchInput.vue
@@ -21,7 +21,7 @@
         <b-table
           :striped="isStriped"
           :hoverable="isHoverabble"
-          :data="filteredList"
+          :data="students"
           :selected.sync="selectedStudent"
           :columns="columns"
           focusable
@@ -38,12 +38,10 @@
 </template>
 
 <script>
-import { Toast } from 'buefy/dist/components/toast'
 import studentComboBox from '../components/studentComboBox'
-
+import debounce from '../shared/debounce.js'
 export default {
   name: 'SearchInput',
-  // props: ['title', 'students', 'thead'],
   components: {
     studentComboBox
   },
@@ -76,7 +74,8 @@ export default {
           field: 'email',
           label: 'Email'
         }
-      ]
+      ],
+      filterOrderBy: false
     }
   },
 
@@ -96,11 +95,31 @@ export default {
 
   mounted() {
     if (this.students.length < 1) {
-      Toast.open({
+      this.$toast.open({
         message: 'Não há alunos ativos neste curso',
         type: 'is-danger'
       })
     }
+  },
+  methods: {
+    getStudentsFilters: debounce(() => {
+      this.$axios
+        .get('/api/students/', {
+          params: {
+            course: this.courseTag,
+            isActive: 1
+          }
+        })
+        .then(res => {
+          this.students = res.data
+        })
+        .catch(() => {
+          this.$toast.open({
+            message: 'Falha ao carregar a lista de alunos.',
+            type: 'is-danger'
+          })
+        })
+    }, 500)
   }
 }
 </script>
diff --git a/layouts/default.vue b/layouts/default.vue
index 9d87afad..a20f764e 100644
--- a/layouts/default.vue
+++ b/layouts/default.vue
@@ -128,9 +128,7 @@ export default {
       currrentUser: 'auth/currentUser'
     }),
     courseNameUppercase() {
-      if (this.courseTag) {
-        return this.courseTag.toUpperCase()
-      } else return false
+      return this.courseTag && this.courseTag.toUpperCase()
     }
   },
   methods: {
diff --git a/middleware/course.js b/middleware/course.js
index d68336f6..4319d1e4 100644
--- a/middleware/course.js
+++ b/middleware/course.js
@@ -2,8 +2,6 @@ import { Toast } from 'buefy/dist/components/toast'
 export default function({ store, redirect }) {
   if (!store.state.courseTag) {
     Toast.open('Selecione um Curso')
-    // eslint-disable-next-line no-console
-    console.log('trying')
     redirect('/')
   }
 }
diff --git a/pages/activeStudents.vue b/pages/activeStudents.vue
index a5714fcd..6b635ffe 100644
--- a/pages/activeStudents.vue
+++ b/pages/activeStudents.vue
@@ -7,7 +7,6 @@
 <script>
 import { mapState } from 'vuex'
 import SearchInput from '~/components/searchInput'
-import '@nuxtjs/axios'
 
 export default {
   name: 'Active',
@@ -34,9 +33,12 @@ export default {
   methods: {
     async getStudents() {
       try {
-        this.students = await this.$axios.$get(
-          `/api/students/?course=${this.courseTag}&isActive=1`
-        )
+        this.students = await this.$axios.$get('/api/students/', {
+          params: {
+            course: this.courseTag,
+            isActive: 1
+          }
+        })
       } catch (e) {
         this.openErrorNotification(e.response.data.code)
       }
@@ -51,12 +53,14 @@ export default {
           return 'Arquivo csv com numero invalido de colunas'
         case 'IMPORT_CSV_INVALID_FILE':
           return 'Por favor selecione um arquivo do tipo csv'
+        default:
+          return 'Ocorreu um erro'
       }
     },
-    openErrorNotification(errorMessage) {
+    openErrorNotification(errorCode) {
       this.$notification.open({
         duration: 5000,
-        message: errorMessage,
+        message: this.errorMessage(errorCode),
         position: 'is-top',
         type: 'is-danger',
         hasIcon: true
diff --git a/pages/allStudents.vue b/pages/allStudents.vue
index 2b4ebd7a..7d607ccd 100644
--- a/pages/allStudents.vue
+++ b/pages/allStudents.vue
@@ -26,7 +26,11 @@ export default {
   },
   created() {
     this.$axios
-      .get(`/api/students/?course=${this.courseTag}`)
+      .get('/api/students/', {
+        params: {
+          course: this.courseTag
+        }
+      })
       .then(res => {
         this.students = res.data
       })
diff --git a/shared/debounce.js b/shared/debounce.js
new file mode 100644
index 00000000..f82f1922
--- /dev/null
+++ b/shared/debounce.js
@@ -0,0 +1,27 @@
+// Credit David Walsh (https://davidwalsh.name/javascript-debounce-function)
+
+// Returns a function, that, as long as it continues to be invoked, will not
+// be triggered. The function will be called after it stops being called for
+// N milliseconds. If `immediate` is passed, trigger the function on the
+// leading edge, instead of the trailing.
+export default function debounce(func, wait, immediate) {
+  let timeout
+
+  return function executedFunction() {
+    const context = this
+    const args = arguments
+
+    const later = function() {
+      timeout = null
+      if (!immediate) func.apply(context, args)
+    }
+
+    const callNow = immediate && !timeout
+
+    clearTimeout(timeout)
+
+    timeout = setTimeout(later, wait)
+
+    if (callNow) func.apply(context, args)
+  }
+}