• All submissions to this site are governed by Second Life Project Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.
Issue Details (XML | Word | Printable)

Key: VWR-1843
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Ramzi Linden
Reporter: Alissa Sabre
Votes: 4
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
1. Second Life Viewer - VWR

Inappropriate default folder name for Make Outfit

Created: 20/Jul/07 07:59 PM   Updated: 06/May/08 07:46 AM
Return to search
Component/s: Internationalization, Inventory, User Interface
Affects Version/s: 1.20 Release Candidate, 1.18.0, 1.18.1.2, 1.18.2.0, 1.18.3.5, 1.18.4 Release Candidate, 1.18.4.3, 1.18.5 Release Candidate, 1.19.0.5, 1.19.1.4
Fix Version/s: 1.20 Release Candidate

File Attachments: 1. Text File newoutfit.patch (1 kB)
Image Attachments:

1. Figures.png
(18 kB)
Environment: Observed on Windows XP SP2, but should be a universal issue.
Issue Links:
Relates
 

Source Version: 1.20.2.85278
Linden Lab Issue ID: SL-53732
Patch attached: Patch attached
Linden Lab Internal Branch: Branch_1-20-Viewer


 Description  « Hide
Edit > Appearances... > Make Outfit pops up a Make New Outfit dialog box. It contains Folder Name field filled with a default name.

If you are running the viewer in English UI, the default name is "New Outfit" and it works fine. However, if you are running in Portuguese UI, the default name is "Novo vestuário", and it creates a folder of name "Novo vestu rio" instead of expected "Novo vestuário". If you are running in Japanese UI, the default name is entirely in Japanese characters that I can't report on JIRA (see WEB-67 , and it creates an empty folder name.

In a long term, the best solution may be to allow non-ASCII characters in inventory item names, but for a short term, the default folder name should be something that is allowed as a folder name. I propose uses of English-based folder names in all languages.

REPRO (Observed behaviour):

  • Start the viewer in Portuguese UI.
  • Choose (Portuguese counterparts of) "Edit > Appearances..." and click on "Make Outfit".
  • The default folder name reads "Novo vestuário". (Fig. 1)
  • Click on "Save".
  • On your Inventory, you get a folder of name "Novo vestu rio". "á" is removed. (Fig. 2)
  • Start the viewer again, in Japanese UI this time.
  • Do the same thing above. The default folder name consists of some Japanese characters. (Fig. 3)
  • Click on "Save" and you get a folder of an empty name.

EXPECTED BEHAVIOUR:

  • The default folder name is "New Outfit" in all languages, and the created folder has that name.

 All   Comments   Change History      Sort Order: Ascending order - Click to sort in descending order
[ Permlink | « Hide ]
Benja Kepler added a comment - 02/Aug/07 02:40 PM
It looks from figure 1.,
that the viewer is handling the accented character correctly during editing Appearance,
but not in displaying the contents of the Inventory.

Does the Inventory display any accented characters?


[ Permlink | « Hide ]
Paul Churchill added a comment - 05/Aug/07 01:24 AM
A quick test indicates that renaming an inventory item ( I used a notecard) does not permit entry of (spanish) accented characters.

Paul Churchill made changes - 05/Aug/07 03:54 AM
Field Original Value New Value
Affects Version/s 1.18.1.2 [ 10120 ]
[ Permlink | « Hide ]
Alissa Sabre added a comment - 12/Aug/07 12:08 AM
My understanding is that non-ASCII characters are not allowed in names for any inventory items. I think it is intentional and is by design. The restriction is primarily by the server side code. We could ask LL to change it, but that's another issue.

This bug filing is on the default folder name of a new outfit folder. I believe that, whatever restriction is enforced for inventory item names, the default names for inventory items should be something that are allowed under the restriction.


[ Permlink | « Hide ]
Melanie Milland added a comment - 24/Aug/07 05:48 AM
As inventory items, even folders, can be given to others, the names must be restricted to the character set that is known to be displayable by all.
My vote is to not internationalize default folder or item names.

lindenrobot made changes - 31/Aug/07 02:23 PM
Linden Lab Issue ID SL-53732
Torley Linden made changes - 04/Sep/07 10:28 AM
Assignee WorkingOnIt Linden [ WorkingOnIt Linden ]
[ Permlink | « Hide ]
Alissa Sabre added a comment - 08/Nov/07 03:06 PM
The issue persists in the latest (1.18.4) viewer.

Alissa Sabre made changes - 08/Nov/07 03:06 PM
Affects Version/s 1.18.2.0 [ 10130 ]
Affects Version/s 1.18.4.3 [ 10190 ]
Affects Version/s 1.18.3.5 [ 10160 ]
[ Permlink | « Hide ]
Alissa Sabre added a comment - 14/Nov/07 05:50 AM
Still unfixed in 1.18.5.0

Alissa Sabre made changes - 14/Nov/07 05:50 AM
Affects Version/s 1.18.5 Release Candidate [ 10200 ]
Affects Version/s 1.18.4 Release Candidate [ 10180 ]
Rob Linden made changes - 22/Dec/07 01:34 AM
Workflow jira [ 14035 ] jira-2007-12-21 [ 22490 ]
Rob Linden made changes - 22/Dec/07 01:58 AM
Workflow jira [ 22490 ] jira-2007-12-21 [ 23821 ]
Rob Linden made changes - 22/Dec/07 02:59 PM
Workflow jira-2007-12-21 [ 23821 ] jira-2007-12-22 [ 30543 ]
Rob Linden made changes - 22/Dec/07 03:21 PM
Workflow jira-2007-12-21 [ 30543 ] jira-2007-12-22 [ 32144 ]
Rob Linden made changes - 22/Dec/07 08:05 PM
Workflow jira-2007-12-22 [ 32144 ] jira-2007-12-22a [ 37438 ]
Rob Linden made changes - 22/Dec/07 08:27 PM
Workflow jira-2007-12-22 [ 37438 ] jira-2007-12-22a [ 38728 ]
Rob Linden made changes - 22/Dec/07 09:20 PM
Workflow jira-2007-12-22 [ 38728 ] jira-2007-12-22a [ 41351 ]
Rob Linden made changes - 22/Dec/07 09:42 PM
Workflow jira-2007-12-22 [ 41351 ] jira-2007-12-22a [ 42595 ]
Alissa Sabre made changes - 18/Apr/08 10:26 PM
Attachment newoutfit.patch [ 16212 ]
[ Permlink | « Hide ]
Alissa Sabre added a comment - 18/Apr/08 10:32 PM
This issue persists in 1.19.0, 1.19.1, and 1.20 RCs (0 to 2).

I produced a patch to fix the issue. (newoutfit..patch) It disables translated texts for "New Outfit" for languages in case the texts contain non ASCII characters. As a result, Japanese (ja), Korean (ko), and Chinese (zh) translations use an English name ("New Outfit") for the default folder name, once the attached patch is applied. The patch doesn't change behaviour of other languages, since those words for translated "New Outfit" don't contain non ASCII characters.

NOTE that the attached patch file contain UTF-8 characters. You need a Uniocde capable tool to handle it.


Alissa Sabre made changes - 18/Apr/08 10:32 PM
Source Version 1.20.2.85278
Affects Version/s 1.19.1.4 [ 10290 ]
Affects Version/s 1.19.0.5 [ 10260 ]
Patch attached [Patch attached]
Affects Version/s 1.20 Release Candidate [ 10300 ]
Alissa Sabre made changes - 19/Apr/08 07:43 AM
Link This issue is related to by VWR-6607 [ VWR-6607 ]
[ Permlink | « Hide ]
Ramzi Linden added a comment - 25/Apr/08 06:17 PM
Thanks for reporting this up and through Viewer 1.20!

I have applied the patch which is the short-term fix – simply stop offering a default folder name which is known to not save to Inventory in that form.


Ramzi Linden made changes - 25/Apr/08 06:17 PM
Status Open [ 1 ] Fix Pending [ 10001 ]
Linden Lab Internal Branch Branch_1-20-Viewer
Assignee WorkingOnIt Linden [ WorkingOnIt Linden ] Ramzi Linden [ Ramzi Linden ]
[ Permlink | « Hide ]
Alissa Sabre added a comment - 06/May/08 07:46 AM
I tested this issue on 1.20 RC5 (1.20.5.86279) and found that it is fixed.
Thank you for the fix.

Alissa Sabre made changes - 06/May/08 07:46 AM
Status Fix Pending [ 10001 ] Closed [ 6 ]
Fix Version/s 1.20 Release Candidate [ 10300 ]
Resolution Fixed [ 1 ]
Sue Linden made changes - 13/Nov/08 11:41 AM
Workflow jira-2007-12-22a [ 42595 ] jira-2008-11-14 [ 75448 ]
Sue Linden made changes - 13/Nov/08 06:39 PM
Workflow jira-2008-11-14 [ 75448 ] jira-2008-11-14a [ 133356 ]

Powered by Atlassian JIRA the Professional Issue Tracker. (Enterprise Edition, Version: 3.13.2-#335) - Bug/feature request - Atlassian news - Contact Administrators