[jira] [Closed] (SLING-2710) Define TenantManager API

2013-11-16 Thread Felix Meschberger (JIRA)
[ https://issues.apache.org/jira/browse/SLING-2710?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger closed SLING-2710. Tenant 1.0 has been released, hence closing issues now. Define TenantManager API

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Carsten Ziegeler (JIRA)
interface. Right now, I would opt for removing TenantProvider, especially as this opens some questions when creating a new tenant through the manager. Which provider is used and why and wouldn't we need a way to create a tenant based on a provider? Define TenantManager API

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Felix Meschberger (JIRA)
TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature Components: Extensions Reporter: Felix Meschberger Assignee

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Carsten Ziegeler (JIRA)
TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature Components: Extensions Reporter: Felix Meschberger Assignee: Felix

Re: [jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Felix Meschberger
Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature Components: Extensions Reporter: Felix

Re: [jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Carsten Ziegeler
, there is, of course, also an AdapterFactory for tenants in the implementation. Regards Felix Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling

Re: [jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Carsten Ziegeler
having two separate APIs. In any case, there is, of course, also an AdapterFactory for tenants in the implementation. Regards Felix Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710

Re: [jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Felix Meschberger
) use of reading tenants as opposed to the specialized management of tenants warrants having two separate APIs. In any case, there is, of course, also an AdapterFactory for tenants in the implementation. Regards Felix Define TenantManager API

Re: [jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Felix Meschberger
to the specialized management of tenants warrants having two separate APIs. In any case, there is, of course, also an AdapterFactory for tenants in the implementation. Regards Felix Define TenantManager API Key: SLING-2710 URL

[jira] [Resolved] (SLING-2710) Define TenantManager API

2013-04-22 Thread Carsten Ziegeler (JIRA)
Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature Components: Extensions Reporter: Felix Meschberger

Re: [jira] [Commented] (SLING-2710) Define TenantManager API

2013-04-22 Thread Carsten Ziegeler
APIs. In any case, there is, of course, also an AdapterFactory for tenants in the implementation. Regards Felix Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-03-01 Thread Felix Meschberger (JIRA)
in Rev. 1451514. Unit tests and potential fixes following. Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature

[jira] [Assigned] (SLING-2710) Define TenantManager API

2013-02-28 Thread Felix Meschberger (JIRA)
[ https://issues.apache.org/jira/browse/SLING-2710?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger reassigned SLING-2710: Assignee: Felix Meschberger Define TenantManager API

Re: TenantManager API

2013-02-24 Thread Timothée Maret
Hi, 2013/2/21 Felix Meschberger fmesc...@adobe.com Hi SLING-2710 [1] proposes the addition of a TenantManager service interface. Thanks for the comments so far. One question that came up in the issue is whether the TenantManager should extend the TenantProvider or not. I am a bit

Re: TenantManager API

2013-02-24 Thread Felix Meschberger
Hi Ruben Am 21.02.2013 um 16:47 schrieb Ruben Reusser: i was wondering if tenants should work differently and be more transparent to the user by allowing to swotch the osgi properties per tenant. this would allow for a more transparent implementation without forcing the developer to

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-21 Thread Felix Meschberger (JIRA)
with setProperty I agree that from a orthogonality perspective this might make sense. How about a removeProperties(Tentant, String...) method ? Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse

[jira] [Updated] (SLING-2710) Define TenantManager API

2013-02-21 Thread Felix Meschberger (JIRA)
. Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature Components: Extensions Reporter: Felix Meschberger

TenantManager API

2013-02-21 Thread Felix Meschberger
Hi SLING-2710 [1] proposes the addition of a TenantManager service interface. Thanks for the comments so far. One question that came up in the issue is whether the TenantManager should extend the TenantProvider or not. I am a bit undecided: On the one hand I think that the TenantManager is

Re: TenantManager API

2013-02-21 Thread Ruben Reusser
i was wondering if tenants should work differently and be more transparent to the user by allowing to swotch the osgi properties per tenant. this would allow for a more transparent implementation without forcing the developer to think about tenants that much. Ruben On Feb 21, 2013, at 2:14

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread Amit Gupta (JIRA)
[ https://issues.apache.org/jira/browse/SLING-2710?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13582036#comment-13582036 ] Amit Gupta commented on SLING-2710: --- looks good.. Define TenantManager

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread Shashank Gupta (JIRA)
description TenantManager extends TenantProvider interface but in patch it doesn't. 2. No api to retrieve tenant from TenantManager. In fact TenantProvider has api to retrieve tenant from tenantId. Define TenantManager API Key: SLING-2710

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread Felix Meschberger (JIRA)
from create, update, delete access. Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature Components

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread kannan iyer (JIRA)
: - ArrayListTenant getTenants - Tenant getTenant(String) - removeProperty(Tenant tenant, String propName) - addPermissions (ACLEntry[]) //similarly update, delete, get Permissions. - assuming tenant can be adapted to JCR node. Define TenantManager API

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread Timothee Maret (JIRA)
don't need to be added as it is already available in the TenantProvider. Beside, what would be the use of addPermissions(ACLEntry[]) ? +1 for the proposed API and for adding removeProperty method Define TenantManager API Key: SLING-2710

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread kannan iyer (JIRA)
on getTenants/getTenant. Regarding addPermissions, it is restrict role based access at tenant level (like having write access for admins/moderators and read access for everybody else). Define TenantManager API Key: SLING-2710 URL

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread Timothee Maret (JIRA)
API should change as well). Maybe you could achieve this by setting the ACL in the content referenced by the tenant, wdyt ? Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread Shashank Gupta (JIRA)
name, Object value); Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature Components: Extensions

[jira] [Commented] (SLING-2710) Define TenantManager API

2013-02-20 Thread Timothee Maret (JIRA)
. Thus all good :-) Define TenantManager API Key: SLING-2710 URL: https://issues.apache.org/jira/browse/SLING-2710 Project: Sling Issue Type: New Feature Components: Extensions

[jira] [Updated] (SLING-2710) Define TenantManager API

2013-02-19 Thread Felix Meschberger (JIRA)
[ https://issues.apache.org/jira/browse/SLING-2710?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Felix Meschberger updated SLING-2710: - Attachment: SLING-2710.patch Proposed patch. Define TenantManager API