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

[MdApp] rreturn is not defined #1727

Closed
mgobishop opened this issue May 13, 2018 · 2 comments · Fixed by #1728
Closed

[MdApp] rreturn is not defined #1727

mgobishop opened this issue May 13, 2018 · 2 comments · Fixed by #1728
Labels

Comments

@mgobishop
Copy link

Following error with Beta 10:

[Vue warn]: Error in render: "ReferenceError: rreturn is not defined"

I look at my bundle and see 'rreturn' the following function:

function shouldRenderSlot(data, componentOptions) {
rreturn(data && componentTypes.includes(data.slot)) || isValidChild(componentOptions);
}

@VdustR VdustR changed the title rreturn is not defined [MdApp] rreturn is not defined May 13, 2018
@VdustR VdustR added the bug label May 13, 2018
marcosmoura pushed a commit that referenced this issue May 13, 2018
This was referenced May 14, 2018
@appleby
Copy link

appleby commented May 14, 2018

[Edit]: Oops, just noticed that someone already reported this as a new issue in #1735. Please ignore.

I'm still seeing the [Vue warn]: Error in render: "ReferenceError: rreturn is not defined" error with vue-material@1.0.0-beta-10.1. I'm an npm/vue/vue-material noob, so sorry if I'm misunderstanding.

Here is a shell session demonstrating (I think) the issue. This is in a fresh directory where I just ran npm init. Note that the rreturn is gone from src/components/MdApp.vue but is still present in the dist files. I assume this should not be the case.

[ma@march vm2]? npm --version
5.5.1
[ma@march vm2]? npm install --save vue vue-material
npm notice created a lockfile as package-lock.json. You should commit this file.
+ vue-material@1.0.0-beta-10.1
+ vue@2.5.16
added 2 packages in 4.121s
[ma@march vm2]? fgrep -lr rreturn node_modules/vue-material/*
node_modules/vue-material/dist/vue-material.min.js
node_modules/vue-material/dist/vue-material.js
node_modules/vue-material/dist/components/MdApp/index.js
node_modules/vue-material/dist/components/index.js

@NightravenJames
Copy link

Thanks this will really help me alot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants