Page MenuHomeDevCentral
Differential D302

Refactoring: Service::getInstanceName()
ClosedPublic
Actions

Authored by dereckson on Feb 23 2016, 19:22.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 4, 20:06
Unknown Object (File)
Sun, May 4, 00:34
Unknown Object (File)
Sat, May 3, 11:55
Unknown Object (File)
Sun, Apr 27, 21:52
Unknown Object (File)
Sun, Apr 27, 19:17
Unknown Object (File)
Fri, Apr 25, 13:49
Unknown Object (File)
Sun, Apr 20, 22:41
Unknown Object (File)
Sat, Apr 19, 15:42
View All Files
Subscribers
alken-orin

Details

Reviewers
dereckson
Commits
rNOTIFa8685fea32e2: Refactoring: Service::getInstanceName()
Summary

The config:show command would like a string to represent the instance
in an array. We transfer the responsability to provide this string to
the Service class.

Test Plan
  • A new test is provided to test the method in the two cases
  • php artisan config:show

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 709.Feb 23 2016, 19:22
dereckson retitled this revision from to Refactoring: Service::getInstanceName().
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
alken-orin added a subscriber: alken-orin.Feb 23 2016, 19:35

Build started: https://ci.nasqueron.org/job/test-notifications-phab/86/ (console: https://ci.nasqueron.org/job/test-notifications-phab/86/console)

alken-orin added a comment.Feb 23 2016, 19:42

Build is green https://ci.nasqueron.org/job/test-notifications-phab/86/ for more details.

dereckson accepted this revision.Feb 23 2016, 20:25
dereckson edited edge metadata.
This revision is now accepted and ready to land.Feb 23 2016, 20:25
Closed by commit rNOTIFa8685fea32e2: Refactoring: Service::getInstanceName() (authored by dereckson, committed by dereckson). · Explain WhyFeb 23 2016, 20:26
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
app/
Console/
Commands/
7 lines
Services/
13 lines
tests/
Services/
45 lines

Diff 712

View Options

app/Console/Commands/ConfigShow.php

Loading...
View Options

app/Services/Service.php

Loading...
View Options

tests/Services/ServiceTest.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator