From c080346751b1316c04ea7602cf2b51150465a6ea Mon Sep 17 00:00:00 2001
From: Keith Grant <keithjgrant@redhat.com>
Date: Wed, 17 Jul 2019 09:25:40 -0700
Subject: [PATCH] start on InventoriesLookup

---
 awx/ui_next/src/api/index.js                  |  3 +
 awx/ui_next/src/api/models/Inventories.js     | 10 +++
 awx/ui_next/src/app.scss                      |  5 --
 awx/ui_next/src/components/Lookup/Lookup.jsx  | 14 +++-
 .../shared/InstanceGroupsLookup.jsx           |  1 +
 .../Template/shared/InventoriesLookup.jsx     | 71 +++++++++++++++++++
 .../Template/shared/JobTemplateForm.jsx       | 35 ++++++++-
 awx/ui_next/src/util/strings.js               | 11 ++-
 awx/ui_next/src/util/strings.test.js          |  4 ++
 9 files changed, 144 insertions(+), 10 deletions(-)
 create mode 100644 awx/ui_next/src/api/models/Inventories.js
 create mode 100644 awx/ui_next/src/screens/Template/shared/InventoriesLookup.jsx

diff --git a/awx/ui_next/src/api/index.js b/awx/ui_next/src/api/index.js
index 204985b31d..5e7b221c77 100644
--- a/awx/ui_next/src/api/index.js
+++ b/awx/ui_next/src/api/index.js
@@ -1,5 +1,6 @@
 import Config from './models/Config';
 import InstanceGroups from './models/InstanceGroups';
+import Inventories from './models/Inventories';
 import JobTemplates from './models/JobTemplates';
 import Jobs from './models/Jobs';
 import Me from './models/Me';
@@ -13,6 +14,7 @@ import WorkflowJobTemplates from './models/WorkflowJobTemplates';
 
 const ConfigAPI = new Config();
 const InstanceGroupsAPI = new InstanceGroups();
+const InventoriesAPI = new Inventories();
 const JobTemplatesAPI = new JobTemplates();
 const JobsAPI = new Jobs();
 const MeAPI = new Me();
@@ -27,6 +29,7 @@ const WorkflowJobTemplatesAPI = new WorkflowJobTemplates();
 export {
   ConfigAPI,
   InstanceGroupsAPI,
+  InventoriesAPI,
   JobTemplatesAPI,
   JobsAPI,
   MeAPI,
diff --git a/awx/ui_next/src/api/models/Inventories.js b/awx/ui_next/src/api/models/Inventories.js
new file mode 100644
index 0000000000..9c9f86754a
--- /dev/null
+++ b/awx/ui_next/src/api/models/Inventories.js
@@ -0,0 +1,10 @@
+import Base from '../Base';
+
+class Inventories extends Base {
+  constructor(http) {
+    super(http);
+    this.baseUrl = '/api/v2/inventories/';
+  }
+}
+
+export default Inventories;
diff --git a/awx/ui_next/src/app.scss b/awx/ui_next/src/app.scss
index 22a874d349..4f0b81df21 100644
--- a/awx/ui_next/src/app.scss
+++ b/awx/ui_next/src/app.scss
@@ -155,11 +155,6 @@
 // and bem style, as well as moved into component-based scss files
 //
 
-.awx-lookup .pf-c-form-control {
-  --pf-c-form-control--Height: 90px;
-  overflow-y: auto;
-}
-
 .at-c-listCardBody {
   --pf-c-card__footer--PaddingX: 0;
   --pf-c-card__footer--PaddingY: 0;
diff --git a/awx/ui_next/src/components/Lookup/Lookup.jsx b/awx/ui_next/src/components/Lookup/Lookup.jsx
index 92c72a8856..b9a417cb55 100644
--- a/awx/ui_next/src/components/Lookup/Lookup.jsx
+++ b/awx/ui_next/src/components/Lookup/Lookup.jsx
@@ -5,11 +5,12 @@ import { SearchIcon } from '@patternfly/react-icons';
 import {
   Button,
   ButtonVariant,
-  InputGroup,
+  InputGroup as PFInputGroup,
   Modal,
 } from '@patternfly/react-core';
 import { withI18n } from '@lingui/react';
 import { t } from '@lingui/macro';
+import styled from 'styled-components';
 
 import PaginatedDataList from '../PaginatedDataList';
 import DataListToolbar from '../DataListToolbar';
@@ -18,6 +19,13 @@ import SelectedList from '../SelectedList';
 import { ChipGroup, Chip } from '../Chip';
 import { getQSConfig, parseNamespacedQueryString } from '../../util/qs';
 
+const InputGroup = styled(PFInputGroup)`
+  ${props => props.multiple && (`
+    --pf-c-form-control--Height: 90px;
+    overflow-y: auto;
+  `)}
+`;
+
 class Lookup extends React.Component {
   constructor(props) {
     super(props);
@@ -145,7 +153,7 @@ class Lookup extends React.Component {
 
     return (
       <Fragment>
-        <InputGroup className="awx-lookup">
+        <InputGroup>
           <Button
             aria-label="Search"
             id={id}
@@ -220,12 +228,14 @@ Lookup.propTypes = {
   onLookupSave: PropTypes.func.isRequired,
   value: PropTypes.arrayOf(PropTypes.object).isRequired,
   sortedColumnKey: PropTypes.string.isRequired,
+  multiple: PropTypes.bool,
 };
 
 Lookup.defaultProps = {
   id: 'lookup-search',
   lookupHeader: null,
   name: null,
+  multiple: false,
 };
 
 export { Lookup as _Lookup };
diff --git a/awx/ui_next/src/screens/Organization/shared/InstanceGroupsLookup.jsx b/awx/ui_next/src/screens/Organization/shared/InstanceGroupsLookup.jsx
index 33a2fc2ea8..66ed5f706b 100644
--- a/awx/ui_next/src/screens/Organization/shared/InstanceGroupsLookup.jsx
+++ b/awx/ui_next/src/screens/Organization/shared/InstanceGroupsLookup.jsx
@@ -35,6 +35,7 @@ class InstanceGroupsLookup extends React.Component {
           value={value}
           onLookupSave={onChange}
           getItems={getInstanceGroups}
+          multiple
           columns={[
             { name: i18n._(t`Name`), key: 'name', isSortable: true },
             {
diff --git a/awx/ui_next/src/screens/Template/shared/InventoriesLookup.jsx b/awx/ui_next/src/screens/Template/shared/InventoriesLookup.jsx
new file mode 100644
index 0000000000..2f44a716db
--- /dev/null
+++ b/awx/ui_next/src/screens/Template/shared/InventoriesLookup.jsx
@@ -0,0 +1,71 @@
+import React, { Fragment } from 'react';
+import PropTypes from 'prop-types';
+import { withI18n } from '@lingui/react';
+import { t } from '@lingui/macro';
+import { FormGroup, Tooltip } from '@patternfly/react-core';
+import { QuestionCircleIcon } from '@patternfly/react-icons';
+
+import { InventoriesAPI } from '@api';
+import Lookup from '@components/Lookup';
+
+const getInventories = async params => InventoriesAPI.read(params);
+
+class InventoriesLookup extends React.Component {
+  render() {
+    const { value, tooltip, onChange, i18n } = this.props;
+
+    return (
+      <FormGroup
+        label={
+          <Fragment>
+            {i18n._(t`Inventories`)}{' '}
+            {tooltip && (
+              <Tooltip position="right" content={tooltip}>
+                <QuestionCircleIcon />
+              </Tooltip>
+            )}
+          </Fragment>
+        }
+        fieldId="inventories-lookup"
+      >
+        <Lookup
+          id="inventories-lookup"
+          lookupHeader={i18n._(t`Inventories`)}
+          name="inventories"
+          value={value}
+          onLookupSave={onChange}
+          getItems={getInventories}
+          multiple
+          columns={[
+            { name: i18n._(t`Name`), key: 'name', isSortable: true },
+            {
+              name: i18n._(t`Modified`),
+              key: 'modified',
+              isSortable: false,
+              isNumeric: true,
+            },
+            {
+              name: i18n._(t`Created`),
+              key: 'created',
+              isSortable: false,
+              isNumeric: true,
+            },
+          ]}
+          sortedColumnKey="name"
+        />
+      </FormGroup>
+    );
+  }
+}
+
+InventoriesLookup.propTypes = {
+  value: PropTypes.arrayOf(PropTypes.object).isRequired,
+  tooltip: PropTypes.string,
+  onChange: PropTypes.func.isRequired,
+};
+
+InventoriesLookup.defaultProps = {
+  tooltip: '',
+};
+
+export default withI18n()(InventoriesLookup);
diff --git a/awx/ui_next/src/screens/Template/shared/JobTemplateForm.jsx b/awx/ui_next/src/screens/Template/shared/JobTemplateForm.jsx
index 1ff5179a25..e823024044 100644
--- a/awx/ui_next/src/screens/Template/shared/JobTemplateForm.jsx
+++ b/awx/ui_next/src/screens/Template/shared/JobTemplateForm.jsx
@@ -10,6 +10,7 @@ import AnsibleSelect from '@components/AnsibleSelect';
 import FormActionGroup from '@components/FormActionGroup';
 import FormField from '@components/FormField';
 import FormRow from '@components/FormRow';
+import Lookup from '@components/Lookup';
 import { required } from '@util/validators';
 import styled from 'styled-components';
 import { JobTemplate } from '@types';
@@ -105,7 +106,37 @@ class JobTemplateForm extends Component {
                   </FormGroup>
                 )}
               />
-              <FormField
+              <Field
+                name="inventory"
+                validate={required(null, i18n)}
+                render={({ field, form }) => (
+                  <FormGroup
+                    fieldId="template-inventory"
+                    helperTextInvalid={form.errors.inventory}
+                    isRequired
+                    label={i18n._(t`Inventory`)}
+                  >
+                    <Lookup
+                      id="template-inventory"
+                      lookupHeader={i18n._(t`Inventory`)}
+                      name="inventory"
+                      value={[field.value]}
+                      onLookupSave={(value) => {console.log(value)}}
+                      getItems={() => ({
+                        data: {
+                          results: [{id: 1, name: 'foo'}],
+                          count: 1
+                        }
+                      })}
+                      columns={[
+                        { name: i18n._(t`Name`), key: 'name', isSortable: true},
+                      ]}
+                      sortedColumnsKey="name"
+                    />
+                  </FormGroup>
+                )}
+              />
+              {/* <FormField
                 id="template-inventory"
                 name="inventory"
                 type="number"
@@ -114,7 +145,7 @@ class JobTemplateForm extends Component {
                 you want this job to manage.`)}
                 isRequired
                 validate={required(null, i18n)}
-              />
+              /> */}
               <FormField
                 id="template-project"
                 name="project"
diff --git a/awx/ui_next/src/util/strings.js b/awx/ui_next/src/util/strings.js
index 588bc8b7b6..ea08d10655 100644
--- a/awx/ui_next/src/util/strings.js
+++ b/awx/ui_next/src/util/strings.js
@@ -1,7 +1,16 @@
+// TODO: switch to using Lingui i18n for pluralization
 export function pluralize(str) {
-  return str[str.length - 1] === 's' ? `${str}es` : `${str}s`;
+  const lastChar = str[str.length - 1];
+  if (lastChar === 's') {
+    return `${str}es`
+  }
+  if (lastChar === 'y') {
+    return `${str.substr(0, str.length - 1)}ies`;
+  }
+  return `${str}s`;
 }
 
+// TODO: switch to using Lingui i18n for articles
 export function getArticle(str) {
   const first = str[0];
   if ('aeiou'.includes(first)) {
diff --git a/awx/ui_next/src/util/strings.test.js b/awx/ui_next/src/util/strings.test.js
index b69db2f489..6ebddc219b 100644
--- a/awx/ui_next/src/util/strings.test.js
+++ b/awx/ui_next/src/util/strings.test.js
@@ -9,6 +9,10 @@ describe('string utils', () => {
     test('should add an "es"', () => {
       expect(pluralize('class')).toEqual('classes');
     });
+
+    test('should handle word ending in y', () => {
+      expect(pluralize('inventory')).toEqual('inventories');
+    });
   });
 
   describe('getArticle', () => {