Merge pull request #1895 from wheels-dev/add-error-logging

Add pre logging to wheels

Author: zainforbjs

public/Application.cfc

@@ -261,8 +261,20 @@ component output="false" {
 	}
 
 	public void function onError( any Exception, string EventName ) {
-		injector = new wheels.Injector("wheels.Bindings");
-		application.wo = injector.getInstance("global");
+		try {
+			injector = new wheels.Injector("wheels.Bindings");
+			application.wo = injector.getInstance("global");
+
+			// Make exception available to the event template
+			request.wheels = request.wheels ?: {};
+			request.wheels.exception = Exception;
+			request.wheels.eventName = EventName;
+
+			// Run early error event if it exists
+			application.wo.$include(template = "/wheels/events/onerror/onerrorstart.cfm");
+		} catch (any e) {
+			// Must never break error handling
+		}
 
 		// In case the error was caused by a timeout we have to add extra time for error handling.
 		// We have to check if onErrorRequestTimeout exists since errors can be triggered before the application.wheels struct has been created.

vendor/wheels/events/onerror/onerrorstart.cfm

@@ -0,0 +1,8 @@
+<cfif structKeyExists(request.wheels, "exception")>
+	<cflog
+		file="wheels-errors"
+		type="error"
+		text="#request.wheels.exception.message#
+#request.wheels.exception.detail#"
+	>
+</cfif>