-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
add IUserMountCache->getMountsForFileId #21858
Conversation
@@ -53,6 +55,8 @@ class UserMountCache implements IUserMountCache { | |||
*/ | |||
private $logger; | |||
|
|||
private $cacheInfoCache = []; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hahaha, why not a cacheCacheCache ?
So for system-wide mounts, Code looks good 👍 |
Seems like this makes PGSQL unhappy?
|
The problem is autoincrement numbers can not be negative? |
64d51fe
to
eaccf33
Compare
$builder = $this->connection->getQueryBuilder(); | ||
$query = $builder->select('storage', 'path') | ||
->from('filecache') | ||
->where($builder->expr()->eq('fileid', $builder->createNamedParameter($fileId, \PDO::PARAM_INT))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use the constant IQueryBuilder::PARAM_INT
eaccf33
to
1eedb51
Compare
1eedb51
to
dc915e3
Compare
|
dc915e3
to
2563c3f
Compare
All tests pass, ready for review ? @nickvergessen @icewind1991 |
👍 for @nickvergessen's part |
👍 for the latest changes. @nickvergessen one last thumbs up for Robin's commits ? |
Works and makes it possible to do the stuff I need to do: |
add IUserMountCache->getMountsForFileId
For updating the docs, is |
I don't know. I guess it's fine. |
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. |
Fixes #21846
Usage example:
cc @PVince81 @nickvergessen