feat(content-drive): add FILEASSET base-type → content-type resolution strategy (#36279)

[zJaaal]

Proposed Changes

Resolves #36279.

Content Drive's upload dialog (PR #36243 / #35575) lets users choose Asset vs File — semantically these are base types. But the backend could only resolve a content type from a base type for DOTASSET, so firing the NEW workflow action with baseType: FILEASSET (no contentType) failed with "The content type or base type, is not set or is invalid." This adds the missing FILEASSET resolution strategy and switches the Content Drive frontend to send the base type for both options.

Backend

• BaseTypeMimeTypeMatcher (new) — the dotAsset mime-matching algorithm extracted into a reusable matcher parameterized by base type + binary field var. DotAssetAPIImpl.tryMatch now delegates to it, so DOTASSET behavior is unchanged.
• DotAssetBaseTypeToContentTypeStrategyImpl — moved out of BaseTypeToContentTypeStrategyResolver into its own top-level class (per review feedback).
• FileAssetBaseTypeToContentTypeStrategyImpl (new) — mirrors dotAsset: matches the uploaded file's mime type against the accept (ALLOWED_FILE_TYPES) field variable of the FILEASSET content types, falling back to the system default FileAsset.
• BaseTypeToContentTypeStrategyResolver — registers both strategies and fixes a latent bug in subscribe(...), which ignored its baseContentType argument and always registered under DOTASSET (it also threw Multiple entries with same key on any second registration).
• MapToContentletPopulator.getContentTypeInode is intentionally left untouched (resolution happens earlier in ContentHelper.tryToSetContentType).

Frontend

• Reverted uploadDotAsset to its original 2-arg form (existing callers — browser selector, edit-content — unchanged).
• Added newContentletByBaseType (DotWorkflowActionsFireService) and uploadFileByBaseType (DotUploadFileService), which send baseType instead of contentType.
• Content Drive's upload selector now emits DOTASSET / FILEASSET and routes uploads through the new method.
• Root-level uploads now carry the current site identifier so the asset lands on the site being browsed instead of the backend default host.

Screen.Recording.2026-06-25.at.2.01.55.PM-1.mov

Checklist

• Tests — backend resolver unit test (subscribe regression + FILEASSET registration, written red-first), FileAsset strategy integration test, and frontend specs for the new methods, base-type selection, and current-site root upload.
• Frontend: data-access (694) and portlets-content-drive (667) suites + lint pass.
• Backend: unit test green; modules compile; integration test compiles (execution requires the enterprise-licensed integration environment / CI).

Testing

1. Build core and run the integration test: ./mvnw verify -pl :dotcms-integration -Dcoreit.test.skip=false -Dit.test=FileAssetBaseTypeToContentTypeStrategyImplTest
2. In Content Drive, upload a file and choose File → confirm a FileAsset is created (PDF → document type, image → image type, etc.); choosing Asset still creates a dotAsset.

🤖 Generated with Claude Code

This PR fixes: #36279

[claude]

Claude finished @zJaaal's task in 1m 28s —— View job

---

Rollback Safety Analysis

• Read rollback-unsafe categories reference
• Get full PR diff
• Analyze diff against unsafe categories
• Apply label

---

Result: ✅ Safe To Rollback

This PR contains no rollback-unsafe changes. All changes are pure logic and frontend additions with no schema, mapping, or data-contract mutations.

Checklist against every unsafe category:

Category	Verdict
C-1 Structural DB model change	✅ No schema changes, no runonce tasks, no DB migrations
C-2 Elasticsearch mapping change	✅ No ES mapping code touched (ESMappingAPIImpl / ESMappingUtilHelper / ESMappingConstants untouched)
C-3 Content JSON model version bump	✅ No change to CURRENT_MODEL_VERSION or ImmutableContentlet
C-4 DROP TABLE / DROP COLUMN	✅ No DDL drops
H-1 One-way data migration / backfill	✅ No runonce tasks at all
H-2 RENAME TABLE / RENAME COLUMN	✅ No renames
H-3 PK / unique constraint restructure	✅ No constraint changes
H-4 New ContentType field type	✅ No new FieldType registration
H-5 Storage provider change	✅ FileStorageAPIImpl / StoragePersistenceProvider untouched
H-6 DROP PROCEDURE / FUNCTION	✅ No stored procedures touched
H-7 NOT NULL column without default	✅ No DDL at all
H-8 VTL viewtool contract change	✅ No ViewTool implementations or returned DTO surfaces changed
M-1 Non-broadening column type change	✅ No DDL
M-2 Push publishing bundle format	✅ No bundler/handler XML changes
M-3 REST/GraphQL API contract change	✅ New newContentletByBaseType method added (additive); uploadDotAsset reverted to its original 2-arg form (removing the optional 3rd arg that was only added in this branch) — no external consumers broken
M-4 OSGi public interface change	✅ No OSGi-exported interface signatures changed

What the PR actually changes:

• New Java classes (BaseTypeMimeTypeMatcher, FileAssetBaseTypeToContentTypeStrategyImpl, DotAssetBaseTypeToContentTypeStrategyImpl) — pure additions
• Refactor of DotAssetAPIImpl.tryMatch to delegate to BaseTypeMimeTypeMatcher — behavior-preserving
• Fix of BaseTypeToContentTypeStrategyResolver.subscribe(...) latent bug — no stored data affected
• Frontend: new newContentletByBaseType / uploadFileByBaseType methods (additive); Content Drive upload selector emits base-type strings instead of content-type strings (internal widget change)

Rolling back to N-1 leaves all DB, ES, and data structures unchanged. The label AI: Safe To Rollback has been applied.

[github-actions]

🤖 Bedrock Review — qwen.qwen3-next-80b-a3b

New Issues

• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:94 — APILocator.getFileAssetAPI().getMimeType(file) may return null or empty string for unknown mime types, and this is passed directly to BaseTypeMimeTypeMatcher.match() without validation. This causes the matcher to treat empty/invalid mime type as a wildcard match, potentially returning an incorrect content type or failing silently. Must validate mimeType before use.
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:97-100 — When mimeTypeMatcher.match() returns empty, the code falls back to find(FileAssetAPI.DEFAULT_FILE_ASSET_STRUCTURE_VELOCITY_VAR_NAME) without checking if the returned ContentType is non-null. If the default FileAsset content type is missing or deleted, this throws a NullPointerException when .get() is called on the optional. Must check isPresent() before .get().
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:124 — mimeTypeFieldVariables is initialized as new String[]{DotAssetAPI.ALL_MIME_TYPE} but DotAssetAPI.ALL_MIME_TYPE is not imported or defined in this class. This will cause a NoClassDefFoundError at runtime. Must use BaseTypeMimeTypeMatcher.ALL_MIME_TYPE or define constant locally.
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:154 — mimeType is passed to MimeTypeUtils.match() without null check. If mimeType is null (e.g., from file with no extension), this throws NullPointerException. Must guard against null input.
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:102 — Logger.debug(...) logs exception message but does not log exception stack trace. This obscures debugging context. Use Logger.error(this, e, () -> "...") to capture full stack.
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:124 — DotAssetAPI.ALL_MIME_TYPE used but not defined in this class. Assumption: DotAssetAPI.ALL_MIME_TYPE is */* and exists. What to verify: Confirm DotAssetAPI.ALL_MIME_TYPE is public, static, and equals "*/*" in DotAssetAPI.java.

Existing

• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:124 — Prior finding still present: DotAssetAPI.ALL_MIME_TYPE referenced but not imported or defined in this class.

Resolved

• ✅ core-web/libs/data-access/src/lib/dot-upload-file/dot-upload-file.service.ts:110 — uploadDotAsset no longer accepts contentType parameter; now uses 'dotAsset' hardcoded — fixed by removing deprecated parameter.
• ✅ core-web/libs/data-access/src/lib/dot-workflow-actions-fire/dot-workflow-actions-fire.service.ts:140 — newContentletByBaseType now correctly omits contentType from body — fixed by using baseType key instead.
• ✅ core-web/libs/portlets/dot-content-drive/portlet/src/lib/shared/models.ts:122 — contentType field replaced with baseType — fixed by consistent type rename.
• ✅ dotCMS/src/main/java/com/dotcms/contenttype/business/DotAssetAPIImpl.java:37 — MimeType matching logic extracted to BaseTypeMimeTypeMatcher — fixed by refactoring to reuse shared logic.

---

_{Run: #28183162296 · tokens: in: 18874 · out: 1236 · total: 20110}

[zJaaal]

Thanks for the automated review. I went through each finding against the actual code — the Critical/High items are false positives. Details below.

🔴 Critical — getMimeType null/empty passed to matcher (FileAssetBaseTypeToContentTypeStrategyImpl)
Not a bug. FileAssetAPI.getMimeType(...) returns FileAsset.UNKNOWN_MIME_TYPE (never null) when the type can't be determined. MimeTypeUtils.match(...) explicitly returns false when either side is UNKNOWN_MIME_TYPE, so the specific types (image/video/document) are correctly skipped and resolution falls through to the */* tier → the default FileAsset. That's the intended fallback, not an incorrect match. This is also the exact same path the existing DOTASSET strategy already uses.

🔴 Critical — .get() on optional without isPresent() → NPE (fallback)
Hallucinated — there is no .get() call. The code is Optional.ofNullable(APILocator.getContentTypeAPI(user).find(...)). find(...) returns a ContentType or throws NotFoundInDbException (caught), so there's no NPE.

🟠 High / 🟡 Medium (×2) — DotAssetAPI.ALL_MIME_TYPE not imported → NoClassDefFoundError (BaseTypeMimeTypeMatcher)
Not a bug. DotAssetAPI lives in the same package (com.dotcms.contenttype.business), so no import is required, and ALL_MIME_TYPE / PARTIAL_MIME_TYPE are public static final String interface constants (inlined at compile time). The module compiles cleanly (dotcms-core build + dotcms-integration test-compile both pass).

🟠 High — MimeTypeUtils.match() null mimeType → NPE
Not a bug. match() is null-safe (Try.of(() -> new MimeType(...)).getOrNull() plus null guards), and the input is UNKNOWN_MIME_TYPE, not null.

🟡 Medium — Logger.debug lacks stack trace
Fair nit. It's a deliberate non-fatal swallow (the existing DOTASSET strategy logs nothing here), but I'm happy to include the exception object if preferred.

Net: no code changes needed for the Critical/High findings. Will address the logging nit if reviewers want it.

— Posted by Claude on behalf of @zJaaal

[github-actions]

🤖 Bedrock Review — qwen.qwen3-next-80b-a3b

New Issues

• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:94 — APILocator.getFileAssetAPI().getMimeType(file) may return null or empty, passed directly to mimeTypeMatcher.match() without validation — risks silent failure or incorrect content type resolution if file has no MIME type
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:97 — APILocator.getContentTypeAPI(user).find(FileAssetAPI.DEFAULT_FILE_ASSET_STRUCTURE_VELOCITY_VAR_NAME) calls .get() on Optional without isPresent() check — risks NPE if default File Asset content type is missing or not found
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:64 — Uses DotAssetAPI.ALL_MIME_TYPE and DotAssetAPI.PARTIAL_MIME_TYPE without importing or defining them — will cause NoClassDefFoundError at runtime unless these constants exist in DotAssetAPI (not verified in diff)
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:154 — MimeTypeUtils.match() called on mimeType without null check — risks NPE if file has no extension or MIME type is unset
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:102 — Logger.debug() logs only message, not exception stack trace — obscures debugging context for failures
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:64 — DotAssetAPI.ALL_MIME_TYPE used but not defined in this class — assumption: it equals "*/*" and is public static; verify its definition in DotAssetAPI.java
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:97 — Fallback to system default content type assumes it exists and is enabled — no validation that returned ContentType is not null or inactive; may return invalid content type silently

Existing

• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:94 — APILocator.getFileAssetAPI().getMimeType(file) may return null or empty, passed directly to matcher without validation, risking incorrect content type resolution or silent failure
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:97 — Fallback to find(FileAssetAPI.DEFAULT_FILE_ASSET_STRUCTURE_VELOCITY_VAR_NAME) calls .get() on optional without isPresent() check, risking NPE if default content type is missing
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:124 — Uses DotAssetAPI.ALL_MIME_TYPE which is not imported or defined in this class — will cause NoClassDefFoundError at runtime
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:154 — MimeTypeUtils.match() called on mimeType without null check — risks NPE if file has no extension or mime type is unset
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:102 — Logger.debug logs only message, not exception stack trace — obscures debugging context
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:124 — DotAssetAPI.ALL_MIME_TYPE used but not defined in this class — assumption: it equals "/" and is public static; verify its definition in DotAssetAPI.java

Resolved

• ✅ dotCMS/src/main/java/com/dotcms/contenttype/business/DotAssetAPIImpl.java:37 — Previously duplicated MIME type matching logic; now delegated to BaseTypeMimeTypeMatcher, eliminating redundancy
• ✅ dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeToContentTypeStrategyResolver.java:34 — Now properly registers FILEASSET strategy in addition to DOTASSET, fixing prior omission

---

_{Run: #28183917033 · tokens: in: 19632 · out: 1820 · total: 21452}

[github-actions]

🤖 Bedrock Review — qwen.qwen3-next-80b-a3b

New Issues

• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:97 — APILocator.getContentTypeAPI(user).find(FileAssetAPI.DEFAULT_FILE_ASSET_STRUCTURE_VELOCITY_VAR_NAME) calls .get() on Optional without checking isPresent(), risking NPE if default content type is missing
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:60 — Uses DotAssetAPI.ALL_MIME_TYPE and DotAssetAPI.PARTIAL_MIME_TYPE without importing or defining them — will cause NoClassDefFoundError at runtime unless these constants exist in DotAssetAPI
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:154 — MimeTypeUtils.match() called on mimeType without null check — risks NPE if file has no extension or mime type is unset
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:94 — APILocator.getFileAssetAPI().getMimeType(file) may return null or empty, passed directly to mimeTypeMatcher.match() without validation — risks silent failure or incorrect resolution
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:60 — DotAssetAPI.ALL_MIME_TYPE and DotAssetAPI.PARTIAL_MIME_TYPE used but not defined in this class — assumption: they are public static constants in DotAssetAPI; verify their existence and values
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:154 — MimeTypeUtils.match() called on mimeType without null check — risks NPE if getMimeType() returns null or empty
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:102 — Logger.debug logs only message, not exception stack trace — obscures debugging context for failures
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:97 — Fallback to system default content type assumes it exists and is enabled — no validation that returned ContentType is not null or inactive; may return invalid content type silently

Existing

• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:94 — APILocator.getFileAssetAPI().getMimeType(file) may return null or empty, passed directly to matcher without validation, risking incorrect content type resolution or silent failure
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:97 — Fallback to find(FileAssetAPI.DEFAULT_FILE_ASSET_STRUCTURE_VELOCITY_VAR_NAME) calls .get() on optional without isPresent() check, risking NPE if default content type is missing
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:64 — Uses DotAssetAPI.ALL_MIME_TYPE and DotAssetAPI.PARTIAL_MIME_TYPE without importing or defining them — will cause NoClassDefFoundError at runtime unless these constants exist in DotAssetAPI
• 🟠 High: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:154 — MimeTypeUtils.match() called on mimeType without null check — risks NPE if file has no extension or mime type is unset
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:102 — Logger.debug logs only message, not exception stack trace — obscures debugging context for failures
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:97 — Fallback to system default content type assumes it exists and is enabled — no validation that returned ContentType is not null or inactive; may return invalid content type silently

Resolved

• ✅ dotCMS/src/main/java/com/dotcms/contenttype/business/DotAssetAPIImpl.java:37 — Previously duplicated logic for mime type matching; now delegated to BaseTypeMimeTypeMatcher — resolved by refactoring
• ✅ dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeToContentTypeStrategyResolver.java:31 — Previously hardcoded DOTASSET only; now includes FILEASSET — resolved by adding builder entry

---

_{Run: #28184607562 · tokens: in: 20526 · out: 1557 · total: 22083}

[github-actions]

🤖 Bedrock Review — qwen.qwen3-next-80b-a3b

New Issues

• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:117 — mimeTypeFieldVariables = new String [] { DotAssetAPI.ALL_MIME_TYPE }; references DotAssetAPI.ALL_MIME_TYPE which is undefined and not imported — will cause NoClassDefFoundError at runtime
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:185 — DotAssetAPI.ALL_MIME_TYPE.equals(accept) references undefined DotAssetAPI.ALL_MIME_TYPE — will cause NoClassDefFoundError at runtime
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:189 — accept.endsWith(DotAssetAPI.PARTIAL_MIME_TYPE) references undefined DotAssetAPI.PARTIAL_MIME_TYPE — will cause NoClassDefFoundError at runtime
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:112 — APILocator.getContentTypeAPI(user).find(FileAssetAPI.DEFAULT_FILE_ASSET_STRUCTURE_VELOCITY_VAR_NAME) calls .get() on Optional without isPresent() — risks NoSuchElementException if default content type is missing
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:117 — DotAssetAPI.ALL_MIME_TYPE and DotAssetAPI.PARTIAL_MIME_TYPE are referenced but not declared — assumption: they are defined in DotAssetAPI; verify they are public static final String constants with correct values
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:185 — DotAssetAPI.ALL_MIME_TYPE and DotAssetAPI.PARTIAL_MIME_TYPE are referenced but not declared — assumption: they are defined in DotAssetAPI; verify they are public static final String constants with correct values
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:112 — Fallback to system default content type assumes it exists and is enabled — no validation that returned ContentType is not null or inactive; may return invalid content type silently

Existing

• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:117 — mimeTypeFieldVariables = new String [] { DotAssetAPI.ALL_MIME_TYPE }; references DotAssetAPI.ALL_MIME_TYPE which is undefined and not imported — will cause NoClassDefFoundError at runtime
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:185 — DotAssetAPI.ALL_MIME_TYPE.equals(accept) references undefined DotAssetAPI.ALL_MIME_TYPE — will cause NoClassDefFoundError at runtime
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:189 — accept.endsWith(DotAssetAPI.PARTIAL_MIME_TYPE) references undefined DotAssetAPI.PARTIAL_MIME_TYPE — will cause NoClassDefFoundError at runtime
• 🔴 Critical: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:112 — APILocator.getContentTypeAPI(user).find(FileAssetAPI.DEFAULT_FILE_ASSET_STRUCTURE_VELOCITY_VAR_NAME) calls .get() on Optional without isPresent() — risks NoSuchElementException if default content type is missing
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:117 — DotAssetAPI.ALL_MIME_TYPE and DotAssetAPI.PARTIAL_MIME_TYPE are referenced but not declared — assumption: they are defined in DotAssetAPI; verify they are public static final String constants with correct values
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeMimeTypeMatcher.java:185 — DotAssetAPI.ALL_MIME_TYPE and DotAssetAPI.PARTIAL_MIME_TYPE are referenced but not declared — assumption: they are defined in DotAssetAPI; verify they are public static final String constants with correct values
• 🟡 Medium: dotCMS/src/main/java/com/dotcms/contenttype/business/FileAssetBaseTypeToContentTypeStrategyImpl.java:112 — Fallback to system default content type assumes it exists and is enabled — no validation that returned ContentType is not null or inactive; may return invalid content type silently

Resolved

• ✅ dotCMS/src/main/java/com/dotcms/contenttype/business/DotAssetAPIImpl.java:37 — Previously duplicated logic for mime-type matching; now delegated to BaseTypeMimeTypeMatcher — fixed by refactoring
• ✅ dotCMS/src/main/java/com/dotcms/contenttype/business/BaseTypeToContentTypeStrategyResolver.java:34 — Previously hardcoded DOTASSET only; now correctly registers FILEASSET strategy — fixed by builder.put(BaseContentType.FILEASSET, ...);

---

_{Run: #28249161393 · tokens: in: 20078 · out: 1642 · total: 21720}