Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Group Background Interface should work with ids #4972

Closed
friscoMad opened this issue Sep 24, 2013 · 7 comments
Closed

Group Background Interface should work with ids #4972

friscoMad opened this issue Sep 24, 2013 · 7 comments

Comments

@friscoMad
Copy link
Contributor

Currently Group Backends use the name as the only identifier, that is great for the basic setup and Backend provided but aside of names being a really bad id as they can change over time and not great for DB indexes, there are use cases where different groups can have the same name.

In our case we are binding the backend with our external app backed where the users can create their own groups and thus several users can have a group called 'friends' and this is incompatible with the current interface.

Think of it as your Gmail group for 'friends' and it makes sense to be able to share files or events with your own groups.

Anyway I am not asking to change the current DB group backend and UI to support that only add a getDisplayName to group backend interface and add the corresponding functions for searching as they are in the User Backend and use them where is needed (Share App mainly, not sure how it deals now with group renaming).

@karlitschek
Copy link
Contributor

@blizzz

@blizzz
Copy link
Contributor

blizzz commented Sep 24, 2013

Basically it is extending the group backend infrastructure to allow display names as well. Probably too little time to get it into OC 6.

cc @schiesbn

@jancborchardt jancborchardt modified the milestones: ownCloud 8, ownCloud 7 Jul 16, 2014
@jancborchardt
Copy link
Member

@blizzz @schiesbn is this already fixed? Or something for ownCloud 8?

@blizzz
Copy link
Contributor

blizzz commented Jul 16, 2014

OC 8

@MorrisJobke
Copy link
Contributor

Maybe something for #12998

@MorrisJobke MorrisJobke removed this from the ownCloud 8 milestone Jan 23, 2015
@MorrisJobke MorrisJobke mentioned this issue Jan 23, 2015
27 tasks
@DeepDiver1975 DeepDiver1975 modified the milestone: backlog Mar 21, 2015
@ghost
Copy link

ghost commented Oct 11, 2016

I’m closing this issue because it has been inactive for a few months. This probably means that it is not reproducible or it has been fixed in a newer version. If it’s an enhancement and hasn’t been taken on for so long, then it seems no one has the time to implement this.

Please reopen if you still encounter this issue with the latest stable version and then please use the issue template. You can also contribute directly by providing a patch – see the developer manual. :)

Thank you!

@ghost ghost closed this as completed Oct 11, 2016
@lock
Copy link

lock bot commented Aug 3, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 3, 2019
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants