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 139692 - Some characters cannot be given as the theme name
Some characters cannot be given as the theme name
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: [obsolete] theme-manager
2.9.x
Other All
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-04-11 02:14 UTC by Tristan Alexander Mc Leay
Modified: 2006-04-26 20:08 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
this is the patch to confirming the user,attaching a dialog box. (1.76 KB, patch)
2005-03-15 13:36 UTC, probir
rejected Details | Review
this patch accepts the charecters which include ' " / ? * \ | as the theme-name (1.30 KB, patch)
2005-03-31 08:47 UTC, probir
needs-work Details | Review

Description Tristan Alexander Mc Leay 2004-04-11 02:14:22 UTC
1. Open the themes control panel.
2. Choose 'Save Theme...'.
3. Type in some characters. Include ' " / ? * \ | in the characters you type.
4. Observe how the characters are not entered; absolutely no feedback is given
and no explanation as to why is around. It looks like your keyboard's broken or
badly configured. This makes it impossible to give a theme a name like
'Tristan's theme'.

(Only confirmed in Gnome 2.6 on Linux.)
Comment 1 Kjartan Maraas 2004-04-14 17:29:48 UTC
Confirming. It's ok to escape the unwanted chars I think, but just making it
impossible to type them in without explanation is a bit weird.
Comment 2 Kjartan Maraas 2004-04-14 17:35:50 UTC
Confirming. 
Comment 3 probir 2005-03-15 13:36:38 UTC
Created attachment 38748 [details] [review]
this is the patch to confirming the user,attaching a dialog box.

hi,
i have attached a dialogbox to confirm the user not to use those of invalid
charecters.

thanking ,

probir paul
Comment 4 shakti 2005-03-18 04:38:25 UTC
I don't think showing an error dialog is a good solution rather it should allow
to type those chars and should be able to save them.

For example: we can create a file named "Tristan's theme" by "vi Tristan\'s\
theme". So, theme manager should be able to handle this.
Comment 5 probir 2005-03-31 08:47:24 UTC
Created attachment 39482 [details] [review]
this patch accepts the charecters which include  ' " / ? * \ | as the theme-name

patch to accept the charecter like ' " / ? * \ | as the theme-name
Comment 6 Thomas Wood 2006-04-25 21:48:42 UTC
The patch doesn't include any escaping mechanism for the backslash (/) character, which means if the user enters a theme name with a backslash, it doesn't get saved properly. I don't think there are any similar problems with the other characters when used as a part of the filename.
Comment 7 Thomas Wood 2006-04-25 22:36:55 UTC
I am, of course, talking about forward slash (/), not backslash (\).
Comment 8 Thomas Wood 2006-04-26 20:08:49 UTC
2006-04-26  Thomas Wood  <thos@gnome.org>

	* gnome-theme-save.c: (str_remove_slash),
	(setup_directory_structure), (write_theme_to_disk),
	(gnome_theme_save_show_dialog):

	Fix bug 139692 - Some characters cannot be given as the theme name