MartinPham / todo-ddd

symfony/src/Web/FrontendBundle/Controller/TaskController.php

Unused Use Statements -3 removed lines

@@ -4,7 +4,6 @@ 
 
 use Symfony\Component\Form\FormFactoryInterface;
 use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\HttpFoundation\Session\Session;
 use Symfony\Component\OptionsResolver\Exception\InvalidOptionsException;
 use Todo\Application\Task\Command;
 use Todo\Application\Task\Exception\TaskCannotBeRemovedException;
@@ -14,8 +13,6 @@ 
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
-use Todo\Domain\Exception\TaskNameIsAlreadyExistedException;
-use Todo\Domain\Exception\TaskNameIsEmptyException;
 use Todo\Domain\Exception\TaskNotFoundException;
 use Todo\Domain\Task;
 use Web\FrontendBundle\Form\CreateTaskForm;

symfony/var/SymfonyRequirements.php

Doc Comments +2 added lines, -2 removed lines

@@ -223,7 +223,7 @@ 
      * Adds a mandatory requirement in form of a php.ini configuration.
      *
      * @param string        $cfgName           The configuration name used for ini_get()
-     * @param bool|callback $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
+     * @param boolean|string $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
      *                                         or a callback function receiving the configuration value as parameter to determine the fulfillment of the requirement
      * @param bool          $approveCfgAbsence If true the Requirement will be fulfilled even if the configuration option does not exist, i.e. ini_get() returns false.
      *                                         This is helpful for abandoned configs in later PHP versions or configs of an optional extension, like Suhosin.
@@ -241,7 +241,7 @@ 
      * Adds an optional recommendation in form of a php.ini configuration.
      *
      * @param string        $cfgName           The configuration name used for ini_get()
-     * @param bool|callback $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
+     * @param string|false $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
      *                                         or a callback function receiving the configuration value as parameter to determine the fulfillment of the requirement
      * @param bool          $approveCfgAbsence If true the Requirement will be fulfilled even if the configuration option does not exist, i.e. ini_get() returns false.
      *                                         This is helpful for abandoned configs in later PHP versions or configs of an optional extension, like Suhosin.

Todo/Infrastructure/Persistence/Eloquent/Repository/TaskRepository.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
         $return = [];
         $results = DB::select('SELECT * FROM tasks');
 
-        foreach($results as $taskObject) {
+        foreach ($results as $taskObject) {
             $return[] = $this->hydrateTask($taskObject);
         }
 
@@ -77,7 +77,7 @@ 
         $return = [];
         $results = DB::table('tasks')->where('status', $status)->get();
 
-        foreach($results as $taskObject) {
+        foreach ($results as $taskObject) {
             $return[] = $this->hydrateTask($taskObject);
         }
 

laravel/routes/console.php

Spacing +1 added lines, -1 removed lines

@@ -13,6 +13,6 @@
 |
 */
 
-Artisan::command('inspire', function () {
+Artisan::command('inspire', function() {
     $this->comment(Inspiring::quote());
 })->describe('Display an inspiring quote');

laravel/routes/api.php

Spacing +1 added lines, -1 removed lines

@@ -13,6 +13,6 @@
 |
 */
 
-Route::middleware('auth:api')->get('/user', function (Request $request) {
+Route::middleware('auth:api')->get('/user', function(Request $request) {
     return $request->user();
 });

laravel/routes/web.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 |
 */
 
-Route::get('/', function () {
+Route::get('/', function() {
     return view('welcome');
 });
 

laravel/routes/channels.php

Spacing +1 added lines, -1 removed lines

@@ -11,6 +11,6 @@
 |
 */
 
-Broadcast::channel('App.User.{id}', function ($user, $id) {
+Broadcast::channel('App.User.{id}', function($user, $id) {
     return (int) $user->id === (int) $id;
 });

laravel/database/migrations/2014_10_12_000000_create_users_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('users', function (Blueprint $table) {
+        Schema::create('users', function(Blueprint $table) {
             $table->increments('id');
             $table->string('name');
             $table->string('email')->unique();

laravel/database/migrations/2014_10_12_100000_create_password_resets_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('password_resets', function (Blueprint $table) {
+        Schema::create('password_resets', function(Blueprint $table) {
             $table->string('email')->index();
             $table->string('token')->index();
             $table->timestamp('created_at')->nullable();