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

client.send() always include : in first text #420

Closed
anonprophet opened this issue Oct 26, 2015 · 3 comments
Closed

client.send() always include : in first text #420

anonprophet opened this issue Oct 26, 2015 · 3 comments

Comments

@anonprophet
Copy link

case '!topgames':
client.send('s', 'chat', nick, 'Top Games :');
db.query('SELECT player, games FROM stats ORDER BY games DESC LIMIT 13', function(err, rows) {
    if (err) throw err;

    for (var i = 0; i < rows.length; i++) {
        client.send('s', 'chat', nick, i+1 + '. ' + rows[i].player + ' : ' + rows[i].games);
    };
});
break;

its always include : in first text
screenshot_1

its the debug message

26 Oct 15:26:22 - SEND: s chat mushi :1. miracle- : 13891

the message should 1. miracle- : 13891
not :1. miracle- : 13891

how/where i can to remote that : ?

@anonprophet anonprophet changed the title client.send() always include : client.send() always include : in first text Oct 26, 2015
@moshmage
Copy link

@anonprophet we are not your personal StackOverflow nor is this a issue what you are seeing is a log of what was sent to the server, which is composed of a : character.

What the SERVER sent to the user was 1. miracle- : 13891.

motions for closure

@anonprophet
Copy link
Author

@moshmage sorry, but its : not show if use mIRC (using /raw)
im ask , someone maybe know why : its show and where i can delete it

@ghost
Copy link

ghost commented Oct 26, 2015

@anonprophet This is only cosmetic to the debug messages, you are receiving the raw output as expected, you should be fine, but we cannot help if its not a node-irc issue. Also do you mind closing your other issue #416 before opening a new one?

@moshmage, i second your motion to close, but i suggest that novice requests get treated with more guidance, this was an honest mistake as oppose to someone trying to offload work to someone else.

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

No branches or pull requests

2 participants