From: Amaury Forgeot d'Arc Date: Fri, 17 Sep 2010 23:27:09 +0000 (+0000) Subject: #1730136: Fix comparison between a tk Font object and an object of a different type. X-Git-Tag: v3.2a3~235 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d61d077bf57796555f0d8747718f60b35f136d6b;p=thirdparty%2FPython%2Fcpython.git #1730136: Fix comparison between a tk Font object and an object of a different type. --- diff --git a/Lib/tkinter/font.py b/Lib/tkinter/font.py index 1ec760ee885f..5425b060132a 100644 --- a/Lib/tkinter/font.py +++ b/Lib/tkinter/font.py @@ -97,7 +97,7 @@ class Font: return self.name def __eq__(self, other): - return self.name == other.name and isinstance(other, Font) + return isinstance(other, Font) and self.name == other.name def __getitem__(self, key): return self.cget(key) diff --git a/Lib/tkinter/test/test_tkinter/test_font.py b/Lib/tkinter/test/test_tkinter/test_font.py new file mode 100644 index 000000000000..56e0c02b1604 --- /dev/null +++ b/Lib/tkinter/test/test_tkinter/test_font.py @@ -0,0 +1,20 @@ +import unittest +import tkinter +from tkinter import font +from test.support import requires, run_unittest + +requires('gui') + +class FontTest(unittest.TestCase): + def test_font_eq(self): + font1 = font.nametofont("system") + font2 = font.nametofont("system") + self.assertIsNot(font1, font2) + self.assertEqual(font1, font2) + self.assertNotEqual(font1, font1.copy()) + self.assertNotEqual(font1, 0) + +tests_gui = (FontTest, ) + +if __name__ == "__main__": + run_unittest(*tests_gui) diff --git a/Misc/NEWS b/Misc/NEWS index 1ccc8b5fc6c7..bfa8bace8cc4 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -52,6 +52,9 @@ Core and Builtins Library ------- +- Issue #1730136: Fix the comparison between a tk.font.Font and an object of + another kind. + - Issue #9441: logging has better coverage for rotating file handlers. - Issue #9865: collections.OrderedDict now has a __sizeof__ method.