Skip to content

Only execute query in propagateChange once - #12855

Merged
MorrisJobke merged 1 commit into
masterfrom
fix/do_not_update_filecache_twice
Dec 5, 2018
Merged

Only execute query in propagateChange once#12855
MorrisJobke merged 1 commit into
masterfrom
fix/do_not_update_filecache_twice

Conversation

@rullzer

Copy link
Copy Markdown
Member

The second execute statement should be inside the if block. Else it gets
executed twice which makes no sense.

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

The second execute statement should be inside the if block. Else it gets
executed twice which makes no sense.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

@ChristophWurstChristophWurst 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.

Whoops. Makes sense!

@ChristophWurst

Copy link
Copy Markdown
Member

backport=

@MorrisJobkeMorrisJobke 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.

Makes sense 👍

@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable15

@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable14

@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable13

@MorrisJobkeMorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Dec 5, 2018
@MorrisJobkeMorrisJobke mentioned this pull request Dec 5, 2018
12 tasks
@MorrisJobke
MorrisJobke merged commit 45294cb into masterDec 5, 2018
@MorrisJobke
MorrisJobke deleted the fix/do_not_update_filecache_twice branch December 5, 2018 20:43
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable15 in #12861

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable14 in #12862

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable13 in #12864

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@ChristophWurst@MorrisJobke