Project

General

Profile

Actions

Bug #21022

open

Bulk Actions dont handle confirmation properly

Added by Partha Aji over 6 years ago. Updated over 5 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Web UI
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Many of our bulk actions don't handle confirmations correctly. For example -
https://github.com/Katello/katello/blob/master/engines/bastion_katello/app/assets/javascripts/bastion_katello/content-hosts/bulk/views/content-hosts-bulk-environment-modal.html#L18
has code that looks like

   <div bst-alert="info" ng-show="showConfirm">
      <span translate>
        Are you sure you want to assign the {{ table.numSelected }} content host(s) selected to {{ selected.contentView.name }} in {{ selected.environment.name }}?
      </span>
      <div>
        <button type="button" class="btn btn-default" ng-click="showConfirm = false; performAction()" translate>Yes</button>
        <button type="button" class="btn btn-default" ng-click="showConfirm = false;" translate>No</button>
      </div>
    </div>

Note there is no "ng-disabled" on 'Yes' when clicked. So we end up with issues where users will end up double clicking 'yes' because its waiting on an ajax call to happen. I see this in all of the views in https://github.com/Katello/katello/blob/master/engines/bastion_katello/app/assets/javascripts/bastion_katello/content-hosts/bulk/views/

Actions #1

Updated by Partha Aji over 6 years ago

  • Category set to Web UI
Actions #2

Updated by Walden Raines over 6 years ago

  • Target version set to 219
Actions #3

Updated by Justin Sherrill over 6 years ago

  • translation missing: en.field_release set to 114
Actions #4

Updated by Walden Raines almost 6 years ago

  • Target version deleted (219)
Actions

Also available in: Atom PDF