Merge pull request #1227 from NicoPennec/fix/various

Various fixes

package.json

@@ -39,7 +39,7 @@
     "three": "0.157.0",
     "uuid": "9.0.1",
     "v-autocomplete": "1.8.2",
-    "vue": "2.7.14",
+    "vue": "2.7.15",
     "vue-at": "2.5.1",
     "vue-chartkick": "0.6.1",
     "vue-drag-drop": "1.1.4",
@@ -59,13 +59,13 @@
   },
   "devDependencies": {
     "@vitejs/plugin-vue2": "2.2.0",
-    "@vue/compiler-sfc": "2.7.14",
+    "@vue/compiler-sfc": "2.7.15",
     "@vue/test-utils": "1.3.6",
     "autoprefixer": "10.4.16",
-    "eslint": "8.51.0",
+    "eslint": "8.52.0",
     "eslint-config-prettier": "9.0.0",
     "eslint-plugin-prettier": "5.0.1",
-    "eslint-plugin-vue": "9.17.0",
+    "eslint-plugin-vue": "9.18.0",
     "esm": "3.2.25",
     "husky": "8.0.3",
     "jsdom": "22.1.0",
@@ -78,7 +78,7 @@
     "vite-plugin-eslint": "1.8.1",
     "vitest": "0.34.6",
     "vitest-localstorage-mock": "0.0.1",
-    "vue-template-compiler": "2.7.14"
+    "vue-template-compiler": "2.7.15"
   },
   "engines": {
     "node": ">=16.0"

src/components/pages/Breakdown.vue

@@ -584,7 +584,7 @@ export default {
     },
 
     csvColumns() {
-      return this.isTVShow
+      return this.isTVShow && this.currentEpisode?.id !== 'all'
         ? ['Episode', 'Parent', 'Name', 'Asset Type', 'Asset', 'Occurences']
         : ['Parent', 'Name', 'Asset Type', 'Asset', 'Occurences']
     },

src/components/pages/MainSchedule.vue

@@ -128,6 +128,9 @@ export default {
     ...mapActions(['editProduction', 'loadScheduleItems', 'saveScheduleItem']),
 
     reset() {
+      if (!this.openProductions.length) {
+        return
+      }
       this.scheduleItems = this.convertScheduleItems(this.openProductions)
       this.startDate = getFirstStartDate(this.scheduleItems)
       this.endDate = getLastEndDate(this.scheduleItems)

src/components/pages/ProductionNewsFeed.vue

@@ -448,7 +448,7 @@ export default {
 
     params() {
       const params = {
-        productionId: this.currentProduction.id,
+        productionId: this.currentProduction?.id,
         only_preview: this.previewMode === 'previews',
         page_size: this.previewMode === 'previews' ? 6 : 50,
         task_type_id: this.taskTypeId !== '' ? this.taskTypeId : undefined,
@@ -501,11 +501,8 @@ export default {
     },
 
     team() {
-      return sortPeople(
-        this.currentProduction.team.map(personId =>
-          this.personMap.get(personId)
-        )
-      )
+      const team = this.currentProduction?.team || []
+      return sortPeople(team.map(personId => this.personMap.get(personId)))
     },
 
     renderedStats() {

src/components/pages/Settings.vue

@@ -36,11 +36,14 @@
       <text-field
         class="mt2"
         :label="$t('settings.fields.name')"
+        :required="true"
         v-model.trim="form.name"
       />
       <text-field
         :label="$t('settings.fields.hours_by_day')"
+        :min="1"
         :max="24"
+        :required="true"
         type="number"
         v-model="form.hours_by_day"
       />
@@ -243,7 +246,6 @@ export default {
             ? 'true'
             : 'false'
         }
-        console.log(this.form.has_avatar)
       }
     }
   },

src/components/pages/schedule/Schedule.vue

@@ -65,13 +65,15 @@
                 />
               </span>
               <span
-                class="filler flexrow-item root-element-name"
+                class="filler flexrow-item root-element-name ellipsis"
+                :title="rootElement.name"
                 v-if="!rootElement.route"
               >
                 {{ rootElement.name }}
               </span>
               <router-link
-                class="filler flexrow-item root-element-name"
+                class="filler flexrow-item root-element-name ellipsis"
+                :title="rootElement.name"
                 :to="rootElement.route"
                 v-else
               >