server/AyaNova/biz/Search.cs

@@ -108,7 +108,6 @@ namespace AyaNova.Biz
 
 
 
-
         public static async Task<List<SearchResult>> DoSearch(AyContext ct, long localeId, AuthorizationRoles currentUserRoles, SearchRequestParameters searchParameters)
         {
             List<SearchResult> ResultList = new List<SearchResult>();
@@ -181,12 +180,11 @@ namespace AyaNova.Biz
 
             //In name?
             if (searchParameters.NameOnly)
-                q.Where(m => m.InName == true);
+                q = q.Where(m => m.InName == true);
 
             //Of type?
             if (searchParameters.TypeOnly != AyaType.NoType)
-                q.Where(m => m.ObjectType == searchParameters.TypeOnly);
-
+                q = q.Where(m => m.ObjectType == searchParameters.TypeOnly);
 
 
             //Find the records that have the search terms in searchkey
@@ -296,7 +294,9 @@ namespace AyaNova.Biz
 
             //REMOVE ANY ITEMS THAT USER IS NOT PERMITTED TO READ
             //If it's a name only search then all is allowed
-            //If it's not a name only search then rights need to be checked for full read because even if it's just a tags search that's part of the full record of the object
+            //If it's not a name only search then rights need to be checked for full read because even if it's just a tags search that's part of the full record of the object           
+            //Note: I have decided in the interests of simplicity that even if the result was only found in the name, the user still needs full rights to read the object if the type of search
+            //was not InNameOnly type.  This greatly simplifies processing.
             if (!searchParameters.NameOnly)
             {
                 //list to hold temporary matches