CB-11259: Improving prepare and build logging
authorRichard Knoll <richard.b.knoll@gmail.com>
Mon, 2 May 2016 17:54:21 +0000 (10:54 -0700)
committerRichard Knoll <richard.b.knoll@gmail.com>
Thu, 12 May 2016 23:06:05 +0000 (16:06 -0700)
This closes #439

src/ConfigChanges/ConfigFile.js
src/PluginInfo/PluginInfo.js
src/events.js

index ad12f38..9c540b9 100644 (file)
@@ -105,13 +105,13 @@ ConfigFile.prototype.graft_child = function ConfigFile_graft_child(selector, xml
         var xml_to_graft = [modules.et.XML(xml_child.xml)];
         result = modules.xml_helpers.graftXML(self.data, xml_to_graft, selector, xml_child.after);
         if ( !result) {
-            throw new Error('grafting xml at selector "' + selector + '" from "' + filepath + '" during config install went bad :(');
+            throw new Error('Unable to graft xml at selector "' + selector + '" from "' + filepath + '" during config install');
         }
     } else {
         // plist file
         result = modules.plist_helpers.graftPLIST(self.data, xml_child.xml, selector);
         if ( !result ) {
-            throw new Error('grafting to plist "' + filepath + '" during config install went bad :(');
+            throw new Error('Unable to graft plist "' + filepath + '" during config install');
         }
     }
     self.is_changed = true;
index e1a1b31..77199ec 100644 (file)
@@ -331,7 +331,7 @@ function PluginInfo(dirname) {
     ///// PluginInfo Constructor logic  /////
     self.filepath = path.join(dirname, 'plugin.xml');
     if (!fs.existsSync(self.filepath)) {
-        throw new CordovaError('Cannot find plugin.xml for plugin \'' + path.basename(dirname) + '\'. Please try adding it again.');
+        throw new CordovaError('Cannot find plugin.xml for plugin "' + path.basename(dirname) + '". Please try adding it again.');
     }
 
     self.dir = dirname;
index 8751976..e702bd8 100644 (file)
@@ -40,7 +40,7 @@ module.exports.forwardEventsTo = function (eventEmitter) {
     }
 
     if (!(eventEmitter instanceof EventEmitter))
-        throw new Error('Cordova events could be redirected to another EventEmitter instance only');
+        throw new Error('Cordova events can be redirected to another EventEmitter instance only');
 
     // CB-10940 Skipping forwarding to self to avoid infinite recursion.
     // This is the case when the modules are npm-linked.