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

Deleting share response does not contain data field #721

Closed
individual-it opened this issue Oct 20, 2020 · 4 comments
Closed

Deleting share response does not contain data field #721

individual-it opened this issue Oct 20, 2020 · 4 comments
Labels
Category:Defect Existing functionality is not working as expected Type:Bug

Comments

@individual-it
Copy link
Member

delete a share response in oc10:

<?xml version="1.0"?>
<ocs>
 <meta>
  <status>ok</status>
  <statuscode>200</statuscode>
  <message/>
  <totalitems></totalitems>
  <itemsperpage></itemsperpage>
 </meta>
 <data/>
</ocs>

delete a share response on OCIS:

<?xml version="1.0" encoding="UTF-8"?>
<ocs><meta><status>ok</status><statuscode>100</statuscode><message>OK</message></meta></ocs>
@refs
Copy link
Member

refs commented Jan 12, 2021

confirm it happens on the current master

@settings settings bot removed the bug label Jan 12, 2021
@refs refs changed the title deleting share response does not contain data field Deleting share response does not contain data field Jan 13, 2021
@refs refs added the Category:Defect Existing functionality is not working as expected label Jan 13, 2021
@settings settings bot removed the p3-medium label Apr 7, 2021
@ScharfViktor
Copy link
Contributor

re-tested with local server oCIS version 1.11.0 vs web 4.1.0
and oC10 version 10.8.0 vs web 4.1.0

both have the same behavior and response <ocs> <meta> <status>ok</status> <statuscode>100</statuscode> <message/> <totalitems></totalitems> <itemsperpage></itemsperpage> </meta> <data/> </ocs>

@individual-it I suggest to close

@individual-it
Copy link
Member Author

Then it looks like it was changed in oc10. Need to see if the case is covered by tests

  • ToDo QA:team double-check if there are API tests for that case

@phil-davis
Copy link
Contributor

Should be fixed by cs3org/reva#2054

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category:Defect Existing functionality is not working as expected Type:Bug
Projects
None yet
Development

No branches or pull requests

5 participants