Skip to content

gh-75666: Fix Misc.unbind behaviour when funcid is given #17954

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 9 commits into from
Closed

gh-75666: Fix Misc.unbind behaviour when funcid is given #17954

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
95c4404
Fix: https://bugs.python.org/issue31485
GiovaLomba Jan 6, 2020
ac6aeef
Merge pull request #1 from GiovaLomba/issue31485
GiovaLomba Jan 6, 2020
ebbdc6f
📜🤖 Added by blurb_it.
blurb-it[bot] Jan 11, 2020
8e03203
Fix: tests on Ubuntu bionic
GiovaLomba Jan 15, 2020
0583640
Update 2020-01-11-22-09-29.bpo-31485.DSvjsY.rst
GiovaLomba Jan 15, 2020
597ef39
Update __init__.py
terryjreedy Jan 16, 2020
082db44
Fix whitespace, delete extra '.'
terryjreedy Jan 16, 2020
af41122
Leave no-funcid call as it was.
terryjreedy Jan 16, 2020
530e1d5
Remove redundant split
terryjreedy Jan 16, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix: tests on Ubuntu bionic 
Fix `_tkinter.TclError: can't delete Tcl command` on Ubuntu bionic.
Change `bound` to `keep` as suggested.
  • Loading branch information
@GiovaLomba
GiovaLomba authored Jan 15, 2020
commit 8e0320300613dde233eb5eaec18f0d8dd438636f
8 changes: 4 additions & 4 deletions Lib/tkinter/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1387,12 +1387,12 @@ def bind(self, sequence=None, func=None, add=None):
def unbind(self, sequence, funcid=None):
"""Unbind for this widget the event SEQUENCE. if
FUNCID is given, delete the command also."""
bound = ''
funcs = self.tk.call('bind', self._w, sequence, None)
self.tk.call('bind', self._w, sequence, '')
if funcid:
self.deletecommand(funcid)
funcs = self.tk.call('bind', self._w, sequence, None).split('\n')
bound = '\n'.join([f for f in funcs if not f.startswith(f'if {{"[{funcid}')])
self.tk.call('bind', self._w, sequence, bound)
keep = '\n'.join(f for f in funcs.split('\n') if not f.startswith(f'if {{"[{funcid}'))
self.tk.call('bind', self._w, sequence, keep)

def bind_all(self, sequence=None, func=None, add=None):
"""Bind to all widgets at an event SEQUENCE a call to function FUNC.
Expand Down
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy