The function on Match.Where is called two times, why?
Is normal or maybe a bug ?
checkFn = Match.Where(function(x) {
console.log('checkFn');
return true;
})
check('', checkFn);
output:
=> App running at: http://localhost:3000/
I20151107-17:19:27.946(1)? checkFn
I20151107-17:19:27.946(1)? checkFnWell, where did you copy this code from? What file in your project?
I think thatâs not a Meteor issue, just a misunderstanding on how Meteor deal with the files.
Hi @borges
is not a production file. Just a bug reproducction.
Create a clean project, create a file on root project with this code line and test your self.
But thatâs the problem: you canât run this piece of code on a root file on Meteor.
You probably want to have the checkFn function on that âroot fileâ (a file in a common directory, that will be copied to both client and server) and use check('', checkFn) inside a Meteor Method or Template event.
Code that runs on âroot filesâ is no reliable to run once. If you really want that to run a single time, you should move it to a Meteor.startup() block.
And if that works for you, I suggest you to close your issue on github.
Meteor doc not say nothing about use check on Methods or template only. I think it can be use everywhere.
I recreate the example again, a little different and it confirm that checkFn is called twice what is a wrong behaviour.
With this example:
checkFn = Match.Where(function(x) {
console.log('checkFn');
return true;
});
function callmeOne() {
console.log('start callmeOne');
check(true, checkFn);
console.log('end callmeOne');
}
callmeOne();
and the output is:
=> App running at: http://localhost:3000/
I20151108-20:39:28.198(1)? start callmeOne
I20151108-20:39:28.199(1)? checkFn
I20151108-20:39:28.199(1)? checkFn
I20151108-20:39:28.200(1)? end callmeOne
The function containing the check call is called one time but check is called twice.
Youâre not getting it. Thatâs the correct behaviour.
Meteor apps, unlike Node projects, shouldnât have functions or code on any âroot fileâ. Thatâs not how it works. If you do this, itâs not guaranteed how many times it would run.
Meteor merges and wraps everything thatâs on âroot filesâ on javascriptâs immediate functions. However, this immediate function can be called many times, thatâs why your code is called twice.
If you want to check an object, you want to do it, for instance, on the client, after you received userâs input, right? So, you can:
checkFn = Match.Where(function(x) {
console.log('checkFn');
return true;
});
if (Meteor.isClient) {
Template.hello.events({
'submit #form': function (e) {
check(e.target.value, checkFn);
}
});
}
@borges. Thanks for take the time answer me but your are wrong.
First when I say âroot fileâ I mean a file on meteor project root folder not inside any folder.
I don´t think that meteor not guaranteed how many time a function is called.
You can use check on every place: server or client code. Is like put a
if condition and throw an Error is condition fail but encapsulate on a convenient API.
Check is for break code execution is value not passed pattern validation.
I you test your own code example, when you click submit form and check get passed, you will get the console output âcheckFnâ, twice.
Thanks anyway.
Now I get what you said. However, you still havenât understood what I meant.
Anyway, whatâs the matter with check being called twice? If it throws an error or donât only the logic break matters. Itâs not like youâll be calling console.log there.
Yet, it could be a library error.
Matter cause any piece of code can´t be execute more time that is design for.
You can use console.log or any log system you need to track what and where things happend and Match.Where is not different than any other Meteor function.
I you have the same twice behaviour like me, this confirm check package bug and we need MDG accept it and fix on next release.
I had reproduce too with Match.test function and Match.Where is runned twice there too.
I belive that this bug is important for all of us that use check or Match.test function.
For anyone that need track this issue, https://github.com/meteor/meteor/issues/5630
Hey can you create a reproduction repo, so people from here or MDG will know what exactly youâre talking.
I assume youâll get response from MDG saying âplease submit reproduction of the issue in a repoâ, it will be best if you do that even before they are ask. Itâll be helpful if community or MDG want to help you.
NOTE Iâm also worried bcz Iâm using Match package in production tooâŚ
I can make a repo if MDG or anyone here needed but I suppose is matter of copy paste the example lines above. More faster than clone a github repo.
I don´t know is a simple check like, Ex:
check(str, String);
is runned twice too. I only tested with Match.Where functions.
So, Iâve tried my hardest to reproduce this (copied your code into a fresh project) and canât.
The closest I can get is a log in the browser (client) and one on the console (server) if I put the code in a shared location. Otherwise I get one logged line only.
What OS and Meteor version are you on?
I clean meteor and install it again. Remove ~\.meteor is the correct way to doit??
I create a repo for bug reproduction, is here: https://github.com/bySabi/twice-match-where-call
The Match.Where function is called twice on all use cases: client, server, and both.
I have latest meteor 1.2.1 on OSX.
Maybe is a problem on my environment.
What is the recommended way to clean and start from scratch?
Your repo has three calls - one from the lib/ folder (which will execute on the client and the server), one in the client/ folder (which will execute on the client only) and one in the server/ folder (which will execute on the server only).
So, you end up with:
called from | logged on client | logged on server
----------------+----------------------+------------------
lib/ | yes | yes
client/ | yes |
server/ | | yes
And thereâs your two logs on the client and two on the server
I thought that at first too @robfallows
but if you look more carefully at the repo, or even run it as I did, you realise @bysabi differentiates the text in console.log() for each file
If you comment out the entire contents of â/lib/file,jsâ so that on the client it is only called once, the result is still
start callmeOne _ client only
checkFn _ client only
checkFn _ client only
end callmeOne _ client only
It seens @garrilla have the same wrong behaviour than me. Then is not related to my environment but a Meteor bug and we need track github issue: https://github.com/meteor/meteor/issues/5630
Thanks for take the time to test it.
Youâre right of course. My bad.
However, I have got this to behave (pre 1.2.1). So, it looks like a new introduction. Iâm just trying to bottom out whatâs causing it.