Configurable preview path

[MorrisJobke]

• like the path of the cache folder:

  core/config/config.sample.php

  Lines 936 to 942 in 750ec93

  	/**
  	* Location of the cache folder, defaults to ``data/$user/cache`` where
  	* ``$user`` is the current user. When specified, the format will change to
  	* ``$cache_path/$user`` where ``$cache_path`` is the configured cache directory
  	* and ``$user`` is the user.
  	*/
  	'cache_path' => '',

Background:

Often the user data is stored on a different storage server and it makes the access to those often needed files faster, because they are on the same server. Also it makes the redundancy easier. Those files are not that critical if they get lost and could be easily re-generated. Therefore it makes sense to put them on a storage that is not that redundant available but a lot faster to access.

cc @MTRichards for this feature request (was requested as a side note in a blue ticket)

cc @oparoz @icewind1991 for the implementation details - it shouldn't be that hard, or am I wrong here?

[oparoz]

First thoughts:

• Move the thumbnails folder to the cache folder if cache users also create folders inside that folder. The one caveat would be that a cache wipe would wipe all thumbnails, but I don't expect the cache to be wiped unless absolutely necessary
• Change the path here: https://github.com/owncloud/core/blob/master/lib/private/preview.php#L40

If that's not a good solution, then much more work would be required from my pov because Preview is still using \OC\Files\View ($this->userView->mkdir($this->getThumbnailsFolder() . '/');) instead of the Node API.
So the best thing would be to first move it to the AppFramework #13610, but that issue is a year old and of low priority. That will never happen, but the move for preview generation from ajax to dav may trigger some Preview refactoring.

[oparoz]

Thinking about it some more, just changing the path in Preview to make it a subfolder of cache will also have problems linked to the fact that Preview is still using \OC\Files\View...

[PVince81]

discussed separating metadata from files here #18029

[PVince81]

and #28939

[PVince81]

closing in favor of #19548

[lock]

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.