More for #273

PlexRequests.Services.Tests/PlexAvailabilityCheckerTests.cs

@@ -25,19 +25,15 @@
 //  ************************************************************************/
 #endregion
 using System;
-using System.Collections.Generic;
 
 using Moq;
 
 using NUnit.Framework;
 
 using PlexRequests.Api.Interfaces;
-using PlexRequests.Api.Models.Plex;
 using PlexRequests.Core;
 using PlexRequests.Core.SettingModels;
-using PlexRequests.Helpers.Exceptions;
 using PlexRequests.Services.Interfaces;
-using PlexRequests.Store;
 using PlexRequests.Helpers;
 using PlexRequests.Services.Jobs;
 using PlexRequests.Store.Models;

PlexRequests.Services/Jobs/StoreBackup.cs

@@ -152,7 +152,6 @@ namespace PlexRequests.Services.Jobs
             if (names.Length > 1)
             {
                 DateTime parsed;
-                //DateTime.TryParseExcat(names[1], "yyyy-MM-dd hh.mm.ss",CultureInfo.CurrentUICulture, DateTimeStyles.None, out parsed);
                 DateTime.TryParse(names[2], out parsed);
                 return parsed;
 

PlexRequests.UI/Content/requests-1.7.js

@@ -654,10 +654,8 @@ function buildRequestContext(result, type) {
         released: result.released,
         available: result.available,
         admin: result.admin,
-        issues: result.issues,
-        otherMessage: result.otherMessage,
+        issueId: result.issueId,
         requestId: result.id,
-        adminNote: result.adminNotes,
         imdb: result.imdbId,
         seriesRequested: result.tvSeriesRequestType,
         coverArtUrl: result.coverArtUrl,

PlexRequests.UI/Models/RequestViewModel.cs

@@ -49,9 +49,7 @@ namespace PlexRequests.UI.Models
         public string ReleaseYear { get; set; }
         public bool Available { get; set; }
         public bool Admin { get; set; }
-        public string Issues { get; set; }
-        public string OtherMessage { get; set; }
-        public string AdminNotes { get; set; }
+        public int IssueId { get; set; }
         public string TvSeriesRequestType { get; set; }
         public string MusicBrainzId { get; set; }
         public QualityModel[] Qualities { get; set; }

PlexRequests.UI/Modules/BaseAuthModule.cs

@@ -87,21 +87,21 @@ namespace PlexRequests.UI.Modules
 
         protected BaseAuthModule(ISettingsService<PlexRequestSettings> pr) : base(pr)
         {
-            Service = pr;
+            PlexRequestSettings = pr;
             Before += (ctx) => CheckAuth();
         }
 
         protected BaseAuthModule(string modulePath, ISettingsService<PlexRequestSettings> pr) : base(modulePath, pr)
         {
-            Service = pr;
+            PlexRequestSettings = pr;
             Before += (ctx) => CheckAuth();
         }
 
-        private ISettingsService<PlexRequestSettings> Service { get; }
+        protected ISettingsService<PlexRequestSettings> PlexRequestSettings { get; }
 
         private Response CheckAuth()
         {
-            var settings = Service.GetSettings();
+            var settings = PlexRequestSettings.GetSettings();
             var baseUrl = settings.BaseUrl;
 
             var redirectPath = string.IsNullOrEmpty(baseUrl) ? "~/userlogin" : $"~/{baseUrl}/userlogin";

PlexRequests.UI/Modules/IssuesModule.cs

@@ -1,4 +1,6 @@
-using System.Linq;
+using System;
+using System.Collections.Generic;
+using System.Linq;
 using System.Threading.Tasks;
 
 using Nancy;
@@ -23,7 +25,7 @@ namespace PlexRequests.UI.Modules
 
             Get["/issuecount", true] = async (x, ct) => await IssueCount();
 
-            Get["/details/{id}", true] = async (x, ct) => await Details(x.id);
+            Get["/{id}", true] = async (x, ct) => await Details(x.id);
 
             Post["/issue", true] = async (x, ct) => await ReportIssue((int)Request.Form.requestId, (IssueState)(int)Request.Form.issue, null);
             Post["/issuecomment", true] = async (x, ct) => await ReportIssue((int)Request.Form.requestId, IssueState.Other, (string)Request.Form.commentArea);
@@ -40,13 +42,35 @@ namespace PlexRequests.UI.Modules
         public async Task<Response> IssueCount()
         {
             var issues = await IssuesService.GetAllAsync();
-            var count = issues.Count(x => x.Deleted == false);
+            var settings = PlexRequestSettings.GetSettings();
+
+            IEnumerable<IssueModel> myIssues;
+
+            if (IsAdmin)
+            {
+                myIssues = issues.Where(x => x.Deleted == false).SelectMany(i => i.Issues);
+            }
+            else if (settings.UsersCanViewOnlyOwnRequests)
+            {
+                myIssues = (from issuesModel in issues
+                            from i in issuesModel.Issues
+                            where i.UserReported.Equals(Username, StringComparison.CurrentCultureIgnoreCase)
+                            select i).ToList();
+            }
+            else
+            {
+                myIssues = issues.Where(x => x.Deleted == false).SelectMany(i => i.Issues);
+            }
+
+
+            var count = myIssues.Count();
 
             return Response.AsJson(count);
         }
 
         public async Task<Negotiator> Details(int id)
         {
+
             var issue = await IssuesService.GetAsync(id);
 
             return issue == null

PlexRequests.UI/Modules/RequestsModule.cs

@@ -76,7 +76,7 @@ namespace PlexRequests.UI.Modules
             Get["/"] = _ => LoadRequests();
             Get["/movies", true] = async (x, ct) => await GetMovies();
             Get["/tvshows", true] = async (c, ct) => await GetTvShows();
-            Get["/albums", true] = async (x,ct) => await GetAlbumRequests();
+            Get["/albums", true] = async (x, ct) => await GetAlbumRequests();
             Post["/delete", true] = async (x, ct) => await DeleteRequest((int)Request.Form.id);
             Post["/reportissue", true] = async (x, ct) => await ReportIssue((int)Request.Form.requestId, (IssueState)(int)Request.Form.issue, null);
             Post["/reportissuecomment", true] = async (x, ct) => await ReportIssue((int)Request.Form.requestId, IssueState.Other, (string)Request.Form.commentArea);
@@ -156,9 +156,7 @@ namespace PlexRequests.UI.Modules
                 ReleaseYear = movie.ReleaseDate.Year.ToString(),
                 Available = movie.Available,
                 Admin = IsAdmin,
-                Issues = movie.Issues.ToString().CamelCaseToWords(),
-                OtherMessage = movie.OtherMessage,
-                AdminNotes = movie.AdminNote,
+                IssueId = movie.IssueId,
                 Qualities = qualities.ToArray()
             }).ToList();
 
@@ -223,9 +221,7 @@ namespace PlexRequests.UI.Modules
                     ReleaseYear = tv.ReleaseDate.Year.ToString(),
                     Available = tv.Available,
                     Admin = IsAdmin,
-                    Issues = tv.Issues.ToString().CamelCaseToWords(),
-                    OtherMessage = tv.OtherMessage,
-                    AdminNotes = tv.AdminNote,
+                    IssueId = tv.IssueId,
                     TvSeriesRequestType = tv.SeasonsRequested,
                     Qualities = qualities.ToArray()
                 };
@@ -266,9 +262,7 @@ namespace PlexRequests.UI.Modules
                     ReleaseYear = album.ReleaseDate.Year.ToString(),
                     Available = album.Available,
                     Admin = IsAdmin,
-                    Issues = album.Issues.ToString().CamelCaseToWords(),
-                    OtherMessage = album.OtherMessage,
-                    AdminNotes = album.AdminNote,
+                    IssueId = album.IssueId,
                     TvSeriesRequestType = album.SeasonsRequested,
                     MusicBrainzId = album.MusicBrainzId,
                     ArtistName = album.ArtistName

PlexRequests.UI/PlexRequests.UI.csproj

@@ -510,6 +510,9 @@
     <Content Include="Views\Admin\SlackNotifications.cshtml">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </Content>
+    <Content Include="Views\Issues\Index.cshtml">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </Content>
     <None Include="Web.Debug.config">
       <DependentUpon>web.config</DependentUpon>
     </None>

PlexRequests.UI/Views/Issues/Index.cshtml

@@ -0,0 +1 @@
+

PlexRequests.UI/Views/Requests/Index.cshtml

@@ -65,7 +65,7 @@
                             <li><a href="#" class="filter" data-filter=".approved-true"><i class="fa fa-square-o"></i>  Approved</a></li>
                             <li><a href="#" class="filter" data-filter=".approved-false"><i class="fa fa-square-o"></i>  Not Approved</a></li>
                             <li><a href="#" class="filter" data-filter=".available-true"><i class="fa fa-square-o"></i>  Available</a></li>
-	                        <li><a href="#" class="filter" data-filter=".available-false"><i class="fa fa-square-o"></i> Not Available</a></li>
+                            <li><a href="#" class="filter" data-filter=".available-false"><i class="fa fa-square-o"></i> Not Available</a></li>
                             <li><a href="#" class="filter" data-filter=".released-true"><i class="fa fa-square-o"></i>  Released</a></li>
                             <li><a href="#" class="filter" data-filter=".released-false"><i class="fa fa-square-o"></i>  Not Released</a></li>
                         </ul>
@@ -135,7 +135,7 @@
             <div class="col-sm-2">
                 {{#if_eq type "movie"}}
                 {{#if posterPath}}
-                <img class="img-responsive" src="https://image.tmdb.org/t/p/w150/{{posterPath}}"  alt="poster">
+                <img class="img-responsive" src="https://image.tmdb.org/t/p/w150/{{posterPath}}" alt="poster">
                 {{/if}}
                 {{/if_eq}}
                 {{#if_eq type "tv"}}
@@ -178,17 +178,13 @@
                 <div>Requested By: {{requestedUsers}}</div>
                 {{/if}}
                 <div>Requested Date: {{requestedDate}}</div>
-                <div id="issueArea{{requestId}}">
-                    {{#if otherMessage}}
-                    <div>Message: {{otherMessage}}</div>
+                <div>
+                    Issue:
+                    {{#if_eq issueId 0}}
+                    <i class="fa fa-times"></i>
                     {{else}}
-                    <div>Issue: {{issues}}</div>
-                    {{/if}}
-                </div>
-                <div id="adminNotesArea{{requestId}}">
-                    {{#if adminNote}}
-                    <div>Note from Admin: {{adminNote}}</div>
-                    {{/if}}
+                    <a href="/issues/{{issueId}}"><i class="fa fa-check"></i></a>
+                    {{/if_eq}}
                 </div>
             </div>
             <div class="col-sm-3 col-sm-push-3">
@@ -197,20 +193,20 @@
                 <form method="POST" action="@formAction/approval/approve" id="approve{{requestId}}">
                     <input name="requestId" type="text" value="{{requestId}}" hidden="hidden" />
                     {{#if_eq hasQualities true}}
-                        <div class="btn-group btn-split">
-                            <button type="button" class="btn btn-sm btn-success-outline approve" id="{{requestId}}" custom-button="{{requestId}}"><i class="fa fa-plus"></i> Approve</button>
-                            <button type="button" class="btn btn-success-outline dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
-                                <span class="caret"></span>
-                                <span class="sr-only">Toggle Dropdown</span>
-                            </button>
-                            <ul class="dropdown-menu">
-                                {{#each qualities}}
-                                    <li><a href="#" class="approve-with-quality" id="{{id}}">{{name}}</a></li>
-                                {{/each}}
-                            </ul>
-                        </div>
+                    <div class="btn-group btn-split">
+                        <button type="button" class="btn btn-sm btn-success-outline approve" id="{{requestId}}" custom-button="{{requestId}}"><i class="fa fa-plus"></i> Approve</button>
+                        <button type="button" class="btn btn-success-outline dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+                            <span class="caret"></span>
+                            <span class="sr-only">Toggle Dropdown</span>
+                        </button>
+                        <ul class="dropdown-menu">
+                            {{#each qualities}}
+                            <li><a href="#" class="approve-with-quality" id="{{id}}">{{name}}</a></li>
+                            {{/each}}
+                        </ul>
+                    </div>
                     {{else}}
-                        <button id="{{requestId}}" custom-button="{{requestId}}" style="text-align: right" class="btn btn-sm btn-success-outline approve" type="submit"><i class="fa fa-plus"></i> Approve</button>
+                    <button id="{{requestId}}" custom-button="{{requestId}}" style="text-align: right" class="btn btn-sm btn-success-outline approve" type="submit"><i class="fa fa-plus"></i> Approve</button>
                     {{/if_eq}}
                 </form>
                 {{/if_eq}}
@@ -405,7 +401,7 @@
             </div>
             <form method="POST" action="@formAction/requests/addnote" id="noteForm">
                 <div class="modal-body">
-                    <input name="requestId" class="noteId" type="text" hidden="hidden" value=""/>
+                    <input name="requestId" class="noteId" type="text" hidden="hidden" value="" />
                     <textarea class="form-control form-control-custom" rows="3" id="noteArea" name="noteArea"></textarea>
                 </div>
                 <div class="modal-footer">

PlexRequests.UI/Views/Shared/_Layout.cshtml

@@ -39,7 +39,7 @@
                 <ul class="nav navbar-nav">
                     @Html.GetNavbarUrl(Context, "/search", "Search", "search")
                     @Html.GetNavbarUrl(Context, "/requests", "Requests", "plus-circle")
-                    @Html.GetNavbarUrl(Context, "/issues", "Issues", "exclamation", "<span id=\"issueCount\" class=\"badge\">0</span>")
+                    @Html.GetNavbarUrl(Context, "/issues", "Issues", "exclamation", "<span id=\"issueCount\"></span>")
 
                 </ul>
                 <ul class="nav navbar-nav navbar-right">
@@ -133,7 +133,9 @@
             dataType: "json",
             success: function (response) {
                 if (response) {
-                    $('#issueCount').html(response);
+                    if(response > 0)
+                        $('#issueCount').addClass("badge");
+                        $('#issueCount').html(+response);
                 }
             },
             error: function (e) {