Open New/Custom Channel

Printable View

01-06-2015, 05:51 PM
Riku

Open New/Custom Channel
I searched and found some old threads about this, here, here and one that's a little bit off but still in the same discussion here.

I did not feel like these people expressed what they wanted to do with it
clear enough so here goes my try to convince you that my suggestion is awesome.

http://i.imgur.com/DZv4PyS.png

So we all know this lovely function ingame (CTRL+O) located in the bottom right corner that looks like a folder,
it opens a little window with all the different channels inside aswell as writeable box that opens a Private Message Channel with the named person.

New Functions:
- pmsay(string name, string message, optional boolean openwindow, optional boolean ctrl)
- openchannel(string channelname, optional boolean ctrl)
- openpmchannel(string name, optional boolean ctrl)
Further Explanation:
- pmsay()
  Will switch focus to the channel name and message an optional message to that channel, optional boolean to open the channel window if one is not already opened (will use *name*[message] otherwise),
  optional boolean to open via CTRL+O or clicking the Folder icon in the bottom right with a mouse click.
- openchannel()
  Will open a channel (World Chat, English Chat, Advertising, Help, Private Chat Channels, NPCs), optional boolean to open via CTRL+O or clicking the Folder icon in the bottom right with a mouse click.
- openpmchannel()
  Will open a private message channel with given name, optional boolean to open via CTRL+O or clicking the Folder icon in the bottom right with a mouse click.
```
--> Do not edit anything below!

-- // Riku Scripts >>>>



init start

	local sendToPlayer = "Lucas Terra"

	local seenNames = {$name}

init end



auto(1000)

	

foreach creature m 'ps' do

	if (not table.find(seenNames, m.name)) then

		if ischannel(sendToPlayer) then

			pmsay(sendToPlayer, "New Player in my screen.", true, true)

			wait(3000, 5000)

   		        table.insert(seenNames, m.name)

		else

			pmsay(sendToPlayer, "New Player in my screen.", false, true)

			wait(3000, 5000)

			table.insert(seenNames, m.name)

		end

	end

end
```
What is there not to love? If you need further explanations or ideas then tell me. It could be used for countless ideas.
01-06-2015, 07:48 PM
Raphael

Normally, I'd be bashing about how you're wasting everyone's time creating another thread with something that has already been suggested, but you did it so well, specially when compared to the other threads that, actually, I'm not even mad, that's amazing. I wish we had more suggestion threads like this one.

If we ever implement this, I'd like to see say() and npcsay() having the same optional parameters as the proposed pmsay(). One thing to consider, though, is what the default values for the optional booleans should be. Personally, I'd prefer a default of true for openwindow and false for usectrl, since that seems to be the desired behavior for the majority of the use cases.

That being said, as you pointed out in your explanation for pmsay(), this functionality can already be achieved by calling say('*<name>* message'), so I don't think we'll be prioritizing this now.

function pmsay(name, msg)

    if ischannel(name) then

        return say(name, msg)

    end



    return say(sprintf("*%s* %s", name, msg))

end
This deals with everything you need, I guess. And openchannel is not so necessary, as the main channels are kept opened after you log off.

01-07-2015, 03:52 PM
Raphael
Quote:
Originally Posted by Leonardo

function pmsay(name, msg)
if ischannel(name) then
return say(name, msg)
end

return say(sprintf("*%s* %s", name, msg))
end

This deals with everything you need, I guess. And openchannel is not so necessary, as the main channels are kept opened after you log off.
That actually fucks up for long names, as they get cropped in the chat window name.
01-07-2015, 04:00 PM
Leonardo

Quote:

Originally Posted by Raphael

That actually fucks up for long names, as they get cropped in the chat window name.

I didn't know if ischannel could deal with that by itself or not, so I let it right as it's, but the 'name' can be cropped as a string too if you know the maximum channel name's length.

edit:

Thinking about it again: it wouldn't fuck up, for example:

ischannel("Cavaleiro Lendario of Luminera") would return false even with channel "Cavaleiro len..." opened, and it would skip this step.
01-07-2015, 05:06 PM
Raphael

Quote:

Originally Posted by Leonardo

I didn't know if ischannel could deal with that by itself or not, so I let it right as it's, but the 'name' can be cropped as a string too if you know the maximum channel name's length.

edit:

Thinking about it again: it wouldn't fuck up, for example:

ischannel("Cavaleiro Lendario of Luminera") would return false even with channel "Cavaleiro len..." opened, and it would skip this step.

Ye, I'm sorry, it wouldn't "fuck up", but it wouldn't be as efficient as it could.
No, ischannel can't handle the cropped names.
Yes, you could account for the cropping if you knew the correct length, I was just reminding you of that.
01-08-2015, 05:57 PM
Riku

Quote:

Originally Posted by Raphael

Normally, I'd be bashing about how you're wasting everyone's time creating another thread with something that has already been suggested, but you did it so well, specially when compared to the other threads that, actually, I'm not even mad, that's amazing. I wish we had more suggestion threads like this one.

If we ever implement this, I'd like to see say() and npcsay() having the same optional parameters as the proposed pmsay(). One thing to consider, though, is what the default values for the optional booleans should be. Personally, I'd prefer a default of true for openwindow and false for usectrl, since that seems to be the desired behavior for the majority of the use cases.

That being said, as you pointed out in your explanation for pmsay(), this functionality can already be achieved by calling say('*<name>* message'), so I don't think we'll be prioritizing this now.

:confused: Speechless about this reply. Thank you very much and for the linking in the Suggestions Posting Guidelines!

Quote:

Originally Posted by Leonardo

This deals with everything you need, I guess. And openchannel is not so necessary, as the main channels are kept opened after you log off.

Well, yeah I guess it does but I'm a fan of simplicity aka not filling every script I have with homebrewed functions everywhere, i'm stubborn and will say this still doesn't deal with everything I need, it's just a patch on my flesh wound. :rolleyes: But thanks a lot for your effort! Will be using that until my ideas get implemented, if they actually do.

I was also thinking about some focuschannel(channelname) command also that switches focus to a certain channel. Nothing of everything i've proposed is necesarrily needed for WindBot, it's just some small details I personally would like to see implemented and would definitely have a lot of fun mixing together scripts with those functions!
01-08-2015, 07:06 PM
Leonardo
Quote:

Originally Posted by aZe

Well, yeah I guess it does but I'm a fan of simplicity aka not filling every script I have with homebrewed functions everywhere, i'm stubborn and will say this still doesn't deal with everything I need, it's just a patch on my flesh wound. :rolleyes: But thanks a lot for your effort! Will be using that until my ideas get implemented, if they actually do.

I was also thinking about some focuschannel(channelname) command also that switches focus to a certain channel. Nothing of everything i've proposed is necesarrily needed for WindBot, it's just some small details I personally would like to see implemented and would definitely have a lot of fun mixing together scripts with those functions!

I guess that can be "patched" too.
```
function focuschannel(channel)

    return say(channel, '')

end
```