Here, also, Scene.Permissions.IsAdministrator should be checked, then the parcel owner should be compared. Checking CanEditParcelProperties would have too broad a scope. Some of these functions are DANGEROUS and for instance renters should not be able to use them. So, -1 on broadening this check.
Melanie On 13/04/2012 13:12, Oren Hurvitz wrote: > I found another place that should be changed: if the permission level is set > to "PARCEL_OWNER" then we should check > World.Permissions.CanEditParcelProperties() instead of just checking if this > is the parcel's owner. That would allow the function to be called by Gods, > Estate Managers, and possibly group members (if the parcel is group-owned). > > In addition, there are 3 functions that check parcel ownership permissions > on their own. These checks should be deleted, since they've been replaced by > the general OSSL permissions mechanism: > * osSetParcelDetails > * osSetParcelMediaURL > * osSetparcelSIPAddress > > -- > View this message in context: > http://opensim-dev.2196679.n2.nabble.com/Remove-check-for-IsGod-in-some-OSSL-functions-tp7462127p7462349.html > Sent from the opensim-dev mailing list archive at Nabble.com. > _______________________________________________ > Opensim-dev mailing list > [email protected] > https://lists.berlios.de/mailman/listinfo/opensim-dev > > _______________________________________________ Opensim-dev mailing list [email protected] https://lists.berlios.de/mailman/listinfo/opensim-dev
