kkeisuke / plantuml-editor

PlantUML online demo client
https://github.com/kkeisuke/plantuml-editor2
MIT License
435 stars 75 forks source link

Not working for me #1

Closed mcondarelli closed 6 years ago

mcondarelli commented 7 years ago

Here is the log of my installation attempt:

curl -sL https://deb.no
mcon@black:~/projects$ git clone https://github.com/kkeisuke/plantuml-editor.git
Cloning into 'plantuml-editor'...
remote: Counting objects: 1237, done.
remote: Compressing objects: 100% (33/33), done.
remote: Total 1237 (delta 16), reused 32 (delta 9), pack-reused 1194
Receiving objects: 100% (1237/1237), 11.33 MiB | 4.94 MiB/s, done.
Resolving deltas: 100% (726/726), done.
mcon@black:~/projects$ cd plantuml-editor
mcon@black:~/projects/plantuml-editor$ npm -v
5.5.1
mcon@black:~/projects/plantuml-editor$ node -v
v9.1.0
mcon@black:~/projects/plantuml-editor$ npm install

> phantomjs-prebuilt@2.1.14 install /home/mcon/projects/plantuml-editor/node_modules/phantomjs-prebuilt
> node install.js

PhantomJS not found on PATH
Download already available at /tmp/phantomjs/phantomjs-2.1.1-linux-x86_64.tar.bz2
Verified checksum of previously downloaded file
Extracting tar contents (via spawned process)
Removing /home/mcon/projects/plantuml-editor/node_modules/phantomjs-prebuilt/lib/phantom
Copying extracted folder /tmp/phantomjs/phantomjs-2.1.1-linux-x86_64.tar.bz2-extract-1510594273211/phantomjs-2.1.1-linux-x86_64 -> /home/mcon/projects/plantuml-editor/node_modules/phantomjs-prebuilt/lib/phantom
Writing location.js file
Done. Phantomjs binary available at /home/mcon/projects/plantuml-editor/node_modules/phantomjs-prebuilt/lib/phantom/bin/phantomjs
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@1.1.2 (node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@1.1.2: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 1096 packages in 11.996s
mcon@black:~/projects/plantuml-editor$ npm run dev

> plantuml-editor@1.0.0 dev /home/mcon/projects/plantuml-editor
> node build/dev-server.js

> Starting dev server...

 ERROR  Failed to compile with 4 errors                                                                                                                                                                18:32:00

These relative modules were not found:

* ./modules/plantumlEditor in ./src/store/index.js
* ./modules/histories in ./src/store/index.js
* ./modules/umlTemplate in ./src/store/index.js
* ./modules/gistApi in ./src/store/index.js
> Listening at http://localhost:8080

mcon@black:~/projects/plantuml-editor$ 

It seems the following change allows it to compile successfully (was this ever tested on a cases-sensitive filesystem?):

index 8668bbe..44d32ce 100644
--- a/src/store/index.js
+++ b/src/store/index.js
@@ -4,18 +4,18 @@ import Vue from 'vue'
 import Vuex from 'vuex'
 Vue.use(Vuex)

-import plantumlEditor from './modules/plantumlEditor'
+import PlantumlEditor from './modules/PlantumlEditor'
 import Layout from './modules/Layout'
-import histories from './modules/histories'
-import umlTemplate from './modules/umlTemplate'
-import gistApi from './modules/gistApi'
+import Histories from './modules/Histories'
+import UmlTemplate from './modules/UmlTemplate'
+import GistApi from './modules/GistApi'

 export default new Vuex.Store({
   modules: {
-    plantumlEditor,
+    PlantumlEditor,
     Layout,
-    histories,
-    umlTemplate,
-    gistApi
+    Histories,
+    UmlTemplate,
+    GistApi
   }
 })

... but I'm not really sure this is the right fix because the program doesn't seem to work too well:

image

What am I missing? Please advise.

mcondarelli commented 7 years ago

It seems the right fix is to rename the files, instead:

mcon@black:~/projects/plantuml-editor/src/store/modules$ ls -l
total 24
-rw-r--r-- 1 mcon mcon 3412 Nov 13 18:30 gistApi.js
-rw-r--r-- 1 mcon mcon 2216 Nov 13 18:30 histories.js
-rw-r--r-- 1 mcon mcon 1250 Nov 13 18:30 Layout.js
-rw-r--r-- 1 mcon mcon 4862 Nov 13 18:30 plantumlEditor.js
-rw-r--r-- 1 mcon mcon 1198 Nov 13 18:30 umlTemplate.js
mcon@black:~/projects/plantuml-editor/src/store/modules$ 

Now it seems to work (I only played with it a few minutes). Regards.

kkeisuke commented 7 years ago

Thanks for the feedback. Fixed index.js.