Skip to content

Fix RTL resource detection by resolving virtual path to physical path#6955

Merged
mitchelsellers merged 1 commit intodnnsoftware:release/10.2.2from
mnouraei:fixRtl
Feb 3, 2026
Merged

Fix RTL resource detection by resolving virtual path to physical path#6955
mitchelsellers merged 1 commit intodnnsoftware:release/10.2.2from
mnouraei:fixRtl

Conversation

@mnouraei
Copy link
Copy Markdown
Contributor

Summary

Fixes an issue where RTL-specific resources were not detected correctly because
File.Exists was being called with a virtual path.

Details

  • Converts RTL resolved paths from virtual (~/...) to physical paths
  • Uses Path.Combine to safely build the physical file system path
  • Ensures RTL resources are only swapped when the corresponding .rtl file exists

Impact

  • RTL styles/scripts are now correctly loaded when available
  • No behavior change for LTR cultures or CDN/external resources

Notes

This change only affects local resources and preserves existing CDN logic.

Copy link
Copy Markdown
Contributor

@uzmannazari uzmannazari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice improvement, thanks for the contribution!

@bdukes bdukes added this to the 10.2.2 milestone Feb 2, 2026
@bdukes bdukes changed the base branch from develop to release/10.2.2 February 2, 2026 14:50
@mitchelsellers mitchelsellers merged commit 0fe6712 into dnnsoftware:release/10.2.2 Feb 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants