fix VuetifyGenerator

Author: alOneh

src/generators/VuetifyGenerator.js

@@ -44,22 +44,6 @@ export default class extends BaseVueGenerator {
     );
     console.log(
       chalk.green(`
-//Configure your Vuetify plugin
-// src/plugins/vuetify.js
-
-import Vue from 'vue'
-import Vuetify from 'vuetify/lib'
-
-Vue.use(Vuetify)
-
-const opts = {
-  icons: {
-    iconfont: 'mdi'
-  }
-};
-
-export default new Vuetify(opts)
-
 // Register the routes in you router
 // src/router/index.js
 import ${titleLc}Routes from './${titleLc}';
@@ -74,7 +58,7 @@ export default new VueRouter({
 
 // Register the modules in the store
 // src/store/index.js
-import ${titleLc}Service from './services/${titleLc}';
+import ${titleLc}Service from '../services/${titleLc}';
 import makeCrudModule from './modules/crud';
 
 // ...

src/generators/VuetifyGenerator.test.js

@@ -0,0 +1,66 @@
+import { Api, Resource, Field } from "@api-platform/api-doc-parser";
+import fs from "fs";
+import tmp from "tmp";
+import VuetifyGenerator from "./VuetifyGenerator";
+
+test("Generate a Vuetify app", () => {
+  const generator = new VuetifyGenerator({
+    hydraPrefix: "hydra:",
+    templateDirectory: `${__dirname}/../../templates`
+  });
+  const tmpobj = tmp.dirSync({ unsafeCleanup: true });
+
+  const fields = [
+    new Field("bar", {
+      id: "http://schema.org/url",
+      range: "http://www.w3.org/2001/XMLSchema#string",
+      reference: null,
+      required: true,
+      description: "An URL"
+    })
+  ];
+  const resource = new Resource("abc", "http://example.com/foos", {
+    id: "foo",
+    title: "Foo",
+    readableFields: fields,
+    writableFields: fields,
+    getParameters: function getParameters() {
+      return Promise.resolve([]);
+    }
+  });
+  const api = new Api("http://example.com", {
+    entrypoint: "http://example.com:8080",
+    title: "My API",
+    resources: [resource]
+  });
+  generator.generate(api, resource, tmpobj.name).then(() => {
+    [
+      "/components/ActionCell",
+      "/components/Breadcrumb",
+      "/components/ConfirmDelete",
+      "/components/DataFilter",
+      "/components/foo/Filter",
+      "/components/foo/Form.vue",
+      "/components/foo/Layout",
+      "/components/InputDate.vue",
+      "/components/Loading.vue",
+      "/components/Snackbar.vue",
+      "/components/Toolbar.vue",
+      "/config/entrypoint.js",
+      "/error/SubmissionError.js",
+      "/locales/en.js",
+      "/router/foo.js",
+      "/services/api.js",
+      "/services/foo.js",
+      "/utils/dates.js",
+      "/utils/fetch.js",
+      "/utils/hydra.js",
+      "/views/foo/Create.vue",
+      "/views/foo/List.vue",
+      "/views/foo/Show.vue",
+      "/views/foo/Update.vue"
+    ].forEach(file => expect(fs.existsSync(tmpobj.name + file)).toBe(true));
+
+    tmpobj.removeCallback();
+  });
+});

templates/vue-common/mixins/ListMixin.js

@@ -1,3 +1,4 @@
+import isEmpty from 'lodash/isEmpty';
 import { formatDateTime } from '../utils/dates';
 import NotificationMixin from './NotificationMixin';
 
@@ -39,7 +40,8 @@ export default {
       if (itemsPerPage > 0) {
         params = { ...params, itemsPerPage, page };
       }
-      if (sortBy) {
+
+      if (!isEmpty(sortBy)) {
         params[`order[${sortBy}]`] = descending ? 'desc' : 'asc';
       }
 

templates/vuetify/views/foo/List.vue

@@ -51,15 +51,17 @@
               {{/case~}}
               {{#default}}
                 {{#if reference}}
-                {{#unless maxCardinality}}
                 <template slot="item.{{{name}}}" slot-scope="{ item }">
+                {{#if maxCardinality }}
+                  \{{ item['@id'] }}
+                  {{else}}
                   <ul>
-                    <li v-for="_item in item['{{{name}}}']" :key="_item.id">
-                      \{{ _item.id }}
+                    <li v-for="_item in item['{{{name}}}']" :key="_item['@id']">
+                      \{{ _item['@id'] }}
                     </li>
                   </ul>
+                  {{/if}}
                 </template>
-                {{/unless~}}
                 {{/if~}}
               {{/default~}}
             {{/switch}}