better escaping for non-quoted values

Author: bitsofinfo

o365Utils.js

@@ -184,7 +184,7 @@ var o365CommandRegistry = {
             'Type':               {'quoted':false, 'default':'Security'},
             'ManagedBy':          {},
             'Members':            {}, // specifying members on create does not seem to work
-            'ModerationEnabled':              { 'default':'$false', 'quoted':false},
+            'ModerationEnabled':              { 'default':'0', 'quoted':false},
             'MemberDepartRestriction':        { 'default':'Closed'},
             'MemberJoinRestriction':          { 'default':'Closed'},
             'SendModerationNotifications':    { 'default':'Never', 'quoted':false},
@@ -206,7 +206,7 @@ var o365CommandRegistry = {
             'ManagedBy':          {},
             'Members':            {},
             'MailTip':            {},
-            'ModerationEnabled':              { 'default':'$false', 'quoted':false},
+            'ModerationEnabled':              { 'default':'0', 'quoted':false},
             'MemberDepartRestriction':        { 'default':'Closed'},
             'MemberJoinRestriction':          { 'default':'Closed'},
             'SendModerationNotifications':    { 'default':'Never', 'quoted':false},

psCommandService.js

@@ -360,9 +360,13 @@ PSCommandService.prototype._sanitize = function(toSanitize,isQuoted) {
     // escape stuff that could screw up variables
     toSanitize = toSanitize.replace(/([`#])/g, "`$1");
 
-    // fix quote breaking
+    // if quoted, escape all quotes
     if (isQuoted) {
         toSanitize = toSanitize.replace(/(['])/g, "'$1");
+
+    // if not quoted, stop $ and |
+    } else {
+        toSanitize = toSanitize.replace(/([\$\|])/g, "`$1");
     }
 
     return toSanitize;