mirror of
https://github.com/lidarr/lidarr.git
synced 2025-07-16 10:03:51 -07:00
System.Logs view can now be filtered by severity.
This commit is contained in:
parent
af4c351428
commit
9df0ad0bf7
7 changed files with 116 additions and 25 deletions
|
@ -23,6 +23,31 @@ namespace NzbDrone.Api.Logs
|
||||||
pageSpec.SortKey = "id";
|
pageSpec.SortKey = "id";
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (pagingResource.FilterKey == "level")
|
||||||
|
{
|
||||||
|
switch (pagingResource.FilterValue)
|
||||||
|
{
|
||||||
|
case "Fatal":
|
||||||
|
pageSpec.FilterExpression = h => h.Level == "Fatal";
|
||||||
|
break;
|
||||||
|
case "Error":
|
||||||
|
pageSpec.FilterExpression = h => h.Level == "Fatal" || h.Level == "Error";
|
||||||
|
break;
|
||||||
|
case "Warn":
|
||||||
|
pageSpec.FilterExpression = h => h.Level == "Fatal" || h.Level == "Error" || h.Level == "Warn";
|
||||||
|
break;
|
||||||
|
case "Info":
|
||||||
|
pageSpec.FilterExpression = h => h.Level == "Fatal" || h.Level == "Error" || h.Level == "Warn" || h.Level == "Info";
|
||||||
|
break;
|
||||||
|
case "Debug":
|
||||||
|
pageSpec.FilterExpression = h => h.Level == "Fatal" || h.Level == "Error" || h.Level == "Warn" || h.Level == "Info" || h.Level == "Debug";
|
||||||
|
break;
|
||||||
|
case "Trace":
|
||||||
|
pageSpec.FilterExpression = h => h.Level == "Fatal" || h.Level == "Error" || h.Level == "Warn" || h.Level == "Info" || h.Level == "Debug" || h.Level == "Trace";
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
return ApplyToPage(_logService.Paged, pageSpec);
|
return ApplyToPage(_logService.Paged, pageSpec);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -9,6 +9,8 @@ namespace NzbDrone.Api
|
||||||
public int PageSize { get; set; }
|
public int PageSize { get; set; }
|
||||||
public string SortKey { get; set; }
|
public string SortKey { get; set; }
|
||||||
public SortDirection SortDirection { get; set; }
|
public SortDirection SortDirection { get; set; }
|
||||||
|
public string FilterKey { get; set; }
|
||||||
|
public string FilterValue { get; set; }
|
||||||
public int TotalRecords { get; set; }
|
public int TotalRecords { get; set; }
|
||||||
public List<TModel> Records { get; set; }
|
public List<TModel> Records { get; set; }
|
||||||
}
|
}
|
||||||
|
|
|
@ -241,6 +241,16 @@ namespace NzbDrone.Api.REST
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (Request.Query.FilterKey != null)
|
||||||
|
{
|
||||||
|
pagingResource.FilterKey = Request.Query.FilterKey.ToString();
|
||||||
|
|
||||||
|
if (Request.Query.FilterValue != null)
|
||||||
|
{
|
||||||
|
pagingResource.FilterValue = Request.Query.FilterValue.ToString();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
return pagingResource;
|
return pagingResource;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -210,18 +210,20 @@ namespace NzbDrone.Core.Datastore
|
||||||
|
|
||||||
public virtual PagingSpec<TModel> GetPaged(PagingSpec<TModel> pagingSpec)
|
public virtual PagingSpec<TModel> GetPaged(PagingSpec<TModel> pagingSpec)
|
||||||
{
|
{
|
||||||
var pagingQuery = Query.OrderBy(pagingSpec.OrderByClause(), pagingSpec.ToSortDirection())
|
pagingSpec.Records = GetPagedQuery(Query, pagingSpec).ToList();
|
||||||
.Skip(pagingSpec.PagingOffset())
|
pagingSpec.TotalRecords = GetPagedQuery(Query, pagingSpec).GetRowCount();
|
||||||
.Take(pagingSpec.PageSize);
|
|
||||||
|
|
||||||
pagingSpec.Records = pagingQuery.ToList();
|
|
||||||
|
|
||||||
//TODO: Use the same query for count and records
|
|
||||||
pagingSpec.TotalRecords = Count();
|
|
||||||
|
|
||||||
return pagingSpec;
|
return pagingSpec;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
protected virtual SortBuilder<TModel> GetPagedQuery(QueryBuilder<TModel> query, PagingSpec<TModel> pagingSpec)
|
||||||
|
{
|
||||||
|
return query.Where(pagingSpec.FilterExpression)
|
||||||
|
.OrderBy(pagingSpec.OrderByClause(), pagingSpec.ToSortDirection())
|
||||||
|
.Skip(pagingSpec.PagingOffset())
|
||||||
|
.Take(pagingSpec.PageSize);
|
||||||
|
}
|
||||||
|
|
||||||
public void DeleteAll()
|
public void DeleteAll()
|
||||||
{
|
{
|
||||||
DataMapper.Delete<TModel>(c => c.Id > 0);
|
DataMapper.Delete<TModel>(c => c.Id > 0);
|
||||||
|
|
|
@ -67,22 +67,12 @@ namespace NzbDrone.Core.History
|
||||||
.FirstOrDefault();
|
.FirstOrDefault();
|
||||||
}
|
}
|
||||||
|
|
||||||
public override PagingSpec<History> GetPaged(PagingSpec<History> pagingSpec)
|
protected override SortBuilder<History> GetPagedQuery(QueryBuilder<History> query, PagingSpec<History> pagingSpec)
|
||||||
{
|
{
|
||||||
pagingSpec.Records = GetPagedQuery(pagingSpec).ToList();
|
var baseQuery = query.Join<History, Series>(JoinType.Inner, h => h.Series, (h, s) => h.SeriesId == s.Id)
|
||||||
pagingSpec.TotalRecords = GetPagedQuery(pagingSpec).GetRowCount();
|
.Join<History, Episode>(JoinType.Inner, h => h.Episode, (h, e) => h.EpisodeId == e.Id);
|
||||||
|
|
||||||
return pagingSpec;
|
return base.GetPagedQuery(baseQuery, pagingSpec);
|
||||||
}
|
|
||||||
|
|
||||||
private SortBuilder<History> GetPagedQuery(PagingSpec<History> pagingSpec)
|
|
||||||
{
|
|
||||||
return Query.Join<History, Series>(JoinType.Inner, h => h.Series, (h, s) => h.SeriesId == s.Id)
|
|
||||||
.Join<History, Episode>(JoinType.Inner, h => h.Episode, (h, e) => h.EpisodeId == e.Id)
|
|
||||||
.Where(pagingSpec.FilterExpression)
|
|
||||||
.OrderBy(pagingSpec.OrderByClause(), pagingSpec.ToSortDirection())
|
|
||||||
.Skip(pagingSpec.PagingOffset())
|
|
||||||
.Take(pagingSpec.PageSize);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
|
@ -4,9 +4,10 @@ define(
|
||||||
[
|
[
|
||||||
'backbone.pageable',
|
'backbone.pageable',
|
||||||
'System/Logs/LogsModel',
|
'System/Logs/LogsModel',
|
||||||
|
'Mixins/AsFilteredCollection',
|
||||||
'Mixins/AsPersistedStateCollection'
|
'Mixins/AsPersistedStateCollection'
|
||||||
],
|
],
|
||||||
function (PagableCollection, LogsModel, AsPersistedStateCollection) {
|
function (PagableCollection, LogsModel, AsFilteredCollection, AsPersistedStateCollection) {
|
||||||
var collection = PagableCollection.extend({
|
var collection = PagableCollection.extend({
|
||||||
url : window.NzbDrone.ApiRoot + '/log',
|
url : window.NzbDrone.ApiRoot + '/log',
|
||||||
model: LogsModel,
|
model: LogsModel,
|
||||||
|
@ -30,6 +31,14 @@ define(
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
|
||||||
|
// Filter Modes
|
||||||
|
filterModes: {
|
||||||
|
'all' : [null, null],
|
||||||
|
'info' : ['level', 'Info'],
|
||||||
|
'warn' : ['level', 'Warn'],
|
||||||
|
'error' : ['level', 'Error']
|
||||||
|
},
|
||||||
|
|
||||||
parseState: function (resp, queryParams, state) {
|
parseState: function (resp, queryParams, state) {
|
||||||
return {totalRecords: resp.totalRecords};
|
return {totalRecords: resp.totalRecords};
|
||||||
},
|
},
|
||||||
|
@ -43,5 +52,7 @@ define(
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
return AsPersistedStateCollection.call(collection);
|
collection = AsFilteredCollection.apply(collection);
|
||||||
|
|
||||||
|
return AsPersistedStateCollection.apply(collection);
|
||||||
});
|
});
|
||||||
|
|
|
@ -66,7 +66,6 @@ define(
|
||||||
|
|
||||||
onRender: function () {
|
onRender: function () {
|
||||||
this.grid.show(new LoadingView());
|
this.grid.show(new LoadingView());
|
||||||
this.collection.fetch();
|
|
||||||
},
|
},
|
||||||
|
|
||||||
onShow: function () {
|
onShow: function () {
|
||||||
|
@ -88,6 +87,44 @@ define(
|
||||||
},
|
},
|
||||||
|
|
||||||
_showToolbar: function () {
|
_showToolbar: function () {
|
||||||
|
var filterButtons = {
|
||||||
|
type : 'radio',
|
||||||
|
storeState : true,
|
||||||
|
menuKey : 'logs.filterMode',
|
||||||
|
defaultAction: 'all',
|
||||||
|
items :
|
||||||
|
[
|
||||||
|
{
|
||||||
|
key : 'all',
|
||||||
|
title : '',
|
||||||
|
tooltip : 'All',
|
||||||
|
icon : 'icon-circle-blank',
|
||||||
|
callback : this._setFilter
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key : 'info',
|
||||||
|
title : '',
|
||||||
|
tooltip : 'Info',
|
||||||
|
icon : 'icon-info',
|
||||||
|
callback : this._setFilter
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key : 'warn',
|
||||||
|
title : '',
|
||||||
|
tooltip : 'Warn',
|
||||||
|
icon : 'icon-warn',
|
||||||
|
callback : this._setFilter
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key : 'error',
|
||||||
|
title : '',
|
||||||
|
tooltip : 'Error',
|
||||||
|
icon : 'icon-error',
|
||||||
|
callback : this._setFilter
|
||||||
|
}
|
||||||
|
]
|
||||||
|
};
|
||||||
|
|
||||||
var rightSideButtons = {
|
var rightSideButtons = {
|
||||||
type : 'default',
|
type : 'default',
|
||||||
storeState: false,
|
storeState: false,
|
||||||
|
@ -111,6 +148,7 @@ define(
|
||||||
this.toolbar.show(new ToolbarLayout({
|
this.toolbar.show(new ToolbarLayout({
|
||||||
right :
|
right :
|
||||||
[
|
[
|
||||||
|
filterButtons,
|
||||||
rightSideButtons
|
rightSideButtons
|
||||||
],
|
],
|
||||||
context: this
|
context: this
|
||||||
|
@ -125,6 +163,19 @@ define(
|
||||||
buttonContext.ui.icon.spinForPromise(promise);
|
buttonContext.ui.icon.spinForPromise(promise);
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
|
|
||||||
|
_setFilter: function(buttonContext) {
|
||||||
|
var mode = buttonContext.model.get('key');
|
||||||
|
|
||||||
|
this.collection.setFilterMode(mode, { reset: false });
|
||||||
|
|
||||||
|
this.collection.state.currentPage = 1;
|
||||||
|
var promise = this.collection.fetch({ reset: true });
|
||||||
|
|
||||||
|
if (buttonContext) {
|
||||||
|
buttonContext.ui.icon.spinForPromise(promise);
|
||||||
|
}
|
||||||
|
},
|
||||||
|
|
||||||
_commandComplete: function (options) {
|
_commandComplete: function (options) {
|
||||||
if (options.command.get('name') === 'clearlog') {
|
if (options.command.get('name') === 'clearlog') {
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue