classification
Title: Class Browser doesn't show internal classes
Type: enhancement Stage: test needed
Components: IDLE Versions: Python 3.7, Python 3.6
process
Status: open Resolution:
Dependencies: 6691 Superseder:
Assigned To: terry.reedy Nosy List: ajaksu2, csabella, gpolo, roger.serwy, terry.reedy
Priority: high Keywords: patch

Created on 2006-12-09 17:38 by taleinat, last changed 2017-07-09 00:02 by csabella.

Files
File name Uploaded Description Edit
classbrowser_nesting_support.diff gpolo, 2009-08-12 22:51 review
Pull Requests
URL Status Linked Edit
PR 2573 open csabella, 2017-07-04 23:41
Messages (8)
msg30777 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2006-12-09 17:38
If I define a class within a class, like this:

class A:
    class B:
        pass
    
    def foo(self):
        pass

The class browser shows that A contains foo, but it doesn't show B at all.
msg84594 - (view) Author: Daniel Diniz (ajaksu2) Date: 2009-03-30 17:39
Confirmed on trunk and py3k.
msg89549 - (view) Author: Roger Serwy (roger.serwy) * (Python committer) Date: 2009-06-20 20:48
The class browser relies on the pyclbr module to scan the code. This
module doesn't support classes within classes. Both pyclbr and IDLE's
class browser need to be modified.
msg91499 - (view) Author: Guilherme Polo (gpolo) * (Python committer) Date: 2009-08-12 16:43
The attached patch adds support for nested objects in pyclbr and also in
ClassBrowser.

I have yet to find an optimal way to test this on test_pyclbr (I did
some things here, none very nice), so you will notice tests are missing
on this patch.
msg91509 - (view) Author: Guilherme Polo (gpolo) * (Python committer) Date: 2009-08-12 22:51
I've removed the previous patch from here and attached an updated one on
issue6691.

Now the patch attached here only contains changes related to
idlelib.ClassBrowser.
msg250596 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2015-09-14 02:29
Class Browser is now a module browser as it includes functions not in classes. I guess pyclbr does this. Enhancing this for current versions would require adding a private copy of enhanced pyclbr to idlelib.
msg297297 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2017-06-29 19:37
I believe the patch for #6691 will also add detection of nested functions.  If  so, the IDLE patch must also make use of that new information.

If the patch for #6691 is only applied to 3.7, I want to put a temporary copy of the revised pyclbr in 3.6 idlelib, perhaps as _pyclbr, and change the 3.6 import accordingly.
msg297974 - (view) Author: Cheryl Sabella (csabella) * Date: 2017-07-09 00:02
I created issue 30881 to add docstrings to browser.py to make creating user tests easier.
History
Date User Action Args
2017-07-09 00:02:55csabellasetmessages: + msg297974
2017-07-04 23:41:11csabellasetpull_requests: + pull_request2644
2017-06-29 19:37:13terry.reedysetpriority: normal -> high

nosy: + csabella
versions: + Python 3.7, - Python 2.7, Python 3.5
messages: + msg297297

assignee: terry.reedy
2015-09-14 02:29:36terry.reedysetpriority: low -> normal
versions: + Python 2.7, Python 3.5, Python 3.6, - Python 3.2
nosy: + terry.reedy

messages: + msg250596
2014-02-04 12:05:48taleinatsetnosy: - taleinat
2010-08-25 14:37:43BreamoreBoysetversions: + Python 3.2, - Python 3.1, Python 2.7
2009-08-12 22:51:23gpolosetfiles: - show_nested_objects.diff
2009-08-12 22:51:14gpolosetfiles: + classbrowser_nesting_support.diff

dependencies: + Support for nested classes and function for pyclbr
messages: + msg91509
2009-08-12 16:43:57gpolosetfiles: + show_nested_objects.diff

nosy: + gpolo
messages: + msg91499

keywords: + patch
2009-06-20 20:48:19roger.serwysetnosy: + roger.serwy
messages: + msg89549
2009-03-30 17:39:48ajaksu2setpriority: normal -> low

type: enhancement
versions: + Python 3.1, Python 2.7, - Python 2.6
nosy: + ajaksu2

messages: + msg84594
stage: test needed
2006-12-09 17:38:55taleinatcreate