Depend on client-side package installation root (#28)

Author: andersevenrud

src/packages.js

@@ -156,12 +156,16 @@ class Packages {
    * @param {object} user
    */
   async installPackage(url, options, user) {
+    if (!options.root) {
+      throw new Error('Missing package installation root path');
+    }
+
     const {realpath} = this.core.make('osjs/vfs');
 
     const name = path.basename(url.split('?')[0])
       .replace(/\.[^/.]+$/, '');
 
-    const userRoot = options.root || 'home:/.packages'; // FIXME: Client-side
+    const userRoot = options.root;
     const target = await realpath(`${userRoot}/${name}`, user);
     const root = await realpath(userRoot, user);
     const manifest = await realpath(`${userRoot}/metadata.json`, user);
@@ -194,6 +198,10 @@ class Packages {
     const metadatas = await Promise.all(filenames.map(f => fs.readJson(f)));
 
     await fs.writeJson(manifest, metadatas);
+
+    return {
+      reload: !options.system
+    };
   }
 
   /**

src/providers/packages.js

@@ -79,7 +79,7 @@ class PackageServiceProvider extends ServiceProvider {
 
     routeAuthenticated('POST', '/api/packages/install', (req, res) => {
       this.packages.installPackage(req.body.url, req.body.options, req.session.user)
-        .then(() => res.json({success: true}))
+        .then(body => res.json(body))
         .catch((error) => {
           console.error(error);
           res.status(400).json({error: 'Package installation failed'});