refactor: None checks anti patterns

Author: ceccopierangiolieugenio

apps/dumbPaintTool/dumbPaintTool/app/palette.py

@@ -94,14 +94,14 @@ def _getPos(col):
             return None
         pw, ph = len(self._palette[0]),len(self._palette)
         if fg!=ttk.TTkColor.RST:
-            if (pos:=_getPos(fg)) != None:
+            if (pos:=_getPos(fg)) is not None:
                 self._fg = pos
             else:
                 self._fg = (pw-2,ph-1)
                 self._palette[ph-1][pw-2] = (fg,fg.invertFgBg())
 
         if bg!=ttk.TTkColor.RST:
-            if (pos:=_getPos(bg)) != None:
+            if (pos:=_getPos(bg)) is not None:
                 self._bg = pos
             else:
                 self._bg = (pw-1,ph-1)

apps/ttkDesigner/ttkDesigner/app/superobj/superlayoutgrid.py

@@ -254,7 +254,7 @@ def _processMouseOver(self, x, y):
         # iy, ih = verSizes[row]
 
         wid = self.layout().itemAtPosition(row,col)
-        if wid == None:
+        if wid is None:
             return dir,pos,wid,placesSpan
 
         col = wid._col
@@ -336,7 +336,7 @@ def _processDragOver(self, x, y):
         dr = ix+iw-x-1
         dmin = min(dt,db,dl,dr)
 
-        if self.layout().itemAtPosition(row,col) == None:
+        if self.layout().itemAtPosition(row,col) is None:
             ret = (ix, iy, iw, ih)
         else:
             #Top - we are closer to this edge

libs/pyTermTk/TermTk/TTkCore/filebuffer.py

@@ -101,7 +101,7 @@ def getLine(self, line):
             return ""
         page = line//self._window
         offset = line%self._window
-        if self._pages[page] == None:
+        if self._pages[page]is None:
             # Dispose of the pages to the bottom
             dispose = self._buffer.pop(0)
             if dispose is not None:

libs/pyTermTk/TermTk/TTkCore/string.py

@@ -485,7 +485,7 @@ def completeColor(self, color:TTkColor, match=None, posFrom=None, posTo=None) ->
                 start = pos+lenMatch
                 for i in range(pos, pos+lenMatch):
                     ret._colors[i] |= color
-        elif posFrom == posTo == None:
+        elif posFrom is posTo is None:
             ret._colors = [c|color for c in self._colors]
         elif posFrom < posTo:
             ret._colors = self._colors.copy()
@@ -523,7 +523,7 @@ def setColor(self, color, match=None, posFrom=None, posTo=None) -> TTkString:
             while None != (pos := self._text.index(match, start) if match in self._text[start:] else None):
                 start = pos+lenMatch
                 ret._colors[pos: pos+lenMatch] = [color]*lenMatch
-        elif posFrom == posTo == None:
+        elif posFrom is posTo is None:
             ret._colors = [color]*len(self._text)
         elif posFrom < posTo:
             ret._colors += self._colors

libs/pyTermTk/TermTk/TTkGui/clipboard.py

@@ -137,7 +137,7 @@ def text():
                 TTkLog.error("Clipboard error, try to export X11 if you are running this UI via SSH")
                 for line in str(e).split("\n"):
                     TTkLog.error(line)
-            if txt == None or txt == str(TTkClipboard._clipboard):
+            if txt is None or txt == str(TTkClipboard._clipboard):
                 return TTkClipboard._clipboard
             return txt
         return TTkClipboard._clipboard

libs/pyTermTk/TermTk/TTkWidgets/apptemplate.py

@@ -236,7 +236,7 @@ def setWidget(self,
             self.layout().addWidget(widget)
             if border!=None:
                 self._panels[position].border = border
-            if fixed != None:
+            if fixed is not None:
                self._panels[position].fixed = fixed
             self._panels[position].title = TTkString(title)
             self._panels[position].size = ( size if size is not None else
@@ -277,7 +277,7 @@ def setItem(self,
             self.layout().addItem(item)
             if border!=None:
                 self._panels[position].border = border
-            if fixed != None:
+            if fixed is not None:
                self._panels[position].fixed = fixed
             self._panels[position].title = TTkString(title)
             self._panels[position].size = ( size if size is not None else

libs/pyTermTk/TermTk/TTkWidgets/checkbox.py

@@ -117,7 +117,7 @@ def __init__(self, *,
 
         self._text = TTkString(text)
 
-        if checkStatus != None :
+        if checkStatus is not None :
             self._checkStatus = checkStatus
         else:
             self._checkStatus = TTkK.Checked if checked else TTkK.Unchecked

libs/pyTermTk/TermTk/TTkWidgets/radiobutton.py

@@ -95,7 +95,7 @@ def __init__(self, *,
         self._radiogroup = radiogroup
         # self.checked = pyTTkSignal()
         self._checked = checked
-        if checkStatus != None :
+        if checkStatus is not None :
             self._checked = checkStatus==TTkK.Checked
         else:
             self._checked = checked
@@ -190,7 +190,7 @@ def paintEvent(self, canvas):
     def _checkEvent(self):
         # Uncheck the radio already checked;
         for radio in TTkRadioButton._radioLists[self._radiogroup]:
-            if self != radio != None:
+            if self != radio is not None:
                 if radio.isChecked():
                     radio.setCheckState(TTkK.Unchecked)
         self._checked = True

tests/pytest/modelView/test_tablemodellist.py

@@ -49,7 +49,7 @@ def test_init_default(self):
 
         assert model.rowCount() == 0
         assert model.columnCount() == 0
-        assert model.data(0, 0) == None
+        assert model.data(0, 0) is None
 
     def test_init_with_data(self):
         """Test initialization with data"""
@@ -298,7 +298,7 @@ def test_remove_rows_boundary_conditions(self):
         assert model.data(0, 0) == 'Alice'
         assert model.data(1, 0) == 'Bob'
         # Charlie should be gone
-        assert model.data(2, 0) == None
+        assert model.data(2, 0) is None
 
         # Reset for next test
         model = ttk.TTkTableModelList(data=[row[:] for row in self.test_data])
@@ -344,7 +344,7 @@ def test_remove_columns_boundary_conditions(self):
         assert model.columnCount() == original_column_count - 1
         assert model.data(0, 0) == 'Alice'  # Name still there
         assert model.data(0, 1) == 25       # Age still there
-        assert model.data(0, 2) == None     # Role should be gone
+        assert model.data(0, 2) is None     # Role should be gone
 
         # Reset for next test
         model = ttk.TTkTableModelList(data=[row[:] for row in self.test_data])
@@ -362,7 +362,7 @@ def test_remove_columns_boundary_conditions(self):
         # Should remove columns 1 and 2 (Age and Role), leaving only Name
         assert model.columnCount() == 1
         assert model.data(0, 0) == 'Alice'
-        assert model.data(0, 1) == None     # Age should be gone
+        assert model.data(0, 1) is None     # Age should be gone
 
         # Reset for next test
         model = ttk.TTkTableModelList(data=[row[:] for row in self.test_data])

tests/pytest/modelView/test_tablemodelsqlite3.py

@@ -773,7 +773,7 @@ def test_empty_table_initialization(self):
         assert model.columnCount() == 2  # name, age (id is primary key)
 
         # Test data access on empty table
-        assert model.data(0, 0) == None or model.data(0, 0) is None
+        assert model.data(0, 0) is None or model.data(0, 0) is None
 
         # Test header data still works
         assert model.headerData(0, ttk.TTkK.HORIZONTAL) == 'name'