Skip to content

Conversation

@icewind1991
Copy link
Member

Allows app to also listen to updates and not just inserts

@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Feb 18, 2019
@icewind1991 icewind1991 added this to the Nextcloud 16 milestone Feb 18, 2019
@icewind1991 icewind1991 added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Feb 18, 2019
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991 icewind1991 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Feb 18, 2019
Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

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

Code makes sense 👍

@MorrisJobke MorrisJobke merged commit 50d7f87 into master Feb 19, 2019
@MorrisJobke MorrisJobke deleted the cache-update-event branch February 19, 2019 10:08
public function put($file, array $data) {
if (($id = $this->getId($file)) > -1) {
$this->update($id, $data);
$this->update($id, $data, $file);
Copy link
Contributor

Choose a reason for hiding this comment

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

/**
* update the metadata of an existing file or folder in the cache
*
* @param int $id the fileid of the existing file or folder
* @param array $data [$key => $value] the metadata to update, only the fields provided in the array will be updated, non-provided values will remain unchanged
*/
public function update($id, array $data) {

💥

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

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.

5 participants