CITKParus/P8-Panels
2
0
Fork 2
Code Issues Pull Requests 1 Projects Releases Wiki Activity

ЦИТК-929 - Вывод сообщения об ошибке без стэка вызова #32

Closed
Dollerok wants to merge 0 commits from Dollerok/P8-Panels:main into main
pull from: Dollerok/P8-Panels:main
merge into: CITKParus:main
Conversation 1 Commits - Files Changed - +85 -8
Dollerok commented 2025-03-28 13:41:49 +03:00
Member
Copy Link
No description provided.
Dollerok added 1 commit 2025-03-28 13:41:51 +03:00
Dollerok added 1 commit 2025-03-28 13:57:42 +03:00
Mim commented 2025-03-28 18:39:16 +03:00
Owner
Copy Link
  1. Для функций showMsgErr и showMsg неверно изменен интерфейс вызова
    Необязательные позиционные параметры можно добавлять только в конец списка, чтобы не ломать существующие вызовы.
    В педагогических целях попробуй, не исправляя эту ошибку, сделать вызов в какой-нибудь панели по кнопке:
    showMsgErr("Какая-то ошибка", () => console.log("Диалог ошибки закрыт..."));
    Вангую:
  • Раньше - он показывал сообщение об ошибке, а при закрытии - писал в консоль.
  • Сейчас - будет показывать в "Подробнее" что-то типа "Object[function]".

  1. В документации внесены изменения только для showMsg

  2. Название константы ERR_MORE не отражает сущности app.text.js
    Должно быть общим. Например - DETAIL

  3. В компоненте P8PAppMessage нет проверки необязательных свойств "showErrMoreCaption", "hideErrMoreCaption"
    Вот именно отсюда и возникают мемы про "Нажмите кнопку undefined"

1. Для функций showMsgErr и showMsg неверно изменен интерфейс вызова Необязательные позиционные параметры можно добавлять только в конец списка, чтобы не ломать существующие вызовы. В педагогических целях попробуй, не исправляя эту ошибку, сделать вызов в какой-нибудь панели по кнопке: showMsgErr("Какая-то ошибка", () => console.log("Диалог ошибки закрыт...")); Вангую: - Раньше - он показывал сообщение об ошибке, а при закрытии - писал в консоль. - Сейчас - будет показывать в "Подробнее" что-то типа "Object[function]". 2. В документации внесены изменения только для showMsg 3. Название константы ERR_MORE не отражает сущности app.text.js Должно быть общим. Например - DETAIL 4. В компоненте P8PAppMessage нет проверки необязательных свойств "showErrMoreCaption", "hideErrMoreCaption" Вот именно отсюда и возникают мемы про "Нажмите кнопку undefined"
Mim closed this pull request 2025-03-28 18:39:16 +03:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
documentation
Improvements or additions to documentation
duplicate
This issue or pull request already exists
enhancement
New feature or request
good first issue
Good for newcomers
help wanted
Extra attention is needed
invalid
This doesn't seem right
question
Further information is requested
wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Mim
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: CITKParus/P8-Panels#32
No description provided.
Delete Branch "Dollerok/P8-Panels:main"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?