Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

The diagnostic message displayed literal {0} instead of the method name. Resource bundles used MessageFormat syntax ({0}) while the code uses String.formatted() which requires %s.

Changes

  • Updated ServerCallsInFormEventsDiagnostic_ru.properties: "{0}""%s"
  • Updated ServerCallsInFormEventsDiagnostic_en.properties: "{0}""%s"

Result

Before:

Процедура "{0}" выполняется на сервере, что может привести к проблемам.

After:

Процедура "ПараметрыФормуУказанияСерий" выполняется на сервере, что может привести к проблемам.
Original prompt

This section details on the original issue you should resolve

<issue_title>[MOD] Исправить текст сообщения диагностики</issue_title>
<issue_description>## Диагностика
ServerCallsInFormEvents

Описание

В диагностику ошибочно выводится параметр вместо текста.
Я как понимаю, вместо {0} должно выводиться имя метода с ошибкой.

Image

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 15, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix diagnostic message text output Fix parameter substitution in ServerCallsInFormEvents diagnostic message Jan 15, 2026
Copilot AI requested a review from nixel2007 January 15, 2026 18:25
@nixel2007 nixel2007 marked this pull request as ready for review January 15, 2026 18:27
Copilot AI review requested due to automatic review settings January 15, 2026 18:27
@nixel2007 nixel2007 enabled auto-merge January 15, 2026 18:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a bug where the ServerCallsInFormEventsDiagnostic was displaying the literal placeholder {0} instead of the actual method name in its diagnostic message. The issue occurred because the resource bundles used MessageFormat syntax ({0}) while the code uses String.formatted() which requires %s placeholders.

Changes:

  • Updated message format placeholders from {0} to %s in both Russian and English resource bundles

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
ServerCallsInFormEventsDiagnostic_ru.properties Changed message format from {0} to %s for method name substitution
ServerCallsInFormEventsDiagnostic_en.properties Changed message format from {0} to %s for method name substitution

@github-actions
Copy link
Contributor

Test Results

 2 907 files  ±0   2 907 suites  ±0   1h 3m 34s ⏱️ - 1m 9s
 1 227 tests ±0   1 227 ✅ ±0  0 💤 ±0  0 ❌ ±0 
11 043 runs  ±0  11 043 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 1ad9a2e. ± Comparison against base commit a76f764.

@nixel2007 nixel2007 merged commit 7e5f646 into develop Jan 15, 2026
44 checks passed
@nixel2007 nixel2007 deleted the copilot/fix-diagnostic-message-text branch January 15, 2026 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MOD] Исправить текст сообщения диагностики

2 participants