After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 701085 - terminal: Change Directory command is broken
terminal: Change Directory command is broken
Status: RESOLVED FIXED
Product: gedit-plugins
Classification: Other
Component: General
git master
Other Linux
: Normal normal
: ---
Assigned To: Gedit maintainers
Gedit maintainers
Depends on:
Blocks:
 
 
Reported: 2013-05-27 13:51 UTC by Adam Dingle
Modified: 2019-03-23 20:43 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Adam Dingle 2013-05-27 13:51:24 UTC
The embedded terminal is supposed to have a Change Directory context menu item that performs a cd to the active document's directory.  You can see the code for this in change_directory() in terminal.py.  Unfortunately this is currently broken.  When you first start gedit, the Change Directory context menu item is visible but disabled (which is reasonable, since no document is active).  But once you open any document, Change Directory vanishes from the terminal's context menu.
Comment 1 Adam Dingle 2013-05-27 15:08:23 UTC
Ah - when Change Directory is missing from the context menu we see this on the console:

Traceback (most recent call last):
  • File "/home/adam/local/lib/gedit/plugins/terminal.py", line 321 in on_panel_populate_popup
    path = self.get_active_document_directory()
  • File "/home/adam/local/lib/gedit/plugins/terminal.py", line 314 in get_active_document_directory
    if location and location.has_uri_scheme(location, "file"):
  • File "/usr/lib/python3/dist-packages/gi/types.py", line 113 in function
    return info.invoke(*args, **kwargs)
TypeError: has_uri_scheme() takes exactly 2 arguments (3 given)

The fix is trivial, so I've simply committed it.
Comment 2 Ignacio Casal Quinteiro (nacho) 2013-05-27 15:11:15 UTC
Please also backport it.
Comment 3 Adam Dingle 2013-05-27 15:22:40 UTC
Done.