Skip to content
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

Component does not render in prod but does in debug, no errors or warnings #3297

Closed
Trojaner opened this issue Feb 24, 2021 · 1 comment · Fixed by #3299
Closed

Component does not render in prod but does in debug, no errors or warnings #3297

Trojaner opened this issue Feb 24, 2021 · 1 comment · Fixed by #3299
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

Comments

@Trojaner
Copy link

Trojaner commented Feb 24, 2021

Version

3.0.6

Reproduction link

http://plnkr.co/edit/9KJo5ozJ86xh9I8u

Steps to reproduce

  1. Open repro
  2. Observe how the bug component does not get rendered (no errors or warnings) (it should output 6 "a"'s side by side)
  3. Comment the following in line 17 in bug.js, see how the component gets rendered:
if(this.values) { }
  1. Uncomment the related code again to break it again
  2. Switch to debug module: replace vue.esm-browser.prod.js with vue.esm-browser.js in index.html
  3. Component gets rendered again despite the related "if" still being there

What is expected?

Component should render if using prod module; behavior should not be inconsistent with prod or debug module

What is actually happening?

Component does not render in prod mode, no errors or warnings in prod or debug mode

@edison1105
Copy link
Member

edison1105 commented Feb 25, 2021

just tip here.
accessCache['values'] should not equal AccessTypes.OTHER

image

if if(this.values) { } exec before return { values: vals, autoFocusIndex };
the accessCache['values'] will be set to AccessTypes.OTHER in PublicInstanceProxyHandlers.get.
so values is incorrect when next time access values

should we just remove

} else if (!__FEATURE_OPTIONS_API__ || !isInBeforeCreate) {
   accessCache![key] = AccessTypes.OTHER
}

or retry to get value when accessCache[key] === AccessTypes.OTHER

@HcySunYang HcySunYang added 🐞 bug Something isn't working 🔨 p3-minor-bug Priority 3: this fixes a bug, but is an edge case that only affects very specific usage. labels Feb 25, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Oct 23, 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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants