<div dir="ltr">About gods: ok. I don't allow any users to be gods for security reasons, but I had assumed that if someone *does* have god permissions then they should be able to do anything. Since that is not the case, I won't be making any changes related to gods.<div>

<br></div><div>I assume by "-1 on changing existing functions" you mean not to change IsAdministrator(). Ok, but that means we still have two methods that do the same thing: IsGod() and IsAdministrator(). This doesn't affect what I'm doing so I have no reason to push for this change, but it looks like a longstanding bug. If anyone is interested in pursuing this, Godspeed :)</div>

<div><br></div><div>Regarding whether ESTATE_MANAGER includes the estate owner: it doesn't, because of this code: "World.RegionInfo.EstateSettings.EstateOwner != ownerID". It explicitly disallows the estate owner, while allowing all the other estate managers. That's the code that should be removed.</div>

<div><br></div><div>Oren</div><div><br><br><div class="gmail_quote">On Fri, Apr 13, 2012 at 3:56 PM, Melanie-2 [via opensim-dev] <span dir="ltr"><<a href="/user/SendEmail.jtp?type=node&node=7462694&i=0" target="_top" rel="nofollow" link="external">[hidden email]</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

        First, HARD -1 on allowing gods all of OSSL. I run a grid where

certain functions just CANNOT be allowed to run, no way, no how.

<br>There must be a way to achieve that, you can't force the

availability of OSSL for gods on grids.

<br><br>Also, HARD -1 on changing existing functions, it will break things

in too many ways for established users.

The check for estate managers already allows estate owners.

<br><br>OpenSim/Region/ScriptEngine/Shared/Api/Implementation/OSSL_Api.cs:377

<br><br>//Only Estate Managers may use the function

<br>if(m_FunctionPerms[function].AllowedOwnerClasses.Contains("ESTATE_MANAGER"))

<br>{

<br>    //Only Estate Managers may use the function

<br>    if (World.RegionInfo.EstateSettings.IsEstateManager(ownerID) &&

<br>World.RegionInfo.EstateSettings.EstateOwner != ownerID)

<br>    {

<br>        return;

<br>    }

<br>}

<br><br>A god check _could_ be added in the parcel owner, parcel group,

<br>estate manager and estate owner clauses as it can be assumed that if

<br>anyone is allowed to run the function gods should be allowed to run

<br>it, too. However, if a function is indeed disabled it must be

disabled for gods as well.

<br><br>Melanie

<br><br></blockquote></div>

</div></div>

<br/><hr align="left" width="300" />

View this message in context: <a href="http://opensim-dev.2196679.n2.nabble.com/Remove-check-for-IsGod-in-some-OSSL-functions-tp7462127p7462694.html">Re: Remove check for IsGod in some OSSL functions</a><br/>

Sent from the <a href="http://opensim-dev.2196679.n2.nabble.com/">opensim-dev mailing list archive</a> at Nabble.com.<br/>