Resolved merge conflicts against master.

evennia/objects/manager.py

@@ -207,7 +207,7 @@ class ObjectDBManager(TypedObjectManager):
             return []
         except ValueError:
             from evennia.utils import logger
-            logger.log_errmsg("The property '%s' does not support search criteria of the type %s." % (property_name, type(property_value)))
+            logger.log_err("The property '%s' does not support search criteria of the type %s." % (property_name, type(property_value)))
             return []
 
     @returns_typeclass_list

evennia/objects/models.py

@@ -269,8 +269,8 @@ class ObjectDB(TypedObject):
             raise #RuntimeError(errmsg)
         except Exception as e:
             errmsg = "Error (%s): %s is not a valid location." % (str(e), location)
-            logger.log_errmsg(errmsg)
-            raise #Exception(errmsg)
+            logger.log_trace(errmsg)
+            raise
 
     def __location_del(self):
         "Cleanly delete the location reference"

evennia/objects/objects.py

@@ -17,7 +17,7 @@ from evennia.scripts.scripthandler import ScriptHandler
 from evennia.commands import cmdset, command
 from evennia.commands.cmdsethandler import CmdSetHandler
 from evennia.commands import cmdhandler
-from evennia.utils.logger import log_trace, log_errmsg
+from evennia.utils import logger
 from evennia.utils.utils import (variable_from_module, lazy_property,
                              make_iter, to_str, to_unicode)
 
@@ -432,13 +432,13 @@ class DefaultObject(ObjectDB):
             try:
                 from_obj.at_msg_send(text=text, to_obj=self, **kwargs)
             except Exception:
-                log_trace()
+                logger.log_trace()
         try:
             if not self.at_msg_receive(text=text, **kwargs):
                 # if at_msg_receive returns false, we abort message to this object
                 return
         except Exception:
-            log_trace()
+            logger.log_trace()
 
         # session relay
         kwargs['_nomulti'] = kwargs.get('_nomulti', True)
@@ -522,7 +522,7 @@ class DefaultObject(ObjectDB):
             "Simple log helper method"
             trc = traceback.format_exc()
             errstring = "%s%s" % (trc, string)
-            log_trace()
+            logger.log_trace()
             self.msg(errstring)
 
         errtxt = _("Couldn't perform move ('%s'). Contact an admin.")
@@ -549,7 +549,6 @@ class DefaultObject(ObjectDB):
             except Exception:
                 logerr(errtxt % "at_before_move()")
                 #emit_to_obj.msg(errtxt % "at_before_move()")
-                #logger.log_trace()
                 return False
 
         # Save the old location
@@ -570,7 +569,6 @@ class DefaultObject(ObjectDB):
             except Exception:
                 logerr(errtxt % "at_object_leave()")
                 #emit_to_obj.msg(errtxt % "at_object_leave()")
-                #logger.log_trace()
                 return False
 
         if not quiet:
@@ -580,16 +578,14 @@ class DefaultObject(ObjectDB):
             except Exception:
                 logerr(errtxt % "at_announce_move()")
                 #emit_to_obj.msg(errtxt % "at_announce_move()" )
-                #logger.log_trace()
                 return False
 
         # Perform move
         try:
-            #print "move_to location:", destination
             self.location = destination
         except Exception:
             emit_to_obj.msg(errtxt % "location change")
-            log_trace()
+            logger.log_trace()
             return False
 
         if not quiet:
@@ -599,7 +595,6 @@ class DefaultObject(ObjectDB):
             except Exception:
                 logerr(errtxt % "announce_move_to()")
                 #emit_to_obj.msg(errtxt % "announce_move_to()")
-                #logger.log_trace()
                 return  False
 
         if move_hooks:
@@ -610,7 +605,6 @@ class DefaultObject(ObjectDB):
             except Exception:
                 logerr(errtxt % "at_object_receive()")
                 #emit_to_obj.msg(errtxt % "at_object_receive()")
-                #logger.log_trace()
                 return False
 
         # Execute eventual extra commands on this object after moving it
@@ -621,7 +615,6 @@ class DefaultObject(ObjectDB):
             except Exception:
                 logerr(errtxt % "at_after_move")
                 #emit_to_obj.msg(errtxt % "at_after_move()")
-                #logger.log_trace()
                 return False
         return True
 
@@ -649,7 +642,7 @@ class DefaultObject(ObjectDB):
                 default_home = None
         except Exception:
             string = _("Could not find default home '(#%d)'.")
-            log_errmsg(string % default_home_id)
+            logger.log_err(string % default_home_id)
             default_home = None
 
         for obj in self.contents:
@@ -665,7 +658,7 @@ class DefaultObject(ObjectDB):
                 string += "now has a null location."
                 obj.location = None
                 obj.msg(_("Something went wrong! You are dumped into nowhere. Contact an admin."))
-                log_errmsg(string % (obj.name, obj.dbid))
+                logger.log_err(string % (obj.name, obj.dbid))
                 return
 
             if obj.has_player: