@bagelink/vue 1.6.7 → 1.6.17

package/src/utils/index.ts

@@ -20,7 +20,7 @@ export function slugify(str: string) {
 }
 
 export function keyToLabel(key?: string): string | undefined {
-	if (key === undefined) {return key}
+	if (key === undefined) { return key }
 	return (
 		key
 			.split('_')
@@ -31,7 +31,7 @@ export function keyToLabel(key?: string): string | undefined {
 
 export async function copyText(text: string, cb?: (msg: string) => void) {
 	await navigator.clipboard.writeText(text)
-	if (cb) {cb('Copied to clipboard')}
+	if (cb) { cb('Copied to clipboard') }
 }
 
 export function initials(...strArr: string[]) {
@@ -40,7 +40,7 @@ export function initials(...strArr: string[]) {
 }
 
 export function useEscape(event: KeyboardEvent, closeModel: () => void) {
-	if ('Escape' === event.key) {
+	if (event.key === 'Escape') {
 		closeModel()
 	}
 }
@@ -48,7 +48,7 @@ export function useEscape(event: KeyboardEvent, closeModel: () => void) {
 export function classify(fieldVal?: any, row?: any, ...classes: any[]) {
 	return classes
 		.map((cls) => {
-			if ('function' === typeof cls) {return cls(fieldVal, row)}
+			if (typeof cls === 'function') { return cls(fieldVal, row) }
 			return cls
 		})
 		.join(' ')
@@ -60,14 +60,14 @@ export function bindAttrs<T, P extends Path<T>>(
 	row?: T
 ) {
 // TODO: Fix this so that you don't have to return a fn for other on* event handlers
-	if (!attrs) {return {}}
+	if (!attrs) { return {} }
 
 	const exclude = ['class', 'onClick', '$el'] as const
 	const arr = Object.entries(attrs)
 		.filter(([key]) => !exclude.includes(key as typeof exclude[number]))
 		.map(([key, value]) => [
 			key,
-			 'function' === typeof value ? value(fieldVal, row) : value,
+			typeof value === 'function' ? value(fieldVal, row) : value,
 		])
 
 	const resolvedAttrs: { [key: string]: any } = Object.fromEntries(arr)
@@ -84,16 +84,16 @@ export function bindAttrs<T, P extends Path<T>>(
 }
 
 export function iffer(field: any, itemData: any) {
-	if (field['v-if'] === undefined) {return true}
-	if ('boolean' === typeof field['v-if']) {return field['v-if']}
-	if ('string' === typeof field['v-if']) {return true}
-	if ('function' === typeof field['v-if'])
-		{return field['v-if']?.(itemData?.[field.id], itemData)}
+	if (field['v-if'] === undefined) { return true }
+	if (typeof field['v-if'] === 'boolean') { return field['v-if'] }
+	if (typeof field['v-if'] === 'string') { return true }
+	if (typeof field['v-if'] === 'function')
+	{ return field['v-if']?.(itemData?.[field.id], itemData) }
 	return true
 }
 
 export function denullify(itemData?: { [key: string]: any }, fieldID?: string) {
-	if (!fieldID) {return undefined}
+	if (!fieldID) { return undefined }
 	return itemData ? itemData[fieldID] : undefined
 }
 
@@ -110,7 +110,7 @@ export function getFallbackSchema<T>(
 		label: keyToLabel(id),
 		transform: (val: any) => {
 			const dateFields = ['created_at', 'updated_at']
-			if (dateFields.includes(id)) {return val ? new Date(val).toLocaleString() : val}
+			if (dateFields.includes(id)) { return val ? new Date(val).toLocaleString() : val }
 			return val
 		},
 	})) as BglFormSchemaT<T>
@@ -133,13 +133,13 @@ export async function appendScript(src: string, options?: { id?: string }): Prom
 	// If this script is already loading, return the existing promise
 	if (scriptsLoading.has(scriptId)) {
 		const existingPromise = scriptsLoading.get(scriptId)
-		if (existingPromise) {return existingPromise}
+		if (existingPromise) { return existingPromise }
 	}
 
 	// Check if script is already in the document
 	if (options?.id && document.getElementById(options.id)) {
 		return Promise.resolve()
-	}if (document.querySelector(`script[src="${src}"]`)) {
+	} if (document.querySelector(`script[src="${src}"]`)) {
 		return Promise.resolve()
 	}
 
@@ -188,13 +188,13 @@ export function appendStyle(src: string): Promise<void> {
 }
 
 export function normalizeURL(url: string) {
-	if (url.startsWith('https://')) {return url}
+	if (url.startsWith('https://')) { return url }
 	url = url.replace(/http:\/\//, '')
 	return `https://${url}`
 }
 
 export function normalizeDimension(value: string | number | undefined, defaultMetric = 'px'): string | undefined {
-	if ('number' === typeof value) {return `${value}${defaultMetric}`}
+	if (typeof value === 'number') { return `${value}${defaultMetric}` }
 	return value
 }
 
@@ -211,7 +211,7 @@ import { getBagelInstance } from '../composables/useBagel'
 const URL_REGEX = /^https?:\/\/|^\/\//
 
 export function pathKeyToURL(pathKey?: string) {
-	if (pathKey === undefined || '' === pathKey || URL_REGEX.test(pathKey)) {return pathKey}
+	if (pathKey === undefined || pathKey === '' || URL_REGEX.test(pathKey)) { return pathKey }
 
 	const bagel = getBagelInstance()
 	if (!bagel) {
@@ -220,7 +220,7 @@ export function pathKeyToURL(pathKey?: string) {
 	}
 
 	if (pathKey.startsWith('static/')) {
-		if (bagel.host === undefined || '' === bagel.host) {
+		if (bagel.host === undefined || bagel.host === '') {
 			console.warn('Bagel host is not configured. Please set it in BagelVue options.')
 			return pathKey
 		}
@@ -231,12 +231,12 @@ export function pathKeyToURL(pathKey?: string) {
 }
 
 export function getNestedValue(obj: any, path?: string, defaultValue: any = undefined): any {
-	if (!path) {return obj}
+	if (!path) { return obj }
 	const keys = path.split(/[.[]/)
 	let current = obj
 
 	for (const key of keys) {
-		if (!current || 'object' !== typeof current || !(key in current)) {
+		if (!current || typeof current !== 'object' || !(key in current)) {
 			return defaultValue
 		}
 		current = current[key]
@@ -247,19 +247,40 @@ export function getNestedValue(obj: any, path?: string, defaultValue: any = unde
 
 const upgradeHeaders = (url: string) => url.replace(/http:\/\//, '//')
 
+export function tryRun<T>(func: () => T, callback?: (error: Error) => void): T | undefined {
+	try {
+		return func()
+	} catch (error) {
+		if (callback) {
+			callback(error as Error)
+		}
+	}
+}
+
 export function downloadFile(source: string | Blob, fileName?: string) {
+	if (typeof document === 'undefined') { return } // SSR / non-browser guard
+
 	const link = document.createElement('a')
 	link.target = '_blank'
+	link.rel = 'noopener noreferrer'
+
+	const hasProvidedFileName
+		= fileName !== undefined
+			&& fileName !== ''
 
-	if ('string' === typeof source) {
+	if (typeof source === 'string') {
 		link.href = upgradeHeaders(source)
 		const fileNameFromSource = source.split('/').pop()
-		link.download = fileName || fileNameFromSource || 'download'
+		const hasFileNameFromSource = fileNameFromSource !== undefined && fileNameFromSource !== ''
+		link.download = hasProvidedFileName
+			? fileName
+			: (hasFileNameFromSource ? fileNameFromSource : 'download')
 	} else {
 		const url = URL.createObjectURL(source)
 		link.href = url
-		link.download = fileName || 'download'
-		URL.revokeObjectURL(url)
+		link.download = hasProvidedFileName ? fileName : 'download'
+		// IMPORTANT: revoke after the browser started consuming (revoking immediately can break the download .
+		setTimeout(() => tryRun(() => { URL.revokeObjectURL(url) }), 0)
 	}
 
 	document.body.appendChild(link)

package/vite.config.ts

@@ -28,7 +28,7 @@ export default defineConfig(() => ({
 		lib: {
 			entry: resolve(indexDir, 'index.ts'),
 			formats: ['es', 'cjs'],
-			fileName: (module, entry) => `${entry}.${ 'es' === module ? 'mjs' : module}`,
+			fileName: (module, entry) => `${entry}.${module === 'es' ? 'mjs' : module}`,
 			cssFileName: 'style',
 		},
 		rollupOptions: {