BuiltinUnknown does not have a get_returned_object method, but it's called about rope HOT 7 CLOSED

(I wasn't signed in, but this issue is one that I created)

from rope.

Hi,

angri, I think something like the attached patch should fix
it (I have not tested it).

Thanks,
Ali

from rope.

That patch seems to address the issue.

from rope.

Hi,

Thanks for reporting and testing it. As you reported in
issue #6, AbstractFunction should be pyobjects.AbstractFunction
in my patch. So, angri please update the patch before applying.

Thanks,
Ali

from rope.

Could you please provide some testcase for this issue? I don't know how to reproduce crash.

from rope.

Hi angri,

The first traceback seems to be related to the new
BuiltinUnknown class. It triggered a bug in evaluate.py.
_call_function() should only call get_returned_object()
for instances of AbstractFunction. So the patch I
submitted checked that. But that patch had a small mistake;
AbstractFunction should have been pyobjects.AbstractFunction.
I'm attaching a fixed patch (not tested).

Thanks,
Ali

PS. sorry for changing issue state everytime I post! my
small text-based browser does it automatically ;-)

from rope.

Patch applied.

from rope.