From 19fa320c88812444e5d95378999cfb5333172a20 Mon Sep 17 00:00:00 2001
From: Dave Halter <davidhalter88@gmail.com>
Date: Sat, 11 Jan 2014 01:48:59 +0100
Subject: [PATCH] a hopefully simple integration of PyObject into Instance

---
 jedi/evaluate/__init__.py          |  9 ++++--
 jedi/evaluate/compiled/__init__.py | 47 ++++++++++++++++++------------
 jedi/evaluate/representation.py    |  2 ++
 3 files changed, 36 insertions(+), 22 deletions(-)

diff --git a/jedi/evaluate/__init__.py b/jedi/evaluate/__init__.py
index c9791d86..978202a4 100644
--- a/jedi/evaluate/__init__.py
+++ b/jedi/evaluate/__init__.py
@@ -361,7 +361,6 @@ class Evaluator(object):
                     result = typ.get_index_types(current)
             elif current.type not in [pr.Array.DICT]:
                 # Scope must be a class or func - make an instance or execution.
-                debug.dbg('exe', typ)
                 result = self.execute(typ, current)
             else:
                 # Curly braces are not allowed, because they make no sense.
@@ -382,13 +381,17 @@ class Evaluator(object):
         if obj.isinstance(er.Function):
             obj = obj.get_decorated_func()
 
+        debug.dbg('execute:', obj, params)
         try:
             return stdlib.execute(self, obj, params)
         except stdlib.NotInStdLib:
             pass
 
         if isinstance(obj, compiled.PyObject):
-            return list(obj.execute(self, params))
+            if obj.is_executable_class():
+                return [er.Instance(self, obj, params)]
+            else:
+                return list(obj.execute_function(self, params))
         elif obj.isinstance(er.Class):
             # There maybe executions of executions.
             return [er.Instance(self, obj, params)]
@@ -409,7 +412,7 @@ class Evaluator(object):
             else:
                 stmts = er.FunctionExecution(self, obj, params).get_return_types(evaluate_generator)
 
-            debug.dbg('execute: %s in %s' % (stmts, obj))
+            debug.dbg('execute result: %s in %s' % (stmts, obj))
             return imports.strip_imports(self, stmts)
 
     def goto(self, stmt, call_path=None):
diff --git a/jedi/evaluate/compiled/__init__.py b/jedi/evaluate/compiled/__init__.py
index 112a6141..390716e2 100644
--- a/jedi/evaluate/compiled/__init__.py
+++ b/jedi/evaluate/compiled/__init__.py
@@ -19,15 +19,16 @@ from . import fake
 # if not hasattr(func, "__name__"):
 
 class PyObject(Base):
+    # comply with the parser
+    start_pos = 0, 0
+    asserts = []
+
     def __init__(self, obj, parent=None, instantiated=False):
         self.obj = obj
         self.parent = parent
         self.instantiated = instantiated
         self.doc = inspect.getdoc(obj)
 
-        # comply with the parser
-        self.start_pos = 0, 0
-
     def __repr__(self):
         return '<%s: %s>' % (type(self).__name__, self.obj)
 
@@ -52,6 +53,9 @@ class PyObject(Base):
                 or inspect.ismethoddescriptor(cls):
             return 'def'
 
+    def is_executable_class(self):
+        return inspect.isclass(self.obj)
+
     @underscore_memoization
     def _cls(self):
         # Ensures that a PyObject is returned that is not an instance (like list)
@@ -64,28 +68,33 @@ class PyObject(Base):
         for name in dir(cls.obj):
             yield PyName(cls, name)
 
+    def instance_names(self):
+        # TODO REMOVE (temporary until the Instance method is removed)
+        return self.get_defined_names()
+
+    def get_subscope_by_name(self, name):
+        if name in dir(self._cls().obj):
+            return PyName(self._cls, name).parent
+        else:
+            raise KeyError("CompiledObject doesn't have an attribute '%s'." % name)
+
     @property
     def name(self):
         # might not exist sometimes (raises AttributeError)
         return self._cls().obj.__name__
 
-    def execute(self, evaluator, params):
-        t = self.type()
-        if t == 'class':
-            if not self.instantiated:
-                yield PyObject(self.obj, self.parent, True)
-        elif t == 'def':
-            for name in self._parse_function_doc()[1].split():
-                try:
-                    bltn_obj = create(getattr(_builtins, name), builtin, module=builtin)
-                except AttributeError:
-                    continue
+    def execute_function(self, evaluator, params):
+        for name in self._parse_function_doc()[1].split():
+            try:
+                bltn_obj = create(getattr(_builtins, name), builtin, module=builtin)
+            except AttributeError:
+                continue
+            else:
+                if isinstance(bltn_obj, PyObject):
+                    yield bltn_obj
                 else:
-                    if isinstance(bltn_obj, PyObject):
-                        yield bltn_obj
-                    else:
-                        for result in evaluator.execute(bltn_obj, params):
-                            yield result
+                    for result in evaluator.execute(bltn_obj, params):
+                        yield result
 
     def get_self_attributes(self):
         return []  # Instance compatibility
diff --git a/jedi/evaluate/representation.py b/jedi/evaluate/representation.py
index bc834834..79916736 100644
--- a/jedi/evaluate/representation.py
+++ b/jedi/evaluate/representation.py
@@ -87,6 +87,8 @@ class Instance(use_metaclass(CachedMetaClass, Executable)):
             n.names = n.names[1:]
             names.append(InstanceElement(self._evaluator, self, n))
 
+        if isinstance(self.base, compiled.PyObject):
+            return []
         names = []
         # This loop adds the names of the self object, copies them and removes
         # the self.