Fix for STOCK_COUNT event (#12281)

* Fix for STOCK_COUNT event

- Prevent double emission of event type string

* adjust unit test

src/backend/InvenTree/stock/models.py

@@ -2663,7 +2663,7 @@ class StockItem(
         return True
 
     @transaction.atomic
-    def stocktake(self, count, user, **kwargs):
+    def stocktake(self, count, user, **kwargs) -> None:
         """Perform item stocktake.
 
         Arguments:
@@ -2678,7 +2678,7 @@ class StockItem(
         try:
             count = Decimal(count)
         except InvalidOperation:
-            return False
+            return
 
         if count < 0:
             return False
@@ -2729,13 +2729,10 @@ class StockItem(
 
             trigger_event(
                 StockEvents.ITEM_COUNTED,
-                'stockitem.counted',
                 id=self.id,
                 quantity=1 if self.serialized else float(self.quantity),
             )
 
-        return True
-
     @transaction.atomic
     def add_stock(self, quantity, user, **kwargs):
         """Add a specified quantity of stock to this item.

src/backend/InvenTree/stock/tests.py

@@ -378,10 +378,8 @@ class StockTest(StockTestBase):
         self.assertEqual(it.status, StockStatus.OK.value)
 
         # Next, perform a valid stocktake
-        self.assertTrue(
+        it.stocktake(
-            it.stocktake(
+            100, None, notes='test stocktake', status=StockStatus.DAMAGED.value
-                100, None, notes='test stocktake', status=StockStatus.DAMAGED.value
-            )
         )
 
         it.refresh_from_db()