Skip to content

Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) #15007

Description

@werner-freytag

Bug description

Bug description

Deleting a Bard set that contains a nested field whose value is stripped on save
(a revealer field, or a field hidden by an if/show_when condition) causes the
Publish form to crash on the next save with:

 TypeError: Cannot use 'in' operator to search for '<key>' in null

(WebKit phrases it as TypeError: I is not an Object. (evaluating 'N in I').)

The save request never completes; the user only sees the generic "Something went wrong" toast.

Root cause

Deleting a Bard set leaves a null node at that index in the in-memory field value
(it is not persisted to the file). On save, Values.forgetValue() is called for every
conditionally-hidden / revealer field path and calls data_delete(values, path).

The missingValue() guard that is supposed to prevent this does not catch a null
intermediate, because the reduce short-circuits to null (not undefined):

 // resources/js/components/publish/Values.js
missingValue(dottedKey) {
var properties = Array.isArray(dottedKey) ? dottedKey : dottedKey.split('.');
var value = properties.reduce((prev, curr) => prev && prev[curr], clone(this.values));
return value === undefined; // null === undefined -> false, so guard is skipped
}

forgetValue() therefore proceeds into data_delete(), which walks the path and hits
the null node:

function data_delete(obj, path) {
var parts = path.split('.');
while (parts.length - 1) {
var key = parts.shift();
...
obj = obj[key]; // becomes null
}
delete obj[parts[0]]; // or: `key in null` throws on the next iteration
}

How to reproduce

  1. Create a Bard field whose set contains a nested Replicator, and give that Replicator's
    set a revealer field plus a field conditioned on it. Minimal blueprint:

    fields:
    -
    handle: text
    field:
    type: bard
    sets:
    enumeration:
    display: Enumeration
    fields:
    -
    handle: bullet_points
    field:
    type: replicator
    sets:
    bullet_point:
    display: Point
    fields:
    -
    handle: show_readmore
    field:
    type: revealer
    -
    handle: readmore_text
    field:
    type: text
    if:
    show_readmore: 'equals 1'
    
  2. On an entry, add the "Enumeration" Bard set and at least one bullet_point row. Save.

  3. Delete that Bard set.

  4. Click Save.

Expected behavior

The entry saves. Deleting a set should not be able to crash the save flow.

Actual behavior

TypeError in data_delete (Values.js); save aborts with "Something went wrong".

Notes

Call stack: saveperformSaveRequestValues.forgetValuedata_delete.
The stored content is never corrupted — the null node exists only in the editor's
in-memory value.

Logs

Environment

- Statamic: 5.74.0 (code is identical on `6.x`)
- Laravel: 12.x
- Browser: reproduced in Safari/WebKit and Chrome

Installation

Fresh statamic/statamic site via CLI

Additional details

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      , 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all
       blocks
      (function() {
      function addCopyButtons() {
      document.querySelectorAll('pre code').forEach(function(codeBlock) {
      if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
      codeBlock.parentElement.setAttribute('data-copy-added', 'true');
      var btn = document.createElement('button');
      btn.textContent = 'Copy';
      btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
      btn.onmouseover = function() { this.style.opacity = '1'; };
      btn.onmouseout = function() { this.style.opacity = '0.7'; };
      btn.onclick = function() {
      navigator.clipboard.writeText(codeBlock.textContent).then(function() {
      btn.textContent = 'Copied!';
      setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
      });
      };
      codeBlock.parentElement.style.position = 'relative';
      codeBlock.parentElement.appendChild(btn);
      });
      }
      addCopyButtons();
      // Re-run on dynamic content
      var observer = new MutationObserver(addCopyButtons);
      observer.observe(document.body, { childList: true, subtree: true });
      })();
      }
      } catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
      })();
      (function(){
      try {
      var __m = "github.com";
      var __re = new RegExp('^' + "github\\.com" + '
      Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) · Issue #15007 · statamic/cms · GitHub
      Skip to content

      Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) #15007

      Description

      @werner-freytag

      Bug description

      Bug description

      Deleting a Bard set that contains a nested field whose value is stripped on save
      (a revealer field, or a field hidden by an if/show_when condition) causes the
      Publish form to crash on the next save with:

       TypeError: Cannot use 'in' operator to search for '<key>' in null
      

      (WebKit phrases it as TypeError: I is not an Object. (evaluating 'N in I').)

      The save request never completes; the user only sees the generic "Something went wrong" toast.

      Root cause

      Deleting a Bard set leaves a null node at that index in the in-memory field value
      (it is not persisted to the file). On save, Values.forgetValue() is called for every
      conditionally-hidden / revealer field path and calls data_delete(values, path).

      The missingValue() guard that is supposed to prevent this does not catch a null
      intermediate, because the reduce short-circuits to null (not undefined):

       // resources/js/components/publish/Values.js
      missingValue(dottedKey) {
      var properties = Array.isArray(dottedKey) ? dottedKey : dottedKey.split('.');
      var value = properties.reduce((prev, curr) => prev && prev[curr], clone(this.values));
      return value === undefined; // null === undefined -> false, so guard is skipped
      }
      

      forgetValue() therefore proceeds into data_delete(), which walks the path and hits
      the null node:

      function data_delete(obj, path) {
      var parts = path.split('.');
      while (parts.length - 1) {
      var key = parts.shift();
      ...
      obj = obj[key]; // becomes null
      }
      delete obj[parts[0]]; // or: `key in null` throws on the next iteration
      }
      

      How to reproduce

      1. Create a Bard field whose set contains a nested Replicator, and give that Replicator's
        set a revealer field plus a field conditioned on it. Minimal blueprint:

        fields:
        -
        handle: text
        field:
        type: bard
        sets:
        enumeration:
        display: Enumeration
        fields:
        -
        handle: bullet_points
        field:
        type: replicator
        sets:
        bullet_point:
        display: Point
        fields:
        -
        handle: show_readmore
        field:
        type: revealer
        -
        handle: readmore_text
        field:
        type: text
        if:
        show_readmore: 'equals 1'
        
      2. On an entry, add the "Enumeration" Bard set and at least one bullet_point row. Save.

      3. Delete that Bard set.

      4. Click Save.

      Expected behavior

      The entry saves. Deleting a set should not be able to crash the save flow.

      Actual behavior

      TypeError in data_delete (Values.js); save aborts with "Something went wrong".

      Notes

      Call stack: saveperformSaveRequestValues.forgetValuedata_delete.
      The stored content is never corrupted — the null node exists only in the editor's
      in-memory value.

      Logs

      Environment

      - Statamic: 5.74.0 (code is identical on `6.x`)
      - Laravel: 12.x
      - Browser: reproduced in Safari/WebKit and Chrome

      Installation

      Fresh statamic/statamic site via CLI

      Additional details

      No response

      Metadata

      Metadata

      Assignees

      No one assigned

        Labels

        Type

        No type

        Projects

        No projects

          Milestone

          No milestone

          Relationships

          None yet

          Development

          No branches or pull requests

          Issue actions

          , 'i'); if (__m === '*' || __re.test(location.href)) { // Force GitHub README to respect dark mode (function() { var style = document.createElement('style'); style.textContent = ' .markdown-body { color-scheme: dark light; } .markdown-body pre { background: #161b22 !important; } .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; } .markdown-body table th, .markdown-body table td { border-color: #30363d !important; } .markdown-body img { background: #0d1117; } .markdown-body blockquote { border-left-color: #8b949e; } .markdown-body hr { border-color: #30363d; } '; document.head.appendChild(style); })(); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) · Issue #15007 · statamic/cms · GitHub
          Skip to content

          Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) #15007

          Description

          @werner-freytag

          Bug description

          Bug description

          Deleting a Bard set that contains a nested field whose value is stripped on save
          (a revealer field, or a field hidden by an if/show_when condition) causes the
          Publish form to crash on the next save with:

           TypeError: Cannot use 'in' operator to search for '<key>' in null
          

          (WebKit phrases it as TypeError: I is not an Object. (evaluating 'N in I').)

          The save request never completes; the user only sees the generic "Something went wrong" toast.

          Root cause

          Deleting a Bard set leaves a null node at that index in the in-memory field value
          (it is not persisted to the file). On save, Values.forgetValue() is called for every
          conditionally-hidden / revealer field path and calls data_delete(values, path).

          The missingValue() guard that is supposed to prevent this does not catch a null
          intermediate, because the reduce short-circuits to null (not undefined):

           // resources/js/components/publish/Values.js
          missingValue(dottedKey) {
          var properties = Array.isArray(dottedKey) ? dottedKey : dottedKey.split('.');
          var value = properties.reduce((prev, curr) => prev && prev[curr], clone(this.values));
          return value === undefined; // null === undefined -> false, so guard is skipped
          }
          

          forgetValue() therefore proceeds into data_delete(), which walks the path and hits
          the null node:

          function data_delete(obj, path) {
          var parts = path.split('.');
          while (parts.length - 1) {
          var key = parts.shift();
          ...
          obj = obj[key]; // becomes null
          }
          delete obj[parts[0]]; // or: `key in null` throws on the next iteration
          }
          

          How to reproduce

          1. Create a Bard field whose set contains a nested Replicator, and give that Replicator's
            set a revealer field plus a field conditioned on it. Minimal blueprint:

            fields:
            -
            handle: text
            field:
            type: bard
            sets:
            enumeration:
            display: Enumeration
            fields:
            -
            handle: bullet_points
            field:
            type: replicator
            sets:
            bullet_point:
            display: Point
            fields:
            -
            handle: show_readmore
            field:
            type: revealer
            -
            handle: readmore_text
            field:
            type: text
            if:
            show_readmore: 'equals 1'
            
          2. On an entry, add the "Enumeration" Bard set and at least one bullet_point row. Save.

          3. Delete that Bard set.

          4. Click Save.

          Expected behavior

          The entry saves. Deleting a set should not be able to crash the save flow.

          Actual behavior

          TypeError in data_delete (Values.js); save aborts with "Something went wrong".

          Notes

          Call stack: saveperformSaveRequestValues.forgetValuedata_delete.
          The stored content is never corrupted — the null node exists only in the editor's
          in-memory value.

          Logs

          Environment

          - Statamic: 5.74.0 (code is identical on `6.x`)
          - Laravel: 12.x
          - Browser: reproduced in Safari/WebKit and Chrome

          Installation

          Fresh statamic/statamic site via CLI

          Additional details

          No response

          Metadata

          Metadata

          Assignees

          No one assigned

            Labels

            Type

            No type

            Projects

            No projects

              Milestone

              No milestone

              Relationships

              None yet

              Development

              No branches or pull requests

              Issue actions

              , 'i'); if (__m === '*' || __re.test(location.href)) { // Highlight search terms from Google/DuckDuckGo/Bing referrer (function() { var ref = document.referrer; var terms = []; if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) { var url = new URL(ref); var q = url.searchParams.get('q') || url.searchParams.get('p'); if (q) { terms = q.split(/\s+/).filter(function(t) { return t.length > 2; }); } } if (terms.length === 0) return; var style = document.createElement('style'); style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }'; document.head.appendChild(style); function highlight(node) { if (node.nodeType === 3) { // text node var text = node.textContent; var found = false; terms.forEach(function(term) { var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) · Issue #15007 · statamic/cms · GitHub
              Skip to content

              Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) #15007

              Description

              @werner-freytag

              Bug description

              Bug description

              Deleting a Bard set that contains a nested field whose value is stripped on save
              (a revealer field, or a field hidden by an if/show_when condition) causes the
              Publish form to crash on the next save with:

               TypeError: Cannot use 'in' operator to search for '<key>' in null
              

              (WebKit phrases it as TypeError: I is not an Object. (evaluating 'N in I').)

              The save request never completes; the user only sees the generic "Something went wrong" toast.

              Root cause

              Deleting a Bard set leaves a null node at that index in the in-memory field value
              (it is not persisted to the file). On save, Values.forgetValue() is called for every
              conditionally-hidden / revealer field path and calls data_delete(values, path).

              The missingValue() guard that is supposed to prevent this does not catch a null
              intermediate, because the reduce short-circuits to null (not undefined):

               // resources/js/components/publish/Values.js
              missingValue(dottedKey) {
              var properties = Array.isArray(dottedKey) ? dottedKey : dottedKey.split('.');
              var value = properties.reduce((prev, curr) => prev && prev[curr], clone(this.values));
              return value === undefined; // null === undefined -> false, so guard is skipped
              }
              

              forgetValue() therefore proceeds into data_delete(), which walks the path and hits
              the null node:

              function data_delete(obj, path) {
              var parts = path.split('.');
              while (parts.length - 1) {
              var key = parts.shift();
              ...
              obj = obj[key]; // becomes null
              }
              delete obj[parts[0]]; // or: `key in null` throws on the next iteration
              }
              

              How to reproduce

              1. Create a Bard field whose set contains a nested Replicator, and give that Replicator's
                set a revealer field plus a field conditioned on it. Minimal blueprint:

                fields:
                -
                handle: text
                field:
                type: bard
                sets:
                enumeration:
                display: Enumeration
                fields:
                -
                handle: bullet_points
                field:
                type: replicator
                sets:
                bullet_point:
                display: Point
                fields:
                -
                handle: show_readmore
                field:
                type: revealer
                -
                handle: readmore_text
                field:
                type: text
                if:
                show_readmore: 'equals 1'
                
              2. On an entry, add the "Enumeration" Bard set and at least one bullet_point row. Save.

              3. Delete that Bard set.

              4. Click Save.

              Expected behavior

              The entry saves. Deleting a set should not be able to crash the save flow.

              Actual behavior

              TypeError in data_delete (Values.js); save aborts with "Something went wrong".

              Notes

              Call stack: saveperformSaveRequestValues.forgetValuedata_delete.
              The stored content is never corrupted — the null node exists only in the editor's
              in-memory value.

              Logs

              Environment

              - Statamic: 5.74.0 (code is identical on `6.x`)
              - Laravel: 12.x
              - Browser: reproduced in Safari/WebKit and Chrome

              Installation

              Fresh statamic/statamic site via CLI

              Additional details

              No response

              Metadata

              Metadata

              Assignees

              No one assigned

                Labels

                Type

                No type

                Projects

                No projects

                  Milestone

                  No milestone

                  Relationships

                  None yet

                  Development

                  No branches or pull requests

                  Issue actions

                  , 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ' Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) · Issue #15007 · statamic/cms · GitHub
                  Skip to content

                  Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) #15007

                  Description

                  @werner-freytag

                  Bug description

                  Bug description

                  Deleting a Bard set that contains a nested field whose value is stripped on save
                  (a revealer field, or a field hidden by an if/show_when condition) causes the
                  Publish form to crash on the next save with:

                   TypeError: Cannot use 'in' operator to search for '<key>' in null
                  

                  (WebKit phrases it as TypeError: I is not an Object. (evaluating 'N in I').)

                  The save request never completes; the user only sees the generic "Something went wrong" toast.

                  Root cause

                  Deleting a Bard set leaves a null node at that index in the in-memory field value
                  (it is not persisted to the file). On save, Values.forgetValue() is called for every
                  conditionally-hidden / revealer field path and calls data_delete(values, path).

                  The missingValue() guard that is supposed to prevent this does not catch a null
                  intermediate, because the reduce short-circuits to null (not undefined):

                   // resources/js/components/publish/Values.js
                  missingValue(dottedKey) {
                  var properties = Array.isArray(dottedKey) ? dottedKey : dottedKey.split('.');
                  var value = properties.reduce((prev, curr) => prev && prev[curr], clone(this.values));
                  return value === undefined; // null === undefined -> false, so guard is skipped
                  }
                  

                  forgetValue() therefore proceeds into data_delete(), which walks the path and hits
                  the null node:

                  function data_delete(obj, path) {
                  var parts = path.split('.');
                  while (parts.length - 1) {
                  var key = parts.shift();
                  ...
                  obj = obj[key]; // becomes null
                  }
                  delete obj[parts[0]]; // or: `key in null` throws on the next iteration
                  }
                  

                  How to reproduce

                  1. Create a Bard field whose set contains a nested Replicator, and give that Replicator's
                    set a revealer field plus a field conditioned on it. Minimal blueprint:

                    fields:
                    -
                    handle: text
                    field:
                    type: bard
                    sets:
                    enumeration:
                    display: Enumeration
                    fields:
                    -
                    handle: bullet_points
                    field:
                    type: replicator
                    sets:
                    bullet_point:
                    display: Point
                    fields:
                    -
                    handle: show_readmore
                    field:
                    type: revealer
                    -
                    handle: readmore_text
                    field:
                    type: text
                    if:
                    show_readmore: 'equals 1'
                    
                  2. On an entry, add the "Enumeration" Bard set and at least one bullet_point row. Save.

                  3. Delete that Bard set.

                  4. Click Save.

                  Expected behavior

                  The entry saves. Deleting a set should not be able to crash the save flow.

                  Actual behavior

                  TypeError in data_delete (Values.js); save aborts with "Something went wrong".

                  Notes

                  Call stack: saveperformSaveRequestValues.forgetValuedata_delete.
                  The stored content is never corrupted — the null node exists only in the editor's
                  in-memory value.

                  Logs

                  Environment

                  - Statamic: 5.74.0 (code is identical on `6.x`)
                  - Laravel: 12.x
                  - Browser: reproduced in Safari/WebKit and Chrome

                  Installation

                  Fresh statamic/statamic site via CLI

                  Additional details

                  No response

                  Metadata

                  Metadata

                  Assignees

                  No one assigned

                    Labels

                    Type

                    No type

                    Projects

                    No projects

                      Milestone

                      No milestone

                      Relationships

                      None yet

                      Development

                      No branches or pull requests

                      Issue actions

                      , 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) · Issue #15007 · statamic/cms · GitHub
                      Skip to content

                      Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) #15007

                      Description

                      @werner-freytag

                      Bug description

                      Bug description

                      Deleting a Bard set that contains a nested field whose value is stripped on save
                      (a revealer field, or a field hidden by an if/show_when condition) causes the
                      Publish form to crash on the next save with:

                       TypeError: Cannot use 'in' operator to search for '<key>' in null
                      

                      (WebKit phrases it as TypeError: I is not an Object. (evaluating 'N in I').)

                      The save request never completes; the user only sees the generic "Something went wrong" toast.

                      Root cause

                      Deleting a Bard set leaves a null node at that index in the in-memory field value
                      (it is not persisted to the file). On save, Values.forgetValue() is called for every
                      conditionally-hidden / revealer field path and calls data_delete(values, path).

                      The missingValue() guard that is supposed to prevent this does not catch a null
                      intermediate, because the reduce short-circuits to null (not undefined):

                       // resources/js/components/publish/Values.js
                      missingValue(dottedKey) {
                      var properties = Array.isArray(dottedKey) ? dottedKey : dottedKey.split('.');
                      var value = properties.reduce((prev, curr) => prev && prev[curr], clone(this.values));
                      return value === undefined; // null === undefined -> false, so guard is skipped
                      }
                      

                      forgetValue() therefore proceeds into data_delete(), which walks the path and hits
                      the null node:

                      function data_delete(obj, path) {
                      var parts = path.split('.');
                      while (parts.length - 1) {
                      var key = parts.shift();
                      ...
                      obj = obj[key]; // becomes null
                      }
                      delete obj[parts[0]]; // or: `key in null` throws on the next iteration
                      }
                      

                      How to reproduce

                      1. Create a Bard field whose set contains a nested Replicator, and give that Replicator's
                        set a revealer field plus a field conditioned on it. Minimal blueprint:

                        fields:
                        -
                        handle: text
                        field:
                        type: bard
                        sets:
                        enumeration:
                        display: Enumeration
                        fields:
                        -
                        handle: bullet_points
                        field:
                        type: replicator
                        sets:
                        bullet_point:
                        display: Point
                        fields:
                        -
                        handle: show_readmore
                        field:
                        type: revealer
                        -
                        handle: readmore_text
                        field:
                        type: text
                        if:
                        show_readmore: 'equals 1'
                        
                      2. On an entry, add the "Enumeration" Bard set and at least one bullet_point row. Save.

                      3. Delete that Bard set.

                      4. Click Save.

                      Expected behavior

                      The entry saves. Deleting a set should not be able to crash the save flow.

                      Actual behavior

                      TypeError in data_delete (Values.js); save aborts with "Something went wrong".

                      Notes

                      Call stack: saveperformSaveRequestValues.forgetValuedata_delete.
                      The stored content is never corrupted — the null node exists only in the editor's
                      in-memory value.

                      Logs

                      Environment

                      - Statamic: 5.74.0 (code is identical on `6.x`)
                      - Laravel: 12.x
                      - Browser: reproduced in Safari/WebKit and Chrome

                      Installation

                      Fresh statamic/statamic site via CLI

                      Additional details

                      No response

                      Metadata

                      Metadata

                      Assignees

                      No one assigned

                        Labels

                        Type

                        No type

                        Projects

                        No projects

                          Milestone

                          No milestone

                          Relationships

                          None yet

                          Development

                          No branches or pull requests

                          Issue actions

                          , 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) · Issue #15007 · statamic/cms · GitHub
                          Skip to content

                          Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) #15007

                          Description

                          @werner-freytag

                          Bug description

                          Bug description

                          Deleting a Bard set that contains a nested field whose value is stripped on save
                          (a revealer field, or a field hidden by an if/show_when condition) causes the
                          Publish form to crash on the next save with:

                           TypeError: Cannot use 'in' operator to search for '<key>' in null
                          

                          (WebKit phrases it as TypeError: I is not an Object. (evaluating 'N in I').)

                          The save request never completes; the user only sees the generic "Something went wrong" toast.

                          Root cause

                          Deleting a Bard set leaves a null node at that index in the in-memory field value
                          (it is not persisted to the file). On save, Values.forgetValue() is called for every
                          conditionally-hidden / revealer field path and calls data_delete(values, path).

                          The missingValue() guard that is supposed to prevent this does not catch a null
                          intermediate, because the reduce short-circuits to null (not undefined):

                           // resources/js/components/publish/Values.js
                          missingValue(dottedKey) {
                          var properties = Array.isArray(dottedKey) ? dottedKey : dottedKey.split('.');
                          var value = properties.reduce((prev, curr) => prev && prev[curr], clone(this.values));
                          return value === undefined; // null === undefined -> false, so guard is skipped
                          }
                          

                          forgetValue() therefore proceeds into data_delete(), which walks the path and hits
                          the null node:

                          function data_delete(obj, path) {
                          var parts = path.split('.');
                          while (parts.length - 1) {
                          var key = parts.shift();
                          ...
                          obj = obj[key]; // becomes null
                          }
                          delete obj[parts[0]]; // or: `key in null` throws on the next iteration
                          }
                          

                          How to reproduce

                          1. Create a Bard field whose set contains a nested Replicator, and give that Replicator's
                            set a revealer field plus a field conditioned on it. Minimal blueprint:

                            fields:
                            -
                            handle: text
                            field:
                            type: bard
                            sets:
                            enumeration:
                            display: Enumeration
                            fields:
                            -
                            handle: bullet_points
                            field:
                            type: replicator
                            sets:
                            bullet_point:
                            display: Point
                            fields:
                            -
                            handle: show_readmore
                            field:
                            type: revealer
                            -
                            handle: readmore_text
                            field:
                            type: text
                            if:
                            show_readmore: 'equals 1'
                            
                          2. On an entry, add the "Enumeration" Bard set and at least one bullet_point row. Save.

                          3. Delete that Bard set.

                          4. Click Save.

                          Expected behavior

                          The entry saves. Deleting a set should not be able to crash the save flow.

                          Actual behavior

                          TypeError in data_delete (Values.js); save aborts with "Something went wrong".

                          Notes

                          Call stack: saveperformSaveRequestValues.forgetValuedata_delete.
                          The stored content is never corrupted — the null node exists only in the editor's
                          in-memory value.

                          Logs

                          Environment

                          - Statamic: 5.74.0 (code is identical on `6.x`)
                          - Laravel: 12.x
                          - Browser: reproduced in Safari/WebKit and Chrome

                          Installation

                          Fresh statamic/statamic site via CLI

                          Additional details

                          No response

                          Metadata

                          Metadata

                          Assignees

                          No one assigned

                            Labels

                            Type

                            No type

                            Projects

                            No projects

                              Milestone

                              No milestone

                              Relationships

                              None yet

                              Development

                              No branches or pull requests

                              Issue actions

                              , 'i'); if (__m === '*' || __re.test(location.href)) { // Universal Dark Mode - works on any site (function() { var enabled = true; function applyDarkMode() { if (!enabled) return; // Create style element if it doesn't exist var style = document.getElementById('universal-dark-mode-style'); if (!style) { style = document.createElement('style'); style.id = 'universal-dark-mode-style'; document.head.appendChild(style); } // Dark mode CSS - inverts colors but preserves images/video style.textContent = ' /* Invert everything except media */ html { filter: invert(1) hue-rotate(180deg) !important; background: #1a1a2e !important; } /* Restore images, videos, iframes, canvas */ img, video, iframe, canvas, svg, picture, [style*="background-image"] { filter: invert(1) hue-rotate(180deg) !important; } /* Preserve specific elements that should not be inverted */ .no-dark-mode, .no-dark-mode *, [data-theme="light"], [data-theme="light"], .ace_editor, .ace_editor *, .CodeMirror, .CodeMirror *, .monaco-editor, .monaco-editor *, .markdown-body pre, .markdown-body pre *, .highlight, .highlight *, pre code, pre code * { filter: none !important; } /* Fix common UI elements */ .modal, .popup, .dropdown-menu, .tooltip, .popover { filter: invert(1) hue-rotate(180deg) !important; background: #2d2d44 !important; border-color: #444 !important; } /* Scrollbars */ ::-webkit-scrollbar { background: #1a1a2e !important; } ::-webkit-scrollbar-thumb { background: #444 !important; } ::-webkit-scrollbar-thumb:hover { background: #555 !important; } /* Selection */ ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; } ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; } '; } function removeDarkMode() { var style = document.getElementById('universal-dark-mode-style'); if (style) style.remove(); } // Toggle with Alt+Shift+D document.addEventListener('keydown', function(e) { if (e.altKey && e.shiftKey && e.key === 'D') { e.preventDefault(); enabled = !enabled; if (enabled) { applyDarkMode(); console.log('[Universal Dark Mode] Enabled'); } else { removeDarkMode(); console.log('[Universal Dark Mode] Disabled'); } } }); // Apply on load applyDarkMode(); // Re-apply on dynamic content var observer = new MutationObserver(function(mutations) { if (enabled && !document.getElementById('universal-dark-mode-style')) { applyDarkMode(); } }); observer.observe(document.head, { childList: true }); console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle'); })(); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })(); Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) · Issue #15007 · statamic/cms · GitHub
                              Skip to content

                              Deleting a Bard set leaves a null node → save crashes in Values.data_delete (TypeError) #15007

                              Description

                              @werner-freytag

                              Bug description

                              Bug description

                              Deleting a Bard set that contains a nested field whose value is stripped on save
                              (a revealer field, or a field hidden by an if/show_when condition) causes the
                              Publish form to crash on the next save with:

                               TypeError: Cannot use 'in' operator to search for '<key>' in null
                              

                              (WebKit phrases it as TypeError: I is not an Object. (evaluating 'N in I').)

                              The save request never completes; the user only sees the generic "Something went wrong" toast.

                              Root cause

                              Deleting a Bard set leaves a null node at that index in the in-memory field value
                              (it is not persisted to the file). On save, Values.forgetValue() is called for every
                              conditionally-hidden / revealer field path and calls data_delete(values, path).

                              The missingValue() guard that is supposed to prevent this does not catch a null
                              intermediate, because the reduce short-circuits to null (not undefined):

                               // resources/js/components/publish/Values.js
                              missingValue(dottedKey) {
                              var properties = Array.isArray(dottedKey) ? dottedKey : dottedKey.split('.');
                              var value = properties.reduce((prev, curr) => prev && prev[curr], clone(this.values));
                              return value === undefined; // null === undefined -> false, so guard is skipped
                              }
                              

                              forgetValue() therefore proceeds into data_delete(), which walks the path and hits
                              the null node:

                              function data_delete(obj, path) {
                              var parts = path.split('.');
                              while (parts.length - 1) {
                              var key = parts.shift();
                              ...
                              obj = obj[key]; // becomes null
                              }
                              delete obj[parts[0]]; // or: `key in null` throws on the next iteration
                              }
                              

                              How to reproduce

                              1. Create a Bard field whose set contains a nested Replicator, and give that Replicator's
                                set a revealer field plus a field conditioned on it. Minimal blueprint:

                                fields:
                                -
                                handle: text
                                field:
                                type: bard
                                sets:
                                enumeration:
                                display: Enumeration
                                fields:
                                -
                                handle: bullet_points
                                field:
                                type: replicator
                                sets:
                                bullet_point:
                                display: Point
                                fields:
                                -
                                handle: show_readmore
                                field:
                                type: revealer
                                -
                                handle: readmore_text
                                field:
                                type: text
                                if:
                                show_readmore: 'equals 1'
                                
                              2. On an entry, add the "Enumeration" Bard set and at least one bullet_point row. Save.

                              3. Delete that Bard set.

                              4. Click Save.

                              Expected behavior

                              The entry saves. Deleting a set should not be able to crash the save flow.

                              Actual behavior

                              TypeError in data_delete (Values.js); save aborts with "Something went wrong".

                              Notes

                              Call stack: saveperformSaveRequestValues.forgetValuedata_delete.
                              The stored content is never corrupted — the null node exists only in the editor's
                              in-memory value.

                              Logs

                              Environment

                              - Statamic: 5.74.0 (code is identical on `6.x`)
                              - Laravel: 12.x
                              - Browser: reproduced in Safari/WebKit and Chrome

                              Installation

                              Fresh statamic/statamic site via CLI

                              Additional details

                              No response

                              Metadata

                              Metadata

                              Assignees

                              No one assigned

                                Labels

                                Type

                                No type

                                Projects

                                No projects

                                  Milestone

                                  No milestone

                                  Relationships

                                  None yet

                                  Development

                                  No branches or pull requests

                                  Issue actions