Plugins » Katello

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/