Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Misleading title "Error" on popup notification when creating manual backup #3460

Closed
ErminMerdanovic opened this issue Apr 12, 2023 · 1 comment · Fixed by #3461
Closed
Labels
bug Something isn't working

Comments

@ErminMerdanovic
Copy link

Describe the bug

Misleading title "Error" on popup notification when creating manual backup.
image

Tell us how to replicate the bug

  1. Click on the Backups tab.
  2. Click on Create Backup.
  3. Backup notification will popup with the Misleading title "Error"

Which components are affected by this bug?

Control Panel Web Interface

Hestia Control Panel Version

1.7.2

Operating system

Ubuntu 22.04

Log capture

No response

@ErminMerdanovic ErminMerdanovic added the bug Something isn't working label Apr 12, 2023
@ErminMerdanovic ErminMerdanovic changed the title Misleading title "Error" on popup notification when creating manual backup [Bug] Misleading title "Error" on popup notification when creating manual backup Apr 12, 2023
@AlecRust
Copy link
Contributor

AlecRust commented Apr 12, 2023

This is arguably a regression in #3401, as we now show titles in dialogs in more places.

Simply removing this line should revert this "global dialog" code to previous behaviour (no title):

title: '<?= _("Error") ?>',

Perhaps that's the best fix for now, this is included in #3461. But it will mean legitimate "error" dialogs don't show the "error" title.

As you can see from surrounding code such as if (!empty($_SESSION["error_msg"])): it looks like this "Task has been added to queue" message is being set to error_msg. So this code is currently specific to error messages, hence the "Error" title. Ideally a wider refactor would take place here, perhaps allowing these "global dialogs" to be of type notice or warn or error, instead of just error.

@AlecRust AlecRust mentioned this issue Apr 12, 2023
@jaapmarcus jaapmarcus linked a pull request Apr 13, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants