Fixes for issues #110 and #111

Author: chris-steele

README.md

@@ -258,5 +258,5 @@ See [Developing plug-ins](https://github.com/adaptlearning/adapt_framework/wiki/
 <div float align=right><a href="#top">Back to Top</a></div>  
 
 ----------------------------
-**Version number:**  2.1.4   <a href="https://community.adaptlearning.org/" target="_blank"><img src="https://github.com/adaptlearning/documentation/blob/master/04_wiki_assets/plug-ins/images/adapt-logo-mrgn-lft.jpg" alt="adapt learning logo" align="right"></a>   
+**Version number:**  2.1.7   <a href="https://community.adaptlearning.org/" target="_blank"><img src="https://github.com/adaptlearning/documentation/blob/master/04_wiki_assets/plug-ins/images/adapt-logo-mrgn-lft.jpg" alt="adapt learning logo" align="right"></a>   
 **Author / maintainer:** Adapt Core Team with [contributors](https://github.com/adaptlearning/adapt-contrib-hotgraphic/graphs/contributors)  

lib/commands/register.js

@@ -9,13 +9,18 @@ var bower = require('bower'),
     JsonWriter = require('../JsonWriter'),
     Project = require('../Project'),
     Plugin = require('../Plugin');
+    semver = require('semver');
 
 module.exports = {
     register: function (renderer) {
         var done = arguments[arguments.length-1] || function () {};
 
         renderer.log(chalk.yellow('This will publish this plugin to', Constants.Registry));
 
+        var c = semver.coerce('>=2.2')
+        console.log(c);
+        console.log(semver.valid(c));
+
         loadPluginProperties('./bower.json', {
             name: undefined,
             repository: undefined,
@@ -110,16 +115,29 @@ function confirm(properties) {
                     default: plugin.toString() || 'not specified',
                     required: true
                 },
-                repository: {
-                    description: chalk.cyan('repository'),
-                    pattern: /([A-Za-z0-9]+@|http(|s)|git\:\/\/)([A-Za-z0-9.]+)(:|\/)([A-Za-z0-9\-\.\/]+)(\.git)?/,
+                repositoryType: {
+                    description: chalk.cyan('repository type (e.g. git)'),
+                    message: 'Please provide a repository type',
+                    pattern: /.+/,
+                    type: 'string',
+                    default: properties.repository ? properties.repository.type : undefined,
+                    required: true
+                },
+                repositoryUrl: {
+                    description: chalk.cyan('repository URL'),
+                    message: 'Please provide a repository URL of the form [git@][<protocol>]<domain><path>.git',
+                    pattern: /((git|ssh|http(s)?)|(git@[\w\.]+))(:(\/\/)?)([\w\.@\:/\-~]+)(\.git)(\/)?/,
                     type: 'string',
-                    default: properties.repository || 'not specified',
+                    default: properties.repository ? properties.repository.url : undefined,
                     required: true
                 },
                 framework: {
                     description: chalk.cyan('framework'),
-                    pattern: /\bv?(?:0|[1-9][0-9]*)\.(?:0|[1-9][0-9]*)\.(?:0|[1-9][0-9]*)(?:-[\da-z\-]+(?:\.[\da-z\-]+)*)?(?:\+[\da-z\-]+(?:\.[\da-z\-]+)*)?\b/ig,
+                    message: 'Please provide a valid semver (see https://semver.org/)',
+                    conform:function(v) {
+                        return semver.valid(semver.coerce(v));
+                    },
+                    errors:['fooble'],
                     type: 'string',
                     default: properties.framework || '~2.0.0',
                     required: false
@@ -143,8 +161,13 @@ function confirm(properties) {
         if(!confirmation.ready) deferred.reject(new Error('Aborted. Nothing has been registered.'));
 
         properties.name = confirmation.name;
-        properties.repository = confirmation.repository;
+
+        if (confirmation.repositoryType && confirmation.repositoryUrl) {
+            properties.repository = {type:confirmation.repositoryType, url:confirmation.repositoryUrl};
+        }
+
         properties.framework = confirmation.framework;
+
         deferred.resolve(properties);
     });
     return deferred.promise;

package.json

@@ -1,6 +1,6 @@
 {
   "name": "adapt-cli",
-  "version": "2.1.6",
+  "version": "2.1.7",
   "description": "Command line tools for Adapt",
   "main": "./lib/cli.js",
   "directories": {