Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Adding symfony events for shares #30570

Closed
wants to merge 1 commit into from

Conversation

sharidas
Copy link
Contributor

Adding before and after events for shares
using symfony's event dispatcher.

Signed-off-by: Sujith H [email protected]

Description

Adding before and after symfony dispatcher events for shares

Related Issue

Motivation and Context

Adding before and after symfony dispatcher events for shares.

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@sharidas sharidas added this to the development milestone Feb 21, 2018
@sharidas sharidas self-assigned this Feb 21, 2018
@sharidas sharidas changed the title Adding symfony events for shares [WIP] Adding symfony events for shares Feb 21, 2018
@sharidas
Copy link
Contributor Author

This PR is a deviation from #30268. This doesn't replace the old hooks, instead adds one.

@codecov
Copy link

codecov bot commented Feb 21, 2018

Codecov Report

Merging #30570 into master will increase coverage by 0.01%.
The diff coverage is 92.48%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #30570      +/-   ##
============================================
+ Coverage     61.61%   61.62%   +0.01%     
- Complexity    18507    18508       +1     
============================================
  Files          1090     1090              
  Lines         61112    61136      +24     
============================================
+ Hits          37653    37677      +24     
  Misses        23459    23459
Impacted Files Coverage Δ Complexity Δ
lib/private/Share20/Manager.php 96.99% <100%> (+0.02%) 213 <1> (+1) ⬆️
lib/private/Share/Share.php 67.3% <90%> (+0.43%) 502 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e1d0f2f...0ce660c. Read the comment docs.

@sharidas sharidas force-pushed the new-symfony-events-updated branch 2 times, most recently from 3280205 to 6aa850f Compare February 22, 2018 10:59
Adding before and after events for shares
using symfony's event dispatcher.

Signed-off-by: Sujith H <[email protected]>
@sharidas sharidas force-pushed the new-symfony-events-updated branch from 6aa850f to 0ce660c Compare February 22, 2018 12:05
@felixboehm felixboehm modified the milestones: development, planned Apr 10, 2018
@PVince81
Copy link
Contributor

is this obsolete ?

@sharidas
Copy link
Contributor Author

is this obsolete ?

Yes this is obsolete PR. I am closing this one.

@sharidas sharidas closed this May 28, 2018
@PVince81 PVince81 deleted the new-symfony-events-updated branch September 27, 2018 13:35
@lock lock bot locked as resolved and limited conversation to collaborators Sep 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants