inviton-powerduck 0.0.331 → 0.0.333

package/app/powerduck-system-resources.ts

@@ -37,6 +37,7 @@ export interface IPowerduckSystemResources {
 	fileDownloadTitle: string;
 	imageCrop: string;
 	uploadImage: string;
+	uploadFailed: string;
 	copyToClipboard: string;
 	copyToClipboardSuccess: string;
 	loginExpired: string;

package/common/extensions/temporal-extensions.ts

@@ -1,4 +1,5 @@
 /* eslint-disable func-style */
+import { Temporal } from '@js-temporal/polyfill';
 import PowerduckState from '../../app/powerduck-state';
 
 export const toDisplayString = Symbol('toDisplayString');

package/common/utils/file-helper.ts

@@ -3,16 +3,32 @@ import PowerduckState from '../../app/powerduck-state';
 import { AppHttpProvider } from '../api-http';
 import { DomainHelper } from './domain-helper';
 
+const ABSOLUTE_URL_RE = /^https?:\/\//i;
+const LEADING_SLASH_ABSOLUTE_RE = /^\/https?:\/\//i;
+
 export default class FileHelper {
 	static getFullPath(file: Photo) {
-		if (file == null) {
+		if (file == null || file.path == null) {
 			return null;
 		}
 
+		const rawPath = file.path.trim();
+
+		// 1. Already-absolute URL (Bunny CDN, S3, data:, protocol-relative) — return as-is.
+		if (ABSOLUTE_URL_RE.test(rawPath) || rawPath.startsWith('//') || rawPath.startsWith('data:')) {
+			return rawPath;
+		}
+
+		// 2. Stray leading-slash before scheme ("/https://..."): strip the slash.
+		if (LEADING_SLASH_ABSOLUTE_RE.test(rawPath)) {
+			return rawPath.slice(1);
+		}
+
+		// 3. Legacy relative path (e.g. "/photos/resorts/file.png") — resolve against backend domain.
 		if (AppHttpProvider.enforceDomain == '/') {
-			return PowerduckState.getCdnPath() + PowerduckState.getFilesPath() + file.path;
+			return PowerduckState.getCdnPath() + PowerduckState.getFilesPath() + rawPath;
 		}
 
-		return DomainHelper.getDomainFromUrl(AppHttpProvider.enforceDomain, true) + PowerduckState.getFilesPath() + file.path;
+		return DomainHelper.getDomainFromUrl(AppHttpProvider.enforceDomain, true) + PowerduckState.getFilesPath() + rawPath;
 	}
 }

package/common/utils/temporal-utils.ts

@@ -1,3 +1,4 @@
+import { Temporal } from '@js-temporal/polyfill';
 import { isNullOrEmpty } from './is-null-or-empty';
 
 const isDigit = (c: string): boolean => c >= '0' && c <= '9';

package/common/utils/upload-image-helper.ts

@@ -1,6 +1,7 @@
 import type { ImageResponse, OnUploadImageResponse } from '../../data/image';
 import { globalState } from '../../app/global-state';
 import PowerduckState from '../../app/powerduck-state';
+import NotificationProvider from '../../components/ui/notification';
 import { AppHttpProvider } from '../api-http';
 import { BrowserImageCompression } from './broswer-image-compression';
 import { isNullOrEmpty } from './is-null-or-empty';
@@ -25,7 +26,7 @@ export class UploadImageHelper {
         disableCompression: boolean = false,
         compressionMaxMb: number = 0.5,
         url?: string,
-	): Promise<ImageResponse> {
+	): Promise<ImageResponse | null> {
 		if (!(disableCompression || file.type == 'image/svg+xml')) {
 			file = await BrowserImageCompression.compress(file, {
 				maxSizeMB: compressionMaxMb,
@@ -49,6 +50,12 @@ export class UploadImageHelper {
 			credentials: UploadImageHelperConfig.includeCredentials ? PowerduckState.getIsDebugMode() ? 'include' : 'same-origin' : 'omit',
 		});
 
+		if (!response.ok) {
+			NotificationProvider.showErrorMessage(PowerduckState.getResourceValue('uploadFailed'));
+
+			return null;
+		}
+
 		return (await response.json()) as ImageResponse;
 	}
 

package/components/datatable/css/datatable.css

@@ -10,7 +10,11 @@
 	position: relative;
 }
 
-.dt-table.dt-table-mode table {
+/* Direct-child selector — without `>`, this rule leaks into any nested
+   <table> inside a cell (e.g. the date-range picker's month tables), forcing
+   them to min-width: 100% of the DataTable container and breaking their
+   layout. Only the DataTable's own root <table> needs this styling. */
+.dt-table.dt-table-mode > table {
 	position: relative;
 	border-collapse: collapse;
 	min-width: 100%;