Implementing Customizable Dashboards - V8.0 #7997

4 Answer(s)

• 

  0

  ismcagdas created 5 years ago

  Support Team

  Hi @dmux

  I couldn't find AppPermissions.Pages_Administration_AuditLogs permission usage for any widget in our code base. Is it possible that someone on your team accidentally did that ?

  Save Cancel
• 

  1

  dmux created 5 years ago

  Hi ismcagdas,

  Thank you for following up.

  I have had a chance to dig a bit deeper and I think I have found the issue.

  In Core/DashboardCustomization/Definitions/DashboardConfiguration.cs this is the pattern provided in the default project:

  var tenantWidgetsDefaultPermission = new List
  {
  AppPermissions\.Pages\_Tenant\_Dashboard
  };
  
  var dailySales = new WidgetDefinition(
      WebPortalDashboardCustomizationConsts.Widgets.Tenant.DailySales,
      "WidgetDailySales",
      side: MultiTenancySides.Tenant,
      usedWidgetFilters: new List<string> { dateRangeFilter.Id },
      permissions: tenantWidgetsDefaultPermission
  );
  
  var generalStats = new WidgetDefinition(
      WebPortalDashboardCustomizationConsts.Widgets.Tenant.GeneralStats,
      "WidgetGeneralStats",
      side: MultiTenancySides.Tenant,
      permissions: tenantWidgetsDefaultPermission
      );
  
  generalStats.Permissions.Add(AppPermissions.Pages_Administration_AuditLogs);
  

  ... and so forth. I haven't yet removed all thses default widgets from my project so they are all still there.

  What happens is that each new widget definition sets its "permissions" to be a pointer to the list called "tenantWidgetsDefaultPermission". The result is that all widgets end up with the same list, which includes any added permissions like the one listed on the last line here. This is why all my widget users required AuditLogs permission for every widget, when they should only require that permission for GeneralStats.

  What I did to stop this behaviour is I added ".ToList()" when setting the widget "permissions" property, so that the permissions list would be a new object. This resolved the issue, like this:

  var generalStats = new WidgetDefinition(
      WebPortalDashboardCustomizationConsts.Widgets.Tenant.GeneralStats,
      "WidgetGeneralStats",
      side: MultiTenancySides.Tenant,
      permissions: tenantWidgetsDefaultPermission.ToList()
      );
  

  There might be nicer ways to do it, but this works for me for the moment.

  Save Cancel
• 

  0

  ismcagdas created 5 years ago

  Support Team

  Thanks @dmux

  Save Cancel
• 

  0

  ISTeam created 4 years ago

  @ismcagdas this issue is still present in MVC Core & jQuery template. I followed exact steps to create my first dashboard widget mentioned in the documentation then I got an error :

  abpexception there is no permission with name "HelloWorld".

  And in fact in the documentation URL there is no mention of how to setup the permission for this particular Widget which they are describing.

  So, in addition to adding ".ToList()" change for generalStats, I had to completely remove the HelloWorld permission. Because all the permissions setup are for pages and there is no specific permission for particular Widget. (i.e. tenantWidgetsDefaultPermission is nothing but AppPermissions.Pages_Tenant_Dashboard)

  And yes even today, the tenant dashboard is empty because of the AuditLog permission mentioned above!

  Thanks @dmux for your explaination and sharing the solution as well!

  Save Cancel