Merge remote-tracking branch 'thehapax/593-remove-write-order=log' in…

…to pr-598-merge

dexbot/orderengines/bitshares_engine.py

@@ -111,10 +111,6 @@ def __init__(self,
         # buy/sell actions will return order id by default
         self.returnOrderId = 'head'
 
-        self.orders_log = logging.LoggerAdapter(
-            logging.getLogger('dexbot.orders_log'), {}
-        )
-
     def _callbackPlaceFillOrders(self, d):
         """ This method distinguishes notifications caused by Matched orders from those caused by placed orders
         """

dexbot/strategies/base.py

@@ -189,6 +189,8 @@ def __init__(self,
             }
         )
 
+        self.orders_log = logging.LoggerAdapter(logging.getLogger('dexbot.orders_log'), {})
+
     def pause(self):
         """ Pause the worker
 

dexbot/strategies/relative_orders.py

@@ -536,10 +536,6 @@ def check_orders(self, *args, **kwargs):
                 if not current_order:
                     need_update = True
                     self.log.debug('Could not found order on the market, it was filled, expired or cancelled')
-                    # Write a trade log entry only when we are not using custom expiration because we cannot
-                    # distinguish an expired order from filled
-                    if not self.is_custom_expiration:
-                        self.write_order_log(self.worker_name, order)
                 elif self.is_reset_on_partial_fill:
                     # Detect partially filled orders;
                     # on fresh order 'for_sale' is always equal to ['base']['amount']