Add alert for org. delete.

__tests__/components/DataListToolbar.test.jsx

@@ -221,4 +221,32 @@ describe('<DataListToolbar />', () => {
     const upAlphaIcon = toolbar.find(upAlphaIconSelector);
     expect(upAlphaIcon.length).toBe(1);
   });
+
+  test('trash can button triggers correct function', () => {
+    const columns = [{ name: 'Name', key: 'name', isSortable: true }];
+    const onOpenDeleteModal = jest.fn();
+    const deleteModal = 'button.pf-c-button.pf-m-plain.awx-ToolBarBtn';
+    const onSearch = jest.fn();
+    const onSort = jest.fn();
+    const onSelectAll = jest.fn();
+
+    toolbar = mount(
+      <I18nProvider>
+        <DataListToolbar
+          isAllSelected={false}
+          selected={() => ([1, 2, 3, 4])}
+          sortedColumnKey="name"
+          sortOrder="ascending"
+          columns={columns}
+          onSearch={onSearch}
+          onSort={onSort}
+          onSelectAll={onSelectAll}
+          onOpenDeleteModal={() => {}}
+        />
+      </I18nProvider>
+    );
+
+    toolbar.find(deleteModal).simulate('click');
+    expect(onOpenDeleteModal).toBeCalled();
+  });
 });

src/api.js

@@ -56,6 +56,11 @@ class APIClient {
     return this.http.get(API_CONFIG);
   }
 
+  destroyOrganization (id) {
+    const endpoint = `${API_ORGANIZATIONS}${id}/`;
+    return (this.http.delete(endpoint));
+  }
+
   getOrganizations (params = {}) {
     return this.http.get(API_ORGANIZATIONS, { params });
   }

src/app.scss

@@ -279,6 +279,10 @@
   margin-bottom: 10px;
 }
 
+.orgListAlert-actionBtn{
+  margin:0 10px;
+}
+
 .awx-c-form-action-group {
   float: right;
   display: block;
@@ -287,4 +291,4 @@
     margin-top: 20px;
     margin-right: 20px;
   }
-}
+}

src/components/DataListToolbar/DataListToolbar.jsx

@@ -24,29 +24,31 @@ import ExpandCollapse from '../ExpandCollapse';
 import Search from '../Search';
 import Sort from '../Sort';
 import VerticalSeparator from '../VerticalSeparator';
+// import SelectedList from '../SelectedList';
 
 class DataListToolbar extends React.Component {
   render () {
     const {
+      addUrl,
       columns,
-      isAllSelected,
+      disableTrashCanIcon,
       onSelectAll,
       sortedColumnKey,
       sortOrder,
-      addUrl,
       showDelete,
       showSelectAll,
+      isAllSelected,
       isLookup,
       isCompact,
       onSort,
       onSearch,
       onCompact,
       onExpand,
-      add
+      add,
+      onOpenDeleteModal
     } = this.props;
 
     const showExpandCollapse = (onCompact && onExpand);
-
     return (
       <I18n>
         {({ i18n }) => (
@@ -115,10 +117,13 @@ class DataListToolbar extends React.Component {
                     position="top"
                   >
                     <Button
+                      className="awx-ToolBarBtn"
                       variant="plain"
                       aria-label={i18n._(t`Delete`)}
+                      onClick={onOpenDeleteModal}
+                      isDisabled={disableTrashCanIcon}
                     >
-                      <TrashAltIcon />
+                      <TrashAltIcon className="awx-ToolBarTrashCanIcon" />
                     </Button>
                   </Tooltip>
                 )}

src/components/DataListToolbar/styles.scss

@@ -80,3 +80,16 @@
 .awx-toolbar .pf-l-toolbar__item .pf-c-button.pf-m-plain {
   font-size: 18px;
 }
+.pf-c-button--disabled--BackgroundColor{
+  background-color: #b7b7b7;
+}
+.awx-ToolBarBtn{
+  width: 30px;
+}
+.awx-ToolBarBtn:hover{
+  .awx-ToolBarTrashCanIcon {
+    color:white;
+  }
+  background-color:#d9534f;
+}
+

src/pages/Organizations/components/OrganizationListItem.jsx

@@ -4,61 +4,72 @@ import { Trans, t } from '@lingui/macro';
 import {
   Badge,
   Checkbox,
+  Button,
 } from '@patternfly/react-core';
+import {
+  TrashAltIcon,
+} from '@patternfly/react-icons';
 import {
   Link
 } from 'react-router-dom';
 
 import VerticalSeparator from '../../../components/VerticalSeparator';
 
-export default ({
-  itemId,
-  name,
-  userCount,
-  teamCount,
-  isSelected,
-  onSelect,
-  detailUrl,
-}) => (
-  <li key={itemId} className="pf-c-data-list__item" aria-labelledby="check-action-item1">
-    <I18n>
-      {({ i18n }) => (
-        <Checkbox
-          checked={isSelected}
-          onChange={onSelect}
-          aria-label={i18n._(t`select organization ${itemId}`)}
-          id={`select-organization-${itemId}`}
-        />
-      )}
-    </I18n>
-    <VerticalSeparator />
-    <div className="pf-c-data-list__cell">
-      <span id="check-action-item1">
-        <Link
-          to={`${detailUrl}`}
-        >
-          <b>{name}</b>
-        </Link>
-      </span>
-    </div>
-    <div className="pf-c-data-list__cell">
-      <Link to={`${detailUrl}/access`}>
-        <Trans>Users</Trans>
-      </Link>
-      <Badge isRead>
-        {' '}
-        {userCount}
-        {' '}
-      </Badge>
-      <Link to={`${detailUrl}/teams`}>
-        <Trans>Teams</Trans>
-      </Link>
-      <Badge isRead>
-        {' '}
-        {teamCount}
-        {' '}
-      </Badge>
-    </div>
-    <div className="pf-c-data-list__cell" />
-  </li>
-);
+class OrganizationListItem extends React.Component {
+  render () {
+    const {
+      itemId,
+      name,
+      userCount,
+      teamCount,
+      isSelected,
+      onSelect,
+      detailUrl,
+    } = this.props;
+    return (
+      <li key={itemId} className="pf-c-data-list__item" aria-labelledby="check-action-item1">
+        <I18n>
+          {({ i18n }) => (
+            <Checkbox
+              checked={isSelected}
+              onChange={onSelect}
+              aria-label={i18n._(t`select organization ${itemId}`)}
+              id={`select-organization-${itemId}`}
+            />
+          )}
+        </I18n>
+        <VerticalSeparator />
+        <div className="pf-c-data-list__cell">
+          <span id="check-action-item1">
+            <Link
+              to={`${detailUrl}`}
+            >
+              <b>{name}</b>
+            </Link>
+          </span>
+        </div>
+        <div className="pf-c-data-list__cell">
+          <Link to={`${detailUrl}/access`}>
+            <Trans>Users</Trans>
+          </Link>
+          <Badge isRead>
+            {' '}
+            {userCount}
+            {' '}
+          </Badge>
+          <Link to={`${detailUrl}/teams`}>
+            <Trans>Teams</Trans>
+          </Link>
+          <Badge isRead>
+            {' '}
+            {teamCount}
+            {' '}
+          </Badge>
+        </div>
+        <div className="pf-c-data-list__cell" />
+      </li>
+    );
+  }
+}
+export default OrganizationListItem;
+

src/pages/Organizations/screens/OrganizationsList.jsx

@@ -12,9 +12,11 @@ import {
   EmptyState,
   EmptyStateIcon,
   EmptyStateBody,
+  Modal,
   PageSection,
   PageSectionVariants,
-  Title
+  Title,
+  Button
 } from '@patternfly/react-core';
 import { CubesIcon } from '@patternfly/react-icons';
 import DataListToolbar from '../../../components/DataListToolbar';
@@ -54,6 +56,8 @@ class OrganizationsList extends Component {
       loading: true,
       results: [],
       selected: [],
+      isModalOpen: false,
+      orgsToDelete: []
     };
 
     this.onSearch = this.onSearch.bind(this);
@@ -64,6 +68,9 @@ class OrganizationsList extends Component {
     this.onSelect = this.onSelect.bind(this);
     this.updateUrl = this.updateUrl.bind(this);
     this.fetchOrganizations = this.fetchOrganizations.bind(this);
+    this.handleOrgDelete = this.handleOrgDelete.bind(this);
+    this.handleOpenOrgDeleteModal = this.handleOpenOrgDeleteModal.bind(this);
+    this.handleClearOrgsToDelete = this.handleClearOrgsToDelete.bind(this);
   }
 
   componentDidMount () {
@@ -129,6 +136,50 @@ class OrganizationsList extends Component {
     return Object.assign({}, this.defaultParams, searchParams, overrides);
   }
 
+  handleClearOrgsToDelete () {
+    this.setState(({ isModalOpen }) => ({ isModalOpen: !isModalOpen, orgsToDelete: [] }));
+    this.onSelectAll();
+  }
+
+  handleOpenOrgDeleteModal () {
+    const { results, selected } = this.state;
+    const warningTitle = i18nMark('Delete Organization');
+    const warningMsg = i18nMark('Are you sure you want to delete:');
+
+    const orgsToDelete = [];
+    results.forEach((result) => {
+      selected.forEach((selectedOrg) => {
+        if (result.id === selectedOrg) {
+          orgsToDelete.push({ name: result.name, id: selectedOrg });
+        }
+      });
+    });
+    this.setState({
+      orgsToDelete,
+      isModalOpen: true,
+      warningTitle,
+      warningMsg,
+      loading: false });
+  }
+
+  async handleOrgDelete (event) {
+    const { orgsToDelete } = this.state;
+    const { api } = this.props;
+    try {
+      const deleteOrgsApiCalls = [];
+
+      orgsToDelete.forEach((org) => {
+        deleteOrgsApiCalls.push(api.destroyOrganization(org.id));
+      });
+      await Promise.all(deleteOrgsApiCalls);
+    } finally {
+      this.handleClearOrgsToDelete();
+      const queryParams = this.getQueryParams();
+      this.fetchOrganizations(queryParams);
+    }
+    event.preventDefault();
+  }
+
   updateUrl (queryParams) {
     const { history, location } = this.props;
     const pathname = '/organizations';
@@ -196,18 +247,47 @@ class OrganizationsList extends Component {
       error,
       loading,
       noInitialResults,
+      orgsToDelete,
       page,
       pageCount,
       page_size,
+      selected,
       sortedColumnKey,
       sortOrder,
       results,
-      selected,
+      isModalOpen,
+      warningTitle,
+      warningMsg,
     } = this.state;
     const { match } = this.props;
     return (
       <PageSection variant={medium}>
         <Card>
+          { isModalOpen && (
+            <Modal
+              className="orgListAlert"
+              width="50%"
+              title={warningTitle}
+              isOpen={isModalOpen}
+              style={{ width: '1000px' }}
+              variant="danger"
+              onClose={this.handleClearOrgsToDelete}
+            >
+              {warningMsg}
+              <br />
+              {orgsToDelete.map((org) => (
+                <strong key={org.id}>
+                  {org.name}
+                  <br />
+                </strong>
+              ))}
+              <br />
+              <span className="awx-c-form-action-group">
+                <Button className="orgListAlert-actionBtn" keys="cancel" variant="secondary" aria-label="cancel-delete" onClick={this.handleClearOrgsToDelete}>Cancel</Button>
+                <Button className="orgListAlert-actionBtn" keys="cancel" variant="danger" aria-label="confirm-delete" onClick={this.handleOrgDelete}>Delete</Button>
+              </span>
+            </Modal>
+          )}
           {noInitialResults && (
             <EmptyState>
               <EmptyStateIcon icon={CubesIcon} />
@@ -229,6 +309,8 @@ class OrganizationsList extends Component {
                 onSearch={this.onSearch}
                 onSort={this.onSort}
                 onSelectAll={this.onSelectAll}
+                onOpenDeleteModal={this.handleOpenOrgDeleteModal}
+                disableTrashCanIcon={selected.length === 0}
                 showDelete
                 showSelectAll
               />
@@ -244,7 +326,8 @@ class OrganizationsList extends Component {
                         userCount={o.summary_fields.related_field_counts.users}
                         teamCount={o.summary_fields.related_field_counts.teams}
                         isSelected={selected.includes(o.id)}
-                        onSelect={() => this.onSelect(o.id)}
+                        onSelect={() => this.onSelect(o.id, o.name)}
+                        onOpenOrgDeleteModal={this.handleOpenOrgDeleteModal}
                       />
                     ))}
                   </ul>