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

Bogus shared::$path storage on master #18455

Closed
PVince81 opened this issue Aug 20, 2015 · 1 comment · Fixed by #18468
Closed

Bogus shared::$path storage on master #18455

PVince81 opened this issue Aug 20, 2015 · 1 comment · Fixed by #18468

Comments

@PVince81
Copy link
Contributor

Steps

  1. Create two users "user1" and "user2"
  2. Login as "user1"
  3. Create a folder "test"
  4. Share "test" with "user2"
  5. Login as "user2"
  6. Check "oc_storages"

Expected result

No entries for local shares.

Actual result

An extra entry "shared::test" is created.

MariaDB [owncloud]> select * from oc_storages;
+---------------------------------------+------------+-----------+--------------+
| id                                    | numeric_id | available | last_checked |
+---------------------------------------+------------+-----------+--------------+
| home::root                            |          1 |         1 |         NULL |
| local::/srv/www/htdocs/owncloud/data/ |          2 |         1 |         NULL |
| home::user1                           |          3 |         1 |         NULL |
| home::user2                           |          4 |         1 |         NULL |
| shared::/test                         |          5 |         1 |         NULL |
+---------------------------------------+------------+-----------+--------------+

Versions

ownCloud 8.2 pre-alpha git master (d93bf35)

Regression from 8.1.1

I'm going to bisect this.

@PVince81 PVince81 added this to the 8.2-current milestone Aug 20, 2015
@PVince81
Copy link
Contributor Author

First breaking commit df19cab from #13641

@Xenopathic can you have a look ? Not all storage types should have their availability written in the DB.
At least not the local shares.
Remote shares (federation) might be fine, IIRC they will get an entry there.

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

Successfully merging a pull request may close this issue.

1 participant