[Owncloud] getInternalPath and absolute User Homes

Arthur Schiwon blizzz at owncloud.com
Thu Oct 18 19:17:36 UTC 2012


Hi Victor,

OC_User::getHome($uid) is a method provided for user backends to return the 
home folder for the user given. For exmaple, instead of 
/var/www/owncloud/data/USER/ it could be /home/USER/ or 
/mnt/someotherstorage/owncloud/USER 

It needs to be in OC_User as it is a core function. User backends who want to 
implement this functionality need to implement such a method.

Cheers
Arthur

On Thursday, October 18, 2012 09:02:43 PM Victor Dubiniuk wrote:
> Hi Arthur,
> 
> The only thing looks unclear to me. What is context of the OC_User::getHome
> invocation?
> It is not legal to call OC_User methods from an application as it's not a
> public API function.
> I'd suggest to implement internal getHome function for an app and refactor
> the app to use this function.
> 
> ---
> Victor
> 
> On Tue, Oct 16, 2012 at 9:31 PM, Arthur Schiwon <blizzz at owncloud.com> wrote:
> > Hey,
> > 
> > a while ago, Georg implemented OC_User::getHome($uid) which gives back the
> > user home folder. It returns an absolute path. I implemented the function
> > for
> > the LDAP backend as well and, analog to the original method, also return
> > an
> > absolute value.
> > 
> > Now I found out that not everywhere OC_User::getHome($uid) is used when
> > the
> > home folder is established. Recently I adjusted files_version. Today I
> > came
> > across OC_Cache_File, thus i wanted to adjust this. However, I came across
> > a
> > problem.
> > 
> > The User File Cache stores everything in the sub folder 'cache' in ther
> > user
> > directory. Now when
> > $dh=$storage->opendir('/');
> > is executed – with $storage being an instance of
> > Filesystemview('/path/to/user/home/cache') – ownCloud tries to read from
> > /pattodatadir/path/to/user/home/cache which of course won't work since the
> > path to the home dir is already an absolute one.
> > 
> > This path is created in Filesystem::getInternalPath(), because the
> > mountpoint
> > is determined as (/).
> > 
> > For now I have no idea how to improve it;  or if it is even possible.
> > Furthermore it's a central part and i don't want to break it :)
> > So, ideas?
> > 
> > Cheers
> > Arthur
> > _______________________________________________
> > Owncloud mailing list
> > Owncloud at kde.org
> > https://mail.kde.org/mailman/listinfo/owncloud



More information about the Owncloud mailing list