0007219: Export Permission: Default Library Textures are not exportable - MantisBT

Mantis Bug Tracker

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

ID

Project

Category

View Status

Date Submitted

Last Update

0007219

opensim

[GRID] Inventory Service

public

2014-06-16 14:32

2014-06-27 15:50

Reporter

Jim Tarber

Assigned To

Priority

normal

Severity

minor

Reproducibility

have not tried

Status

new

Resolution

open

Platform

OS

OS Version

Product Version

master (dev code)

Target Version

Fixed in Version

Summary

0007219: Export Permission: Default Library Textures are not exportable

Description

For regions where ExportSupported=true, the Export checkbox is often disabled (greyed out) in viewers that support the Export permission. This seems to be due to the textures used (e.g. default plywood texture) having PermissionMask.All rather than PermissionMask.All|PermissionMask.Export

Steps To Reproduce

1. Start a viewer that supports the Export permission, such as a current Singularity release.
2. Login to a grid with ExportSupported=true in the region configuration.
3. Rez a plywood cube.
4. Enable full permission for Next Owner.

The [x] Export checkbox should be enabled, but it is not. Instead, the field is disabled (greyed out).

Additional Information

See Singularity problem report: https://code.google.com/p/singularity-viewer/issues/detail?id=1543 [^]

Although this turned out to be an issue at the server end, with non-exportable Library resources.

It could be argued that the entire Library should or should not default to exportable, but I think it is without question that the 4 (plus one reassigned UUID) predefined textures (e.g. http://wiki.secondlife.com/wiki/TEXTURE_PLYWOOD [^] ) should be marked exportable.

Tags

No tags attached.

Git Revision or version number

Run Mode

Standalone (1 Region) , Standalone (Multiple Regions) , Grid (1 Region per Sim) , Grid (Multiple Regions per Sim)

Physics Engine

ODE

Script Engine

Environment

Unknown

Mono Version

None

Viewer

Singularity 1.8.5 (5617)

Relationships

Notes
(0026287) Jim Tarber (reporter) 2014-06-16 14:34	I've resolved this in the InWorldz code, and although it is quite different, I found the corresponding OpenSim code searching for the Library root folder UUID. My change was to add PermissionMask.Export to the the B/C/E/N masks when adding new Library items. In the OpenSim code, that means changing ReadItemFromConfig() in LibraryService.cs to code like: item.CurrentPermissions = (uint)config.GetLong("currentPermissions", (uint)(PermissionMask.All\|PermissionMask.Export)); This should be done for all four of the masks above. Similarly, the CreateItem() method in that file should be updated to match the same defaults, rather than have the old-school 0x7FFFFFFF masks. In the OpenSim code, both are missing an explicit initialization of GroupMask as well, although presumably would end up being PermissionsMask.None (0).

(0026288) aiaustin (developer) 2014-06-17 02:09	Thanks for tracking down this issue Jim.. its been a pain in moving items between out experimental and live grids... even for items I thought were completely created afresh. I do think that the library should be all set as exportable to prevent simple slip ups and encourage sharing in the core OpenSim code. Its all open source anyway and export would be consistent with that.

Issue History
Date Modified	Username	Field	Change
2014-06-16 14:32	Jim Tarber	New Issue
2014-06-16 14:34	Jim Tarber	Note Added: 0026287
2014-06-17 02:09	aiaustin	Note Added: 0026288

Copyright © 2000 - 2012 MantisBT Group