refactor(SharingSidebarView): Use NC logger & sort imports

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>

package.json

@@ -63,6 +63,7 @@
     "@nextcloud/files": "^3.1.1",
     "@nextcloud/initial-state": "^2.1.0",
     "@nextcloud/l10n": "^2.2.0",
+    "@nextcloud/logger": "^3.0.2",
     "@nextcloud/router": "^3.0.0",
     "@nextcloud/vue": "^8.12.0",
     "nextcloud-server": "^0.15.10",

src/client.js

@@ -22,6 +22,7 @@
 
 import ACL_PROPERTIES from './model/Properties.js'
 import Rule from './model/Rule.js'
+import logger from './services/logger.ts'
 
 /**
  * @member {OC.Files.Client} client
@@ -263,16 +264,16 @@ class AclDavService {
 					return response
 				} else if (response.status === 403) {
 					// Handle permission denied scenario
-					console.error('Permission denied:', response.status, response.statusText)
+					logger.error('Permission denied:', { responseStatus: response.status, responseStatusText: response.statusText })
 					throw new Error(t('groupfolders', 'Permission denied. User does not have sufficient permissions.'))
 				} else {
 					// Handle unexpected status codes
-					console.error('Unexpected status:', response.status, response.statusText)
+					logger.error('Unexpected status:', { responseStatus: response.status, responseStatusText: response.statusText })
 					throw new Error(t('groupfolders', 'Unexpected status from server'))
 				}
 		  }).catch(error => {
 			// Handle network errors or exceptions
-				console.error('Error in propPatch:', error)
+				logger.error('Error in propPatch:', { error })
 				throw error
 		  })
 	  }

src/components/SharingSidebarView.vue

@@ -172,20 +172,21 @@
 </template>
 
 <script>
-import Vue from 'vue'
 import axios from '@nextcloud/axios'
-import { generateUrl } from '@nextcloud/router'
-import AclStateButton, { STATES } from './AclStateButton.vue'
 import { showError } from '@nextcloud/dialogs'
-import Rule from './../model/Rule.js'
-import BinaryTools from './../BinaryTools.js'
-import client from './../client.js'
-import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
+import { generateUrl } from '@nextcloud/router'
 import NcAvatar from '@nextcloud/vue/dist/Components/NcAvatar.js'
+import NcButton from '@nextcloud/vue/dist/Components/NcButton.js'
 import NcSelect from '@nextcloud/vue/dist/Components/NcSelect.js'
 import Tooltip from '@nextcloud/vue/dist/Directives/Tooltip.js'
-import Plus from 'vue-material-design-icons/Plus.vue'
+import Vue from 'vue'
 import Close from 'vue-material-design-icons/Close.vue'
+import Plus from 'vue-material-design-icons/Plus.vue'
+import logger from '../services/logger.ts'
+import BinaryTools from './../BinaryTools.js'
+import client from './../client.js'
+import Rule from './../model/Rule.js'
+import AclStateButton, { STATES } from './AclStateButton.vue'
 
 let searchRequestCancelSource = null
 
@@ -320,7 +321,7 @@ export default {
 				})
 			}).catch((error) => {
 				if (!axios.isCancel(error)) {
-					console.error('Failed to search results for groupfolder ACL')
+					logger.error('Failed to search results for groupfolder ACL')
 				}
 			})
 		},
@@ -379,9 +380,9 @@ export default {
 			// TODO: Block UI during save
 			try {
 				await client.propPatch(this.model, this.list.filter(rule => !rule.inherited))
-				console.debug('Permissions updated successfully')
+				logger.debug('Permissions updated successfully')
 			} catch (error) {
-				console.error('Failed to save changes:', error)
+				logger.error('Failed to save changes:', { error })
 				Vue.set(this.list, index, itemRestorePoint)
 				showError(error)
 			} finally {

src/services/logger.ts

@@ -0,0 +1,6 @@
+import { getLoggerBuilder } from '@nextcloud/logger'
+
+export default getLoggerBuilder()
+	.setApp('groupfolders')
+	.detectUser()
+	.build()