Bug Fix #83: Correct case in attachment usernames
This commit is contained in:
@@ -20,7 +20,7 @@ namespace Disco.BI.Extensions
|
|||||||
return true;
|
return true;
|
||||||
|
|
||||||
if (UserService.CurrentAuthorization.Has(Claims.Device.Actions.RemoveOwnAttachments)
|
if (UserService.CurrentAuthorization.Has(Claims.Device.Actions.RemoveOwnAttachments)
|
||||||
&& da.TechUserId == UserService.CurrentUserId)
|
&& da.TechUserId.Equals(UserService.CurrentUserId, StringComparison.OrdinalIgnoreCase))
|
||||||
return true;
|
return true;
|
||||||
|
|
||||||
return false;
|
return false;
|
||||||
@@ -45,7 +45,7 @@ namespace Disco.BI.Extensions
|
|||||||
return true;
|
return true;
|
||||||
|
|
||||||
if (UserService.CurrentAuthorization.Has(Claims.Job.Actions.RemoveOwnAttachments)
|
if (UserService.CurrentAuthorization.Has(Claims.Job.Actions.RemoveOwnAttachments)
|
||||||
&& ja.TechUserId == UserService.CurrentUserId)
|
&& ja.TechUserId.Equals(UserService.CurrentUserId, StringComparison.OrdinalIgnoreCase))
|
||||||
return true;
|
return true;
|
||||||
|
|
||||||
return false;
|
return false;
|
||||||
@@ -70,7 +70,7 @@ namespace Disco.BI.Extensions
|
|||||||
return true;
|
return true;
|
||||||
|
|
||||||
if (UserService.CurrentAuthorization.Has(Claims.User.Actions.RemoveOwnAttachments)
|
if (UserService.CurrentAuthorization.Has(Claims.User.Actions.RemoveOwnAttachments)
|
||||||
&& ua.TechUserId == UserService.CurrentUserId)
|
&& ua.TechUserId.Equals(UserService.CurrentUserId, StringComparison.OrdinalIgnoreCase))
|
||||||
return true;
|
return true;
|
||||||
|
|
||||||
return false;
|
return false;
|
||||||
|
|||||||
@@ -490,7 +490,7 @@ namespace Disco.Web.Areas.API.Controllers
|
|||||||
var da = new DeviceAttachment()
|
var da = new DeviceAttachment()
|
||||||
{
|
{
|
||||||
DeviceSerialNumber = d.SerialNumber,
|
DeviceSerialNumber = d.SerialNumber,
|
||||||
TechUserId = UserService.CurrentUserId,
|
TechUserId = UserService.CurrentUser.UserId,
|
||||||
Filename = file.FileName,
|
Filename = file.FileName,
|
||||||
MimeType = contentType,
|
MimeType = contentType,
|
||||||
Timestamp = DateTime.Now,
|
Timestamp = DateTime.Now,
|
||||||
|
|||||||
@@ -558,7 +558,7 @@ namespace Disco.Web.Areas.API.Controllers
|
|||||||
{
|
{
|
||||||
var undetectedLocation = DataStore.CreateLocation(Database, "DocumentDropBox_Unassigned");
|
var undetectedLocation = DataStore.CreateLocation(Database, "DocumentDropBox_Unassigned");
|
||||||
var filename = System.IO.Path.Combine(undetectedLocation, string.Concat(id, ".pdf"));
|
var filename = System.IO.Path.Combine(undetectedLocation, string.Concat(id, ".pdf"));
|
||||||
if (BI.Interop.Pdf.PdfImporter.ProcessPdfAttachment(filename, Database, DocumentTemplateId, DataId, UserService.CurrentUserId, DateTime.Now))
|
if (BI.Interop.Pdf.PdfImporter.ProcessPdfAttachment(filename, Database, DocumentTemplateId, DataId, UserService.CurrentUser.UserId, DateTime.Now))
|
||||||
{
|
{
|
||||||
// Delete File
|
// Delete File
|
||||||
System.IO.File.Delete(filename);
|
System.IO.File.Delete(filename);
|
||||||
|
|||||||
@@ -75,7 +75,7 @@ namespace Disco.Web.Areas.API.Controllers
|
|||||||
var ua = new Disco.Models.Repository.UserAttachment()
|
var ua = new Disco.Models.Repository.UserAttachment()
|
||||||
{
|
{
|
||||||
UserId = u.UserId,
|
UserId = u.UserId,
|
||||||
TechUserId = UserService.CurrentUserId,
|
TechUserId = UserService.CurrentUser.UserId,
|
||||||
Filename = file.FileName,
|
Filename = file.FileName,
|
||||||
MimeType = contentType,
|
MimeType = contentType,
|
||||||
Timestamp = DateTime.Now,
|
Timestamp = DateTime.Now,
|
||||||
|
|||||||
Reference in New Issue
Block a user