for each if break

Steven Langston

I need this code to not add the draft document to the list if the loged in user is not the documents author.

Should it work?

4 Replies

Know the answer? Post it — somebody with the same question will find it here.

Jaimin Shethiya5y ago
Hello Steven,

can you please try with below code.

var filtered = new List();
var loggedInUser = User.FindFirst(ClaimTypes.NameIdentifier);
if(loggedInUser != null)
{
filtered = dbContext.Documents.Where(x=> x.IsDraft == true && x.Author.Equals(loggedInUser.Value, StringComparison.InvariantCultureIgnoreCase)).ToList();
}

let me know if you have any other doubts so will definetly help you.

Thanks
+1
Satya Karki5y ago
Simple way
var filtered = new List () ;

var loggedInUser= User.FindFirst(ClaimTypes.NameIdentifier).Value;
if(loggedInUser != null)
  filtered = dbContext.Documents.where(x=> x.IsDraft == true && x.Author == loggedInUser).toList()
+1
Rijwan Ansari5y ago
Better way to do in this way.
var filtered = new List () ;

var currentUser = User.FindFirst(ClaimTypes.NameIdentifier).Value;
if(currentUser != null)
  filtered = dbContext.Documents.where(x=> x.IsDraft == true && x.Author == currentUser).toList()
+1
Sachin Singh5y ago
don't create additional list when it is not that necessary,
var loogedInUser=User.FindFirst(ClaimTypes.NameIdentifier).Value;
var listofdocuments = dbContext.Documents.where(x=>x.IsDraft==true && x.Author==loogedInUser).ToList();
+2