Remove unused dependency injection parameter
ClosedPublic
Actions

Authored by dereckson on Jul 11 2016, 19:48.

Details

Reviewers

Commits

rGROVEb0bded210442: Remove unused dependency injection parameter
rGROVEda04911527b3: Remove unused dependency injection parameter.

Summary

Detected by phpmd.

Test Plan

phpmd
run tests
try a recover request

Diff Detail

Repository

rGROVE Auth Grove

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 1231.Jul 11 2016, 19:48

dereckson retitled this revision from to Remove unused dependency injection parameter..

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

dereckson accepted this revision.Jul 11 2016, 19:49

dereckson edited edge metadata.

dereckson added inline comments.

app/Services/ResetsPasswords.php
25–26	? whitespace issue to fix in another commit

This revision is now accepted and ready to land.Jul 11 2016, 19:49

Closed by commit rGROVEda04911527b3: Remove unused dependency injection parameter. (authored by dereckson, committed by dereckson). · Explain WhyJul 11 2016, 19:53

This revision was automatically updated to reflect the committed changes.

dereckson retitled this revision from Remove unused dependency injection parameter. to Remove unused dependency injection parameter.Jul 11 2016, 19:54

dereckson edited edge metadata.

Revision Contents
Changeset List

Path

Size

app/

Services/

ResetsPasswords.php

3 lines

Diff 1232

View Options

app/Services/ResetsPasswords.php

Remove unused dependency injection parameterClosedPublicActions