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

Renaming a received share effects in sharees list api but not on single resource #4322

Closed
AlexAndBear opened this issue Aug 3, 2022 · 4 comments · Fixed by #4539
Closed
Assignees
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug

Comments

@AlexAndBear
Copy link
Contributor

AlexAndBear commented Aug 3, 2022

Describe the bug

Renaming a recieved share effects in sharees list api but not on single resource

Steps to reproduce

Steps to reproduce the behavior:

  1. Albert shares with Alice a folder named folder
  2. Alice renames the shared folder (mountpoint) to sharedFolder
  3. Calling https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares?include_tags=false&state=all&shared_with_me=true shows the new name in the path
  4. Calling https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares/21da5f9d-2b7d-47e5-bf07-e3843c33d88f shows the old (original) name in path

Expected behavior

New name should be listed in path calling single get share

Actual behavior

Old (original) name listed in path calling single get share

NOTE: There is also a slight difference in the path property calling:
https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares?include_tags=false&state=all&shared_with_me=true
path contains /Shares/sharedFolder
while calling https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares/21da5f9d-2b7d-47e5-bf07-e3843c33d88f
path contains /folder

Setup

Please describe how you started the server and provide a list of relevant environment variables or configuration files.

OCIS_XXX=somevalue
OCIS_YYY=somevalue
PROXY_XXX=somevalue

Additional context

Add any other context about the problem here.

@AlexAndBear
Copy link
Contributor Author

FYI @C0rby @micbar

@micbar
Copy link
Contributor

micbar commented Aug 3, 2022

Needs quick fix.

@micbar micbar added the Priority:p2-high Escalation, on top of current planning, release blocker label Aug 3, 2022
@micbar micbar added this to the 2.0.0 General Availability milestone Aug 3, 2022
@AlexAndBear
Copy link
Contributor Author

@micbar Is this still in planning?

@C0rby C0rby self-assigned this Sep 6, 2022
@C0rby
Copy link
Contributor

C0rby commented Sep 6, 2022

@janackermann, can you provide me with an example curl request to rename a share?
I currently get a 409 Conflict response and don't know if my request is missing anything.

Nevermind... I found my mistake.

@phil-davis phil-davis changed the title Renaming a recieved share effects in sharees list api but not on single resource Renaming a received share effects in sharees list api but not on single resource Sep 6, 2022
@micbar micbar mentioned this issue Sep 12, 2022
29 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants