fix: Replace console with our logger that also handles log levels

Signed-off-by: Ferdinand Thiessen <[email protected]>

src/components/Folder.vue

@@ -36,6 +36,7 @@ import { getCurrentUser } from '@nextcloud/auth'
 import FolderTagPreview from './FolderTagPreview.vue'
 import getAlbumContent from '../services/AlbumContent.js'
 import AbortControllerMixin from '../mixins/AbortControllerMixin.js'
+import logger from '../services/logger'
 
 export default {
 	name: 'Folder',
@@ -122,7 +123,9 @@ export default {
 				this.$store.dispatch('updateFiles', { folder, files, folders })
 			} catch (error) {
 				if (error.response && error.response.status) {
-					console.error('Failed to get folder content', filename, error.response)
+					logger.error('Failed to get folder content', { error, filename })
+				} else {
+					logger.debug(error)
 				}
 			}
 		},

src/mixins/GridConfig.js

@@ -21,6 +21,7 @@
  */
 
 import getGridConfig from '../services/GridConfig.js'
+import logger from '../services/logger.js'
 
 /**
  * Get the current used grid config
@@ -36,7 +37,7 @@ export default {
 		getGridConfig.$on('changed', val => {
 			this.gridConfig = val
 		})
-		console.debug(`[${appName}]`, 'Grid config', Object.assign({}, getGridConfig.gridConfig))
+		logger.debug('Grid config', { gridConfig: getGridConfig.gridConfig })
 		this.gridConfig = getGridConfig.gridConfig
 	},
 

src/services/Albums.js

@@ -185,8 +185,6 @@ export async function fetchAlbumContent(path, options, client = davClient) {
 		}
 
 		logger.error('Error fetching album files', { error })
-		console.error(error)
-
 		throw error
 	}
 }

src/services/collectionFetcher.js

@@ -222,8 +222,6 @@ export async function fetchCollectionFiles(path, options, extraProps = [], clien
 		}
 
 		logger.error('Error fetching collection files', { error })
-		console.error(error)
-
 		throw error
 	}
 }

src/store/files.js

@@ -193,7 +193,6 @@ const actions = {
 				} catch (error) {
 					logger.error(t('photos', 'Failed to delete {fileId}.', { fileId }), { error })
 					showError(t('photos', 'Failed to delete {fileName}.', { fileName: file.basename }))
-					console.error(error)
 					context.dispatch('appendFiles', [file])
 				} finally {
 					semaphore.release(symbol)

src/store/systemtags.js

@@ -23,6 +23,7 @@ import Vue from 'vue'
 import { sortCompare } from '../utils/fileUtils.js'
 import getTaggedImages from '../services/TaggedImages.js'
 import getSystemTags from '../services/SystemTags.js'
+import logger from '../services/logger.js'
 
 const state = {
 	tags: {},
@@ -81,7 +82,7 @@ const mutations = {
 		const list = files.sort((a, b) => sortCompare(a, b, 'filesAssigned'))
 
 		// overwrite list
-		console.info(id, list)
+		logger.debug(`Overwrite list, id: ${id}`, { list })
 		Vue.set(state.tags[id], 'files', list.map(file => file.fileid))
 	},
 }
@@ -136,7 +137,9 @@ const actions = {
 			await context.dispatch('appendFiles', files)
 		} catch (error) {
 			if (error.response && error.response.status) {
-				console.error('Failed to get tag content', id, error.response)
+				logger.error(`Failed to get tag content, id: ${id}`, { error })
+			} else {
+				logger.debug(error)
 			}
 		}
 	},

src/views/Folders.vue

@@ -84,6 +84,7 @@ import getAlbumContent from '../services/AlbumContent.js'
 import AbortControllerMixin from '../mixins/AbortControllerMixin.js'
 import GridConfigMixin from '../mixins/GridConfig.js'
 import { fetchFile } from '../services/fileFetcher'
+import logger from '../services/logger'
 
 export default {
 	name: 'Folders',
@@ -268,7 +269,7 @@ export default {
 					}
 				}
 				// cancelled request, moving on...
-				console.error('Error fetching album data', error)
+				logger.error('Error fetching album data', { error })
 			} finally {
 				// done loading even with errors
 				this.loading = false

src/views/TagContent.vue

@@ -168,7 +168,7 @@ export default {
 					await this.$store.dispatch('fetchTagFiles', { id: this.tagId, signal: this.abortController.signal })
 				}
 			} catch (error) {
-				console.error(error)
+				logger.error(error)
 				this.error = true
 			} finally {
 				// done loading

src/views/Tags.vue

@@ -54,6 +54,7 @@ import { NcEmptyContent, NcLoadingIcon } from '@nextcloud/vue'
 
 import TagCover from '../components/TagCover.vue'
 import AbortControllerMixin from '../mixins/AbortControllerMixin.js'
+import logger from '../services/logger'
 
 export default {
 	name: 'Tags',
@@ -115,7 +116,7 @@ export default {
 					})
 				}
 			} catch (error) {
-				console.error(error)
+				logger.error(error)
 				this.error = true
 			} finally {
 				// done loading