Skip to content

[stable13] Emit event when running ./occ db:add-missing-indices - #9620

Merged
MorrisJobke merged 1 commit into
stable13from
backport/9576/stable13
May 29, 2018
Merged

[stable13] Emit event when running ./occ db:add-missing-indices#9620
MorrisJobke merged 1 commit into
stable13from
backport/9576/stable13

Conversation

@rullzer

Copy link
Copy Markdown
Member

Backport of #9576

@@ -1,4 +1,5 @@
<?php
declare(strict_types=1);

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.

Shouldn't stable13 be compatible with php5.6? Does this line work for that?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

yes you are right... Let me fix that

This allows apps to listen to this event in order to also update
indecies there.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer
rullzerforce-pushed the backport/9576/stable13 branch from b2c610d to dd5995aCompareMay 28, 2018 07:05
@codecov

codecovBot commented May 28, 2018

Copy link
Copy Markdown

Codecov Report

Merging #9620 into stable13 will decrease coverage by <.01%.
The diff coverage is 0%.

@@ Coverage Diff @@## stable13 #9620 +/- ##
==============================================
- Coverage 50.87% 50.87% -0.01% 
Complexity 25097 25097 ==============================================
Files 1547 1547 Lines 87718 87723 +5 ==============================================
Hits 44628 44628 - Misses 43090 43095 +5
Impacted FilesCoverage ΔComplexity Δ
core/register_command.php0% <0%> (ø)0 <0> (ø)⬇️
core/Command/Db/AddMissingIndices.php0% <0%> (ø)7 <1> (ø)⬇️
index.php0% <0%> (ø)0% <0%> (ø)⬇️

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

LGTM

@juliusknorrjuliusknorr added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 29, 2018
@MorrisJobke
MorrisJobke merged commit b03752b into stable13May 29, 2018
@MorrisJobke
MorrisJobke deleted the backport/9576/stable13 branch May 29, 2018 12:30
@MorrisJobkeMorrisJobke mentioned this pull request May 31, 2018
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 finishenhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@rullzer@ChristophWurst@juliusknorr@MorrisJobke