diff --git a/src/Orchard.Web/Modules/Orchard.Indexing/Commands/IndexingCommands.cs b/src/Orchard.Web/Modules/Orchard.Indexing/Commands/IndexingCommands.cs index e199a83cc..a89a41fb6 100644 --- a/src/Orchard.Web/Modules/Orchard.Indexing/Commands/IndexingCommands.cs +++ b/src/Orchard.Web/Modules/Orchard.Indexing/Commands/IndexingCommands.cs @@ -4,7 +4,6 @@ using Orchard.Commands; using Orchard.ContentManagement; using Orchard.Indexing.Services; using Orchard.Tasks.Indexing; -using Orchard.Utility.Extensions; using static Orchard.Indexing.Helpers.IndexingHelpers; namespace Orchard.Indexing.Commands { @@ -92,19 +91,19 @@ namespace Orchard.Indexing.Commands { return; } - if ( !_indexManager.HasIndexProvider() ) { + if (!_indexManager.HasIndexProvider()) { Context.Output.WriteLine(T("No index available")); return; } var searchBuilder = _indexManager.GetSearchIndexProvider().CreateSearchBuilder(index); - var results = searchBuilder.Parse( new [] {"body", "title"}, Query).Search(); + var results = searchBuilder.Parse(new[] { "body", "title" }, Query).Search(); Context.Output.WriteLine("{0} result{1}\r\n-----------------\r\n", results.Count(), results.Count() > 0 ? "s" : ""); Context.Output.WriteLine("┌──────────────────────────────────────────────────────────────┬────────┐"); Context.Output.WriteLine("│ {0} │ {1,6} │", "Title" + new string(' ', 60 - "Title".Length), "Score"); Context.Output.WriteLine("├──────────────────────────────────────────────────────────────┼────────┤"); - foreach ( var searchHit in results ) { + foreach (var searchHit in results) { var contentItem = _contentManager.Get(searchHit.ContentItemId); var metadata = _contentManager.GetItemMetadata(contentItem); var title = String.IsNullOrWhiteSpace(metadata.DisplayText) ? "- no title -" : metadata.DisplayText; @@ -127,7 +126,7 @@ namespace Orchard.Indexing.Commands { return; } - if ( !_indexManager.HasIndexProvider() ) { + if (!_indexManager.HasIndexProvider()) { Context.Output.WriteLine(T("No index available")); return; } @@ -136,11 +135,10 @@ namespace Orchard.Indexing.Commands { } [CommandName("index refresh")] - [CommandHelp("index refresh /ContentItem: \r\n\t" + "Refreshes the index for the specifed ")] + [CommandHelp("index refresh /ContentItem: \r\n\t" + "Refreshes the index for the specified ")] [OrchardSwitches("ContentItem")] public void Refresh() { - int contentItemId; - if ( !int.TryParse(ContentItem, out contentItemId) ) { + if (!int.TryParse(ContentItem, out int contentItemId)) { Context.Output.WriteLine(T("Invalid content item id. Not an integer.")); return; } @@ -148,19 +146,18 @@ namespace Orchard.Indexing.Commands { var contentItem = _contentManager.Get(contentItemId); _indexingTaskManager.CreateUpdateIndexTask(contentItem); - Context.Output.WriteLine(T("Content Item marked for reindexing")); + Context.Output.WriteLine(T("Content Item marked for re-indexing")); } [CommandName("index delete")] - [CommandHelp("index delete /ContentItem:\r\n\t" + "Deletes the specifed from the index")] + [CommandHelp("index delete /ContentItem:\r\n\t" + "Deletes the specified from the index")] [OrchardSwitches("ContentItem")] public void Delete() { - int contentItemId; - if(!int.TryParse(ContentItem, out contentItemId)) { + if (!int.TryParse(ContentItem, out int contentItemId)) { Context.Output.WriteLine(T("Invalid content item id. Not an integer.")); return; } - + var contentItem = _contentManager.Get(contentItemId); _indexingTaskManager.CreateDeleteIndexTask(contentItem); diff --git a/src/Orchard.Web/Modules/Orchard.Indexing/Controllers/AdminController.cs b/src/Orchard.Web/Modules/Orchard.Indexing/Controllers/AdminController.cs index a7868e9e0..ab3c25f96 100644 --- a/src/Orchard.Web/Modules/Orchard.Indexing/Controllers/AdminController.cs +++ b/src/Orchard.Web/Modules/Orchard.Indexing/Controllers/AdminController.cs @@ -2,12 +2,11 @@ using System.Linq; using System.Web.Mvc; using Orchard.Indexing.Services; +using Orchard.Indexing.ViewModels; using Orchard.Localization; using Orchard.Logging; using Orchard.Security; -using Orchard.Indexing.ViewModels; using Orchard.UI.Notify; -using Orchard.Utility.Extensions; using static Orchard.Indexing.Helpers.IndexingHelpers; namespace Orchard.Indexing.Controllers { @@ -16,7 +15,7 @@ namespace Orchard.Indexing.Controllers { private readonly IIndexManager _indexManager; public AdminController( - IIndexingService indexingService, + IIndexingService indexingService, IOrchardServices services, IIndexManager indexManager) { _indexingService = indexingService; @@ -35,23 +34,21 @@ namespace Orchard.Indexing.Controllers { IndexEntries = Enumerable.Empty(), IndexProvider = _indexManager.GetSearchIndexProvider() }; - + if (_indexManager.HasIndexProvider()) { viewModel.IndexEntries = _indexManager.GetSearchIndexProvider().List().Select(x => { try { return _indexingService.GetIndexEntry(x); } - catch(Exception e) { + catch (Exception e) { Logger.Error(e, "Index couldn't be read: " + x); - return new IndexEntry { + return new IndexEntry { IndexName = x, IndexingStatus = IndexingStatus.Unavailable }; } }); } - - // Services.Notifier.Information(T("The index might be corrupted. If you can't recover click on Rebuild.")); return View(viewModel); } @@ -60,7 +57,7 @@ namespace Orchard.Indexing.Controllers { if (!Services.Authorizer.Authorize(StandardPermissions.SiteOwner, T("Not allowed to manage the search index."))) return new HttpUnauthorizedResult(); - return View("Create", String.Empty); + return View("Create"); } [HttpPost, ActionName("Create")] @@ -83,9 +80,9 @@ namespace Orchard.Indexing.Controllers { provider.CreateIndex(id); Services.Notifier.Information(T("Index named {0} created successfully", id)); } - catch(Exception e) { + catch (Exception e) { Services.Notifier.Error(T("An error occurred while creating the index: {0}", id)); - Logger.Error("An error occurred while creatign the index " + id, e); + Logger.Error("An error occurred while creating the index " + id, e); return View("Create", id); }