onBeforeExecCommand always overwrites the beforeBookmark

onBeforeExecCommand always overwrites the beforeBookmark (in bookmarkManager) despite skip_undo being checked further into the call stack. Different variables were being passed to onBefore than onExec. This causes commands that skip_undo to modify the undoManagers state.