diff --git a/build/emojis_plugin.js b/build/emojis_plugin.js index 9872f5331..b21c10048 100644 --- a/build/emojis_plugin.js +++ b/build/emojis_plugin.js @@ -35,7 +35,7 @@ const getAllAccessibleAnnotations = async (projectRoot) => { }), ) ) - .filter((k) => k) + .filter(Boolean) .join(',\n') return ` diff --git a/build/update-emoji.js b/build/update-emoji.js index dd965cf66..b2631ec6e 100644 --- a/build/update-emoji.js +++ b/build/update-emoji.js @@ -3,7 +3,7 @@ import emojis from '@kazvmoe-infra/unicode-emoji-json/data-by-group.json' with { type: 'json', } -Object.keys(emojis).map((k) => { +Object.keys(emojis).forEach((k) => { emojis[k].forEach((e) => { delete e.unicode_version delete e.emoji_version diff --git a/sonar-project.properties b/sonar-project.properties index 15030e3cc..5fae237b8 100644 --- a/sonar-project.properties +++ b/sonar-project.properties @@ -1,3 +1,5 @@ sonar.projectKey=Pleroma-FE sonar.qualitygatesonar.host.url.wait=true sonar.host.url=https://sonarqube.pleroma.dev +sonar.sources=src,build,tools +sonar.tests=test diff --git a/src/api/admin.js b/src/api/admin.js index d4a9e8ff9..8bbee8b4f 100644 --- a/src/api/admin.js +++ b/src/api/admin.js @@ -237,10 +237,10 @@ export const changeStatusScope = ({ credentials, }) => { const payload = {} - if (typeof sensitive !== 'undefined') { + if (sensitive !== undefined) { payload['sensitive'] = sensitive } - if (typeof visibility !== 'undefined') { + if (visibility !== undefined) { payload['visibility'] = visibility } @@ -260,15 +260,15 @@ export const announcementToPayload = ({ }) => { const payload = { content } - if (typeof startsAt !== 'undefined') { + if (startsAt !== undefined) { payload.starts_at = startsAt ? new Date(startsAt).toISOString() : null } - if (typeof endsAt !== 'undefined') { + if (endsAt !== undefined) { payload.ends_at = endsAt ? new Date(endsAt).toISOString() : null } - if (typeof allDay !== 'undefined') { + if (allDay !== undefined) { payload.all_day = allDay } diff --git a/src/api/chats.js b/src/api/chats.js index 5d766dec4..b212380de 100644 --- a/src/api/chats.js +++ b/src/api/chats.js @@ -18,7 +18,7 @@ export const chats = ({ credentials }) => url: PLEROMA_CHATS_URL, credentials, }).then(({ data }) => ({ - data: data.map(parseChat).filter((c) => c), + data: data.map(parseChat).filter(Boolean), })) export const getOrCreateChat = ({ accountId, credentials }) => @@ -40,7 +40,7 @@ export const chatMessages = ({ method: 'GET', credentials, }).then(({ data }) => ({ - data: data.map(parseChatMessage).filter((c) => c), + data: data.map(parseChatMessage).filter(Boolean), })) } diff --git a/src/api/helpers.js b/src/api/helpers.js index cb2f2bdd5..90956a4dc 100644 --- a/src/api/helpers.js +++ b/src/api/helpers.js @@ -37,11 +37,7 @@ export const paramsString = (params = {}) => { arrays.forEach(([k, array]) => { array.forEach((v) => { - if ( - typeof v === 'object' || - typeof v === 'function' || - typeof v === 'undefined' - ) + if (typeof v === 'object' || typeof v === 'function' || v === undefined) throw new TypeError('Array param cannot contain non-primitives!') }) }) diff --git a/src/components/announcement/announcement.js b/src/components/announcement/announcement.js index ee427533f..6b45b2b90 100644 --- a/src/components/announcement/announcement.js +++ b/src/components/announcement/announcement.js @@ -29,9 +29,8 @@ const Announcement = { currentUser: (state) => state.users.currentUser, }), canEditAnnouncement() { - return ( - this.currentUser && - this.currentUser.privileges.has('announcements_manage_announcements') + return this.currentUser?.privileges.has( + 'announcements_manage_announcements', ) }, content() { diff --git a/src/components/announcements_page/announcements_page.js b/src/components/announcements_page/announcements_page.js index 0f7933e5f..802e90cf6 100644 --- a/src/components/announcements_page/announcements_page.js +++ b/src/components/announcements_page/announcements_page.js @@ -33,9 +33,8 @@ const AnnouncementsPage = { return useAnnouncementsStore().announcements }, canPostAnnouncement() { - return ( - this.currentUser && - this.currentUser.privileges.has('announcements_manage_announcements') + return this.currentUser?.privileges.has( + 'announcements_manage_announcements', ) }, }, diff --git a/src/components/attachment/attachment.js b/src/components/attachment/attachment.js index f7d6ebf10..6556b07a0 100644 --- a/src/components/attachment/attachment.js +++ b/src/components/attachment/attachment.js @@ -165,16 +165,16 @@ const Attachment = { useMediaViewerStore().setCurrentMedia(this.attachment) }, onEdit(event) { - this.edit && this.edit(this.attachment, event) + this.edit?.(this.attachment, event) }, onRemove() { - this.remove && this.remove(this.attachment) + this.remove?.(this.attachment) }, onShiftUp() { - this.shiftUp && this.shiftUp(this.attachment) + this.shiftUp?.(this.attachment) }, onShiftDn() { - this.shiftDn && this.shiftDn(this.attachment) + this.shiftDn?.(this.attachment) }, stopFlash() { this.$refs.flash.closePlayer() diff --git a/src/components/chat_message/chat_message.vue b/src/components/chat_message/chat_message.vue index 394cce4b7..88ab2fc0a 100644 --- a/src/components/chat_message/chat_message.vue +++ b/src/components/chat_message/chat_message.vue @@ -1,9 +1,9 @@ - + {{ $t('status.broken_reply') }} @@ -83,7 +86,10 @@ :user="author" /> -
+
@@ -98,7 +104,6 @@ @mouseenter="hovered = true" @mouseleave="hovered = false" > -
- +
@@ -232,7 +240,10 @@ v-else class="chat-message-date-separator" > - +
diff --git a/src/components/chat_message_list/chat_message_list.vue b/src/components/chat_message_list/chat_message_list.vue index 5cdbf6871..7833730e7 100644 --- a/src/components/chat_message_list/chat_message_list.vue +++ b/src/components/chat_message_list/chat_message_list.vue @@ -7,7 +7,7 @@ :previous-item="getPreviousItem(index)" :hovered-message-chain="chatItem.messageChainId === hoveredMessageChainId" :focused="chatItem.id === focusedId" - :repliedTo="chatItem.id === repliedId" + :replied-to="chatItem.id === repliedId" @hover="onMessageHover" @delete="onMessageDelete" @reply-requested="onReplyRequested" diff --git a/src/components/chat_view/chat_view.js b/src/components/chat_view/chat_view.js index f48b75c65..16b2ae1d7 100644 --- a/src/components/chat_view/chat_view.js +++ b/src/components/chat_view/chat_view.js @@ -97,7 +97,7 @@ const Chat = { mounted() { window.addEventListener('resize', this.handleResize) window.addEventListener('scroll', this.handleScroll) - if (typeof document.hidden !== 'undefined') { + if (document.hidden !== undefined) { document.addEventListener( 'visibilitychange', this.handleVisibilityChange, @@ -112,7 +112,7 @@ const Chat = { unmounted() { window.removeEventListener('scroll', this.handleScroll) window.removeEventListener('resize', this.handleResize) - if (typeof document.hidden !== 'undefined') + if (document.hidden !== undefined) document.removeEventListener( 'visibilitychange', this.handleVisibilityChange, diff --git a/src/components/chat_view/chat_view.vue b/src/components/chat_view/chat_view.vue index e6bef0f84..06862ab3c 100644 --- a/src/components/chat_view/chat_view.vue +++ b/src/components/chat_view/chat_view.vue @@ -24,7 +24,7 @@ v-if="messages[0]?.summary_raw_html" :html="messages[0].summary_raw_html" :emoji="messages[0].emojis" - /> + /> diff --git a/src/components/color_input/color_input.vue b/src/components/color_input/color_input.vue index d6667c28f..53396f532 100644 --- a/src/components/color_input/color_input.vue +++ b/src/components/color_input/color_input.vue @@ -11,11 +11,11 @@ {{ label }}
{ } else { conversation = filter(conversation, (status) => status.type !== 'retweet') } - return conversation.filter((_) => _).sort(sortById) + return conversation.filter(Boolean).sort(sortById) } const conversation = { @@ -239,9 +239,9 @@ const conversation = { depth, }, walk(forest, forest[id], depth + 1, processed), - ].reduce((a, b) => a.concat(b), []) + ].flat() }) - .reduce((a, b) => a.concat(b), []) + .flat() const linearized = walk( threads.forest, @@ -305,11 +305,10 @@ const conversation = { topLevel() { const topLevel = this.conversation.reduce( (tl, cur) => - tl.filter( - (k) => - this.getReplies(cur.id) - .map((v) => v.id) - .indexOf(k.id) === -1, + tl.filter((k) => + this.getReplies(cur.id) + .map((v) => v.id) + .includes(k.id), ), this.conversation, ) @@ -375,7 +374,7 @@ const conversation = { return !!(this.expanded || this.isPage) }, hiddenStyle() { - const height = (this.status && this.status.virtualHeight) || '120px' + const height = this.status?.virtualHeight || '120px' return this.virtualHidden ? { height } : {} }, threadDisplayStatus() { diff --git a/src/components/conversation/conversation.vue b/src/components/conversation/conversation.vue index 170ab41d6..0833c4f89 100644 --- a/src/components/conversation/conversation.vue +++ b/src/components/conversation/conversation.vue @@ -14,9 +14,9 @@ v-if="conversation[0]?.summary_raw_html" :html="conversation[0].summary_raw_html" :emoji="conversation[0].emojis" - /> + />