Added initial auto back linking support (#63)

itsff · Oct 24, 2023 · 198c871 · 198c871
1 parent ae3f4bf
commit 198c871
Show file tree

Hide file tree

Showing 3 changed files with 49 additions and 6 deletions.
diff --git a/src/SlownikPolonijny.Dal.Mongo/MongoEntryAuditor.cs b/src/SlownikPolonijny.Dal.Mongo/MongoEntryAuditor.cs
@@ -220,7 +220,14 @@ public void CheckLinks(Entry entry, List<IAuditIssue> problems)
                 {
                     if (!HasBackLink(entry.Name, linkedEntry))
                     {
-                        problems.Add(new GenericAuditIssue($"Jednostronny link. Hasło '{link}' nie jest spokrewnione z '{entry.Name}'"));
+                        var fix = new AutoFixStrategy() { Name = "add-link" };
+                        fix.Parameters["from"] = link;
+                        fix.Parameters["to"] = entry.Name;
+
+                        var issue = new GenericAuditIssue($"Jednostronny link. Hasło '{link}' nie jest spokrewnione z '{entry.Name}'");
+                        issue.Fixes.Add(fix);
+
+                        problems.Add(issue);
                     }
                 }
             }

diff --git a/src/SlownikPolonijny.Dal/IEntryAuditor.cs b/src/SlownikPolonijny.Dal/IEntryAuditor.cs
@@ -8,12 +8,18 @@ public interface IEntryAuditor
     IList<IAuditIssue> PerformEntryAudit(Entry entry);
 }
 
+public class AutoFixStrategy
+{
+    public string Name { get; set; }
+    public Dictionary<string, string> Parameters { get; init; } = new();
+}
+
 public interface IAuditIssue
 {
     string Description { get; }
     bool CanAutoFix { get; }
 
-    void AutoFix ();
+    IList<AutoFixStrategy> Fixes { get; }
 }
 
 public class GenericAuditIssue : IAuditIssue
@@ -25,11 +31,9 @@ public GenericAuditIssue(string description)
 
     public string Description { get; init; }
 
-    public bool CanAutoFix => false;
+    public bool CanAutoFix => this.Fixes.Count > 0;
 
-    public void AutoFix()
-    {
-    }
+    public IList<AutoFixStrategy> Fixes { get; init; } = new List<AutoFixStrategy>();
 
     public override string ToString()
     {

diff --git a/src/SlownikPolonijny.Web/Controllers/AdminController.cs b/src/SlownikPolonijny.Web/Controllers/AdminController.cs
@@ -146,6 +146,38 @@ public async Task<IActionResult> Remove(string id)
         return Json(r);
     }
 
+    [Route("/admin/add-link/{from}/{to}")]
+    [HttpPost]
+    public async Task<IActionResult> AddRelatedLink(string from, string to)
+    {
+        var r = new AddEntryResultModel();
+
+        try
+        {
+            Entry entry = _repo.GetEntryById(from);
+            Entry toEntry = _repo.GetEntryById(to);
+            if (entry != null && toEntry != null)
+            {
+                var user = await this.GetCurrentUserAsync();
+
+                entry.ApprovedBy = user.UserName;
+                entry.SeeAlso.Add(to);
+                _repo.UpdateEntry(entry);
+            }
+            else
+            {
+                r.Problems.Add("Nie ma takiego hasła");
+            }
+        }
+        catch (System.Exception ex)
+        {
+            _logger.LogError(ex.ToString());
+            r.Problems.Add("Błąd bazy danych");
+        }
+
+        return Json(r);
+    }
+
     [Route("/admin/usunieto/{id}/{name}")]
     public IActionResult RemoveConfirmation(string id, string name)
     {