Skip to content
Snippets Groups Projects
Commit 546c7882 authored by Benedikt Heinrichs's avatar Benedikt Heinrichs
Browse files

Remove Build Error

parent cac899da
No related tags found
1 merge request!28Fix: Don't use null project object
......@@ -128,6 +128,11 @@ namespace Coscine.Api.Pid.Controllers
private void LogAnalyticsPidEnquiry(Project project, Resource resource, string applicationsProfile, string license,
IEnumerable<DisciplineObject> disciplines, IEnumerable<OrganizationObject> organizations, IEnumerable<ProjectRole> owners, string email)
{
string visibility = null;
if (project != null && project.VisibilityId.HasValue)
{
visibility = _visibilityModel.GetById(project.VisibilityId.Value)?.DisplayName;
}
_coscineLogger.AnalyticsLog(
new AnalyticsLogObject
{
......@@ -139,7 +144,7 @@ namespace Coscine.Api.Pid.Controllers
License = license,
Disciplines = disciplines?.Select(x => x.DisplayNameEn).ToList(),
Organizations = organizations?.Select(x => x.DisplayName).ToList(),
Visibility = project?.VisibilityId.HasValue ? _visibilityModel.GetById(project.VisibilityId.Value)?.DisplayName : null,
Visibility = visibility,
UserList = owners?.Select(x => x.UserId.ToString()).ToList(),
ExternalInfo = email == null ? null : HashMail(email),
});
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment