Skip to content

Conversation

@SilverGreen93
Copy link
Contributor

The string returned by StatusCenter_ProcessedItems_Header.GetLocalizedFormatResource is empty and does not display anything in the Status Center.

  • Pass the correct number of parameters to GetLocalizedFormatResource to get the real format string instead of "".
  • Correct plural form for fraction. The divisor must be in accordance to the plural form, not the dividend.

8/20 items processed = Processed 8 out of 20 items.

Steps used to test these changes

image

The string returned by `StatusCenter_ProcessedItems_Header.GetLocalizedFormatResource` is empty and does not display anything in the Status Center.

- Pass the correct number of parameters to `GetLocalizedFormatResource` to get the real string instead of "".
- Correct plural form for fraction. The divisor must be in accordance to the plural form, not the dividend.

8/20 items processed = Processed 8 out of 20 items.
@SilverGreen93 SilverGreen93 changed the title StatusCenter: Item processed fraction is not displayed StatusCenter: Items processed fraction is not displayed Jul 31, 2025
@yaira2
Copy link
Member

yaira2 commented Jul 31, 2025

Can you check how this affects the progress reporting for cloning git repos?

@yaira2 yaira2 added the ready for review Pull requests that are ready for review label Jul 31, 2025
@SilverGreen93
Copy link
Contributor Author

Can you check how this affects the progress reporting for cloning git repos?

All seems to work nicely.

image

Copy link
Member

@yaira2 yaira2 left a comment

Choose a reason for hiding this comment

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

LGTM

@yaira2 yaira2 changed the title StatusCenter: Items processed fraction is not displayed Code Quality: Fixed items processed string in status center Jul 31, 2025
@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed ready for review Pull requests that are ready for review labels Jul 31, 2025
@yaira2 yaira2 merged commit 2360937 into files-community:main Jul 31, 2025
6 checks passed
@SilverGreen93 SilverGreen93 deleted the item_fraction branch July 31, 2025 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Pull requests that are approved and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants