Skip to content

Better error message for ownership_invalid_binding? #13423

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

Closed
thebjorn opened this issue Sep 28, 2024 · 0 comments · Fixed by #15678
Closed

Better error message for ownership_invalid_binding? #13423

thebjorn opened this issue Sep 28, 2024 · 0 comments · Fixed by #15678

Comments

@thebjorn
Copy link

Describe the bug

This is what I'm seeing currently

[svelte] ownership_invalid_binding
C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Rows.svelte passed a value to C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Row.svelte with `bind:`, but the value is owned by C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Row.svelte. Consider creating a binding between C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Row.svelte and C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Rows.svelte

Could we get which value is problematic in the error message?

Suggestion 1: add newlines

[svelte] ownership_invalid_binding
    C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Rows.svelte 
passed a value to 
    C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Row.svelte 
with `bind:`, but the value is owned by 
    C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Row.svelte. 
Consider creating a binding between 
    C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Row.svelte 
and 
    C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Rows.svelte

Suggestion 2: "alias" file references? Something like

[svelte] ownership_invalid_binding
    C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Rows.svelte (Rows) 
passed a value to 
    C:/srv/lib/tilly/packages/grogui/src/lib/components/treetable/Row.svelte (Row) 
with `bind:`, but the value is owned by (Row). 
Consider creating a binding between (Row) and (Rows)

Reproduction

N/A (if I knew what was causing the problem, the error message would probably not be problematic ;-) )

Logs

No response

System Info

System:
    OS: Windows 11 10.0.22631
    CPU: (48) x64 AMD Ryzen Threadripper 3960X 24-Core Processor 
    Memory: 38.66 GB / 63.88 GB
  Binaries:
    Node: 22.6.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.21 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 10.8.2 - C:\Program Files\nodejs\npm.CMD
    pnpm: 9.11.0 - ~\AppData\Local\pnpm\pnpm.CMD
    bun: 1.1.8 - ~\AppData\Roaming\npm\bun.CMD
  Browsers:
    Edge: Chromium (127.0.2651.74)
    Internet Explorer: 11.0.22621.3527

(note: envinfo has found the two browsers I don't use.. there are 4 more installed + canary/dev of chrome and firefox...)

Severity

annoyance

@benmccann benmccann changed the title Better errorr message for ownership_invalid_binding? Better error message for ownership_invalid_binding? Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant