chore: Use getFileInfo helper from utils (#10819)
# Pull Request Template ## Description The PR includes the usage of `getFileInfo` helper from utils https://github.com/chatwoot/utils/pull/40. Fixes https://github.com/chatwoot/chatwoot/pull/10806#discussion_r1937797905 ## How Has This Been Tested? **Screenshot** <img width="490" alt="image" src="https://github.com/user-attachments/assets/f0788e89-b670-47da-b0ca-3765eb424be0" /> ## Checklist: - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my code - [ ] I have commented on my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] Any dependent changes have been merged and published in downstream modules
This commit is contained in:
@@ -1,6 +1,7 @@
|
||||
<script setup>
|
||||
import { computed } from 'vue';
|
||||
import { useI18n } from 'vue-i18n';
|
||||
import { getFileInfo } from '@chatwoot/utils';
|
||||
|
||||
import FileIcon from 'next/icon/FileIcon.vue';
|
||||
import Icon from 'next/icon/Icon.vue';
|
||||
@@ -14,43 +15,20 @@ const { attachment } = defineProps({
|
||||
|
||||
const { t } = useI18n();
|
||||
|
||||
const fileName = computed(() => {
|
||||
const url = attachment.dataUrl;
|
||||
if (!url) return t('CONVERSATION.UNKNOWN_FILE_TYPE');
|
||||
|
||||
try {
|
||||
const encodedFilename = url.substring(url.lastIndexOf('/') + 1);
|
||||
return decodeURIComponent(encodedFilename);
|
||||
} catch {
|
||||
return t('CONVERSATION.UNKNOWN_FILE_TYPE');
|
||||
}
|
||||
});
|
||||
|
||||
const fileType = computed(() => fileName.value.split('.').pop()?.toLowerCase());
|
||||
|
||||
const fileNameWithoutExt = computed(() => {
|
||||
const parts = fileName.value.split('.');
|
||||
|
||||
// If there's no extension (no dots in filename)
|
||||
if (parts.length === 1) {
|
||||
return fileName.value.trim();
|
||||
}
|
||||
|
||||
// Take all parts except the last one (extension)
|
||||
const nameWithoutExt = parts.slice(0, -1).join('.');
|
||||
return nameWithoutExt.trim();
|
||||
const fileDetails = computed(() => {
|
||||
return getFileInfo(attachment?.dataUrl || '');
|
||||
});
|
||||
|
||||
const displayFileName = computed(() => {
|
||||
const name = fileNameWithoutExt.value;
|
||||
const { base, type } = fileDetails.value;
|
||||
const truncatedName = (str, maxLength, hasExt) =>
|
||||
str.length > maxLength
|
||||
? `${str.substring(0, maxLength).trimEnd()}${hasExt ? '..' : '...'}`
|
||||
: str;
|
||||
|
||||
return fileType.value
|
||||
? `${truncatedName(name, 14, true)}.${fileType.value}`
|
||||
: truncatedName(name, 16, false);
|
||||
return type
|
||||
? `${truncatedName(base, 12, true)}.${type}`
|
||||
: truncatedName(base, 14, false);
|
||||
});
|
||||
|
||||
const textColorClass = computed(() => {
|
||||
@@ -73,7 +51,7 @@ const textColorClass = computed(() => {
|
||||
zip: 'dark:text-[#EDEEF0] text-[#2F265F]',
|
||||
};
|
||||
|
||||
return colorMap[fileType.value] || 'text-n-slate-12';
|
||||
return colorMap[fileDetails.value.type] || 'text-n-slate-12';
|
||||
});
|
||||
</script>
|
||||
|
||||
@@ -81,10 +59,10 @@ const textColorClass = computed(() => {
|
||||
<div
|
||||
class="h-9 bg-n-alpha-white gap-2 overflow-hidden items-center flex px-2 rounded-lg border border-n-container"
|
||||
>
|
||||
<FileIcon class="flex-shrink-0" :file-type="fileType" />
|
||||
<FileIcon class="flex-shrink-0" :file-type="fileDetails.type" />
|
||||
<span
|
||||
class="flex-1 min-w-0 text-sm max-w-36"
|
||||
:title="fileName"
|
||||
:title="fileDetails.name"
|
||||
:class="textColorClass"
|
||||
>
|
||||
{{ displayFileName }}
|
||||
|
||||
Reference in New Issue
Block a user