0007356: llGetPrimitiveParams returns transparency instead of alpha for PRIM_TEXT

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0007356

opensim

[REGION] Script Functions

public

2014-10-21 16:10

2015-04-13 11:06

Reporter

Magnuz

Assigned To

justincc

Priority

normal

Severity

minor

Reproducibility

always

Status

closed

Resolution

fixed

Platform

Intel Core i7

Operating System

Ubuntu

Operating System Version

12.04

Product Version

master (dev code)

Target Version

Fixed in Version

master (dev code)

Summary

0007356: llGetPrimitiveParams returns transparency instead of alpha for PRIM_TEXT

Description

llGetPrimitiveParams([PRIM_TEXT]) returns ["text", <r,g,b>, 1-alpha] instead of the expected ["text", <r,g,b>, alpha]. The same goes for llGetLinkPrimitiveParams(LINK_THIS, [PRIM_TEXT]). The result is the same whether the text has been set with llSetText, llSetPrimitiveParms, llSetLinkPrimitiveParms or llSetLinkPrimitiveParmsFast.

Steps To Reproduce

The script

default
{
    state_entry()
    {
        llSetText("ABC", <0.8,0.6,0.4>, 0.8);
        llOwnerSay(llList2CSV(["ST+GPP"]+llGetPrimitiveParams([PRIM_TEXT])));
        llOwnerSay(llList2CSV(["ST+GLPP"]+llGetLinkPrimitiveParams(LINK_THIS, [PRIM_TEXT])));
        llSetPrimitiveParams([PRIM_TEXT, "ABC", <0.8,0.6,0.4>, 0.8]);
        llOwnerSay(llList2CSV(["SPP+GPP"]+llGetPrimitiveParams([PRIM_TEXT])));
        llOwnerSay(llList2CSV(["SPP+GLPP"]+llGetLinkPrimitiveParams(LINK_THIS, [PRIM_TEXT])));
        llSetLinkPrimitiveParams(LINK_THIS, [PRIM_TEXT, "ABC", <0.8,0.6,0.4>, 0.8]);
        llOwnerSay(llList2CSV(["SLPP+GPP"]+llGetPrimitiveParams([PRIM_TEXT])));
        llOwnerSay(llList2CSV(["SLPP+GLPP"]+llGetLinkPrimitiveParams(LINK_THIS, [PRIM_TEXT])));
        llSetLinkPrimitiveParamsFast(LINK_THIS, [PRIM_TEXT, "ABC", <0.8,0.6,0.4>, 0.8]);
        llOwnerSay(llList2CSV(["SLPPF+GPP"]+llGetPrimitiveParams([PRIM_TEXT])));
        llOwnerSay(llList2CSV(["SLPPF+GLPP"]+llGetLinkPrimitiveParams(LINK_THIS, [PRIM_TEXT])));
        //llSetTimerEvent(1.0);
    }
}

returns

[15:44] Text Test: ST+GPP, ABC, <0.800000,0.600000,0.400000>, 0.200000
[15:44] Text Test: ST+GLPP, ABC, <0.800000,0.600000,0.400000>, 0.200000
[15:44] Text Test: SPP+GPP, ABC, <0.800000,0.600000,0.400000>, 0.200000
[15:44] Text Test: SPP+GLPP, ABC, <0.800000,0.600000,0.400000>, 0.200000
[15:44] Text Test: SLPP+GPP, ABC, <0.800000,0.600000,0.400000>, 0.200000
[15:44] Text Test: SLPP+GLPP, ABC, <0.800000,0.600000,0.400000>, 0.200000
[15:44] Text Test: SLPPF+GPP, ABC, <0.800000,0.600000,0.400000>, 0.200000
[15:44] Text Test: SLPPF+GLPP, ABC, <0.800000,0.600000,0.400000>, 0.200000

Tags

No tags attached.

Git Revision or version number

0.8.0.1

Run Mode

Grid (1 Region per Sim)

Physics Engine

BulletSim

Script Engine

Environment

Mono / Linux32

Mono Version

2.10

Viewer

Attached Files

0001-Correct-PRIM_TEXT-return-value-from-trans-to-alpha.patch [^] (1,203 bytes) 2015-02-10 14:07 [Show Content] [Hide Content]

From 0e65db89f598add1f068c3495a239289de6a83be Mon Sep 17 00:00:00 2001
From: Magnuz Binder <magnuz@magnuz-se.com>
Date: Tue, 10 Feb 2015 23:01:31 +0100
Subject: [PATCH] Correct PRIM_TEXT return value from trans to alpha.

---
 .../Shared/Api/Implementation/LSL_Api.cs           |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/LSL_Api.cs b/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/LSL_Api.cs
index 4825556..f7c0820 100644
--- a/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/LSL_Api.cs
+++ b/OpenSim/Region/ScriptEngine/Shared/Api/Implementation/LSL_Api.cs
@@ -9009,7 +9009,7 @@ namespace OpenSim.Region.ScriptEngine.Shared.Api
                         res.Add(new LSL_Vector(textColor.R,
                                                textColor.G,
                                                textColor.B));
-                        res.Add(new LSL_Float(textColor.A));
+                        res.Add(new LSL_Float(1.0 - textColor.A));
                         break;
                     case (int)ScriptBaseClass.PRIM_NAME:
                         res.Add(new LSL_String(part.Name));
-- 
1.7.9.5

Relationships

Notes
(0027478) Magnuz (reporter) 2015-02-10 14:11	Added patch to correct alpha return value for PRIM_TEXT in llGetPrimitiveParams and llGetLinkPrimitiveParams to alpha instead of 1-alpha.

(0027637) justincc (administrator) 2015-02-27 16:53	Committed as git master 158c3f8. Thanks Magnuz!

(0028000) Magnuz (reporter) 2015-04-13 11:06	Verified and closing.

Issue History
Date Modified	Username	Field	Change
2014-10-21 16:10	Magnuz	New Issue
2015-02-10 14:07	Magnuz	File Added: 0001-Correct-PRIM_TEXT-return-value-from-trans-to-alpha.patch
2015-02-10 14:10	Magnuz	Note Added: 0027477
2015-02-10 14:11	Magnuz	Note Added: 0027478
2015-02-10 14:11	Magnuz	Status	new => patch included
2015-02-10 14:11	Magnuz	Note Deleted: 0027477
2015-02-27 16:53	justincc	Note Added: 0027637
2015-02-27 16:53	justincc	Status	patch included => resolved
2015-02-27 16:53	justincc	Resolution	open => fixed
2015-02-27 16:53	justincc	Assigned To	=> justincc
2015-04-13 11:06	Magnuz	Note Added: 0028000
2015-04-13 11:06	Magnuz	Status	resolved => closed
2015-04-13 11:06	Magnuz	Fixed in Version	=> master (dev code)