<div dir="ltr">Regarding estate owner vs. manager: here's the existing implementation of EstateSettings.IsEstateManager():<div><br></div><div><div>        public bool IsEstateManager(UUID avatarID)</div><div>        {</div>
<div>
            if (IsEstateOwner(avatarID))</div><div>                return true;</div><div><br></div><div>            return l_EstateManagers.Contains(avatarID);</div><div>        }</div><div><br></div><div>It explicitly adds the estate owner to the list of managers. The permission ESTATE_MANAGER should match the method IsEstateManager(). Therefore, it should include the estate owner.</div>

<div><br></div><div>Oren</div><div><br></div><br><div class="gmail_quote">On Fri, Apr 13, 2012 at 4:49 PM, Melanie-2 [via opensim-dev] <span dir="ltr"><<a href="/user/SendEmail.jtp?type=node&node=7462795&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">

        Not so. ESTATE_MANAGER MEANS estate MANAGER. This is intentional. I
<br>reread the code and found that it already allows a comma separated
<br>list notation, so you can use ESTATE_OWNER,ESTATE_MANAGER.
<br><br>This allows for finer grained control. Please understand that estate
<br>owners may be significantly different from estate managers, for
<br>instance in the case of a mainland. This division is useful and,
<br>again, making changes could cause privilege escalation in some
<br>grids. So, -1. What you want to achieve can already be achieved.
<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-tp7462127p7462795.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/>