X Tutup
Skip to content

Make it possible to filter by tags with REPORT method#1796

Merged
LukasReschke merged 1 commit intomasterfrom
oc_fav-report
Oct 20, 2016
Merged

Make it possible to filter by tags with REPORT method#1796
LukasReschke merged 1 commit intomasterfrom
oc_fav-report

Conversation

@rullzer
Copy link
Member

@rullzer rullzer commented Oct 19, 2016

From owncloud/core#26099

Enhanced the REPORT method on the Webdav endpoint and added a
"oc:favorite" filter rule. When set, it will return a flat list of
results filtered with only favorite files.

The web UI was also adjusted to use this REPORT method instead of the
private API endpoint.

CC: @nickvergessen @MorrisJobke @icewind1991 @LukasReschke

Test steps:

  1. Favorite a few files in folders and subfolders, share one of them with link (for the share indicator).
  2. Go to the "Favorites" section in the web UI.
  3. Check the network console and see that it's using REPORT to get the results.
  4. Check that the displayed results look fine and also contain the share indicator from the previous link share (it's part of the Webdav response)

Enhanced the REPORT method on the Webdav endpoint and added a
"oc:favorite" filter rule. When set, it will return a flat list of
results filtered with only favorite files.

The web UI was also adjusted to use this REPORT method instead of the
private API endpoint.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer rullzer added 3. to review Waiting for reviews downstream labels Oct 19, 2016
@rullzer rullzer added this to the Nextcloud 11.0 milestone Oct 19, 2016
@mention-bot
Copy link

@rullzer, thanks for your PR! By analyzing the history of the files in this pull request, we identified @LukasReschke, @PVince81 and @nickvergessen to be potential reviewers.

@icewind1991
Copy link
Member

Iirc favorites are just a system tag, wouldn't it be possible to reuse that filtering system

@rullzer
Copy link
Member Author

rullzer commented Oct 19, 2016

@icewind1991 correct. But maybe do that seperatly from this port.

@nickvergessen
Copy link
Member

No favorites are not system tags but the personal tags. otherwise favorites would be shared.

@MorrisJobke
Copy link
Member

Tested and works 👍

@nickvergessen @LukasReschke Could this get reviewed? It blocks https://github.com/owncloud/core/pull/26186/files to be downstreamed ;)

@LukasReschke
Copy link
Member

World didn't explode 👍

@LukasReschke LukasReschke merged commit 0864f53 into master Oct 20, 2016
@LukasReschke LukasReschke deleted the oc_fav-report branch October 20, 2016 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

X Tutup