Ensure we catch empty ID's for single nodes also

I don't think this would have a large effect on the UI whichever but
best to make sure
pull/4331/head
John Cowen 6 years ago
parent a47a0b617f
commit 6398fb02c7

@ -1,6 +1,14 @@
import Adapter from './application';
import { PRIMARY_KEY, SLUG_KEY } from 'consul-ui/models/node';
import { OK as HTTP_OK } from 'consul-ui/utils/http/status';
// TODO: Looks like ID just isn't used at all
// consider just using .Node for the SLUG_KEY
const fillSlug = function(item) {
if (item[SLUG_KEY] === '') {
item[SLUG_KEY] = item['Node'];
}
return item;
};
export default Adapter.extend({
urlForQuery: function(query, modelName) {
return this.appendURL('internal/ui/nodes', [], this.cleanQuery(query));
@ -14,6 +22,7 @@ export default Adapter.extend({
const url = this.parseURL(requestData.url);
switch (true) {
case this.isQueryRecord(url):
response = fillSlug(response);
response = {
...response,
...{
@ -23,9 +32,7 @@ export default Adapter.extend({
break;
default:
response = response.map((item, i, arr) => {
if (item[SLUG_KEY] === '') {
item[SLUG_KEY] = item['Node'];
}
item = fillSlug(item);
return {
...item,
...{

Loading…
Cancel
Save