Formula9/Framework

View on GitHub
F9/Providers/BladeViewServiceProvider.php

Summary

Maintainability
A
3 hrs
Test Coverage

Avoid too many return statements within this method.
Open

                return $view_factory;
Severity: Major
Found in F9/Providers/BladeViewServiceProvider.php - About 30 mins to fix

    Avoid too many return statements within this method.
    Open

                $app['blade'] = $app->factory(function () { return new Blade($this->app['blade.context']); });
    Severity: Major
    Found in F9/Providers/BladeViewServiceProvider.php - About 30 mins to fix

      Avoid too many return statements within this method.
      Open

                      $resolver->register('php', function () { return new PhpEngine; });
      Severity: Major
      Found in F9/Providers/BladeViewServiceProvider.php - About 30 mins to fix

        Avoid too many return statements within this method.
        Open

                        return $resolver;
        Severity: Major
        Found in F9/Providers/BladeViewServiceProvider.php - About 30 mins to fix

          Avoid too many return statements within this method.
          Open

                      $app['blade.view'] = $app->factory(function () { return new BladeView($this->app['blade.context']); });
          Severity: Major
          Found in F9/Providers/BladeViewServiceProvider.php - About 30 mins to fix

            Avoid too many return statements within this method.
            Open

                                function () use ($app) { return new CompilerEngine($app['blade.compiler'], $app['files']); });
            Severity: Major
            Found in F9/Providers/BladeViewServiceProvider.php - About 30 mins to fix

              The variable $view_factory is not named in camelCase.
              Open

                  public function register(Container $app)
                  {
                      // we only need to do this if Blade views are enabled.
                      if ($app['config']['view.blade.enabled']) {
              
              

              CamelCaseVariableName

              Since: 0.2

              It is considered best practice to use the camelCase notation to name variables.

              Example

              class ClassName {
                  public function doSomething() {
                      $data_module = new DataModule();
                  }
              }

              Source

              The variable $view_factory is not named in camelCase.
              Open

                  public function register(Container $app)
                  {
                      // we only need to do this if Blade views are enabled.
                      if ($app['config']['view.blade.enabled']) {
              
              

              CamelCaseVariableName

              Since: 0.2

              It is considered best practice to use the camelCase notation to name variables.

              Example

              class ClassName {
                  public function doSomething() {
                      $data_module = new DataModule();
                  }
              }

              Source

              The variable $view_factory is not named in camelCase.
              Open

                  public function register(Container $app)
                  {
                      // we only need to do this if Blade views are enabled.
                      if ($app['config']['view.blade.enabled']) {
              
              

              CamelCaseVariableName

              Since: 0.2

              It is considered best practice to use the camelCase notation to name variables.

              Example

              class ClassName {
                  public function doSomething() {
                      $data_module = new DataModule();
                  }
              }

              Source

              There are no issues that match your filters.

              Category
              Status