Search API fixes - MaxDocumentsNumber should be a property; facet fie…

…lds do not have to have PreviewFunction defined
Orckestra · Feb 2, 2017 · 927042d · 927042d
1 parent 809f86b
commit 927042d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Composite/Plugins/Search/Endpoint/ConsoleSearchRpcService.cs b/Composite/Plugins/Search/Endpoint/ConsoleSearchRpcService.cs
@@ -192,7 +192,7 @@ where selection.Values.Length > 0
                         Label = StringResourceSystemFacade.ParseString(facetField.Label),
                         Facets = selection.Values.Select(value => new ConsoleSearchResultFacetValue
                         {
-                            Label = facetField.Facet.PreviewFunction(value),
+                            Label = (facetField.Facet.PreviewFunction ?? (v => v))(value),
                             Value = value,
                             HitCount = 0
                         }).ToArray()
@@ -222,7 +222,7 @@ private ConsoleSearchResultFacetField[] GetFacets(SearchResult queryResult, ICol
                     {
                         Value = v.Value,
                         HitCount = v.HitCount,
-                        Label = field.Facet.PreviewFunction(v.Value)
+                        Label = (field.Facet.PreviewFunction ?? (value => value))(v.Value)
                     }).ToArray()
                 });
             }

diff --git a/Composite/Search/SearchQuery.cs b/Composite/Search/SearchQuery.cs
@@ -174,7 +174,7 @@ public void FilterByUser(string userName)
         /// <summary>
         /// Maximum amount of documents returned.
         /// </summary>
-        public int MaxDocumentsNumber;
+        public int MaxDocumentsNumber { get; set; }
 
         /// <summary>
         /// Facets to be returned.