server/AyaNova/biz/WidgetBiz.cs

@@ -13,7 +13,7 @@ namespace AyaNova.Biz
 {
     internal class WidgetBiz : BizObject, IJobObject, ISearchAbleObject, IReportAbleObject
     {
-        
+
 
         internal WidgetBiz(AyContext dbcontext, long currentUserId, long userTranslationId, AuthorizationRoles UserRoles)
         {
@@ -284,9 +284,16 @@ namespace AyaNova.Biz
             //whatever needs to be check to delete this object
         }
 
-
+        //Fulfil report request for data
         public async Task<JArray> GetReportData(long[] idList)
-        {           
+        {
+            //NOTE: Report widget is a superset of biz object widget
+            //Biz objects will add and needed linked records here as extra fields with the data included
+            //for example instead of a userid only there will be username added to the record
+            //so the report designer can just select it as a field, no need to query seperately for it etc
+            //REMEMBER: there is a name display format system and it should honour that so that the report
+            //displays a user the same as it would display in the UI in an input form, no surprises
+
             JArray j = new JArray();
             while (idList.Any())
             {
@@ -299,7 +306,7 @@ namespace AyaNova.Biz
                 foreach (Widget w in orderedList)
                 {
                     j.Add(JObject.FromObject(w));
-                }               
+                }
             }
             return j;
         }

server/AyaNova/models/Report.cs

@@ -29,8 +29,7 @@ namespace AyaNova.Models
         [Required]
         public ReportRenderType RenderType { get; set; }
 
-        //tentative in case of need
-        // public string Locale { get; set; }
+        //tentative in case of need in future
         // public string Header { get; set; }
         // public string Footer { get; set; }