Page MenuHome

Bug Fix: Revision in parameter list of execute() in console_shell.py
AbandonedPublic

Authored by Campbell Barton (campbellbarton) on Dec 27 2013, 6:30 PM.

Details

Summary

Shell console module in blender UI console screen doesn't work. How to test it:

import console_shell
bpy.ops.console.language("shell")
ls

gives you following error:

Traceback (most recent call last):
  File "/home/user/blender-2.69-linux-glibc211-x86_64/2.69/scripts/startup/bl_operators/console.py", line 54, in execute
    return execute(context, self.interactive)
TypeError: execute() takes 1 positional argument but 2 were given

With submitted diff it works without problem, I figured console_python.execute() expects (context, is_interactive) but console_shell.execute() expects only (context), so changed console_shell.execute in order to reflect the same parameter expectation.

It works as expected with this change.

System: archlinux x86_64 & blender 2.69

Diff Detail

Event Timeline

If that has been accepted, why is is still open / not committed?

If that has been accepted, why is is still open / not committed?

As far as I can remember I both commited this report and reported the issue to Campbell on IRC. He didn't realize my bug report and accidentally committed the fix himself. He then realized the bug report and accepted it. So the fix is indeed in the repo but not via this commit. To sum up, the issue is fixed. It can be closed. :)

This revision now requires review to proceed.Aug 2 2014, 8:25 AM