Compare ⋅ danielme85/laravel-log-to-db

No flags found

Use flags to group coverage reports by test type, project and/or folders.
Then setup custom commit statuses and notifications for each flag.

e.g., #unittest #integration

#production #enterprise

#frontend #backend

Learn more about Codecov Flags here.

Showing 2 of 4 files from the diff.

100.00% 100.00% ø 17 0

src/Commands/LogCleanerUpper.php changed.

95.65% ø ø 46 0

src/LogToDB.php changed.

Other files ignored by Codecov

phpunit.xml has changed.

readme.md has changed.

@@ -49,24 +49,22 @@

Loading

        $channels = $this->getLogToDbChannels();

        if (!empty($channels)) {
            foreach ($channels as $channel) {
            foreach ($channels as $name => $channel) {
                $maxRecords = $channel['max_records'] ?? $config['purge_log_when_max_records']  ?? false;
                $maxHours = $channel['max_hours'] ?? $config['purge_log_when_max_records'] ?? false;
                $connection = $channel['connection'] ?? 'default';
                $collection = $channel['collection'] ?? 'log';

                //delete based on numbers of records
                if (!empty($maxRecords) && $maxRecords > 0) {
                    if (LogToDB::model($connection, $collection)->removeOldestIfMoreThan($maxRecords)) {
                        $this->warn("Deleted oldest log records on channel: {$connection}->{$collection}, keep max number of records: {$maxRecords}");
                    if (LogToDB::model($name)->removeOldestIfMoreThan($maxRecords)) {
                        $this->warn("Deleted oldest records on log channel: {$name}, keep max number of records: {$maxRecords}");
                    }
                }

                //delete based on age
                if (!empty($maxHours) && $maxHours > 0) {
                    $time = Carbon::now()->subHours($maxHours)->toDateTimeString();
                    if (LogToDB::model($connection, $collection)->removeOlderThan($time)) {
                        $this->warn("Deleted log records on channel: {$connection}->{$collection}, older than: {$time}");
                    if (LogToDB::model($name)->removeOlderThan($time)) {
                        $this->warn("Deleted oldest records on log channel: {$name}, older than: {$time}");
                    }
                }
            }

@@ -81,9 +79,10 @@

Loading

        $list = [];
        $logging = config('logging');
        if (!empty($logging) && isset($logging['channels']) && !empty($logging['channels'])) {
            foreach ($logging['channels'] as $log) {
                if (isset($log['via']) && $log['via'] === LogToDbHandler::class) {
                    $list[] = $log;
            foreach ($logging['channels'] as $name => $channel) {
                //Only look for the relevant logging class in the config.
                if (isset($channel['via']) && $channel['via'] === LogToDbHandler::class) {
                    $list[$name] = $channel;
                }
            }
        }

@@ -80,6 +80,9 @@

Loading

    /**
     * Return a new LogToDB Module instance.
     *
     * If specifying 'channel, 'connection' and 'collection' would not be needed (they will be extracted from channel).
     * If specifying 'connection' and 'collection', 'channel' is not needed.
     *
     * @param string|null $channel
     * @param string|null $connection
     * @param string|null $collection

Learn more Showing 6 files with coverage changes found.

Changes in src/migrations/2018_08_11_003343_create_log_table.php

→

+1

Loading file...

Changes in src/LogToDbHandler.php

-1

→

Loading file...

New file src/Models/DBLog.php

New

Loading file...

New file src/Models/DBLogMongoDB.php

New

Loading file...

Changes in src/ServiceProvider.php

→

+1

Loading file...

Changes in src/Jobs/SaveNewLogEvent.php

→

+1

Loading file...

96.90% 100.00% +0.05% 116 -2

Fix for issue #28, use channel name/config array indexes to fetch log connection and collection when LogCleanerUpper command uses the static model call: LogToDB::model(channel, connection, collection);

danielme85

e4def4b

CI Passed

+2 Files

+4

96.85% ø ø 118 0

Merge branch 'lumen-support'

danielme85

0e8ad2a

CI Passed

Files		•	•	•	Complexity	Coverage
src	+4	+4	ø	ø	ø	^0.05% 96.90%
Project Totals (13 files)	258	250	0	8	116	96.90%

49	49		$channels = $this->getLogToDbChannels();
50	50
51	51		if (!empty($channels)) {
52		-	foreach ($channels as $channel) {
	52	+	foreach ($channels as $name => $channel) {
53	53		$maxRecords = $channel['max_records'] ?? $config['purge_log_when_max_records'] ?? false;
54	54		$maxHours = $channel['max_hours'] ?? $config['purge_log_when_max_records'] ?? false;
55		-	$connection = $channel['connection'] ?? 'default';
56		-	$collection = $channel['collection'] ?? 'log';
57	55
58	56		//delete based on numbers of records
59	57		if (!empty($maxRecords) && $maxRecords > 0) {
60		-	if (LogToDB::model($connection, $collection)->removeOldestIfMoreThan($maxRecords)) {
61		-	$this->warn("Deleted oldest log records on channel: {$connection}->{$collection}, keep max number of records: {$maxRecords}");
	58	+	if (LogToDB::model($name)->removeOldestIfMoreThan($maxRecords)) {
	59	+	$this->warn("Deleted oldest records on log channel: {$name}, keep max number of records: {$maxRecords}");
62	60		}
63	61		}
64	62
65	63		//delete based on age
66	64		if (!empty($maxHours) && $maxHours > 0) {
67	65		$time = Carbon::now()->subHours($maxHours)->toDateTimeString();
68		-	if (LogToDB::model($connection, $collection)->removeOlderThan($time)) {
69		-	$this->warn("Deleted log records on channel: {$connection}->{$collection}, older than: {$time}");
	66	+	if (LogToDB::model($name)->removeOlderThan($time)) {
	67	+	$this->warn("Deleted oldest records on log channel: {$name}, older than: {$time}");
70	68		}
71	69		}
72	70		}

Navigation	Overlay
`t` Navigate files	`h` Toggle hits
`y` Change url to tip of branch	`m` Toggle misses
`b / v` Jump to prev/next hit line	`p` Toggle partial
`z / x` Jump to prev/next missed or partial line	`1..9` Toggle flags
`shift + o` Open current page in GitHub	`a` Toggle all on
`/ or ?` Show keyboard shortcuts dialog	`c` Toggle context lines or commits

Compare `0e8ad2a ... +0 ... e4def4b`

No flags found

2 Commits

81	79		$list = [];
82	80		$logging = config('logging');
83	81		if (!empty($logging) && isset($logging['channels']) && !empty($logging['channels'])) {
84		-	foreach ($logging['channels'] as $log) {
85		-	if (isset($log['via']) && $log['via'] === LogToDbHandler::class) {
86		-	$list[] = $log;
	82	+	foreach ($logging['channels'] as $name => $channel) {
	83	+	//Only look for the relevant logging class in the config.
	84	+	if (isset($channel['via']) && $channel['via'] === LogToDbHandler::class) {
	85	+	$list[$name] = $channel;
87	86		}
88	87		}
89	88		}

80	80		/**
81	81		* Return a new LogToDB Module instance.
82	82		*
	83	+	* If specifying 'channel, 'connection' and 'collection' would not be needed (they will be extracted from channel).
	84	+	* If specifying 'connection' and 'collection', 'channel' is not needed.
	85	+	*
83	86		* @param string\|null $channel
84	87		* @param string\|null $connection
85	88		* @param string\|null $collection

Compare 0e8ad2a ... +0 ... e4def4b

No flags found

2 Commits

Compare `0e8ad2a ... +0 ... e4def4b`