@livepreso/api 6.44.0 → 6.46.0

package/src/collections/base.js

@@ -38,7 +38,8 @@ export const BaseCollection = Collection.extend(
 			per_page: 1000,
 		},
 
-		constructor(data = {}, opts = {}) {
+		constructor(_data = {}, opts = {}) {
+			let data = _data;
 			const options = _.defaults(opts, {
 				data: {},
 			});
@@ -97,7 +98,7 @@ export const BaseCollection = Collection.extend(
 				const Model = registeredStates[this.model];
 				if (!Model) {
 					throw new Error(
-						`'${this.model}' not found! Did you forget to import it on the '${this.modelName}' model?`
+						`'${this.model}' not found! Did you forget to import it on the '${this.modelName}' model?`,
 					);
 				}
 				this.model = Model;
@@ -122,7 +123,7 @@ export const BaseCollection = Collection.extend(
 					query = _.extend(
 						{},
 						qs.parse(url.slice(queryIndex + 1)),
-						this.queryData
+						this.queryData,
 					);
 				}
 				if (url.slice(-1) !== "/") {
@@ -142,6 +143,10 @@ export const BaseCollection = Collection.extend(
 				resp += "/";
 			}
 
+			if (!resp.includes("{") || !resp.includes("}")) {
+				return this.getFullURLPath(resp);
+			}
+
 			// Replace any values in a url with one from the collection's options.
 			// So if you have /path/to/{model.id}/models/
 			// It will check the collection options for "model.id" and replace
@@ -155,7 +160,7 @@ export const BaseCollection = Collection.extend(
 					resp = resp.replace(match, replacement);
 				} else if (!this.isNew()) {
 					log.error(
-						`Collection ${this.modelName} is missing value ${path} as an option`
+						`Collection ${this.modelName} is missing value ${path} as an option`,
 					);
 				}
 				resp = resp.replace(match, replacement);
@@ -255,7 +260,7 @@ export const BaseCollection = Collection.extend(
 					this.queryData = { ...options.data };
 
 					return this.fetchPage().then((response) => {
-						checkPages(response);
+						return checkPages(response);
 					});
 				} else {
 					// The collection's models are replaced with the latest
@@ -292,7 +297,7 @@ export const BaseCollection = Collection.extend(
 		},
 
 		toServer() {
-			return this.map(function (model) {
+			return this.map((model) => {
 				if (model.toServer) {
 					return model.toServer();
 				} else if (model.serialize) {
@@ -306,6 +311,11 @@ export const BaseCollection = Collection.extend(
 			});
 		},
 
+		// Returns "true" if all child models have been fetched
+		areModelsFetched() {
+			return this.map((model) => model.isFetched()).every(Boolean);
+		},
+
 		isNew() {
 			return true;
 		},
@@ -333,7 +343,7 @@ export const BaseCollection = Collection.extend(
 			return Promise.all(
 				this.map((model) => {
 					return model.save(model.toServer(), options);
-				})
+				}),
 			);
 		},
 
@@ -347,7 +357,7 @@ export const BaseCollection = Collection.extend(
 			return Promise.all(
 				this.map((model) => {
 					return model.delete(model.toServer(), options);
-				})
+				}),
 			);
 		},
 
@@ -363,14 +373,14 @@ export const BaseCollection = Collection.extend(
 				// can be passed to the model.
 				//
 				// TODO: Regex match for URLs
-				return results.map(function (attr) {
+				return results.map((attr) => {
 					return { url: attr };
 				});
 			}
 
 			return results;
 		},
-	}
+	},
 );
 
 // Make matchesURL available as a static method

package/src/collections/base.spec.js

@@ -1,9 +1,9 @@
+import AmpersandCollection from "ampersand-rest-collection";
 import { BaseCollection } from "./base.js";
 import { BaseModel } from "../models/base.js";
-import AmpersandCollection from "ampersand-rest-collection";
 import { G } from "sp-test/api";
-import td from "testdouble";
 import Promise from "bluebird";
+import td from "testdouble";
 
 it("should parse an array of urls into models", () => {
 	G(BaseModel, "TestModel").extend({ modelName: "tests" });
@@ -14,7 +14,7 @@ it("should parse an array of urls into models", () => {
 	expect(collection.models).toHaveLength(2);
 });
 
-it("should fetch the individual models in the collection", async function () {
+it("should fetch the individual models in the collection", async () => {
 	G(BaseModel, "TestModel");
 	const Collection = G(BaseCollection).extend({ model: "TestModel" });
 	const collection = new Collection([{ id: 1 }, { id: 2 }]);
@@ -93,10 +93,10 @@ it("should query multiple values if using an array", () => {
 			data: {
 				foo: [1, 2, null],
 			},
-		}
+		},
 	);
 	expect(collection.url()).toEqual(
-		"/api/appointments/?per_page=1000&foo=1&foo=2&foo="
+		"/api/appointments/?per_page=1000&foo=1&foo=2&foo=",
 	);
 });
 
@@ -122,6 +122,20 @@ it("should be empty on initialize", () => {
 	expect(collection.length).toEqual(0);
 });
 
+it("should be not 'areModelsFetched' if all models are just a url or integer", () => {
+	const AppointmentCollection = G("AppointmentCollection");
+	// A string or an integer should be considered not fetched
+	const collection = new AppointmentCollection(["/api/appointments/1/", 2]);
+	expect(collection.areModelsFetched()).toBe(false);
+});
+
+it("should be 'areModelsFetched' if models are objects", () => {
+	G(BaseModel, "TestModel");
+	const Collection = G(BaseCollection.extend({ model: "TestModel" }));
+	const collection = new Collection([{ id: 1 }, { id: 2 }]);
+	expect(collection.areModelsFetched()).toBe(true);
+});
+
 const mockFetchResults = (dataFn) => {
 	jest
 		.spyOn(AmpersandCollection.prototype, "fetch")

package/src/collections/cms-vals.js

@@ -34,7 +34,11 @@ export const CMSValCollection = BaseCollection.extend(
 
 			return BaseCollection.prototype.fetch.call(this, opts);
 		},
-	}
+
+		toCompactJSON() {
+			return this.models.map((cmsVal) => cmsVal.toCompactJSON());
+		},
+	},
 );
 
 CMSValCollection.canBeEmpty = true;

package/src/collections/contact-phones.js

@@ -0,0 +1,18 @@
+import "../models/contact-phone.js";
+
+import { BaseCollection } from "./base.js";
+import { register } from "../state-register.js";
+
+/**
+ * @constructor
+ * @alias collections.ContactCollection
+ * @extends collections.BaseCollection
+ * @see models.ContactPhone
+ */
+export const ContactPhoneCollection = BaseCollection.extend(
+	/** @lends collections.ContactCollection# */ {
+		model: "ContactPhoneModel",
+	},
+);
+
+register("ContactPhoneCollection", ContactPhoneCollection);

package/src/collections/contacts.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const ContactCollection = BaseCollection.extend(
 	/** @lends collections.ContactCollection# */ {
 		model: "ContactModel",
-	}
+	},
 );
 
 register("ContactCollection", ContactCollection);

package/src/collections/created-assets.js

@@ -20,7 +20,7 @@ export const CreatedAssetCollection = BaseCollection.extend(
 
 			return BaseCollection.prototype.url.call(this);
 		},
-	}
+	},
 );
 
 register("CreatedAssetCollection", CreatedAssetCollection);

package/src/collections/customers.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const CustomerCollection = BaseCollection.extend(
 	/** @lends collections.CustomerCollection# */ {
 		model: "CustomerModel",
-	}
+	},
 );
 
 register("CustomerCollection", CustomerCollection);

package/src/collections/deck-notes.js

@@ -19,7 +19,7 @@ export const DeckNoteCollection = BaseCollection.extend(
 			}
 			return BaseCollection.prototype.url.call(this);
 		},
-	}
+	},
 );
 
 DeckNoteCollection.canBeEmpty = true;

package/src/collections/deck-version-screenshots.js

@@ -0,0 +1,48 @@
+import "../models/deck-version-screenshot.js";
+
+import Auth from "../auth.js";
+import { BaseCollection } from "./base.js";
+import { register } from "../state-register.js";
+
+/**
+ * @constructor
+ * @alias collections.DeckVersionScreenshotCollection
+ * @extends collections.BaseCollection
+ * @see models.DeckVersionScreenshotModel
+ */
+export const DeckVersionScreenshotCollection = BaseCollection.extend(
+	/** @lends collections.DeckVersionScreenshotCollection# */ {
+		model: "DeckVersionScreenshotModel",
+
+		fetch(opts = {}) {
+			const { user } = Auth;
+
+			if (!opts.data) {
+				opts.data = {};
+			}
+
+			// Let's only add this if it's not already being set, in case a global user
+			// DOES want to see a specific team.
+			if (user && !opts.data.team) {
+				//
+				// Filter all global autoadjuncts, and only query teams that
+				// the authorized user is a part of. This is mostly for the case where
+				// a 'global' temp slide admin could end up seeing autoadjuncts for
+				// many teams - which would be increadibly confusing.
+				opts.data.team = [null].concat(user.team_set.map((team) => team.id));
+			}
+
+			return BaseCollection.prototype.fetch.call(this, opts);
+		},
+
+		url() {
+			if (!this._url && this.parent) {
+				this._options.deckversion = this.parent;
+			}
+
+			return BaseCollection.prototype.url.call(this);
+		},
+	},
+);
+
+register("DeckVersionScreenshotCollection", DeckVersionScreenshotCollection);

package/src/collections/deck-versions.js

@@ -50,7 +50,7 @@ export const DeckVersionCollection = BaseCollection.extend(
 				...options,
 			});
 		},
-	}
+	},
 );
 
 register("DeckVersionCollection", DeckVersionCollection);

package/src/collections/decks.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const DeckCollection = BaseCollection.extend(
 	/** @lends collections.DeckCollection# */ {
 		model: "DeckModel",
-	}
+	},
 );
 
 register("DeckCollection", DeckCollection);

package/src/collections/deleted-assets.js

@@ -20,7 +20,7 @@ export const DeletedAssetCollection = BaseCollection.extend(
 
 			return BaseCollection.prototype.url.call(this);
 		},
-	}
+	},
 );
 
 register("DeletedAssetCollection", DeletedAssetCollection);

package/src/collections/dispatches.js

@@ -35,13 +35,13 @@ export const DispatchCollection = BaseCollection.extend(
 		url() {
 			if (!this._url && this.parent) {
 				this._url = this.getFullURLPath(
-					`${this.parent.getPlainURL()}dispatches/`
+					`${this.parent.getPlainURL()}dispatches/`,
 				);
 			}
 
 			return BaseCollection.prototype.url.call(this);
 		},
-	}
+	},
 );
 
 register("DispatchCollection", DispatchCollection);

package/src/collections/event-objects.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const EventObjectCollection = BaseCollection.extend(
 	/** @lends collections.EventObjectCollection# */ {
 		model: "EventObjectModel",
-	}
+	},
 );
 
 register("EventObjectCollection", EventObjectCollection);

package/src/collections/events.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const EventCollection = BaseCollection.extend(
 	/** @lends collections.EventCollection# */ {
 		model: "EventModel",
-	}
+	},
 );
 
 register("EventCollection", EventCollection);

package/src/collections/groups.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const GroupCollection = BaseCollection.extend(
 	/** @lends collections.GroupCollection# */ {
 		model: "GroupModel",
-	}
+	},
 );
 
 register("GroupCollection", GroupCollection);

package/src/collections/images.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const ImageCollection = BaseCollection.extend(
 	/** @lends collections.ImageCollection# */ {
 		model: "ImageModel",
-	}
+	},
 );
 
 register("ImageCollection", ImageCollection);

package/src/collections/locations.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const LocationCollection = BaseCollection.extend(
 	/** @lends collections.LocationCollection# */ {
 		model: "LocationModel",
-	}
+	},
 );
 
 register("LocationCollection", LocationCollection);

package/src/collections/opportunities.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const OpportunityCollection = BaseCollection.extend(
 	/** @lends collections.OpportunityCollection# */ {
 		model: "OpportunityModel",
-	}
+	},
 );
 
 register("OpportunityCollection", OpportunityCollection);

package/src/collections/pack-diffs.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const PackDiffCollection = BaseCollection.extend(
 	/** @lends collections.PackDiffCollection# */ {
 		model: "PackDiffModel",
-	}
+	},
 );
 
 register("PackDiffCollection", PackDiffCollection);

package/src/collections/packs.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const PackCollection = BaseCollection.extend(
 	/** @lends collections.PackCollection# */ {
 		model: "PackModel",
-	}
+	},
 );
 
 register("PackCollection", PackCollection);

package/src/collections/presets.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const PresetCollection = BaseCollection.extend(
 	/** @lends collections.PresetCollection# */ {
 		model: "PresetModel",
-	}
+	},
 );
 
 register("PresetCollection", PresetCollection);

package/src/collections/section-selections.js

@@ -20,7 +20,7 @@ export const SectionSelectionCollection = BaseCollection.extend(
 				},
 			};
 		},
-	}
+	},
 );
 
 register("SectionSelectionCollection", SectionSelectionCollection);

package/src/collections/sections.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const SectionCollection = BaseCollection.extend(
 	/** @lends collections.SectionCollection# */ {
 		model: "SectionModel",
-	}
+	},
 );
 
 register("SectionCollection", SectionCollection);

package/src/collections/slide-selections.js

@@ -20,7 +20,7 @@ export const SlideSelectionCollection = BaseCollection.extend(
 				},
 			};
 		},
-	}
+	},
 );
 
 register("SlideSelectionCollection", SlideSelectionCollection);

package/src/collections/slides.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const SlideCollection = BaseCollection.extend(
 	/** @lends collections.SlideCollection# */ {
 		model: "SlideModel",
-	}
+	},
 );
 
 register("SlideCollection", SlideCollection);

package/src/collections/snapshots.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const SnapshotCollection = BaseCollection.extend(
 	/** @lends collections.SnapshotCollection# */ {
 		model: "SnapshotModel",
-	}
+	},
 );
 
 register("SnapshotCollection", SnapshotCollection);

package/src/collections/sub-slides.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const SubSlideCollection = BaseCollection.extend(
 	/** @lends collections.SubSlideCollection# */ {
 		model: "SubSlideModel",
-	}
+	},
 );
 
 register("SubSlideCollection", SubSlideCollection);

package/src/collections/tags.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const TagCollection = BaseCollection.extend(
 	/** @lends collections.TagCollection# */ {
 		model: "TagModel",
-	}
+	},
 );
 
 register("TagCollection", TagCollection);

package/src/collections/teams.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const TeamCollection = BaseCollection.extend(
 	/** @lends collections.TeamCollection# */ {
 		model: "TeamModel",
-	}
+	},
 );
 
 TeamCollection.canBeEmpty = true;

package/src/collections/templates.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const TemplateCollection = BaseCollection.extend(
 	/** @lends collections.TemplateCollection# */ {
 		model: "TemplateModel",
-	}
+	},
 );
 
 TemplateCollection.canBeEmpty = true;

package/src/collections/uploads.js

@@ -13,7 +13,7 @@ export const UploadCollection = BaseCollection.extend(
 	/** @lends collections.UploadCollection# */ {
 		comparator: "sequence",
 		model: "UploadModel",
-	}
+	},
 );
 
 register("UploadCollection", UploadCollection);

package/src/collections/users.js

@@ -12,7 +12,7 @@ import { register } from "../state-register.js";
 export const UserCollection = BaseCollection.extend(
 	/** @lends collections.UserCollection# */ {
 		model: "UserModel",
-	}
+	},
 );
 
 register("UserCollection", UserCollection);

package/src/collections.js

@@ -11,9 +11,11 @@ export * from "./collections/auto-adjuncts";
 export * from "./collections/base";
 export * from "./collections/cms-vals";
 export * from "./collections/contacts";
+export * from "./collections/contact-phones";
 export * from "./collections/created-assets";
 export * from "./collections/customers";
 export * from "./collections/deck-versions";
+export * from "./collections/deck-version-screenshots";
 export * from "./collections/decks";
 export * from "./collections/deleted-assets";
 export * from "./collections/dispatches";
@@ -44,3 +46,4 @@ export * from "./collections/appointment-deckversion-tags.js";
 export * from "./collections/appointment-section-tags.js";
 export * from "./collections/appointment-slide-tags.js";
 export * from "./collections/appointment-adjunct-tags.js";
+export * from "./collections/appointment-templates.js";