➜
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2019-08-23 13:27 byterry.reedy, last changed2022-04-11 14:59 byadmin. This issue is nowclosed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 15452 | merged | taleinat,2019-08-24 08:17 | |
| PR 15484 | merged | miss-islington,2019-08-25 05:53 | |
| PR 15485 | merged | miss-islington,2019-08-25 05:53 | |
| Messages (6) | |||
|---|---|---|---|
| msg350281 -(view) | Author: Terry J. Reedy (terry.reedy)* | Date: 2019-08-23 13:27 | |
If one opens configdialog when there is no Shell and then hits OK, the dialog fails to close. The following Squeezer-related traceback appears in associated console when there is one.Exception in Tkinter callbackTraceback (most recent call last): File "F:\dev\3x\lib\tkinter\__init__.py", line 1885, in __call__ return self.func(*args) File "F:\dev\3x\lib\idlelib\configdialog.py", line 172, in ok self.apply() File "F:\dev\3x\lib\idlelib\configdialog.py", line 186, in apply self.activate_config_changes() File "F:\dev\3x\lib\idlelib\configdialog.py", line 240, in activate_config_changes klass.reload() File "F:\dev\3x\lib\idlelib\squeezer.py", line 222, in reload instance.load_font() File "F:\dev\3x\lib\idlelib\squeezer.py", line 318, in load_font Font(text, font=text.cget('font')).measure('0') File "F:\dev\3x\lib\idlelib\delegator.py", line 10, in __getattr__ attr = getattr(self.delegate, name) # May raise AttributeErrorAttributeError: 'NoneType' object has no attribute 'cget'Either the squeezer instance should be destroyed and removed along with Shell or, if keeping it is intentional, it should be removed from the update list and reinstated or if not doing that is intentional, it must either check 'if text is not None' before the access or catch the attribute error after. Tal, which? | |||
| msg350356 -(view) | Author: Tal Einat (taleinat)* | Date: 2019-08-24 08:19 | |
See PRGH-15452 with a fix. | |||
| msg350428 -(view) | Author: Tal Einat (taleinat)* | Date: 2019-08-25 05:53 | |
New changesetd4b4c00b57d24f6ee2cf3a96213406bb09953df3 by Tal Einat in branch 'master':bpo-37929: IDLE: avoid Squeezer-related config dialog crashes (GH-15452)https://github.com/python/cpython/commit/d4b4c00b57d24f6ee2cf3a96213406bb09953df3 | |||
| msg350430 -(view) | Author: miss-islington (miss-islington) | Date: 2019-08-25 06:17 | |
New changesetf2b468dd6d0bdbe2e87c0ca7515800a115e95e54 by Miss Islington (bot) in branch '3.7':bpo-37929: IDLE: avoid Squeezer-related config dialog crashes (GH-15452)https://github.com/python/cpython/commit/f2b468dd6d0bdbe2e87c0ca7515800a115e95e54 | |||
| msg350431 -(view) | Author: miss-islington (miss-islington) | Date: 2019-08-25 06:18 | |
New changesete266d062e017b122b9741db2bd5fb99742378623 by Miss Islington (bot) in branch '3.8':bpo-37929: IDLE: avoid Squeezer-related config dialog crashes (GH-15452)https://github.com/python/cpython/commit/e266d062e017b122b9741db2bd5fb99742378623 | |||
| msg350433 -(view) | Author: Tal Einat (taleinat)* | Date: 2019-08-25 06:25 | |
Fixed. | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:19 | admin | set | github: 82110 |
| 2019-08-25 06:25:32 | taleinat | set | status: open -> closed resolution: fixed messages: +msg350433 stage: patch review -> resolved |
| 2019-08-25 06:18:04 | miss-islington | set | messages: +msg350431 |
| 2019-08-25 06:17:23 | miss-islington | set | nosy: +miss-islington messages: +msg350430 |
| 2019-08-25 05:53:47 | miss-islington | set | pull_requests: +pull_request15172 |
| 2019-08-25 05:53:11 | miss-islington | set | pull_requests: +pull_request15171 |
| 2019-08-25 05:53:01 | taleinat | set | messages: +msg350428 |
| 2019-08-24 08:19:11 | taleinat | set | messages: +msg350356 |
| 2019-08-24 08:17:27 | taleinat | set | keywords: +patch stage: needs patch -> patch review pull_requests: +pull_request15144 |
| 2019-08-23 13:27:44 | terry.reedy | create | |