Merge pull request #207 from georgejecook/fix/observe-selection-compl…

…etion

fix(core): fixes completion state callback not firing for selection

src/source/view/BaseView.bs

@@ -235,6 +235,7 @@ namespace mv
       'assumes that you have an appController instance that will handle the selection
       appController = m.getInstance("appController")
       if appController <> invalid
+        m.observeNodeField(selection, "itemCompletionState", m.onSelectionCompletionStateChange, "both", true)
         result = appController@.onExecuteSelection(selection)
         if result = invalid
           m.log.error("Selection could not be executed. Do you have an appController assigned, with an onExecuteSelection method?")
@@ -243,7 +244,6 @@ namespace mv
           else
             m.log.info("selection requires async resolution")
             m.observeNodeField(selection, "state", m.onSelectionStateChange, "both", true)
-            m.observeNodeField(selection, "itemCompletionState", m.onSelectionCompletionStateChange, "both", true)
           end if
         end if
       else