Consolidate logic for actions taken on a workspace

* When renaming a workspace allow validation to handle all cases the same
* Do not display backtrace on console for rename failure, this is still logged to framework.log

lib/msf/core/db_manager/workspace.rb

@@ -97,11 +97,16 @@ def update_workspace(opts)
     Msf::Util::DBManager.delete_opts_workspace(opts)
 
     ::ActiveRecord::Base.connection_pool.with_connection {
-      ws_to_update = workspaces({ id: opts.delete(:id) }).first
+      ws_id = opts.delete(:id)
+      ws_to_update = workspaces({ id: ws_id }).first
       default_renamed = true if ws_to_update.name == DEFAULT_WORKSPACE_NAME
-      updated_ws = ws_to_update.update!(opts)
+      begin
+        ws_to_update.update!(opts) # will raise exception if it fails
+      rescue ActiveRecord::RecordInvalid => e
+        raise ArgumentError.new(e.message)
+      end
       add_workspace({ name: DEFAULT_WORKSPACE_NAME }) if default_renamed
-      updated_ws
+      workspaces({ id: ws_id }).first
     }
   end
 end

lib/msf/ui/console/command_dispatcher/db.rb

@@ -179,10 +179,6 @@ def cmd_workspace(*args)
           name: names.last
       }
       begin
-        if names.last == Msf::DBManager::Workspace::DEFAULT_WORKSPACE_NAME
-          print_error("Unable to rename a workspace to '#{Msf::DBManager::Workspace::DEFAULT_WORKSPACE_NAME}'")
-          return
-        end
         updated_ws = framework.db.update_workspace(opts)
         if updated_ws
           framework.db.workspace = updated_ws if names.first == framework.db.workspace.name
@@ -197,7 +193,6 @@ def cmd_workspace(*args)
         end
       rescue => e
         print_error "Failed to rename workspace: #{e.message}"
-        e.backtrace.each { |line| print_error "#{line}"}
       end
 
     elsif names