-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
[Reactivity Transform] $$
breaks the original semantics
#6312
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Labels
🔨 p3-minor-bug
Priority 3: this fixes a bug, but is an edge case that only affects very specific usage.
🐞 bug
Something isn't working
scope: reactivityTransform
Comments
I've closed my PR. If you decide the best solution is to remove the brackets when transforming |
Unfortunately, Reactivity Transform has been dropped officially now, so I closed this issue/PR. If you want to keep using it, please consider the community version Vue Macros. |
chrislone
pushed a commit
to chrislone/core
that referenced
this issue
Feb 4, 2023
zhangzhonghe
pushed a commit
to zhangzhonghe/core
that referenced
this issue
Apr 12, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
🔨 p3-minor-bug
Priority 3: this fixes a bug, but is an edge case that only affects very specific usage.
🐞 bug
Something isn't working
scope: reactivityTransform
Vue version
v3.2.37
Link to minimal reproduction
https://sfc.vuejs.org/#eNo9TjsOwjAMvUpkdQAJErFWoRIbh8hSKpe2aj6yXRiq3p2kICb7ffz8VrilpF8LQg2WOxqTKEZZUuOCC10MLIrUVVWE/eFydKGqDpSHNV/zbrOCPs2tYEZK2eHS3HGeozV5K9a/DCcYfYokZ98mPXEM+e1ajtxPYAe12pnC5V4FOxhEEtfGcN+VshPrSE+TN01LkNGjRvbnB8U3I+VgByVic2GD7QP4UU0C
Steps to reproduce
Open the reproduction link, there's an error on SFC playground.
Cannot access 'r' before initialization
The code
is transformed to
We can see that
$$
is removed, and only brackets remain. It will break the original semantics.It's actually only one line code:
const r = _ref(1)(r)
.What is expected?
What is actually happening?
System Info
Any additional comments?
IMHO, the brackets should be removed to avoid changing the semantics when using
$$
function.The text was updated successfully, but these errors were encountered: