Avoid generating useless objects on Errors.process

Author: Marsup

lib/errors.js

@@ -119,15 +119,10 @@ exports.process = function (errors, object) {
                 return item.options.error;
             }
 
-            const detail = {
-                message: item.toString(),
-                path: internals.getPath(item),
-                type: item.type,
-                context: item.context
-            };
-
+            let itemMessage;
             if (parent === undefined) {
-                message = message + (message ? '. ' : '') + detail.message;
+                itemMessage = item.toString();
+                message = message + (message ? '. ' : '') + itemMessage;
             }
 
             // Do not push intermediate errors, we're only interested in leafs
@@ -139,7 +134,12 @@ exports.process = function (errors, object) {
                 }
             }
             else {
-                details.push(detail);
+                details.push({
+                    message: itemMessage || item.toString(),
+                    path: internals.getPath(item),
+                    type: item.type,
+                    context: item.context
+                });
             }
         }
     };
@@ -161,17 +161,7 @@ exports.process = function (errors, object) {
 
 internals.getPath = function (item) {
 
-    const recursePath = (it) => {
-
-        const reachedItem = Hoek.reach(it, 'context.reason.0');
-        if (reachedItem && reachedItem.context) {
-            return recursePath(reachedItem);
-        }
-
-        return it.path;
-    };
-
-    return recursePath(item) || item.context.key;
+    return item.path || item.context.key;
 };