blog documentation FAQ followers forum marketplace services youtube

solved Updating Marketplace Item

The Jamroom Network » Forum » Jamroom Developers » Page 61 » Updating Marketplace Item
nate
@nate
11 years ago
917 posts
I got this error.

Quote: ERROR: bad console.log() function call found in file: char_count.js on line 32.
If this console.log() call is needed, add "// OK" as a comment on the same line AFTER the console.log() function call, and it will be ignored

updated by @nate: 09/27/14 07:07:50AM
michael
@michael
11 years ago
7,800 posts
console.log() only works for firefox. It breaks in browsers that dont support it.

Check that its actually needed in the packaged module If it is put the // OK after it.

Thats how I read it. I've seen in some places that there is 
fdebug() // OK
and suspect that what you are seeing is some packaging system check.
nate
@nate
11 years ago
917 posts
I guess I was vague. My mistake. This error happens on your site. Jamroom.net.
michael
@michael
11 years ago
7,800 posts
ah. Got a url? any way that I can see it?
brian
@brian
11 years ago
10,149 posts
Natedogg265:
I guess I was vague. My mistake. This error happens on your site. Jamroom.net.

That is correct. When you upload a file to the marketplace HERE On Jamroom.net, it runs several "tests" on your code:

- it checks all PHP scripts for syntax and syntax errors
- checks for debug()/fdebug() calls left in the code
- checks for console.log calls left in the code

So what this error is telling you is that in YOUR javascript (char_count.js) there is a console.log call that needs to be removed.

We do use the char_count.js file in the jrAction module, but it does not contain any console.log calls.

Hope this helps!


--
Brian Johnson
Founder and Lead Developer - Jamroom
https://www.jamroom.net
nate
@nate
11 years ago
917 posts
Well that's pretty cool but you guys have to figure out a way to get this info out to developers. The way I just found out seems counter predictive.
brian
@brian
11 years ago
10,149 posts
Natedogg265:
Well that's pretty cool but you guys have to figure out a way to get this info out to developers. The way I just found out seems counter predictive.

I thought the error message was pretty clear, but let me know how it could be reworded to make it more intuitive what the issue is.

Thanks!


--
Brian Johnson
Founder and Lead Developer - Jamroom
https://www.jamroom.net
nate
@nate
11 years ago
917 posts
It's clear if you know the code is being checked. I had no idea of that.
brian
@brian
11 years ago
10,149 posts
Natedogg265:
It's clear if you know the code is being checked. I had no idea of that.

OK - I'll try to add some note there that indicates that.

Thanks!


--
Brian Johnson
Founder and Lead Developer - Jamroom
https://www.jamroom.net
nate
@nate
11 years ago
917 posts
I imagine a couple of extra words could fix it. I will say this, the error it found is something I recently fixed but I haven't packaged those updates. So it does work :-)
nate
@nate
11 years ago
917 posts
Errors were found in your uploaded code

might just do it.
updated by @nate: 08/26/14 11:25:49AM
brian
@brian
11 years ago
10,149 posts
Natedogg265:
Errors were found in your uploaded code

might just do it.

Yep that sounds good - I've updated the error messages to include that.

Thanks!


--
Brian Johnson
Founder and Lead Developer - Jamroom
https://www.jamroom.net

Tags