Skip to content

Commit dce4f0b

Browse files
authored
Merge pull request #459 from TechnologyEnhancedLearning/Develop/Feature/TD-2895-2-ActivityStatus
Develop/feature/td 2895 2 activity status
2 parents f629282 + 58aa35d commit dce4f0b

File tree

27 files changed

+536
-89
lines changed

27 files changed

+536
-89
lines changed

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,3 +48,5 @@ obj
4848
/WebAPI/MigrationTool/LearningHub.Nhs.Migration.Staging.Database/LearningHub.Nhs.Migration.Staging.Database.dbmdl
4949
/WebAPI/MigrationTool/LearningHub.Nhs.Migration.Staging.Database/LearningHub.Nhs.Migration.Staging.Database.jfm
5050
/LearningHub.Nhs.WebUI.AutomatedUiTests/appsettings.Development.json
51+
/OpenAPI/LearningHub.Nhs.OpenApi/appsettings.Development.json
52+
/OpenAPI/LearningHub.Nhs.OpenApi/web.config

OpenAPI/LearningHub.Nhs.OpenApi.Models/LearningHub.Nhs.OpenApi.Models.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
</PropertyGroup>
1717

1818
<ItemGroup>
19-
<PackageReference Include="LearningHub.Nhs.Models" Version="2.1.1" />
19+
<PackageReference Include="LearningHub.Nhs.Models" Version="3.0.33" />
2020
<PackageReference Include="NLog.Web.AspNetCore" Version="4.13.0" />
2121
</ItemGroup>
2222

OpenAPI/LearningHub.Nhs.OpenApi.Models/ViewModels/ResourceMetadataViewModel.cs

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
namespace LearningHub.Nhs.OpenApi.Models.ViewModels
22
{
3+
using LearningHub.Nhs.Models.Entities.Activity;
34
using System.Collections.Generic;
45

6+
57
/// <summary>
68
/// Class.
79
/// </summary>
@@ -23,20 +25,25 @@ public ResourceMetadataViewModel()
2325
/// <param name="references"><see cref="References"/>.</param>
2426
/// <param name="resourceType"><see cref="ResourceType"/>.</param>
2527
/// <param name="rating"><see cref="Rating"/>.</param>
28+
/// <param name="userSummaryActivityStatuses"><see cref="UserSummaryActivityStatuses"/>.</param>
2629
public ResourceMetadataViewModel(
2730
int resourceId,
2831
string title,
2932
string description,
3033
List<ResourceReferenceViewModel> references,
3134
string resourceType,
32-
decimal rating)
35+
int? majorVersion,
36+
decimal rating,
37+
List<MajorVersionIdActivityStatusDescription> userSummaryActivityStatuses)
3338
{
3439
this.ResourceId = resourceId;
3540
this.Title = title;
3641
this.Description = description;
3742
this.References = references;
3843
this.ResourceType = resourceType;
44+
this.MajorVersion = majorVersion;
3945
this.Rating = rating;
46+
this.UserSummaryActivityStatuses = userSummaryActivityStatuses;
4047
}
4148

4249
/// <summary>
@@ -64,9 +71,20 @@ public ResourceMetadataViewModel(
6471
/// </summary>
6572
public string ResourceType { get; set; }
6673

74+
/// <summary>
75+
/// Gets or sets <see cref="MajorVersion"/>.
76+
/// </summary>
77+
public int? MajorVersion { get; set; }
78+
79+
6780
/// <summary>
6881
/// Gets or sets <see cref="Rating"/>.
6982
/// </summary>
7083
public decimal Rating { get; set; }
84+
85+
/// <summary>
86+
/// Gets or sets <see cref="UserSummaryActivityStatuses"/>.
87+
/// </summary>
88+
public List<MajorVersionIdActivityStatusDescription> UserSummaryActivityStatuses { get; set; }
7189
}
7290
}

OpenAPI/LearningHub.Nhs.OpenApi.Models/ViewModels/ResourceReferenceWithResourceDetailsViewModel.cs

Lines changed: 23 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
1+
using LearningHub.Nhs.Models.Entities.Activity;
2+
using System.Collections.Generic;
3+
14
namespace LearningHub.Nhs.OpenApi.Models.ViewModels
25
{
36
/// <summary>
@@ -14,26 +17,32 @@ public class ResourceReferenceWithResourceDetailsViewModel
1417
/// <param name="description"><see cref="Description"/>.</param>
1518
/// <param name="catalogueViewModel"><see cref="Catalogue"/>.</param>
1619
/// <param name="resourceType"><see cref="ResourceType"/>.</param>
20+
/// <param name="majorVersion"></param>
1721
/// <param name="rating"><see cref="Rating"/>.</param>
1822
/// <param name="link"><see cref="Link"/>.</param>
23+
/// <param name="userSummaryActivityStatuses"></param>
1924
public ResourceReferenceWithResourceDetailsViewModel(
2025
int resourceId,
2126
int refId,
2227
string title,
2328
string description,
2429
CatalogueViewModel catalogueViewModel,
2530
string resourceType,
31+
int? majorVersion,
2632
decimal rating,
27-
string link)
33+
string link,
34+
List<MajorVersionIdActivityStatusDescription> userSummaryActivityStatuses)
2835
{
2936
this.ResourceId = resourceId;
3037
this.RefId = refId;
3138
this.Title = title;
3239
this.Description = description;
3340
this.Catalogue = catalogueViewModel;
41+
this.MajorVersion = majorVersion;
3442
this.ResourceType = resourceType;
3543
this.Rating = rating;
3644
this.Link = link;
45+
this.UserSummaryActivityStatuses = userSummaryActivityStatuses;
3746
}
3847

3948
/// <summary>
@@ -66,14 +75,27 @@ public ResourceReferenceWithResourceDetailsViewModel(
6675
/// </summary>
6776
public string ResourceType { get; }
6877

78+
79+
/// <summary>
80+
/// Gets <see cref="MajorVersion"/>.
81+
/// </summary>
82+
public int? MajorVersion { get; }
83+
6984
/// <summary>
7085
/// Gets <see cref="Rating"/>.
7186
/// </summary>
87+
///
88+
7289
public decimal Rating { get; }
7390

7491
/// <summary>
7592
/// Gets <see cref="Link"/>.
7693
/// </summary>
7794
public string Link { get; }
95+
96+
/// <summary>
97+
/// Gets <see cref="UserSummaryActivityStatuses"/>.
98+
/// </summary>
99+
public List<MajorVersionIdActivityStatusDescription> UserSummaryActivityStatuses { get; }
78100
}
79101
}

OpenAPI/LearningHub.Nhs.OpenApi.Repositories.Interface/Repositories/IResourceRepository.cs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ namespace LearningHub.Nhs.OpenApi.Repositories.Interface.Repositories
22
{
33
using System.Collections.Generic;
44
using System.Threading.Tasks;
5+
using LearningHub.Nhs.Models.Entities.Activity;
56
using LearningHub.Nhs.Models.Entities.Resource;
67

78
/// <summary>
@@ -23,5 +24,13 @@ public interface IResourceRepository
2324
/// <returns>Resource references.</returns>
2425
public Task<IEnumerable<ResourceReference>> GetResourceReferencesByOriginalResourceReferenceIds(
2526
IEnumerable<int> originalResourceReferenceIds);
27+
28+
/// <summary>
29+
/// Gets resource activity for resourceReferenceIds and userIds.
30+
/// </summary>
31+
/// <param name="resourceReferenceIds"><see cref="resourceReferenceIds"/>.</param>
32+
/// <param name="userIds"></param>
33+
/// <returns>ResourceActivityDTO.</returns>
34+
Task<IEnumerable<ResourceActivityDTO>> GetResourceActivityPerResourceMajorVersion(IEnumerable<int>? resourceReferenceIds, IEnumerable<int>? userIds);
2635
}
2736
}

OpenAPI/LearningHub.Nhs.OpenApi.Repositories/EntityFramework/LearningHubDbContext.cs

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -295,6 +295,11 @@ public LearningHubDbContext(LearningHubDbContextOptions options)
295295
/// </summary>
296296
public virtual DbSet<FileChunkDetail> FileChunkDetail { get; set; }
297297

298+
/// <summary>
299+
/// Gets or sets the ResourceActivityDto. These are not entities. They are returned from the [activity].[GetResourceActivityPerResourceMajorVersion] stored proc..
300+
/// </summary>
301+
public virtual DbSet<ResourceActivityDTO> ResourceActivityDTO { get; set; }
302+
298303
/// <summary>
299304
/// Gets or sets the RecentlyAddedResources. These are not entities. They are returned from the [resources].[GetRecentlyAddedResources] stored proc..
300305
/// </summary>
@@ -312,14 +317,14 @@ public LearningHubDbContext(LearningHubDbContextOptions options)
312317
public virtual DbSet<DashboardResourceDto> DashboardResourceDto { get; set; }
313318

314319
/// <summary>
315-
/// Gets or sets the ScormContentDetailsViewModel.
320+
/// Gets or sets the ExternalContentDetailsViewModel.
316321
/// </summary>
317-
public virtual DbSet<ScormContentDetailsViewModel> ScormContentDetailsViewModel { get; set; }
322+
public virtual DbSet<ExternalContentDetailsViewModel> ExternalContentDetailsViewModel { get; set; }
318323

319324
/// <summary>
320-
/// Gets or sets the ScormContentServerViewModel.
325+
/// Gets or sets the ContentServerViewModel.
321326
/// </summary>
322-
public virtual DbSet<ScormContentServerViewModel> ScormContentServerViewModel { get; set; }
327+
public virtual DbSet<ContentServerViewModel> ContentServerViewModel { get; set; }
323328

324329
/// <summary>
325330
/// Gets or sets the DashboardCatalogueDto
@@ -520,12 +525,12 @@ public LearningHubDbContext(LearningHubDbContextOptions options)
520525
/// <summary>
521526
/// Gets or sets the whole slide image annotation.
522527
/// </summary>
523-
public virtual DbSet<WholeSlideImageAnnotation> WholeSlideImageAnnotation { get; set; }
528+
public virtual DbSet<ImageAnnotation> ImageAnnotation { get; set; }
524529

525530
/// <summary>
526531
/// Gets or sets the whole slide image annotation mark.
527532
/// </summary>
528-
public virtual DbSet<WholeSlideImageAnnotationMark> WholeSlideImageAnnotationMark { get; set; }
533+
public virtual DbSet<ImageAnnotationMark> ImageAnnotationMark { get; set; }
529534

530535
/// <summary>
531536
/// Gets or sets the media block.

OpenAPI/LearningHub.Nhs.OpenApi.Repositories/EntityFramework/ServiceMappings.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,8 +110,8 @@ public static void AddLearningHubMappings(this IServiceCollection services, ICon
110110
services.AddSingleton<IEntityTypeMap, WholeSlideImageBlockMap>();
111111
services.AddSingleton<IEntityTypeMap, WholeSlideImageBlockItemMap>();
112112
services.AddSingleton<IEntityTypeMap, WholeSlideImageMap>();
113-
services.AddSingleton<IEntityTypeMap, WholeSlideImageAnnotationMap>();
114-
services.AddSingleton<IEntityTypeMap, WholeSlideImageAnnotationMarkMap>();
113+
services.AddSingleton<IEntityTypeMap, ImageAnnotationMap>();
114+
services.AddSingleton<IEntityTypeMap, ImageAnnotationMarkMap>();
115115
services.AddSingleton<IEntityTypeMap, WholeSlideImageFileMap>();
116116
services.AddSingleton<IEntityTypeMap, UrlRewritingMap>();
117117
services.AddSingleton<IEntityTypeMap, ResourceSyncMap>();

OpenAPI/LearningHub.Nhs.OpenApi.Repositories/Map/Content/PageSectionDetailMap.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@ protected override void InternalMap(EntityTypeBuilder<PageSectionDetail> entity)
1717
{
1818
entity.ToTable("PageSectionDetail", "content");
1919

20-
entity.Property(e => e.AssetPositionId).HasDefaultValueSql("((2))");
21-
2220
entity.Property(e => e.BackgroundColour).HasMaxLength(20);
2321

2422
entity.Property(e => e.Description).HasMaxLength(512);
@@ -31,7 +29,7 @@ protected override void InternalMap(EntityTypeBuilder<PageSectionDetail> entity)
3129

3230
entity.Property(e => e.TextColour).HasMaxLength(20);
3331

34-
entity.Property(e => e.Title).HasMaxLength(128);
32+
entity.Property(e => e.SectionTitle).HasMaxLength(128);
3533

3634
entity.Property(e => e.DeletePending).IsRequired(false);
3735

OpenAPI/LearningHub.Nhs.OpenApi.Repositories/Map/LogMap.cs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -68,10 +68,6 @@ protected void InternalMap(EntityTypeBuilder<Log> modelBuilder)
6868
modelBuilder.Property(e => e.UserId)
6969
.HasColumnName("UserId");
7070

71-
modelBuilder.HasOne(d => d.User)
72-
.WithMany(p => p.Logs)
73-
.HasForeignKey(d => d.UserId)
74-
.OnDelete(DeleteBehavior.ClientSetNull);
7571
}
7672
}
7773
}

OpenAPI/LearningHub.Nhs.OpenApi.Repositories/Map/Resources/Blocks/WholeSlideImageAnnotationMap.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,23 +7,23 @@
77
/// <summary>
88
/// The whole slide image annotation map.
99
/// </summary>
10-
public class WholeSlideImageAnnotationMap : BaseEntityMap<WholeSlideImageAnnotation>
10+
public class ImageAnnotationMap : BaseEntityMap<ImageAnnotation>
1111
{
1212
/// <summary>
1313
/// The internal map.
1414
/// </summary>
1515
/// <param name="modelBuilder">
1616
/// The model builder.
1717
/// </param>
18-
protected override void InternalMap(EntityTypeBuilder<WholeSlideImageAnnotation> modelBuilder)
18+
protected override void InternalMap(EntityTypeBuilder<ImageAnnotation> modelBuilder)
1919
{
20-
modelBuilder.ToTable("WholeSlideImageAnnotation", "resources");
20+
modelBuilder.ToTable("ImageAnnotation", "resources");
2121

2222
modelBuilder.HasOne(a => a.WholeSlideImage)
23-
.WithMany(i => i.WholeSlideImageAnnotations)
23+
.WithMany(i => i.ImageAnnotations)
2424
.HasForeignKey(a => a.WholeSlideImageId)
2525
.OnDelete(DeleteBehavior.Cascade)
26-
.HasConstraintName("FK_WholeSlideImageAnnotation_WholeSlideImageId");
26+
.HasConstraintName("FK_ImageAnnotation_WholeSlideImageId");
2727
}
2828
}
2929
}

0 commit comments

Comments
 (0)