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

Blaze compile errors completely silent if imported #396

Open
TMeerhof opened this issue Apr 26, 2019 · 12 comments
Open

Blaze compile errors completely silent if imported #396

TMeerhof opened this issue Apr 26, 2019 · 12 comments
Labels
confirmed We want to fix or implement it hacktoberfest
Milestone

Comments

@TMeerhof
Copy link

In version 1.8 when compiling a blaze template from the /imports directory all errors are swallowed breaking the normal development process, because you get no line numbers or error specifications.

In 1.7 this problem is not present.

Reproduction:
Create a new meteor 1.8 project.

create a new malformed blaze template in the client folder.

<template name="test">
   {{#if}
</template>

You will get a compile error

Errors prevented startup:
   
   While processing files with templating-compiler (for target web.browser):
   client/test.html:2: Expected `}}`

Now move the file to /imports/client and add the import statement import '/imports/client/test.html'; to main.js.

There is no compile time error, but when running the app you get the following run time error
Uncaught Error: Cannot find module '/imports/client/test.html'.

@TMeerhof
Copy link
Author

this is also the case for .less files, here you will get the even more criptic error:

internal/process/next_tick.js:131
      callback();

@stale
Copy link

stale bot commented Dec 10, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale
Copy link

stale bot commented Dec 18, 2019

This issue has been automatically closed it has not had recent activity.

@stale stale bot closed this as completed Dec 18, 2019
@zenhack
Copy link

zenhack commented Feb 6, 2021

I'm still seeing this issue as of meteor 2.0; can we reopen this?

@haveaguess
Copy link

Agreed I have to refactor my files out of import to see what the error is and then refactor them back, very tedious

@zodern zodern reopened this Feb 6, 2021
@filipenevola filipenevola added the confirmed We want to fix or implement it label Feb 9, 2021
@StorytellerCZ
Copy link
Collaborator

Seems to be the same issue as in meteor/meteor#10366

@StorytellerCZ
Copy link
Collaborator

@zodern do you have any plans for this issue?

@StorytellerCZ
Copy link
Collaborator

@zodern @jankapunkt maybe we should move this to the Blaze repository?

@jankapunkt
Copy link
Collaborator

I agree 👍

@StorytellerCZ StorytellerCZ transferred this issue from meteor/meteor Oct 1, 2022
@StorytellerCZ StorytellerCZ added this to the Blaze 2.6.2 milestone Oct 1, 2022
@StorytellerCZ
Copy link
Collaborator

@jankapunkt any way to quickly solve this for 2.6.2?

@jankapunkt
Copy link
Collaborator

@StorytellerCZ not quite sure about the "quick". I think the change was not within Blaze but Meteor core that caused it. Could lead to a trail of more issues, which is generally good but that's why "quick" may be not possible.

@StorytellerCZ
Copy link
Collaborator

OK, I'm moving this to v3 to solve.

@StorytellerCZ StorytellerCZ modified the milestones: Blaze 2.6.2, Blaze 3.0 Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
confirmed We want to fix or implement it hacktoberfest
Projects
None yet
Development

No branches or pull requests

7 participants