simeonackermann / OC-User-Conversations

User conversations for ownCloud
13 stars 9 forks source link

Issue with larger attachments for not admin users #38

Closed simeonackermann closed 9 years ago

simeonackermann commented 9 years ago

Spinning wheel after submit messages with bigger attachments (e.g. 2MB) for not admin users.

simeonackermann commented 9 years ago

This bug was based on reports off different users.

But I couldn't recreate this issues with current OC 8...

stonerl commented 9 years ago

I do have a similar problem. Conversation just spins and does not load. I have to clear the browser cache and reload the page to use owncloud again, but as soon as I start conversation I get an endless spinning wheel.

stonerl commented 9 years ago

Some additional information. I attached a 46kb PDF. Me and the other users have non-admin-accounts. I had to delete the message in the database to get conversations working again.

stonerl commented 9 years ago

OK the problem appears when a file from a subfolder gets shared.

e.g. "owcloudroot\FILE_A" is ok but "owncloudroot\subfolder\FILE_B" results in an endless spinning wheel

stonerl commented 9 years ago

I dove in the database and found the problem:

{"type":"internal_file","fileid":165217,"path":"files%2FNew+text+file.txt","owner":"me"}

I changed it to this and now it works:

{"type":"internal_file","fileid":165217,"path":"files%2FSubfolder/New+text+file.txt","owner":"me"}

But I think that the user I shared this file with won't see or will have a spinning wheel because the path is only valid for me but not for him.

stonerl commented 9 years ago

Just an update. This only happens when you add a file from a folder that is shared with you by another user.

e.g. User A shared the Folder 1 with me. Inside this Folder is a file called test.txt. When I add the file to a conversation with User B, the correct path should be

"Folder+1%2Ftest.txt"

but in the database the entry is just "test.txt"