Merge pull request #4 from pnlinh/feature/add-config

Add new config

Author: pnlinh

Diff for: Makefile

@@ -0,0 +1,38 @@
+help:
+	@grep -E '^[a-zA-Z_-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "⚡ \033[34m%-30s\033[0m %s\n", $$1, $$2}'
+
+install: build run
+test: build run test
+
+build: ## Build Docker image for local development
+	docker-compose build
+
+start: run
+run: ## Run application Docker. Run 'make build' first
+	docker-compose up -d
+
+status: ps
+ps: ### Alias of docker-composer ps command
+	docker-compose ps
+
+restart: ## Restart containers
+	docker-compose restart
+
+test: ### Run unit testing
+	docker-compose exec app sh -c "composer test"
+
+stop: ## Stop application running in Docker
+	docker-compose kill
+
+destroy: ## Alias docker-compose down command
+	docker-compose down
+down: destroy
+
+logs: ## View container logs
+	docker-compose logs -ft app
+
+shell: ## Enter bash in running Docker container
+	docker-compose exec app sh
+
+root: ## Enter bash in running Docker container as root user
+	docker-compose exec --user root app sh

Diff for: README.md

@@ -1,4 +1,5 @@
 # Enable/disable query logger in Laravel/Lumen
+
 [![StyleCI](https://github.styleci.io/repos/477752604/shield?branch=master)](https://github.styleci.io/repos/477752604?branch=master)
 [![CircleCI](https://circleci.com/gh/pnlinh/laravel-query-logger/tree/master.svg?style=svg)](https://circleci.com/gh/pnlinh/laravel-query-logger/tree/master)
 
@@ -15,24 +16,40 @@ Require this package with composer.
 composer require pnlinh/laravel-query-logger --dev
 ```
 
+To publishes config `config/query-logger.php`, use command:
+
+```shell
+php artisan vendor:publish --tag="query-logger"
+```
+
 ## Usage
 
 To enable log query, set .env file below
+
 ```
-APP_DEBUG=true
+QUERY_LOGGER_ENABLED=true
 ```
 
 To disable log query, set .env file below
+
 ```
-APP_DEBUG=false
+QUERY_LOGGER_ENABLED=false
 ```
 
 ## Test
 
+- Without Docker
+
 ```bash
 composer test
 ```
 
+- With Docker
+
+```bash
+make test
+```
+
 ## Credits
 
 - [Ngoc Linh Pham](https://github.com/pnlinh)

Diff for: config/query-logger.php

@@ -0,0 +1,12 @@
+<?php
+
+return [
+    /*
+     |--------------------------------------------------------------------------
+     | Query Logger Settings
+     |--------------------------------------------------------------------------
+     | Query logger is disabled by default. You can enable by setting enable to true.
+     |
+     */
+    'enabled' => env('QUERY_LOGGER_ENABLED', false),
+];

Diff for: docker-compose.yml

@@ -1,9 +1,7 @@
-version: '2'
+version: '3'
 
 services:
   app:
-    image: shinsenter/laravel:php8.0
+    image: pnlinh/php:80
     volumes:
       - ./:/var/www/html
-    networks:
-      - localnet

Diff for: src/QueryLoggerServiceProvider.php

@@ -20,11 +20,19 @@ class QueryLoggerServiceProvider extends ServiceProvider
      *
      * @param Request $request
      *
+     * @throws \Illuminate\Contracts\Container\BindingResolutionException
+     *
      * @return void
      */
     public function boot(Request $request)
     {
-        if (!config('app.debug')) {
+        if ($this->app->runningInConsole()) {
+            $this->publishes([
+                __DIR__.'/../config/query-logger.php' => config_path('query-logger.php'),
+            ], 'query-logger');
+        }
+
+        if (!config('query-logger.enabled')) {
             return;
         }
 

Diff for: tests/TestCase.php

@@ -28,7 +28,7 @@ public function getEnvironmentSetUp($app)
             'database' => ':memory:',
             'prefix'   => '',
         ]);
-        $app['config']->set('app.debug', true);
+        $app['config']->set('query-logger.enabled', true);
         $app['config']->set('app.logging.default', 'single');
 
         Schema::create('users', function (Blueprint $table) {