Bug #7735: Cancel button are validating fields in filters and event classes - Mentat - Homeproj: Redmine for CESNET

Custom queries

All issues for next version
Version 2.11

Actions

Copy link

Bug #7735

closed

Cancel button are validating fields in filters and event classes

Added by Rajmund Hruška 7 months ago. Updated 5 months ago.

Status:

Closed

Priority:

Normal

Assignee:

Rajmund Hruška

Category:

Development - GUI

Target version:

2.13

Start date:

10/08/2021

Due date:

10/08/2021

% Done:

Estimated time:

To be discussed:

Description

See #7519.

Files

clipboard-202406121404-lqedw.png (16.3 KB) clipboard-202406121404-lqedw.png

Rajmund Hruška, 06/12/2024 02:04 PM

Related issues

Follows Mentat - Bug #7519: Cancel buttons are validating fields

Closed

Jan Mach

10/07/2021

Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Rajmund Hruška 7 months ago

Due date set to 10/08/2021
Start date changed from 05/13/2024 to 10/08/2021
Follows Bug #7519: Cancel buttons are validating fields added

Actions

Copy link

Updated by Rajmund Hruška 6 months ago

To be discussed changed from No to Yes

Simple solution would be to just remove the cancel buttons from those forms, similar to how to it was done in 1c53383a. That would fix this issue on frontend.

There is a feature on backend which handles action 'cancel': it flashes a banner saying that the action XXX was canceled.

def check_action_cancel(self, form, **kwargs):
    """ 
    *Hook method*. Check the form for *cancel* button press and cancel the action.
    """ 
    if hasattr(form, hawat.const.FORM_ACTION_CANCEL):
        if getattr(form, hawat.const.FORM_ACTION_CANCEL).data:
            self.flash(
                flask.Markup(self.get_message_cancel(**kwargs)),
                hawat.const.FLASH_INFO
            )
            return self.redirect(
                default_url = self.get_url_next()
            )
    return None

But the question is, do we need this functionality? I think we can remove this.

Actions

Copy link

Updated by Rajmund Hruška 6 months ago

Status changed from New to Feedback

Actions

Copy link

Updated by Pavel Kácha 6 months ago

Rajmund Hruška wrote in #note-2:

Simple solution would be to just remove the cancel buttons from those forms, similar to how to it was done in 1c53383a. That would fix this issue on frontend.

There is a feature on backend which handles action 'cancel': it flashes a banner saying that the action XXX was canceled.
[...]

But the question is, do we need this functionality? I think we can remove this.

Well, in login forms, it does not make sense (much) to cancel login, as that means you can't get inside the app altogether, and closing the window (tab) is appropriate.

Is it appropriate here? We don't open new windows for specific functions as this, so the user relies on us correctly returning him to previous state, which is not doable without Cancel, I guess?

Actions

Copy link

Updated by Rajmund Hruška 6 months ago

Pavel Kácha wrote in #note-4:

Rajmund Hruška wrote in #note-2:

Simple solution would be to just remove the cancel buttons from those forms, similar to how to it was done in 1c53383a. That would fix this issue on frontend.

There is a feature on backend which handles action 'cancel': it flashes a banner saying that the action XXX was canceled.
[...]

But the question is, do we need this functionality? I think we can remove this.

Well, in login forms, it does not make sense (much) to cancel login, as that means you can't get inside the app altogether, and closing the window (tab) is appropriate.

Is it appropriate here? We don't open new windows for specific functions as this, so the user relies on us correctly returning him to previous state, which is not doable without Cancel, I guess?

Instead of closing the window, the user could just press 'Back' button in the browser.

Actions

Copy link

Updated by Pavel Kácha 6 months ago

Couldn't Cancel be just link to previous page then?

Actions

Copy link

Updated by Rajmund Hruška 6 months ago

File clipboard-202406121404-lqedw.png clipboard-202406121404-lqedw.png added

Pavel Kácha wrote in #note-6:

Couldn't Cancel be just link to previous page then?

Yes it could, that would be quite simple. We would lose that flash which informs about canceling the action, though. But, in my proposed solution we would lose it anyway

Actions

Copy link

Updated by Pavel Kácha 6 months ago

Rajmund Hruška wrote in #note-7:

Pavel Kácha wrote in #note-6:

Couldn't Cancel be just link to previous page then?

Yes it could, that would be quite simple. We would lose that flash which informs about canceling the action, though. But, in my proposed solution we would lose it anyway

Sure. That is probably also solvable by adding some "flash=" like argument to returning URL, but I guess the whole Cancel flash bussiness is quite superfluous.

Actions

Copy link