Storage API 2.0

[DeepDiver1975]

Remaining from #22069

• remove duplicate implementations with respect to filemtime, filetype and stat
• do we really need the hash() method? Can we drop it? => needed by documents
• file_get_contents is duplicate with respect to the storage implementation - can we drop it?
• define a proper search interface on storage api level

Further considerations:

• split the storage api into two layers: the true storage api which implements a connection to a physical storage and on storage api which is basically the abstract API accessing any storage.

This goes into the direction of the original discussion with the adapter approach as far as I can tell

[MTRichards]

Conversational ok for 9.1, will consider it when we need to break the API anyway (versions for example) and also discuss with external parties.
Related: #12274 and #22330

All three could be combined into a storage API 3.0 if it makes sense.

[PVince81]

oh, yet another "new storage API" ticket 😄

Which one is the main one ? @DeepDiver1975 @butonic

[PVince81]

• throw exceptions Storage API must work with exceptions, not booleans #13792

[PVince81]

Long term I'd still vote for a full revamp for the storage API if possible, like the proposed interfaces from #20394

Then for the old storages if needed we can provide adapters that map the crappy PHP-like functions to the new interface.

[PVince81]

backlog for now.

@pmaier1

[PVince81]

• another ticket from @labkode to look at about improving the API: New Filesystem API: IStorage #25671

[ownclouders]

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 7 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)

[PVince81]

@butonic can you post the result of our Nbg discussion about the storage API ?

[ownclouders]

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 7 days. Feel free to reopen this issue if you deem it appropriate.

(This is an automated comment from GitMate.io.)