We now do not show the text Requested By to the user, we also show a 'success' message instead of a warning when something has already been requested

pull/130/head
tidusjar 9 years ago
parent 14d1d279bb
commit 8256ceec2d

@ -109,7 +109,7 @@ namespace PlexRequests.UI.Modules
Admin = isAdmin, Admin = isAdmin,
Issues = movie.Issues.Humanize(LetterCasing.Title), Issues = movie.Issues.Humanize(LetterCasing.Title),
OtherMessage = movie.OtherMessage, OtherMessage = movie.OtherMessage,
AdminNotes = movie.AdminNote AdminNotes = movie.AdminNote,
}; };
}).ToList(); }).ToList();

@ -192,7 +192,7 @@ namespace PlexRequests.UI.Modules
existingRequest.RequestedUsers.Add(Username); existingRequest.RequestedUsers.Add(Username);
RequestService.UpdateRequest(existingRequest); RequestService.UpdateRequest(existingRequest);
} }
return Response.AsJson(new JsonResponseModel { Result = false, Message = settings.UsersCanViewOnlyOwnRequests ? $"{fullMovieName} was successfully added!" : $"{fullMovieName} has already been requested!" }); return Response.AsJson(new JsonResponseModel { Result = true, Message = settings.UsersCanViewOnlyOwnRequests ? $"{fullMovieName} was successfully added!" : $"{fullMovieName} has already been requested!" });
} }
Log.Debug("movie with id {0} doesnt exists", movieId); Log.Debug("movie with id {0} doesnt exists", movieId);
@ -330,7 +330,7 @@ namespace PlexRequests.UI.Modules
existingRequest.RequestedUsers.Add(Username); existingRequest.RequestedUsers.Add(Username);
RequestService.UpdateRequest(existingRequest); RequestService.UpdateRequest(existingRequest);
} }
return Response.AsJson(new JsonResponseModel { Result = false, Message = settings.UsersCanViewOnlyOwnRequests ? $"{fullShowName} was successfully added!" : $"{fullShowName} has already been requested!" }); return Response.AsJson(new JsonResponseModel { Result = true, Message = settings.UsersCanViewOnlyOwnRequests ? $"{fullShowName} was successfully added!" : $"{fullShowName} has already been requested!" });
} }
try try

@ -140,7 +140,9 @@
{{#if_eq type "tv"}} {{#if_eq type "tv"}}
<div>Series Requested: {{seriesRequested}}</div> <div>Series Requested: {{seriesRequested}}</div>
{{/if_eq}} {{/if_eq}}
{{#if requestedUsers}}
<div>Requested By: {{requestedUsers}}</div> <div>Requested By: {{requestedUsers}}</div>
{{/if}}
<div>Requested Date: {{requestedDate}}</div> <div>Requested Date: {{requestedDate}}</div>
<div id="issueArea{{requestId}}"> <div id="issueArea{{requestId}}">
{{#if otherMessage}} {{#if otherMessage}}

Loading…
Cancel
Save