Jump to content

Admin watch Clan Chat


Recommended Posts

5 hours ago, Solomun said:

As far as i tested yday, it was working...

Magic else you are talking about H5 or so, as on interlude it's  not working.  Only the window pops up, but no chat logs. 

Link to comment
Share on other sites

2 hours ago, Solomun said:

As far as i tested yday, it was working...

It is not working on C6. The window was implemented and the packet structure is there but the messages wont appear in the window.

My guess was that it was just a PoC in the initial C6 release and it as actually finalized in a C6 update patch but i don't have the pre-Kamael Interlude patcht to prove it.

 

26 minutes ago, sepultribe said:

can you answer what I asked you or provide a screenshot?

If i remember correctly, you target the player and press //snoop and then this player's inbound and outbound chat appears in a chat window.

 

 

On the share: Nice, useful piece of code to replace the //snoop functionality that is missing on Interlude. However it is workin on pretty much any other chronicle after that, making it a bit useful.

 

A couple on things on the code: 

General: (ADMIN_COMMANDS[0]) This is not as smart as you think it is because i have to scroll to the top to see what im doing. Either make them constants with reasonable self explanatory name or use the string itself.

Line 127. This whole check is useless.

 

Other than that, it's fine.

Edited by .Elfocrash
Link to comment
Share on other sites

43 minutes ago, .Elfocrash said:

It is not working on C6. The window was implemented and the packet structure is there but the messages wont appear in the window.

My guess was that it was just a PoC in the initial C6 release and it as actually finalized in a C6 update patch but i don't have the pre-Kamael Interlude patcht to prove it.

 

If i remember correctly, you target the player and press //snoop and then this player's inbound and outbound chat appears in a chat window.

 

 

On the share: Nice, useful piece of code to replace the //snoop functionality that is missing on Interlude. However it is workin on pretty much any other chronicle after that, making it a bit useful.

 

A couple on things on the code: 

General: (ADMIN_COMMANDS[0]) This is not as smart as you think it is because i have to scroll to the top to see what im doing. Either make them constants with reasonable self explanatory name or use the string itself.

Line 127. This whole check is useless.

 

Other than that, it's fine.

thanks for taking the time to answer me.

 

link to latest known update patch for C6: http://www25.zippyshare.com/v/35171501/file.html

filename in case this link goes bad in the future:  InterludeUpdate071212full

Link to comment
Share on other sites

On 9/1/2018 at 2:50 PM, .Elfocrash said:

It is not working on C6. The window was implemented and the packet structure is there but the messages wont appear in the window.

My guess was that it was just a PoC in the initial C6 release and it as actually finalized in a C6 update patch but i don't have the pre-Kamael Interlude patcht to prove it.

 

If i remember correctly, you target the player and press //snoop and then this player's inbound and outbound chat appears in a chat window.

 

 

On the share: Nice, useful piece of code to replace the //snoop functionality that is missing on Interlude. However it is workin on pretty much any other chronicle after that, making it a bit useful.

 

A couple on things on the code: 

General: (ADMIN_COMMANDS[0]) This is not as smart as you think it is because i have to scroll to the top to see what im doing. Either make them constants with reasonable self explanatory name or use the string itself.

Line 127. This whole check is useless.

 

Other than that, it's fine.

My bad, i had no idea we are talking for interlude..

Edited by Solomun
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.



×
×
  • Create New...

AdBlock Extension Detected!

Our website is made possible by displaying online advertisements to our members.

Please disable AdBlock browser extension first, to be able to use our community.

I've Disabled AdBlock