• 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
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?


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.

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.


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.

Alissa Sabre added a comment - 08/Nov/07 03:06 PM
The issue persists in the latest (1.18.4) viewer.

Alissa Sabre added a comment - 14/Nov/07 05:50 AM
Still unfixed in 1.18.5.0

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.


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.


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.