Merge pull request #68 from xtr-dev/dev

Fix filterOptions ObjectId casting error and bump version to 0.4.20
2025-12-10 16:23:23 +00:00 · 2025-10-07 22:04:22 +02:00 · 2025-10-07 22:01:23 +02:00 · 2025-10-07 21:38:42 +02:00 · 2025-10-07 21:35:55 +02:00
3 changed files with 7 additions and 3 deletions
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "@xtr-dev/payload-mailing",
-  "version": "0.4.18",
+  "version": "0.4.20",
  "description": "Template-based email system with scheduling and job processing for PayloadCMS",
  "type": "module",
  "main": "dist/index.js",
--- a/src/collections/Emails.ts
+++ b/src/collections/Emails.ts
@@ -206,7 +206,7 @@ const Emails: CollectionConfig = {
        readOnly: true,
      },
      filterOptions: ({ id }) => {
-        const emailId = resolveID({ id })
+        const emailId = resolveID(id)
        return {
          'input.emailId': {
            equals: emailId ? String(emailId) : '',
--- a/src/utils/jobScheduler.ts
+++ b/src/utils/jobScheduler.ts
@@ -129,7 +129,11 @@ export async function updateEmailJobRelationship(
      id: normalizedEmailId,
    })

-    const currentJobs = (currentEmail.jobs || []).map((job: any) => String(job))
+    // Extract IDs from job objects or use the value directly if it's already an ID
+    // Jobs can be populated (objects with id field) or just IDs (strings/numbers)
+    const currentJobs = (currentEmail.jobs || []).map((job: any) =>
+      typeof job === 'object' && job !== null && job.id ? String(job.id) : String(job)
+    )
    const allJobs = [...new Set([...currentJobs, ...normalizedJobIds])] // Deduplicate with normalized strings

    await payload.update({
Author	SHA1	Message	Date
Bas	53ab62ed10	Merge pull request #68 from xtr-dev/dev Fix filterOptions ObjectId casting error and bump version to 0.4.20	2025-10-07 22:04:22 +02:00
Bas van den Aakster	de57dd4102	Fix filterOptions ObjectId casting error and bump version to 0.4.20 Fixed incorrect usage of resolveID in filterOptions where { id } was passed instead of id directly. This caused ObjectId casting errors when the id parameter was a populated object. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-10-07 22:01:23 +02:00
Bas	4633ead274	Merge pull request #67 from xtr-dev/dev Fix ObjectId casting error when jobs relationship is populated	2025-10-07 21:38:42 +02:00
Bas van den Aakster	d69f7c1f98	Fix ObjectId casting error when jobs relationship is populated When the email's jobs relationship is populated with full job objects instead of just IDs, calling String(job) on an object results in "[object Object]", which causes a Mongoose ObjectId casting error. This fix properly extracts the ID from job objects or uses the value directly if it's already an ID. Fixes job scheduler error: "Cast to ObjectId failed for value '[object Object]'" 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-10-07 21:35:55 +02:00