From 14c9ed88ca4f055499016b0b18b8d20d052fb672 Mon Sep 17 00:00:00 2001
From: Dave Halter <davidhalter88@gmail.com>
Date: Fri, 10 Jan 2014 15:14:55 +0100
Subject: [PATCH] fixed the getattr/__getattr__ stuff

---
 jedi/evaluate/__init__.py  | 7 +++----
 jedi/evaluate/compiled.py  | 7 -------
 jedi/evaluate/finder.py    | 2 +-
 jedi/evaluate/stdlib.py    | 1 -
 test/completion/classes.py | 2 +-
 5 files changed, 5 insertions(+), 14 deletions(-)

diff --git a/jedi/evaluate/__init__.py b/jedi/evaluate/__init__.py
index 60d74ef3..81e0e5af 100644
--- a/jedi/evaluate/__init__.py
+++ b/jedi/evaluate/__init__.py
@@ -258,16 +258,15 @@ class Evaluator(object):
                 call.stmt.parent = loop
                 result += self.eval_statement(call.stmt)
             else:
-                if isinstance(call, compiled.PyName):
-                    print call, call.parent
-                    result.append(call.parent)
-                elif isinstance(call, pr.Lambda):
+                if isinstance(call, pr.Lambda):
                     result.append(er.Function(self, call))
                 # With things like params, these can also be functions...
                 elif isinstance(call, pr.Base) and call.isinstance(
                         er.Function, er.Class, er.Instance, iterable.ArrayInstance):
                     result.append(call)
                 # The string tokens are just operations (+, -, etc.)
+                elif isinstance(call, compiled.PyObject):
+                    result.append(call)
                 elif not isinstance(call, (str, unicode)):
                     if isinstance(call, pr.Call) and str(call.name) == 'if':
                         # Ternary operators.
diff --git a/jedi/evaluate/compiled.py b/jedi/evaluate/compiled.py
index a7c4f9a9..82083fee 100644
--- a/jedi/evaluate/compiled.py
+++ b/jedi/evaluate/compiled.py
@@ -91,9 +91,6 @@ class PyName(object):
         self._obj = obj
         self._name = name
         self.start_pos = 0, 0  # an illegal start_pos, to make sorting easy.
-        #if not type(name) is str:
-        #    print obj, name
-        #    raise NotImplementedError()
 
     def __repr__(self):
         return '<%s: (%s).%s>' % (type(self).__name__, repr(self._obj.obj), self._name)
@@ -223,7 +220,3 @@ magic_function_class = PyObject(type(load_module), parent=builtin)
 
 def create(obj):
     return PyObject(obj, builtin)
-
-
-def name_from_string(string):
-    return PyName(create(string), string)
diff --git a/jedi/evaluate/finder.py b/jedi/evaluate/finder.py
index adf90e3b..007e202c 100644
--- a/jedi/evaluate/finder.py
+++ b/jedi/evaluate/finder.py
@@ -84,7 +84,7 @@ class NameFinder(object):
         """Checks for both __getattr__ and __getattribute__ methods"""
         result = []
         # str is important to lose the NamePart!
-        name = compiled.name_from_string(self.name_str)
+        name = compiled.create(str(self.name_str))
         with common.ignored(KeyError):
             result = inst.execute_subscope_by_name('__getattr__', [name])
         if not result:
diff --git a/jedi/evaluate/stdlib.py b/jedi/evaluate/stdlib.py
index 4f1feca7..3d31f30d 100644
--- a/jedi/evaluate/stdlib.py
+++ b/jedi/evaluate/stdlib.py
@@ -51,7 +51,6 @@ def builtins_getattr(evaluator, obj, params):
 
         for name in names:
             s = unicode, str
-            print name
             if isinstance(name, compiled.PyObject) and isinstance(name.obj, s):
                 stmts += evaluator.follow_path(iter([name.obj]), [obj], obj)
             else:
diff --git a/test/completion/classes.py b/test/completion/classes.py
index 9ae49633..44967f6a 100644
--- a/test/completion/classes.py
+++ b/test/completion/classes.py
@@ -339,7 +339,7 @@ class Wrapper2():
         return getattr(Base(), name)
 
 #? int()
-Wrapper(Base()).ret
+Wrapper(Base()).ret(3)
 
 #? int()
 Wrapper2(Base()).ret(3)