Skip to content

Conversation

@aw0lid
Copy link

@aw0lid aw0lid commented Jan 17, 2026

Fixes #108422

Description

This PR addresses the hardcoded "on" string issue in ExtendedPropertyDescriptor by moving it to the centralized resource file (Strings.resx). This change is essential for supporting proper localization (L10N) across different languages where the word order or the preposition might differ.

Changes

  • Localization: Moved the hardcoded formatting string to SR.MetaExtenderName in resources.
  • Code Optimization: - Replaced manually concatenated strings with SR.Format for safer and more localized formatting.
    • Refactored logic to use string.IsNullOrEmpty for better readability and slightly better performance.
  • Wasm Fix: Cleaned up unnecessary test file inclusions in the .csproj that were causing environment-specific build failures.

Impact

  • Enables translators to localize the "on" connector for extender provider names.
  • No breaking changes in functionality; only internal string handling is improved.

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jan 17, 2026
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-componentmodel
See info in area-owners.md if you want to be subscribed.

- Replaced hardcoded "on" string with SR.MetaExtenderName in ExtendedPropertyDescriptor.
- Removed unused ItemGroup and Content inclusion for testFileToReload.json that was causing Wasm build failures.
- Optimized logic with string.IsNullOrEmpty for better maintainability.
….csproj

Description: Restoring the csproj file to its original state to resolve WASM and library test failures, keeping only the localization fix in ExtendedPropertyDescriptor.cs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-System.ComponentModel community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[LOC]The "on" is not translated for Error on errorProvider property on all local languages

1 participant