Default ports to non-required, fixes #200

CHANGES.md

@@ -1,6 +1,10 @@
 NoFlo ChangeLog
 ===============
 
+## 0.5.7 (git master)
+
+* Ports now default to *not required*. Set the port option `required: true` the port needs to be connected in order for the component to work
+
 ## 0.5.6 (June 23rd 2014)
 
 * Custom icon support for subgraphs via the `icon` key in graph properties

component.json

@@ -7,7 +7,7 @@
     "flow"
   ],
   "repo": "noflo/noflo",
-  "version": "0.5.5",
+  "version": "0.5.6",
   "dependencies": {
     "bergie/emitter": "*",
     "jashkenas/underscore": "*",

package.json

@@ -7,7 +7,7 @@
     "flow"
   ],
   "author": "Henri Bergius <henri.bergius@iki.fi>",
-  "version": "0.5.5",
+  "version": "0.5.6",
   "licenses": [
     {
       "type": "MIT",

spec/Component.coffee

@@ -92,6 +92,7 @@ describe 'Component', ->
         outPorts:
           error:
             datatype: 'object'
+            required: true
 
       s1 = new socket.InternalSocket
       c.inPorts.in.attach s1

spec/InPort.coffee

@@ -15,8 +15,8 @@ describe 'Inport Port', ->
     p = new inport
     it 'should be of datatype "all"', ->
       chai.expect(p.getDataType()).to.equal 'all'
-    it 'should be required', ->
-      chai.expect(p.isRequired()).to.equal true
+    it 'should not be required', ->
+      chai.expect(p.isRequired()).to.equal false
     it 'should not be addressable', ->
       chai.expect(p.isAddressable()).to.equal false
     it 'should not be buffered', ->

src/lib/BasePort.coffee

@@ -30,7 +30,7 @@ class BasePort extends EventEmitter
   handleOptions: (options) ->
     options = {} unless options
     options.datatype = 'all' unless options.datatype
-    options.required = true if options.required is undefined
+    options.required = false if options.required is undefined
 
     options.datatype = 'int' if options.datatype is 'integer'
     if validTypes.indexOf(options.datatype) is -1