Skip to content

Commit 78eba39

Browse files
authored
Merge pull request #10 from melihovv/laravel-7-support
Laravel 7 support
2 parents d03e33b + a50236e commit 78eba39

18 files changed

Lines changed: 527 additions & 289 deletions

.coveralls.yml

Lines changed: 0 additions & 3 deletions
This file was deleted.

.github/workflows/default.yml

Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
name: Run tests
2+
3+
on:
4+
push:
5+
schedule:
6+
- cron: '0 0 * * *'
7+
8+
jobs:
9+
php-tests:
10+
runs-on: ${{ matrix.os }}
11+
12+
strategy:
13+
matrix:
14+
php: [7.4, 7.3, 7.2]
15+
laravel: [7.*, 6.*]
16+
dependency-version: [prefer-lowest, prefer-stable]
17+
os: [ubuntu-latest, windows-latest]
18+
include:
19+
- laravel: 7.*
20+
testbench: 5.*
21+
- laravel: 6.*
22+
testbench: 4.*
23+
24+
name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.dependency-version }} - ${{ matrix.os }}
25+
26+
steps:
27+
- name: Checkout code
28+
uses: actions/checkout@v1
29+
30+
- name: Setup PHP
31+
uses: shivammathur/setup-php@v1
32+
with:
33+
php-version: ${{ matrix.php }}
34+
extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick, fileinfo
35+
coverage: none
36+
37+
- name: Install dependencies
38+
run: |
39+
composer require "laravel/framework:${{ matrix.laravel }}" "orchestra/testbench:${{ matrix.testbench }}" --no-interaction --no-update
40+
composer update --${{ matrix.dependency-version }} --prefer-dist --no-interaction --no-suggest
41+
- name: Execute tests
42+
run: composer run phpunit

.styleci.yml

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1 @@
1-
preset: recommended
2-
enabled:
3-
- concat_with_spaces
4-
disabled:
5-
- align_double_arrow
6-
- phpdoc_to_comment
7-
- phpdoc_var_without_name
8-
- concat_without_spaces
9-
- no_multiline_whitespace_around_double_arrow
1+
preset: laravel

.travis.yml

Lines changed: 0 additions & 47 deletions
This file was deleted.

README.md

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,8 @@
11
Laravel log viewer
22
==================
33

4-
[![Build Status](https://travis-ci.org/melihovv/laravel-log-viewer.svg?branch=master)](https://travis-ci.org/melihovv/laravel-log-viewer)
4+
![GitHub Workflow Status](https://img.shields.io/github/workflow/status/melihovv/laravel-log-viewer/run-php-tests?label=Tests)
55
[![styleci](https://styleci.io/repos/78041678/shield)](https://styleci.io/repos/78041678)
6-
[![Coverage Status](https://coveralls.io/repos/github/melihovv/laravel-log-viewer/badge.svg?branch=master)](https://coveralls.io/github/melihovv/laravel-log-viewer?branch=master)
76

87
[![Packagist](https://img.shields.io/packagist/v/melihovv/laravel-log-viewer.svg)](https://packagist.org/packages/melihovv/laravel-log-viewer)
98
[![Packagist](https://poser.pugx.org/melihovv/laravel-log-viewer/d/total.svg)](https://packagist.org/packages/melihovv/laravel-log-viewer)
@@ -28,23 +27,23 @@ composer require melihovv/laravel-log-viewer
2827
```
2928

3029
Add a route in your web routes file:
31-
```php
32-
Route::get('logs', '\Melihovv\LaravelLogViewer\LaravelLogViewerController@index');
30+
```php
31+
Route::get('logs', '\Melihovv\LaravelLogViewer\Controller@index');
3332
```
3433

35-
Go to `http://myapp/logs`
34+
Go to `http://localhost:8000/logs`
3635

3736
Additional
3837
----------
3938

4039
Publish package config if you want customize default values
4140
```
42-
php artisan vendor:publish --tag=config
41+
php artisan vendor:publish --provider="Melihovv\LaravelLogViewerController\ServiceProvider" --tag=config
4342
```
4443

4544
If you want customize package view
4645
```
47-
php artisan vendor:publish --tag=views
46+
php artisan vendor:publish --provider="Melihovv\LaravelLogViewerController\ServiceProvider" --tag=views
4847
```
4948

5049
## Security

composer.json

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,22 +19,28 @@
1919
],
2020
"require": {
2121
"php": ">=7.2",
22-
"illuminate/filesystem": "^6.0",
23-
"illuminate/support": "^6.0"
22+
"ext-fileinfo": "*",
23+
"illuminate/config": "^6.0|^7.0",
24+
"illuminate/filesystem": "^6.0|^7.0",
25+
"illuminate/http": "^6.0|^7.0",
26+
"illuminate/routing": "^6.0|^7.0",
27+
"illuminate/support": "^6.0|^7.0",
28+
"illuminate/view": "^6.0|^7.0"
2429
},
2530
"require-dev": {
26-
"mockery/mockery": "^1.0",
27-
"orchestra/testbench": "^4.0",
28-
"phpunit/phpunit": "^8.0"
31+
"mockery/mockery": "^1.3",
32+
"orchestra/testbench": "^4.0|^5.0",
33+
"orchestra/testbench-browser-kit": "^4.0|^5.0",
34+
"phpunit/phpunit": "^8.4|^9.0"
2935
},
3036
"autoload": {
3137
"psr-4": {
32-
"Melihovv\\LaravelLogViewer\\": "src/LaravelLogViewer"
38+
"Melihovv\\LaravelLogViewer\\": "src"
3339
}
3440
},
3541
"autoload-dev": {
3642
"psr-4": {
37-
"Melihovv\\LaravelLogViewer\\Tests\\": "tests/"
43+
"Melihovv\\LaravelLogViewer\\Tests\\": "tests"
3844
}
3945
},
4046
"scripts": {
@@ -43,7 +49,7 @@
4349
"extra": {
4450
"laravel": {
4551
"providers": [
46-
"Melihovv\\LaravelLogViewer\\LaravelLogViewerServiceProvider"
52+
"Melihovv\\LaravelLogViewer\\ServiceProvider"
4753
]
4854
}
4955
},

src/Controller.php

Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,48 @@
1+
<?php
2+
3+
namespace Melihovv\LaravelLogViewer;
4+
5+
use Illuminate\Http\Request;
6+
use Illuminate\Support\Facades\File;
7+
use Illuminate\Support\Facades\Response;
8+
use Illuminate\Support\Facades\View;
9+
use InvalidArgumentException;
10+
11+
class Controller
12+
{
13+
public function index(Request $request)
14+
{
15+
$viewer = app('log-viewer');
16+
$error = false;
17+
18+
try {
19+
if ($request->has('file')) {
20+
$viewer->setCurrentFile(base64_decode($request->get('file')));
21+
22+
if ($request->has('download')) {
23+
return Response::download($viewer->getCurrentFile());
24+
}
25+
26+
if ($request->has('delete')) {
27+
File::delete($viewer->getCurrentFile());
28+
29+
return Response::redirectTo($request->url());
30+
}
31+
} elseif ($request->has('dir')) {
32+
$viewer->setCurrentDirectory(base64_decode($request->get('dir')));
33+
}
34+
} catch (InvalidArgumentException $e) {
35+
$error = true;
36+
}
37+
38+
return View::make('log-viewer::log', [
39+
'logs' => $viewer->getLogsFromCurrentFile(),
40+
'dirItems' => $viewer->getCurrentDirectoryContent(),
41+
'currentFile' => $viewer->getCurrentFileRelativeToBaseDir(),
42+
'parentDirPath' => $viewer->getRelativePathToCurrentDirectoryParent(),
43+
'isCurrentDirectoryBase' => $viewer->isCurrentDirectoryBase(),
44+
'parentDirs' => $viewer->getParentDirectories(),
45+
'error' => $error,
46+
]);
47+
}
48+
}

src/LaravelLogViewer/Facades/LaravelLogViewer.php renamed to src/Facades/LogViewer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
use Illuminate\Support\Facades\Facade;
66

7-
class LaravelLogViewer extends Facade
7+
class LogViewer extends Facade
88
{
99
protected static function getFacadeAccessor()
1010
{

src/LaravelLogViewer/LaravelLogViewerController.php

Lines changed: 0 additions & 39 deletions
This file was deleted.

0 commit comments

Comments
 (0)