Remove deprecated method argument

src/order/models.py

@@ -203,7 +203,20 @@ def auto_create_orders(self, delivery_date, clients):
         return created_orders
 
     def create_batch_orders(self, delivery_dates, client, items,
-                            override_dates=[], return_created_orders=False):
+                            override_dates=[]):
+        """
+        Create orders for one or multiple days, for a given client.
+        Order items will be created based on client's meals schedule.
+
+        Parameters:
+          delivery_dates : dates on which orders are to be created
+          client : a client object
+          items: the order items
+          override_dates: dates on which existing orders can be overriden
+
+        Returns:
+          An array of created orders.
+        """
         created_orders = []
         messages = []
 
@@ -238,10 +251,7 @@ def create_batch_orders(self, delivery_dates, client, items,
             )
             created_orders.append(order)
 
-        if not return_created_orders:
-            return len(created_orders)   # LXYANG: TO BE DEPRECATED
-        else:
-            return created_orders
+        return created_orders
 
     @transaction.atomic
     def create_order(self, delivery_date, client, items, prices):

src/order/tests.py

@@ -614,9 +614,9 @@ def test_create_batch_orders(self):
         Provide a client, 4 delivery dates.
         """
         client = self.episodic_client[0]
-        counter = Order.objects.create_batch_orders(
+        created_orders = Order.objects.create_batch_orders(
             self.delivery_dates, self.episodic_client[0], self.orditems)
-        self.assertEqual(counter, 4)
+        self.assertEqual(len(created_orders), 4)
 
         # check items
         # 2016-12-12

src/order/views.py

@@ -249,8 +249,7 @@ def form_valid(self, form):
 
         # Place orders using posted datas
         created_orders = Order.objects.create_batch_orders(
-            del_dates, client, items, override_dates=ovr_dates,
-            return_created_orders=True
+            del_dates, client, items, override_dates=ovr_dates
         )
 
         # check created and uncreated dates