VimPlug/jedi-fork
1
0
Fork 0
forked from VimPlug/jedi
Code Pull Requests Activity

finally fixed the most prominent bug, that is being featured on the front page of github-jedi -> class methods shouldn't return their params as a completion

Browse Source
This commit is contained in:
David Halter
2012-11-22 13:01:00 +01:00
parent e60a4dde7b
commit 816d01b543
3 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
jedi/api.py
View File

@@ -108,7 +108,7 @@ class Script(object):
for s in scopes: for s in scopes:
# TODO is this really the right way? just ignore the funcs? \ # TODO is this really the right way? just ignore the funcs? \
# do the magic functions first? and then recheck here? # do the magic functions first? and then recheck here?
if not isinstance(s, evaluate.Function): if not s.isinstance(evaluate.Function):
if isinstance(s, imports.ImportPath): if isinstance(s, imports.ImportPath):
if like == 'import': if like == 'import':
l = self.module.get_line(self.pos[0])[:self.pos[1]] l = self.module.get_line(self.pos[0])[:self.pos[1]]

2
jedi/imports.py
View File

@@ -25,7 +25,7 @@ class ModuleNotFound(Exception):
pass pass
class ImportPath(object): class ImportPath(parsing.Base):
""" """
An ImportPath is the path of a `parsing.Import` object. An ImportPath is the path of a `parsing.Import` object.
""" """

3
test/completion/classes.py
View File

@@ -20,6 +20,9 @@ class FindClass():
#? ['ret'] #? ['ret']
TestClass.ret TestClass.ret
#? []
FindClass().find_class.self
# set variables, which should not be included, because they don't belong to the # set variables, which should not be included, because they don't belong to the
# class # class
second = 1 second = 1