Skip to content

Commit a6efc69

Browse files
committed
feat(user-picker): use logger instead of console
Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
1 parent 743f2ad commit a6efc69

File tree

4 files changed

+18
-6
lines changed

4 files changed

+18
-6
lines changed

apps/user_picker/src/components/ProfilesCustomPicker.vue

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ import NcEmptyContent from '@nextcloud/vue/components/NcEmptyContent'
5959
import NcSelect from '@nextcloud/vue/components/NcSelect'
6060
import AccountOutline from 'vue-material-design-icons/AccountOutline.vue'
6161
import ArrowRightIcon from 'vue-material-design-icons/ArrowRight.vue'
62+
import { logger } from '../utils/logger.ts'
6263
6364
export default {
6465
name: 'ProfilesCustomPicker',
@@ -145,8 +146,8 @@ export default {
145146
isNoUser: userAutocomplete.source.startsWith('users'),
146147
}
147148
})
148-
} catch (err) {
149-
console.error(err)
149+
} catch (error) {
150+
logger.error('user_picker: error while searching for users', { error })
150151
} finally {
151152
this.loading = false
152153
}
@@ -168,8 +169,8 @@ export default {
168169
signal: this.abortController.signal,
169170
})
170171
this.reference = res.data.ocs.data.references[this.resultUrl]
171-
} catch (err) {
172-
console.error(err)
172+
} catch (error) {
173+
logger.error('user_picker: error resolving the user profile link', { error })
173174
} finally {
174175
this.loading = false
175176
}

apps/user_picker/src/reference.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,5 @@ registerCustomPickerElement('profile_picker', async (el, { providerId, accessibl
3737

3838
return new NcCustomPickerRenderResult(el, app)
3939
}, (el, renderResult) => {
40-
console.debug('Users picker custom picker destroy callback. el', el, 'renderResult:', renderResult)
4140
renderResult.object.unmount()
4241
}, 'normal')
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
/*!
2+
* SPDX-FileCopyrightText: 2026 Nextcloud GmbH and Nextcloud contributors
3+
* SPDX-License-Identifier: AGPL-3.0-or-later
4+
*/
5+
6+
import { getLoggerBuilder } from '@nextcloud/logger'
7+
8+
export const logger = getLoggerBuilder()
9+
.detectLogLevel()
10+
.setApp('user_picker')
11+
.build()

apps/user_picker/src/views/ProfilePickerReferenceWidget.vue

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@ import HandshakeOutline from 'vue-material-design-icons/HandshakeOutline.vue'
6464
import MapMarkerOutline from 'vue-material-design-icons/MapMarkerOutline.vue'
6565
import TextAccount from 'vue-material-design-icons/TextAccount.vue'
6666
import Web from 'vue-material-design-icons/Web.vue'
67+
import { logger } from '../utils/logger.ts'
6768
6869
export default {
6970
name: 'ProfilePickerReferenceWidget',
@@ -95,7 +96,7 @@ export default {
9596
},
9697
9798
beforeMount() {
98-
console.debug('ProfilePickerReferenceWidget', this.richObject)
99+
logger.debug('ProfilePickerReferenceWidget', this.richObject)
99100
},
100101
}
101102
</script>

0 commit comments

Comments
 (0)