musora-content-services 1.0.178 → 1.0.179

package/CHANGELOG.md

@@ -2,6 +2,8 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [1.0.179](https://github.com/railroadmedia/musora-content-services/compare/v1.0.178...v1.0.179) (2024-11-19)
+
 ### [1.0.178](https://github.com/railroadmedia/musora-content-services/compare/v1.0.177...v1.0.178) (2024-11-19)
 
 ### [1.0.177](https://github.com/railroadmedia/musora-content-services/compare/v1.0.176...v1.0.177) (2024-11-18)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "musora-content-services",
-  "version": "1.0.178",
+  "version": "1.0.179",
   "description": "A package for Musoras content services ",
   "main": "src/index.js",
   "scripts": {

package/src/contentTypeConfig.js

@@ -86,7 +86,7 @@ let contentTypeConfig = {
         'fields': [
             'enrollment_start_time',
             'enrollment_end_time',
-            'registration_url',
+            "'registration_url': '/' + brand + '/enrollment/' + slug.current",
             '"lesson_count": child_count',
             '"primary_cta_text": select(dateTime(published_on) > dateTime(now()) && dateTime(enrollment_start_time) > dateTime(now()) => "Notify Me", "Start Challenge")',
             'challenge_state',

package/src/filterBuilder.js

@@ -4,6 +4,10 @@ import {fetchUserPermissions} from "./services/userPermissions";
 export class FilterBuilder {
 
     STATUS_SCHEDULED = 'scheduled';
+    STATUS_PUBLISHED = 'published';
+    STATUS_DRAFT = 'draft';
+    STATUS_ARCHIVED = 'archived';
+    STATUS_UNLISTED = 'unlisted';
 
     constructor(
         filter = '',
@@ -13,13 +17,18 @@ export class FilterBuilder {
             pullFutureContent = false,
             getFutureContentOnly = false,
             getFutureScheduledContentsOnly = false,
-
+            bypassStatuses = false,
+            bypassPublishedDateRestriction = false,
+            isSingle = false
         } = {}) {
         this.availableContentStatuses = availableContentStatuses;
         this.bypassPermissions = bypassPermissions;
+        this.bypassStatuses = bypassStatuses;
+        this.bypassPublishedDateRestriction = bypassPublishedDateRestriction;
         this.pullFutureContent = pullFutureContent;
         this.getFutureContentOnly = getFutureContentOnly;
         this.getFutureScheduledContentsOnly = getFutureScheduledContentsOnly;
+        this.isSingle = isSingle;
         this.filter = filter;
         // this.debug = process.env.DEBUG === 'true' || false;
         this.debug = false;
@@ -48,7 +57,17 @@ export class FilterBuilder {
 
     _applyContentStatuses() {
         // This must be run before _applyPublishDateRestrictions()
-        if (this.availableContentStatuses.length === 0) return this;
+        if(this.bypassStatuses) return this;
+        if (this.availableContentStatuses.length === 0) {
+            if (this.userData.isAdmin) {
+                this.availableContentStatuses = [this.STATUS_DRAFT, this.STATUS_SCHEDULED, this.STATUS_PUBLISHED, this.STATUS_ARCHIVED, this.STATUS_UNLISTED];
+            } else if(this.isSingle){
+                this.availableContentStatuses = [this.STATUS_SCHEDULED, this.STATUS_PUBLISHED, this.STATUS_UNLISTED, this.STATUS_ARCHIVED];
+            } else{
+                this.availableContentStatuses = [this.STATUS_SCHEDULED, this.STATUS_PUBLISHED];
+            }
+        }
+
         // I'm not sure if I'm 100% on this logic, but this is my intepretation of the ContentRepository logic
         if (this.getFutureScheduledContentsOnly && this.availableContentStatuses.includes(this.STATUS_SCHEDULED)) {
             // we must pull in future content here, otherwise we'll restrict on content this is published in the past and remove any scheduled content
@@ -77,6 +96,7 @@ export class FilterBuilder {
     }
 
     _applyPublishingDateRestrictions() {
+        if(this.bypassPublishedDateRestriction) return this;
         const now = new Date().toISOString();
         if (this.getFutureContentOnly) {
             this._andWhere(`published_on >= '${now}'`);

package/src/services/sanity.js

@@ -510,6 +510,7 @@ export async function fetchAll(brand, type, {
     let webUrlPathType = config?.slug ?? type;
     const start = (page - 1) * limit;
     const end = start + limit;
+    let bypassStatusAndPublishedValidation = (type == 'instructor');
 
     // Construct the type filter
     const typeFilter = type ? `&& _type == '${type}'` : "";
@@ -574,8 +575,10 @@ export async function fetchAll(brand, type, {
         filter = `brand == "${brand}" ${typeFilter} ${searchFilter} ${includedFieldsFilter} ${progressFilter}`
         entityFieldsString = fieldsString;
     }
+
+    const filterWithRestrictions = await new FilterBuilder(filter,{bypassStatuses:bypassStatusAndPublishedValidation, bypassPermissions: bypassStatusAndPublishedValidation, bypassPublishedDateRestriction: bypassStatusAndPublishedValidation} ).buildFilter();
     query = buildEntityAndTotalQuery(
-        filter,
+        filterWithRestrictions,
         entityFieldsString,
         {
             sortOrder: sortOrder,
@@ -990,7 +993,7 @@ export async function fetchNextPreviousLesson(railcontentId) {
  *   .catch(error => console.error(error));
  */
 export async function fetchLessonContent(railContentId) {
-    const filterParams = {};
+    const filterParams = {isSingle:true};
     // Format changes made to the `fields` object may also need to be reflected in Musora-web-platform SanityGateway.php $fields object
     // Currently only for challenges and challenge lessons
     // If you're unsure, message Adrian, or just add them.

package/test/sanityQueryService.test.js

@@ -630,13 +630,16 @@ describe('Filter Builder', function () {
         let finalFilter = await builder.buildFilter(filter);
         let clauses = spliceFilterForAnds(finalFilter);
         expect(clauses[0].phrase).toBe(filter);
-        expect(clauses[1].field).toBe('published_on');
+        expect(clauses[1].field).toBe('status');
+        expect(clauses[2].field).toBe('published_on');
 
         builder = new FilterBuilder('', {bypassPermissions: true});
         finalFilter = await builder.buildFilter(filter);
         clauses = spliceFilterForAnds(finalFilter);
-        expect(clauses[0].field).toBe('published_on');
-        expect(clauses[0].operator).toBe('<=');
+        expect(clauses[0].field).toBe('status');
+        expect(clauses[0].operator).toBe('in');
+        expect(clauses[1].field).toBe('published_on');
+        expect(clauses[1].operator).toBe('<=');
     });
 
     test('withOnlyFilterAvailableStatuses', async () => {
@@ -665,8 +668,6 @@ describe('Filter Builder', function () {
         expect(clauses[1].operator).toBe('in');
         // getFutureScheduledContentsOnly doesn't make a filter that's splicable, so we match on the more static string
         const expected = "['published','unlisted'] || (status == 'scheduled' && published_on >=";
-        console.log(clauses[1].condition);
-        console.log(expected)
         const isMatch = finalFilter.includes(expected);
         expect(isMatch).toBeTruthy();
     });
@@ -701,7 +702,9 @@ describe('Filter Builder', function () {
         expect(isMatch).toBeFalsy();
         const clauses = spliceFilterForAnds(finalFilter);
         expect(clauses[0].field).toBe('railcontent_id');
-        expect(clauses[1].field).toBe('published_on');
+        expect(clauses[1].field).toBe('status');
+        expect(clauses[2].field).toBe('published_on');
+
     });
 
 
@@ -717,10 +720,11 @@ describe('Filter Builder', function () {
         let finalFilter = await builder.buildFilter();
         let clauses = spliceFilterForAnds(finalFilter);
         expect(clauses[0].phrase).toBe(filter);
-
-        expect(clauses[1].field).toBe('published_on');
-        expect(clauses[1].operator).toBe('<=');
-        const restrictionDate = new Date(clauses[1].condition)
+        expect(clauses[1].field).toBe('status');
+        expect(clauses[1].operator).toBe('in');
+        expect(clauses[2].field).toBe('published_on');
+        expect(clauses[2].operator).toBe('<=');
+        const restrictionDate = new Date(clauses[2].condition)
         const now = new Date();
         expect(now.getTime()).toBeLessThan(restrictionDate.getTime());
 
@@ -732,8 +736,8 @@ describe('Filter Builder', function () {
         finalFilter = await builder.buildFilter();
         clauses = spliceFilterForAnds(finalFilter);
         expect(clauses[0].phrase).toBe(filter);
-        expect(clauses[1].field).toBe('published_on');
-        expect(clauses[1].operator).toBe('>=');
+        expect(clauses[2].field).toBe('published_on');
+        expect(clauses[2].operator).toBe('>=');
     });
 
     function spliceFilterForAnds(filter) {