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 161905 - Don't hard code file name into Nautilus message
Don't hard code file name into Nautilus message
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: File and Folder Operations
unspecified
Other All
: Normal normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-12-21 17:58 UTC by Christian Rose
Modified: 2005-03-25 14:14 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
Proposed patch (1.85 KB, patch)
2005-02-25 02:07 UTC, Michael Terry
accepted-commit_after_freeze Details | Review

Description Christian Rose 2004-12-21 17:58:43 UTC
#. name, stock id, label
#: src/file-manager/fm-directory-view.c:5695
msgid "Run or manage scripts from ~/.gnome2/nautilus-scripts"

Please do NOT hard code file names or directory names into translateable
messages! The next time the path or file name changes, all 72 translations will
have to be updated. That's certainly not good design, and a pain for translators.
Comment 1 Luis Villa 2005-01-03 05:37:15 UTC
Can someone on the i18n team whip up a quick patch for this, you think?

[Not marking high priority because while I understand the sentiment, I'm pretty
sure that location has not changed since nov/dec. 2001.]
Comment 2 Christian Rose 2005-01-03 11:07:15 UTC
It changed recently; previously the message was pointing at ~/Nautilus/scripts
something. However, I too suspect that it is unlikely to soon change again, but
still, when it does I and many other translators are going to start pulling our
hairs again in frustration. Furthermore, in the mean time there is always the
danger of some translator actually translating the path/filename by accident.
Comment 3 Michael Terry 2005-02-25 02:07:19 UTC
Created attachment 37923 [details] [review]
Proposed patch

This patch is slightly complicated by the fact that all the other menu actions
are declared in a static array.  In order to dynamically create the tooltip for
the scripts action, it had to be brought out of the nice array and created
manually.
Comment 4 Martin Wehner 2005-03-21 21:02:59 UTC
Comment on attachment 37923 [details] [review]
Proposed patch

Looks good - Perhaps using char instead of gchar would make it more consistant
with the rest of fm-directory-view.
It'll have to wait till the branch due to the string change though.
Thanks.
Comment 5 Michael Terry 2005-03-21 21:34:49 UTC
Thanks; I don't have CVS write access though.  Please commit this for me after
the branch.
Comment 6 Martin Wehner 2005-03-25 14:14:04 UTC
2005-03-25  Martin Wehner  <martin.wehner@gmail.com>

	* src/file-manager/fm-directory-view.c: (real_merge_menus):
	Remove hard coded script directory path from tooltip message.
	Fixes bug #161905.
	
	Patch from Michael Terry <mike@mterry.name>