Skip to content

bpo-33096 Modify ttk.Treeview.insert#6228

Merged
serhiy-storchaka merged 6 commits into
python:masterfrom
plusminushalf:fix-bpo-33096
Mar 26, 2018
Merged

bpo-33096 Modify ttk.Treeview.insert#6228
serhiy-storchaka merged 6 commits into
python:masterfrom
plusminushalf:fix-bpo-33096

Conversation

@plusminushalf

@plusminushalf plusminushalf commented Mar 25, 2018 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

Allow ttk.Treeview.insert to insert False iid.
Note iid=0 and iid=false would be same

https://bugs.python.org/issue33096

@plusminushalf
bpo-33096 Modify ttk.Treeview.insert
d908060 
Allow ttk.Treeview.insert to insert False iid.
Note iid=0 and iid=false would be same
@plusminushalf plusminushalf force-pushed the fix-bpo-33096 branch 2 times, most recently from da8cef1 to d908060 Compare March 25, 2018 07:34
serhiy-storchaka
serhiy-storchaka reviewed Mar 25, 2018
View reviewed changes

@serhiy-storchaka serhiy-storchaka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A news entry shouldn't be added directly in the NEWS file. Instead it should be added in a separate file in the NEWS.d directory. You can use the blurb utility for creating it.

Comment thread Lib/tkinter/test/test_ttk/test_widgets.py Outdated

@serhiy-storchaka serhiy-storchaka Mar 25, 2018

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to wrap this line, it is not long enough.

Why the second argument of insert() is '' instead of 'end'?

@plusminushalf plusminushalf Mar 25, 2018

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad this should be end only.

@serhiy-storchaka

serhiy-storchaka commented Mar 25, 2018

Copy link
Copy Markdown
Member

Also please add your name in Misc/ACKS and "Patch by your name." at the end of a news entry.

@plusminushalf

plusminushalf commented Mar 25, 2018

Copy link
Copy Markdown
Contributor Author

My name is already in Misc/ACKS should I insert a duplicate entry?

serhiy-storchaka
serhiy-storchaka reviewed Mar 25, 2018
View reviewed changes
Comment thread Misc/NEWS.d/next/Library/2018-03-25-13-18-16.bpo-33096.ofdbe7.rst Outdated

@serhiy-storchaka serhiy-storchaka Mar 25, 2018

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a period at the end.

Comment thread Misc/NEWS.d/next/Library/2018-03-25-13-18-16.bpo-33096.ofdbe7.rst Outdated
@@ -0,0 +1,4 @@
Allow ttk.Treeview.insert to insert False iid.
Note iid=0 and iid=false would be same.

@serhiy-storchaka serhiy-storchaka Mar 25, 2018

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

iid=False

Comment thread Misc/NEWS.d/next/Library/2018-03-25-13-18-16.bpo-33096.ofdbe7.rst Outdated
@@ -0,0 +1,4 @@
Allow ttk.Treeview.insert to insert False iid.

@serhiy-storchaka serhiy-storchaka Mar 25, 2018

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"to insert iid that has a false boolean value."

@serhiy-storchaka

serhiy-storchaka commented Mar 25, 2018

Copy link
Copy Markdown
Member

No, one time is enough. 😄

@serhiy-storchaka serhiy-storchaka added type-bug An unexpected behavior, bug, or error needs backport to 3.6 labels Mar 25, 2018
@serhiy-storchaka serhiy-storchaka merged commit 3ab44c0 into python:master Mar 26, 2018
@miss-islington

miss-islington commented Mar 26, 2018

Copy link
Copy Markdown
Contributor

Thanks @garvitdelhi for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.6, 3.7.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

@bedevere-bot

bedevere-bot commented Mar 26, 2018

Copy link
Copy Markdown

GH-6246 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 26, 2018
@plusminushalf @miss-islington
bpo-33096: Fix ttk.Treeview.insert. (pythonGH-6228)
caa3166 
Allow ttk.Treeview.insert to insert iid that has a false boolean value.
Note iid=0 and iid=False would be same.
(cherry picked from commit 3ab44c0)

Co-authored-by: Garvit Khatri <garvitdelhi@gmail.com>
@miss-islington

miss-islington commented Mar 26, 2018

Copy link
Copy Markdown
Contributor

Sorry, @garvitdelhi and @serhiy-storchaka, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 3ab44c0783eebdff687014f7d14d5dec59b6bd39 2.7

@bedevere-bot

bedevere-bot commented Mar 26, 2018

Copy link
Copy Markdown

GH-6247 is a backport of this pull request to the 3.6 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 26, 2018
@plusminushalf @miss-islington
bpo-33096: Fix ttk.Treeview.insert. (pythonGH-6228)
44de749 
Allow ttk.Treeview.insert to insert iid that has a false boolean value.
Note iid=0 and iid=False would be same.
(cherry picked from commit 3ab44c0)

Co-authored-by: Garvit Khatri <garvitdelhi@gmail.com>
miss-islington added a commit that referenced this pull request Mar 26, 2018
@miss-islington @plusminushalf
bpo-33096: Fix ttk.Treeview.insert. (GH-6228)
a7b1b84 
Allow ttk.Treeview.insert to insert iid that has a false boolean value.
Note iid=0 and iid=False would be same.
(cherry picked from commit 3ab44c0)

Co-authored-by: Garvit Khatri <garvitdelhi@gmail.com>
miss-islington added a commit that referenced this pull request Mar 26, 2018
@miss-islington @plusminushalf
bpo-33096: Fix ttk.Treeview.insert. (GH-6228)
45116d3 
Allow ttk.Treeview.insert to insert iid that has a false boolean value.
Note iid=0 and iid=False would be same.
(cherry picked from commit 3ab44c0)

Co-authored-by: Garvit Khatri <garvitdelhi@gmail.com>
serhiy-storchaka pushed a commit to serhiy-storchaka/cpython that referenced this pull request Mar 31, 2018
@plusminushalf @serhiy-storchaka
[2.7] bpo-33096: Fix ttk.Treeview.insert. (pythonGH-6228)
f5e6287 
Allow ttk.Treeview.insert to insert iid that has a false boolean value.
Note iid=0 and iid=False would be same..
(cherry picked from commit 3ab44c0)

Co-authored-by: Garvit Khatri <garvitdelhi@gmail.com>
@bedevere-bot

bedevere-bot commented Mar 31, 2018

Copy link
Copy Markdown

GH-6326 is a backport of this pull request to the 2.7 branch.

serhiy-storchaka added a commit that referenced this pull request Mar 31, 2018
@serhiy-storchaka @plusminushalf
[2.7] bpo-33096: Fix ttk.Treeview.insert. (GH-6228) (GH-6326)
e80a232 
Allow ttk.Treeview.insert to insert iid that has a false boolean value.
Note iid=0 and iid=False would be same.
(cherry picked from commit 3ab44c0)

Co-authored-by: Garvit Khatri <garvitdelhi@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@serhiy-storchaka serhiy-storchaka serhiy-storchaka approved these changes

Assignees

No one assigned

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@plusminushalf @serhiy-storchaka @miss-islington @bedevere-bot @the-knights-who-say-ni