renamed typing build command from ts-prep-def with build-typedef. fixed null serviceobject default values

Author: gautamsi

.vscode/launch.json

@@ -9,13 +9,31 @@
 			// Type of configuration. Possible values: "node", "mono".
 			"type": "node",
 			// Workspace relative or absolute path to the program.
-			"program": "./build/output/node/test/ManualLiveTesting/app.js",
+			"program": "${workspaceRoot}//build/output/node/test/ManualLiveTesting/app.js",
 			// Automatically stop program after launch.
 			"stopOnEntry": false,
 			// Command line arguments passed to the program.
 			"args": [],
 			// Workspace relative or absolute path to the working directory of the program being debugged. Default is the current workspace.
-			"cwd": ".",
+			"cwd": "${workspaceRoot}",
+			// Workspace relative or absolute path to the runtime executable to be used. Default is the runtime executable on the PATH.
+			"runtimeExecutable": null,
+			// Environment variables passed to the program.
+			"env": { }
+		},
+        {
+			// Name of configuration; appears in the launch configuration drop down menu.
+			"name": "Launch AttachmentTest.js",
+			// Type of configuration. Possible values: "node", "mono".
+			"type": "node",
+			// Workspace relative or absolute path to the program.
+			"program": "${workspaceRoot}/build/output/node/test/ManualLiveTesting/AttachmentTest.js",
+			// Automatically stop program after launch.
+			"stopOnEntry": false,
+			// Command line arguments passed to the program.
+			"args": [],
+			// Workspace relative or absolute path to the working directory of the program being debugged. Default is the current workspace.
+			"cwd": "${workspaceRoot}",
 			// Workspace relative or absolute path to the runtime executable to be used. Default is the runtime executable on the PATH.
 			"runtimeExecutable": null,
 			// Environment variables passed to the program.
@@ -24,10 +42,19 @@
 		{
 			"name": "Attach",
 			"type": "node",
+            "request": "attach",
 			// TCP/IP address. Default is "localhost".
 			"address": "localhost",
 			// Port to attach to.
 			"port": 5858
+		},
+        {
+			"name": "Run Mocha Test",
+			"type": "node",
+            "request": "launch",
+            "cwd": "${workspaceRoot}",
+            "program": "${workspaceRoot}/node_modules/mocha/bin/_mocha",
+            "args": ["${workspaceRoot}/build/output/node/test/mocha/**/*.js"]			
 		}
 	]
 }

gulpfile.js

@@ -222,7 +222,7 @@ gulp.task('ts-def-clean', ["ts-def-concat"], function (done) {
     del(delconfig, done);
 });
 
-gulp.task("ts-prep-def", ['ts-def-clean'])
+gulp.task("build-typedef", ['ts-def-clean'])
 
 
 /**

import fixup script.ps1

@@ -396,3 +396,27 @@ if($filetofix -ne $null){
 $exports = "export {"
 $allfiles.BaseName | %{$exports += "`r`n// $_,"}
 $exports += "`r`n};"
+
+$moduleNames = @()
+$allthrows = @()
+function searchdependencyportstatus($f){
+    $match = dir ($f + ".ts") -Recurse |  Select-String -Pattern  '(import).*\".*\/(?<moduleName>.*)\"\;'
+        if($match)
+        {
+            $moduleNames = $match | %{$_.Matches | %{$_.Groups["moduleName"].Value}}
+            $moduleNames | %{
+                if($global:moduleNames -notcontains $_){
+                    $global:moduleNames += $_
+                    Write-Host $_ -ForegroundColor Red
+                    $m2 = dir ($_ + ".ts") -Recurse |  Select-String -Pattern  '^\s+\w.*throw new Error.*Not implemented.*'
+                    $global:allthrows += $m2
+                    write-host "---------------------------------------------------" -ForegroundColor Yellow
+                    #$list
+                    searchdependencyportstatus $_
+                }
+                #else{"skipping $_"}
+            }
+
+        }
+}
+searchdependencyportstatus "Attachment"

src/js/PropertyDefinitions/ComplexPropertyDefinitionBase.ts

@@ -111,17 +111,20 @@ export abstract class ComplexPropertyDefinitionBase extends PropertyDefinition {
 
         propertyBag._setItem(this, outComplexproperty.outValue);
     }
-    
+
     /**
      * @internal Loads the property value from XMLJsObject.
      *
      * @param   {any}               jsObject         The JSON value.  Can be a JsonObject, string, number, bool, array, or null.
      * @param   {ExchangeService}   service       The service.
      * @param   {PropertyBag}       propertyBag   The property bag.
-     */    
+     */
     LoadPropertyValueFromXmlJsObject(jsObject: any, service: ExchangeService, propertyBag: PropertyBag): void {
-        //debugger;//update: array type detection in next call, can not call GetPropertyInstance multiple time
-        this.InternalLoadFromXmlJsObject(jsObject, service, propertyBag);        
+        // check for null value from empty element
+        if (jsObject) {
+            //debugger;//update: array type detection in next call, can not call GetPropertyInstance multiple time
+            this.InternalLoadFromXmlJsObject(jsObject, service, propertyBag);
+        }
     }
 
     /**