View Issue Details

IDProjectCategoryView StatusLast Update
0005189mantisbtbugtrackerpublic2024-01-24 17:36
Reportertandler Assigned Tovboctor  
PrioritynormalSeveritytrivialReproducibilityalways
Status closedResolutionfixed 
Product Version0.19.2 
Target Version2.26.0Fixed in Version2.26.0 
Summary0005189: "Operation successful." message page slows down interaction
Description

Each time an issues is updated, mantis shows the "Operation successful." page and redirects after a while to the issue page.
This somehow takes a lot of time and is of quite little information, as nearly all operations are successful. So, I feel that this slows down my workflow too much.

I'd recommend either to be able to configure mantis to

  • not show the "Operation successful." at all, only show an error page if there was an error or
  • include the "Operation successful." message at the top of the follow-up page (this is what bugzilla is doing) or
  • keep the current behaviour.

This should be controllable for pages with a single follow-up page only. Pages where the user can select either to go to the issue list or to report the next issue or to view the issue details of the updated issue, this behaviour should be kept (or be made configurable separately).

Steps To Reproduce

Update an issue ... ;-)

TagsNo tags attached.

Relationships

related to 0004462 acknowledged Add preference for default action after bug operations 
has duplicate 0005188 closedgrangeway "Operation successful." message page slows down interaction 
has duplicate 0017845 closedatrol Get rid of the "Proceed" screen 
has duplicate 0032893 closedvboctor Retire "Operation Successful" pages 
related to 0021109 closeddregad Replace "operational successful" pages with flash control 
related to 0033480 closeddregad Blank page when redirecting with print_successful_redirect() 

Activities

tandler

tandler

2005-02-01 06:56

reporter   ~0009161

related to 0004462

dhx

dhx

2010-06-08 00:15

reporter   ~0025750

Reference: http://en.wikipedia.org/wiki/Post/Redirect/Get

It seems that we should return a HTTP 303 response from the server when a form is submitted by POST. This causes the browser to send off a GET query to the landing page.

It isn't correct AFAIK to just remove the redirect completely or else the browser will try to resubmit the form when the page is refreshed (and other weird things happen too).

cas

cas

2010-06-08 04:34

reporter   ~0025753

so why not simply set the redirect delay ($g_default_redirect_delay = 2; )to zero?
Those users that prefer to have a delay can set this is their own prefs.

dhx

dhx

2010-06-09 03:44

reporter   ~0025776

The point is that our current redirect method is still very inefficient if the user doesn't want to see a confirmation message. A HTTP 303 response happens behind the scenes far more efficiently but users also lose the ability to see that their changes have been applied successfully.

From a usability perspective I would have thought that users would appreciate receiving feedback on their form submissions to let them know that their changes have been applied successfully.

dregad

dregad

2023-08-30 05:25

developer   ~0068052

Last edited: 2023-08-30 05:26

Repeating @vboctor's points from 0032893 after closing it as duplicate:

I suggest retiring this for the following reasons:

  • It is very old behavior.
  • It is very slow compared to a direct redirect due to the delay we introduce.
  • It disrupts the user flow, specially if they are doing multiple actions on a page.

I think we can start by having print_successful_redirect() deprecated and make it just call print_header_redirect().

I also think we should retire the one after reporting an issue which gives user options to go to the issue or View Issues page. I would just go to the view issue page of the created issue.

Both I and @atrol are in agreement with this.

I suggested to introduce some other kind of visual feedback that the update was successful ? (also covered by duplicate issue 0021109).

vboctor

vboctor

2023-08-31 03:06

manager   ~0068054

PR: https://github.com/mantisbt/mantisbt/pull/1912

Related Changesets

MantisBT: master e715e9ea

2023-08-31 02:38

vboctor


Details Diff
Remove operations successful from action pages

Fixes 0005189
Affected Issues
0005189
mod - account_prefs_update.php Diff File
mod - account_prof_update.php Diff File
mod - account_sponsor_update.php Diff File
mod - account_update.php Diff File
mod - bug_reminder.php Diff File
mod - manage_config_columns_reset.php Diff File
mod - manage_config_columns_set.php Diff File
mod - manage_config_email_set.php Diff File
mod - manage_config_revert.php Diff File
mod - manage_config_work_threshold_set.php Diff File
mod - manage_config_workflow_set.php Diff File
mod - manage_custom_field_create.php Diff File
mod - manage_custom_field_delete.php Diff File
mod - manage_custom_field_update.php Diff File
mod - manage_filter_delete.php Diff File
mod - manage_proj_cat_delete.php Diff File
mod - manage_proj_cat_update.php Diff File
mod - manage_proj_create.php Diff File
mod - manage_proj_custom_field_add_existing.php Diff File
mod - manage_proj_custom_field_remove.php Diff File
mod - manage_proj_custom_field_update.php Diff File
mod - manage_proj_subproj_add.php Diff File
mod - manage_proj_subproj_delete.php Diff File
mod - manage_proj_user_remove.php Diff File
mod - manage_proj_user_update.php Diff File
mod - manage_proj_ver_add.php Diff File
mod - manage_proj_ver_delete.php Diff File
mod - manage_proj_ver_update.php Diff File
mod - manage_user_delete.php Diff File
mod - manage_user_impersonate.php Diff File
mod - manage_user_proj_delete.php Diff File
mod - manage_user_update.php Diff File
mod - plugins/XmlImportExport/pages/config.php Diff File
mod - print_all_bug_options_reset.php Diff File
mod - print_all_bug_options_update.php Diff File
mod - proj_doc_add.php Diff File
mod - proj_doc_delete.php Diff File
mod - proj_doc_update.php Diff File
mod - set_project.php Diff File

MantisBT: master 4a20beb2

2023-08-31 03:01

vboctor


Details Diff
Show operation successful for pages that may be confusing

Some pages redirect to themselves and they are not refreshed in a way that indicate changes have been applied.

Fixes 0005189
Affected Issues
0005189
mod - account_prefs_update.php Diff File
mod - core/print_api.php Diff File
mod - manage_proj_custom_field_update.php Diff File

MantisBT: master bcec5bc7

2023-09-06 22:08

vboctor


Details Diff
Deprecate `print_successful_redirect*()`

Fixes 0005189
Affected Issues
0005189
mod - account_prefs_update.php Diff File
mod - adm_config_delete.php Diff File
mod - adm_config_set.php Diff File
mod - bug_monitor_add.php Diff File
mod - bug_monitor_delete.php Diff File
mod - bug_revision_drop.php Diff File
mod - bug_stick.php Diff File
mod - bug_update.php Diff File
mod - bugnote_add.php Diff File
mod - bugnote_delete.php Diff File
mod - bugnote_set_view_state.php Diff File
mod - bugnote_update.php Diff File
mod - core/print_api.php Diff File
mod - docbook/Developers_Guide/en-US/Plugins_Building.xml Diff File
mod - manage_plugin_install.php Diff File
mod - manage_plugin_uninstall.php Diff File
mod - manage_plugin_update.php Diff File
mod - manage_proj_update_children.php Diff File
mod - manage_proj_ver_add.php Diff File
mod - manage_user_create.php Diff File
mod - news_add.php Diff File
mod - plugins/MantisCoreFormatting/pages/config_edit.php Diff File
mod - tag_attach.php Diff File
mod - tag_create.php Diff File
mod - tag_delete.php Diff File
mod - tag_detach.php Diff File
mod - tag_update.php Diff File

MantisBT: master bd81ad14

2023-09-06 22:28

vboctor


Details Diff
Don't show a message/options after reporting an issue

Fixes 0005189
Affected Issues
0005189
mod - bug_report.php Diff File

MantisBT: master 2e192bc5

2023-09-06 22:38

vboctor


Details Diff
Short some long lines

Fixes 0005189
Affected Issues
0005189
mod - manage_proj_update_children.php Diff File
mod - manage_proj_ver_add.php Diff File

MantisBT: master 856d05cd

2023-09-16 10:36

dregad


Details Diff
Throw warning in deprecated functions

Also update PHPDoc to indicate in which version they were deprecated.

Issue 0005189
Affected Issues
0005189
mod - core/print_api.php Diff File

MantisBT: master-2.26 90df6dde

2024-01-04 08:37

dregad


Details Diff
Show delayed inline errors from print_header_redirect()

Following the deprecation of print_successful_redirect() function in
issue 0005189, calling it shows a blank page when $g_stop_on_errors = ON
and $g_display_errors[E_USER_DEPRECATED] = DISPLAY_ERROR_INLINE because
Mantis does not complete the redirection due to presence of a handled
error.

print_header_redirect() now displays a basic "proceed" page, with
redirection enabled only when $g_stop_on_errors = ON, allowing any
delayed inline error messages to be shown.

Fixes 0033480
Affected Issues
0005189, 0033480
mod - core/print_api.php Diff File