Skip to content

Add getID function to the simplefile implementation#26003

Merged
MorrisJobke merged 1 commit into
masterfrom
enh/simplefile/getid
Mar 15, 2021
Merged

Add getID function to the simplefile implementation#26003
MorrisJobke merged 1 commit into
masterfrom
enh/simplefile/getid

Conversation

@rullzer

@rullzer rullzer commented Mar 8, 2021

Copy link
Copy Markdown
Member

Sometimes it is useful to get the actual fileid

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

Sometimes it is useful to get the actual fileid

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer rullzer added enhancement 3. to review Waiting for reviews labels Mar 8, 2021
@rullzer rullzer added this to the Nextcloud 22 milestone Mar 8, 2021

@juliusknorr juliusknorr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah wait, should be added to the public interface as well then I guess 😉

@rullzer

rullzer commented Mar 9, 2021

Copy link
Copy Markdown
Member Author

Ah wait, should be added to the public interface as well then I guess wink

Not for now. I'm not sure yet how to do it with the other implementations.
But also would like to backport this part. It is mainly for internal use this getId anyway.

@rullzer rullzer requested a review from juliusknorr March 12, 2021 07:21
@MorrisJobke MorrisJobke merged commit f512705 into master Mar 15, 2021
@MorrisJobke MorrisJobke deleted the enh/simplefile/getid branch March 15, 2021 13:41
@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable21

@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable20

@kesselb

kesselb commented Mar 17, 2021

Copy link
Copy Markdown
Contributor

/backport to stable20

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 enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants