Add dispatcher event for remote fed shares

[sharidas]

Adding dispatcher events for remote fed shares

Signed-off-by: Sujith H sharidasan@owncloud.com

Description

Dispatcher event for remote fed shares.

Related Issue

Motivation and Context

This change will help users to catch the events and log it.

How Has This Been Tested?

• Create a fed1 folder from a remote oC instance1
• Share to another oC instance2.
• When oC instance2 accepts/rejects the share the logs are available at oC2 instance.

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.

[codecov]

Codecov Report

Merging #29481 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master   #29481      +/-   ##
============================================
+ Coverage     60.84%   60.85%   +0.01%     
  Complexity    17238    17238              
============================================
  Files          1032     1032              
  Lines         57365    57380      +15     
============================================
+ Hits          34904    34919      +15     
  Misses        22461    22461

Impacted Files	Coverage Δ	Complexity Δ
...aring/lib/Controllers/ExternalSharesController.php	95.83% <100%> (+1.89%)	15 <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 3da7b8e...404f45f. Read the comment docs.

[codecov]

Codecov Report

Merging #29481 into master will increase coverage by <.01%.
The diff coverage is 88.23%.

@@             Coverage Diff              @@
##             master   #29481      +/-   ##
============================================
+ Coverage     60.84%   60.85%   +<.01%     
  Complexity    17238    17238              
============================================
  Files          1032     1032              
  Lines         57365    57381      +16     
============================================
+ Hits          34904    34919      +15     
- Misses        22461    22462       +1

Impacted Files	Coverage Δ	Complexity Δ
apps/files_sharing/lib/AppInfo/Application.php	44.26% <0%> (-0.74%)	4 <0> (ø)
...aring/lib/Controllers/ExternalSharesController.php	95.83% <100%> (+1.89%)	15 <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 3da7b8e...3749986. Read the comment docs.

[sharidas]

Backport for stable10 : #29482

[PVince81]

why not inject it directly ? you're in the perfect setup to do so as controllers are DI'd automagically

[PVince81]

Fix DI then this looks good

[sharidas]

Injected EventDispatcherInterface.

[PVince81]

👍

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.