Version

3.0.7

Reproduction link

https://codesandbox.io/s/bold-drake-1qwvq?file=/src/App.vue

Steps to reproduce

I found two things which are weird: I get the "Maximum recursive updates exceeded" error only if I write <VEntry mandatory />.

But if I write <VEntry :mandatory="true" /> (or false) it works. (See comment in App.vue in the demo)

(For me a huge indicator that there is something wrong internally, since both should mean the same)

Additionally: If I wrap one of the passed computed refs with a readonly, it outputs other warnings. (See comment in DescriptionListEntry.vue in the demo)

What is expected?

• consistent behaviour with or without ="true" on boolean props (ideally without the recursive error)

• no internal write access on readonly computed refs

What is actually happening?

• Maximum recursive updates exceeded.

• Set operation on key "_ value" failed: target is readonly. Set operation on key "_ dirty" failed: target is readonly.

I am so confused right now. I tried to remove everything around the issue to track it down to a minimal reproduction. Hope this helps.

you should

  const register = (isVisible) => {
    console.log("useRelocatableParent - register");
    const index = nextIndex++;

    // like this
    registeredEntries.value[index] = { isVisible: isVisible.value };
    return index;
  };

On Fri, Mar 5, 2021 at 11:53 PM edison notifications@github.com wrote:

you should

const register = (isVisible) => { console.log("useRelocatableParent - register"); const index = nextIndex++;

// like this
registeredEntries.value[index] = { isVisible: isVisible.value };
return index;

};

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/vuejs/vue-next/issues/3371#issuecomment-791879259, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQR7RC4KIGJ2UYX6Y3Y3JK3TCG7OPANCNFSM4YVP4WKA .

-- Erica K Vance

@edison1105 this would loose reactivity. I need to pass computed refs, which will update as soon as their state change. Do you mean I should watch for changes and then call a injected function again each time? I thought the benifit of exposing computed is that I don't need to do that anymore. (This would still not explain why everything works when I write :mandatory="true" instead of just mandatory)

To get the smaller issue out of cleared up: wrapping a computed in readonly doesn't make much sense as its readonly already. But nonetheless we should probably not have it break.

Should likely be handled in a separate issue. (-> #3376, PR: #3377)

The thing we should concentrate on here is that, yes, it's weird and possibly a bug that this breaks only when relying on the default value.