Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Completed
Pull Request — master (#20)
by
unknown
01:52
created
src/database/migrations/2015_08_04_131614_create_settings_table.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@
 block discarded – undo
12 12
      */
13 13
     public function up()
14 14
     {
15
-        Schema::create('settings', function (Blueprint $table) {
15
+        Schema::create('settings', function(Blueprint $table) {
16 16
             $table->increments('id');
17 17
             $table->string('key');
18 18
             $table->string('name');
Please login to merge, or discard this patch.
Indentation   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -5,32 +5,32 @@
 block discarded – undo
5 5
 
6 6
 class CreateSettingsTable extends Migration
7 7
 {
8
-    /**
9
-     * Run the migrations.
10
-     *
11
-     * @return void
12
-     */
13
-    public function up()
14
-    {
15
-        Schema::create('settings', function (Blueprint $table) {
16
-            $table->increments('id');
17
-            $table->string('key');
18
-            $table->string('name');
19
-            $table->string('description')->nullable();
20
-            $table->string('value')->nullable();
21
-            $table->text('field');
22
-            $table->tinyInteger('active');
23
-            $table->timestamps();
24
-        });
25
-    }
8
+	/**
9
+	 * Run the migrations.
10
+	 *
11
+	 * @return void
12
+	 */
13
+	public function up()
14
+	{
15
+		Schema::create('settings', function (Blueprint $table) {
16
+			$table->increments('id');
17
+			$table->string('key');
18
+			$table->string('name');
19
+			$table->string('description')->nullable();
20
+			$table->string('value')->nullable();
21
+			$table->text('field');
22
+			$table->tinyInteger('active');
23
+			$table->timestamps();
24
+		});
25
+	}
26 26
 
27
-    /**
28
-     * Reverse the migrations.
29
-     *
30
-     * @return void
31
-     */
32
-    public function down()
33
-    {
34
-        Schema::drop('settings');
35
-    }
27
+	/**
28
+	 * Reverse the migrations.
29
+	 *
30
+	 * @return void
31
+	 */
32
+	public function down()
33
+	{
34
+		Schema::drop('settings');
35
+	}
36 36
 }
Please login to merge, or discard this patch.
src/app/Http/Controllers/SettingCrudController.php 3 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
4 4
 
5 5
 use Backpack\CRUD\app\Http\Controllers\CrudController;
6 6
 // VALIDATION
7
-use Backpack\Settings\app\Http\Requests\SettingRequest as StoreRequest;
8 7
 use Backpack\Settings\app\Http\Requests\SettingRequest as UpdateRequest;
9 8
 
10 9
 class SettingCrudController extends CrudController
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
         $this->crud->hasAccessOrFail('update');
65 65
 
66 66
         $this->data['entry'] = $this->crud->getEntry($id);
67
-        $this->crud->addField((array) json_decode($this->data['entry']->field)); // <---- this is where it's different
67
+        $this->crud->addField((array)json_decode($this->data['entry']->field)); // <---- this is where it's different
68 68
         $this->data['crud'] = $this->crud;
69 69
         $this->data['fields'] = $this->crud->getUpdateFields($id);
70 70
         $this->data['title'] = trans('backpack::crud.edit').' '.$this->crud->entity_name;
Please login to merge, or discard this patch.
Indentation   +75 added lines, -75 removed lines patch added patch discarded remove patch
@@ -9,89 +9,89 @@
 block discarded – undo
9 9
 
10 10
 class SettingCrudController extends CrudController
11 11
 {
12
-    public function __construct()
13
-    {
14
-        parent::__construct();
12
+	public function __construct()
13
+	{
14
+		parent::__construct();
15 15
 
16
-        $this->crud->setModel("Backpack\Settings\app\Models\Setting");
17
-        $this->crud->setEntityNameStrings(trans('backpack::settings.setting_singular'), trans('backpack::settings.setting_plural'));
18
-        $this->crud->setRoute(config('backpack.base.route_prefix', 'admin').'/setting');
19
-        $this->crud->denyAccess(['create', 'delete']);
20
-        $this->crud->setColumns([
21
-            [
22
-                'name'  => 'name',
23
-                'label' => trans('backpack::settings.name'),
24
-            ],
25
-            [
26
-                'name'  => 'value',
27
-                'label' => trans('backpack::settings.value'),
28
-            ],
29
-            [
30
-                'name'  => 'description',
31
-                'label' => trans('backpack::settings.description'),
32
-            ],
33
-        ]);
34
-        $this->crud->addField([
35
-            'name'       => 'name',
36
-            'label'      => trans('backpack::settings.name'),
37
-            'type'       => 'text',
38
-            'attributes' => [
39
-                'disabled' => 'disabled',
40
-            ],
41
-        ]);
42
-    }
16
+		$this->crud->setModel("Backpack\Settings\app\Models\Setting");
17
+		$this->crud->setEntityNameStrings(trans('backpack::settings.setting_singular'), trans('backpack::settings.setting_plural'));
18
+		$this->crud->setRoute(config('backpack.base.route_prefix', 'admin').'/setting');
19
+		$this->crud->denyAccess(['create', 'delete']);
20
+		$this->crud->setColumns([
21
+			[
22
+				'name'  => 'name',
23
+				'label' => trans('backpack::settings.name'),
24
+			],
25
+			[
26
+				'name'  => 'value',
27
+				'label' => trans('backpack::settings.value'),
28
+			],
29
+			[
30
+				'name'  => 'description',
31
+				'label' => trans('backpack::settings.description'),
32
+			],
33
+		]);
34
+		$this->crud->addField([
35
+			'name'       => 'name',
36
+			'label'      => trans('backpack::settings.name'),
37
+			'type'       => 'text',
38
+			'attributes' => [
39
+				'disabled' => 'disabled',
40
+			],
41
+		]);
42
+	}
43 43
 
44
-    /**
45
-     * Display all rows in the database for this entity.
46
-     * This overwrites the default CrudController behaviour:
47
-     * - instead of showing all entries, only show the "active" ones.
48
-     *
49
-     * @return Response
50
-     */
51
-    public function index()
52
-    {
53
-        // if view_table_permission is false, abort
54
-        $this->crud->hasAccessOrFail('list');
55
-        $this->crud->addClause('where', 'active', 1); // <---- this is where it's different from CrudController::index()
44
+	/**
45
+	 * Display all rows in the database for this entity.
46
+	 * This overwrites the default CrudController behaviour:
47
+	 * - instead of showing all entries, only show the "active" ones.
48
+	 *
49
+	 * @return Response
50
+	 */
51
+	public function index()
52
+	{
53
+		// if view_table_permission is false, abort
54
+		$this->crud->hasAccessOrFail('list');
55
+		$this->crud->addClause('where', 'active', 1); // <---- this is where it's different from CrudController::index()
56 56
 
57
-        $this->data['entries'] = $this->crud->getEntries();
58
-        $this->data['crud'] = $this->crud;
59
-        $this->data['title'] = ucfirst($this->crud->entity_name_plural);
57
+		$this->data['entries'] = $this->crud->getEntries();
58
+		$this->data['crud'] = $this->crud;
59
+		$this->data['title'] = ucfirst($this->crud->entity_name_plural);
60 60
 
61
-        // load the view from /resources/views/vendor/backpack/crud/ if it exists, otherwise load the one in the package
62
-        return view('crud::list', $this->data);
63
-    }
61
+		// load the view from /resources/views/vendor/backpack/crud/ if it exists, otherwise load the one in the package
62
+		return view('crud::list', $this->data);
63
+	}
64 64
 
65
-    public function store(StoreRequest $request)
66
-    {
67
-        return parent::storeCrud();
68
-    }
65
+	public function store(StoreRequest $request)
66
+	{
67
+		return parent::storeCrud();
68
+	}
69 69
 
70
-    /**
71
-     * Show the form for editing the specified resource.
72
-     *
73
-     * @param int $id
74
-     *
75
-     * @return Response
76
-     */
77
-    public function edit($id)
78
-    {
79
-        $this->crud->hasAccessOrFail('update');
70
+	/**
71
+	 * Show the form for editing the specified resource.
72
+	 *
73
+	 * @param int $id
74
+	 *
75
+	 * @return Response
76
+	 */
77
+	public function edit($id)
78
+	{
79
+		$this->crud->hasAccessOrFail('update');
80 80
 
81
-        $this->data['entry'] = $this->crud->getEntry($id);
82
-        $this->crud->addField((array) json_decode($this->data['entry']->field)); // <---- this is where it's different
83
-        $this->data['crud'] = $this->crud;
84
-        $this->data['fields'] = $this->crud->getUpdateFields($id);
85
-        $this->data['title'] = trans('backpack::crud.edit').' '.$this->crud->entity_name;
81
+		$this->data['entry'] = $this->crud->getEntry($id);
82
+		$this->crud->addField((array) json_decode($this->data['entry']->field)); // <---- this is where it's different
83
+		$this->data['crud'] = $this->crud;
84
+		$this->data['fields'] = $this->crud->getUpdateFields($id);
85
+		$this->data['title'] = trans('backpack::crud.edit').' '.$this->crud->entity_name;
86 86
 
87
-        $this->data['id'] = $id;
87
+		$this->data['id'] = $id;
88 88
 
89
-        // load the view from /resources/views/vendor/backpack/crud/ if it exists, otherwise load the one in the package
90
-        return view('crud::edit', $this->data);
91
-    }
89
+		// load the view from /resources/views/vendor/backpack/crud/ if it exists, otherwise load the one in the package
90
+		return view('crud::edit', $this->data);
91
+	}
92 92
 
93
-    public function update(UpdateRequest $request)
94
-    {
95
-        return parent::updateCrud();
96
-    }
93
+	public function update(UpdateRequest $request)
94
+	{
95
+		return parent::updateCrud();
96
+	}
97 97
 }
Please login to merge, or discard this patch.
src/app/Http/Requests/SettingRequest.php 1 patch
Indentation   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -6,28 +6,28 @@
 block discarded – undo
6 6
 
7 7
 class SettingRequest extends Request
8 8
 {
9
-    /**
10
-     * Determine if the user is authorized to make this request.
11
-     *
12
-     * @return bool
13
-     */
14
-    public function authorize()
15
-    {
16
-        // only allow updates if the user is logged in
17
-        return \Auth::check();
18
-    }
9
+	/**
10
+	 * Determine if the user is authorized to make this request.
11
+	 *
12
+	 * @return bool
13
+	 */
14
+	public function authorize()
15
+	{
16
+		// only allow updates if the user is logged in
17
+		return \Auth::check();
18
+	}
19 19
 
20
-    /**
21
-     * Get the validation rules that apply to the request.
22
-     *
23
-     * @return array
24
-     */
25
-    public function rules()
26
-    {
27
-        return [
28
-            // 'key' => 'required|min:3|max:255',
29
-            // 'name' => 'required|min:3|max:255',
30
-            // 'field' => 'required'
31
-        ];
32
-    }
20
+	/**
21
+	 * Get the validation rules that apply to the request.
22
+	 *
23
+	 * @return array
24
+	 */
25
+	public function rules()
26
+	{
27
+		return [
28
+			// 'key' => 'required|min:3|max:255',
29
+			// 'name' => 'required|min:3|max:255',
30
+			// 'field' => 'required'
31
+		];
32
+	}
33 33
 }
Please login to merge, or discard this patch.
src/app/Models/Setting.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -7,8 +7,8 @@
 block discarded – undo
7 7
 
8 8
 class Setting extends Model
9 9
 {
10
-    use CrudTrait;
10
+	use CrudTrait;
11 11
 
12
-    protected $table = 'settings';
13
-    protected $fillable = ['value'];
12
+	protected $table = 'settings';
13
+	protected $fillable = ['value'];
14 14
 }
Please login to merge, or discard this patch.
src/database/seeds/SettingsTableSeeder.php 1 patch
Indentation   +46 added lines, -46 removed lines patch added patch discarded remove patch
@@ -7,50 +7,50 @@
 block discarded – undo
7 7
 
8 8
 class SettingsTableSeeder extends Seeder
9 9
 {
10
-    /**
11
-     * Run the database seeds.
12
-     *
13
-     * @return void
14
-     */
15
-    public function run()
16
-    {
17
-        DB::table('settings')->insert([
18
-            'key'           => 'contact_email',
19
-            'name'          => 'Contact form email address',
20
-            'description'   => 'The email address that all emails from the contact form will go to.',
21
-            'value'         => '[email protected]',
22
-            'field'         => '{"name":"value","label":"Value","type":"email"}',
23
-            'active'        => 1,
24
-        ]);
25
-
26
-        DB::table('settings')->insert([
27
-            'key'           => 'contact_cc',
28
-            'name'          => 'Contact form CC field',
29
-            'description'   => 'Email adresses separated by comma, to be included as CC in the email sent by the contact form.',
30
-            'value'         => '',
31
-            'field'         => '{"name":"value","label":"Value","type":"email"}',
32
-            'active'        => 1,
33
-
34
-        ]);
35
-
36
-        DB::table('settings')->insert([
37
-            'key'           => 'contact_bcc',
38
-            'name'          => 'Contact form BCC field',
39
-            'description'   => 'Email adresses separated by comma, to be included as BCC in the email sent by the contact form.',
40
-            'value'         => '',
41
-            'field'         => '{"name":"value","label":"Value","type":"email"}',
42
-            'active'        => 1,
43
-
44
-        ]);
45
-
46
-        DB::table('settings')->insert([
47
-            'key'           => 'motto',
48
-            'name'          => 'Motto',
49
-            'description'   => 'Website motto',
50
-            'value'         => 'this is the value',
51
-            'field'         => '{"name":"value","label":"Value", "title":"Motto value" ,"type":"textarea"}',
52
-            'active'        => 1,
53
-
54
-        ]);
55
-    }
10
+	/**
11
+	 * Run the database seeds.
12
+	 *
13
+	 * @return void
14
+	 */
15
+	public function run()
16
+	{
17
+		DB::table('settings')->insert([
18
+			'key'           => 'contact_email',
19
+			'name'          => 'Contact form email address',
20
+			'description'   => 'The email address that all emails from the contact form will go to.',
21
+			'value'         => '[email protected]',
22
+			'field'         => '{"name":"value","label":"Value","type":"email"}',
23
+			'active'        => 1,
24
+		]);
25
+
26
+		DB::table('settings')->insert([
27
+			'key'           => 'contact_cc',
28
+			'name'          => 'Contact form CC field',
29
+			'description'   => 'Email adresses separated by comma, to be included as CC in the email sent by the contact form.',
30
+			'value'         => '',
31
+			'field'         => '{"name":"value","label":"Value","type":"email"}',
32
+			'active'        => 1,
33
+
34
+		]);
35
+
36
+		DB::table('settings')->insert([
37
+			'key'           => 'contact_bcc',
38
+			'name'          => 'Contact form BCC field',
39
+			'description'   => 'Email adresses separated by comma, to be included as BCC in the email sent by the contact form.',
40
+			'value'         => '',
41
+			'field'         => '{"name":"value","label":"Value","type":"email"}',
42
+			'active'        => 1,
43
+
44
+		]);
45
+
46
+		DB::table('settings')->insert([
47
+			'key'           => 'motto',
48
+			'name'          => 'Motto',
49
+			'description'   => 'Website motto',
50
+			'value'         => 'this is the value',
51
+			'field'         => '{"name":"value","label":"Value", "title":"Motto value" ,"type":"textarea"}',
52
+			'active'        => 1,
53
+
54
+		]);
55
+	}
56 56
 }
Please login to merge, or discard this patch.
src/SettingsServiceProvider.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -51,10 +51,10 @@  discard block
 block discarded – undo
51 51
      */
52 52
     public function setupRoutes(Router $router)
53 53
     {
54
-        $router->group(['namespace' => 'Backpack\Settings\app\Http\Controllers'], function ($router) {
54
+        $router->group(['namespace' => 'Backpack\Settings\app\Http\Controllers'], function($router) {
55 55
             // Admin Interface Routes
56 56
             Route::group(['prefix'   => config('backpack.base.route_prefix', 'admin'),
57
-                        'middleware' => ['web', 'admin'], ], function () {
57
+                        'middleware' => ['web', 'admin'], ], function() {
58 58
                             // Settings
59 59
                             Route::resource('setting', 'SettingCrudController');
60 60
                         });
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 
80 80
     private function registerSettings()
81 81
     {
82
-        $this->app->bind('settings', function ($app) {
82
+        $this->app->bind('settings', function($app) {
83 83
             return new Settings($app);
84 84
         });
85 85
     }
Please login to merge, or discard this patch.
Indentation   +68 added lines, -68 removed lines patch added patch discarded remove patch
@@ -11,81 +11,81 @@
 block discarded – undo
11 11
 
12 12
 class SettingsServiceProvider extends ServiceProvider
13 13
 {
14
-    /**
15
-     * Indicates if loading of the provider is deferred.
16
-     *
17
-     * @var bool
18
-     */
19
-    protected $defer = false;
14
+	/**
15
+	 * Indicates if loading of the provider is deferred.
16
+	 *
17
+	 * @var bool
18
+	 */
19
+	protected $defer = false;
20 20
 
21
-    /**
22
-     * Perform post-registration booting of services.
23
-     *
24
-     * @return void
25
-     */
26
-    public function boot()
27
-    {
28
-        // only use the Settings package if the Settings table is present in the database
29
-        if (count(Schema::getColumnListing('settings'))) {
30
-            // get all settings from the database
31
-            $settings = Setting::all();
21
+	/**
22
+	 * Perform post-registration booting of services.
23
+	 *
24
+	 * @return void
25
+	 */
26
+	public function boot()
27
+	{
28
+		// only use the Settings package if the Settings table is present in the database
29
+		if (count(Schema::getColumnListing('settings'))) {
30
+			// get all settings from the database
31
+			$settings = Setting::all();
32 32
 
33
-            // bind all settings to the Laravel config, so you can call them like
34
-            // Config::get('settings.contact_email')
35
-            foreach ($settings as $key => $setting) {
36
-                Config::set('settings.'.$setting->key, $setting->value);
37
-            }
38
-        }
33
+			// bind all settings to the Laravel config, so you can call them like
34
+			// Config::get('settings.contact_email')
35
+			foreach ($settings as $key => $setting) {
36
+				Config::set('settings.'.$setting->key, $setting->value);
37
+			}
38
+		}
39 39
 
40
-        $this->loadTranslationsFrom(realpath(__DIR__.'/resources/lang'), 'backpack');
40
+		$this->loadTranslationsFrom(realpath(__DIR__.'/resources/lang'), 'backpack');
41 41
 
42
-        // publish the migrations and seeds
43
-        $this->publishes([__DIR__.'/database/migrations/' => database_path('migrations')], 'migrations');
44
-        $this->publishes([__DIR__.'/database/seeds/' => database_path('seeds')], 'seeds');
42
+		// publish the migrations and seeds
43
+		$this->publishes([__DIR__.'/database/migrations/' => database_path('migrations')], 'migrations');
44
+		$this->publishes([__DIR__.'/database/seeds/' => database_path('seeds')], 'seeds');
45 45
 
46
-        // publish translation files
47
-        $this->publishes([__DIR__.'/resources/lang' => resource_path('lang/vendor/backpack')], 'lang');
48
-    }
46
+		// publish translation files
47
+		$this->publishes([__DIR__.'/resources/lang' => resource_path('lang/vendor/backpack')], 'lang');
48
+	}
49 49
 
50
-    /**
51
-     * Define the routes for the application.
52
-     *
53
-     * @param \Illuminate\Routing\Router $router
54
-     *
55
-     * @return void
56
-     */
57
-    public function setupRoutes(Router $router)
58
-    {
59
-        $router->group(['namespace' => 'Backpack\Settings\app\Http\Controllers'], function ($router) {
60
-            // Admin Interface Routes
61
-            Route::group(['prefix'   => config('backpack.base.route_prefix', 'admin'),
62
-                        'middleware' => ['web', 'admin'], ], function () {
63
-                            // Settings
64
-                            Route::resource('setting', 'SettingCrudController');
65
-                        });
66
-        });
67
-    }
50
+	/**
51
+	 * Define the routes for the application.
52
+	 *
53
+	 * @param \Illuminate\Routing\Router $router
54
+	 *
55
+	 * @return void
56
+	 */
57
+	public function setupRoutes(Router $router)
58
+	{
59
+		$router->group(['namespace' => 'Backpack\Settings\app\Http\Controllers'], function ($router) {
60
+			// Admin Interface Routes
61
+			Route::group(['prefix'   => config('backpack.base.route_prefix', 'admin'),
62
+						'middleware' => ['web', 'admin'], ], function () {
63
+							// Settings
64
+							Route::resource('setting', 'SettingCrudController');
65
+						});
66
+		});
67
+	}
68 68
 
69
-    /**
70
-     * Register any package services.
71
-     *
72
-     * @return void
73
-     */
74
-    public function register()
75
-    {
76
-        $this->registerSettings();
77
-        $this->setupRoutes($this->app->router);
69
+	/**
70
+	 * Register any package services.
71
+	 *
72
+	 * @return void
73
+	 */
74
+	public function register()
75
+	{
76
+		$this->registerSettings();
77
+		$this->setupRoutes($this->app->router);
78 78
 
79
-        // use this if your package has a config file
80
-        // config([
81
-        //         'config/Settings.php',
82
-        // ]);
83
-    }
79
+		// use this if your package has a config file
80
+		// config([
81
+		//         'config/Settings.php',
82
+		// ]);
83
+	}
84 84
 
85
-    private function registerSettings()
86
-    {
87
-        $this->app->bind('settings', function ($app) {
88
-            return new Settings($app);
89
-        });
90
-    }
85
+	private function registerSettings()
86
+	{
87
+		$this->app->bind('settings', function ($app) {
88
+			return new Settings($app);
89
+		});
90
+	}
91 91
 }
Please login to merge, or discard this patch.