From 0b06d41ea5f86b563ceebec0a4e1c8f1bebb5b67 Mon Sep 17 00:00:00 2001 From: NEB <135718507+techareaone@users.noreply.github.com> Date: Sat, 30 May 2026 17:06:41 +0100 Subject: [PATCH] Remove duplicate minperstatus dispatch in _oncepost_open _oncepost_open and _next_open share functional code. both read self._minperstatus and dispatched to prenext_open / nextstart_open / next_open. _oncepost_open now delegates to _next_open directly. Removes 4 lines of duplicated logic --- backtrader/strategy.py | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/backtrader/strategy.py b/backtrader/strategy.py index e5115688d..bb94566d5 100644 --- a/backtrader/strategy.py +++ b/backtrader/strategy.py @@ -281,13 +281,7 @@ def next_open(self): pass def _oncepost_open(self): - minperstatus = self._minperstatus - if minperstatus < 0: - self.next_open() - elif minperstatus == 0: - self.nextstart_open() # only called for the 1st value - else: - self.prenext_open() + self._next_open() def _oncepost(self, dt): for indicator in self._lineiterators[LineIterator.IndType]: