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

Player to Player Tell/Page #5

Closed
Griatch opened this issue Jan 21, 2014 · 11 comments
Closed

Player to Player Tell/Page #5

Griatch opened this issue Jan 21, 2014 · 11 comments
Labels
feature-request A suggestion for new functionality.

Comments

@Griatch
Copy link
Member

Griatch commented Jan 21, 2014

From squishywaffle@gmail.com on April 23, 2007 17:53:21

Implement player to player paging/telling as per MUX and MUD. Let players
either use 'page' or 'tell' depending on what they're familiar with.

Original issue: http://code.google.com/p/evennia/issues/detail?id=5

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From m.tsal.k...@gmail.com on April 25, 2007 08:16:33

This should be something fairly simple to implement. I'll have some free time this
evening, and will take a stab at it.

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From squishywaffle@gmail.com on April 25, 2007 08:23:38

Ok, I'll try to get in the player alias stuff for you to tie into some time today.

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From squishywaffle@gmail.com on April 25, 2007 20:08:54

Labels: Milestone-Alpha

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From tsalar...@gmail.com on April 26, 2007 16:37:22

I've committed a stab at cmd_page, but I'm having trouble getting the server up and
running under MySQL, so I can't tell if the problem with it (it dumps connection
when you try to "page" someone) is SQLite or MySQL. Needs work, obviously. I might
re-approach it later, I'm sure there's better ways to do what I did. In addition, I
over-simplified error conditions.

Owner: tsalaroth
Cc: squishywaffle

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From m.tsal.k...@gmail.com on April 27, 2007 14:11:49

I'll have to fix a bug I missed before when I get home - I didn't properly put a
couple of things in tuples for string interpolation.

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From tsalar...@gmail.com on April 28, 2007 07:55:39

The page command works now. Just need to add an alias for tell, and all should work
fine, unless we're allowing spaces in character names. But I think there's a
couple of places that might break, if so.

Status: Needs-Information

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From squishywaffle@gmail.com on April 28, 2007 07:59:06

Yes, we will allow spaces in names with 'page', but 'tell' wouldn't work. I haven't
had a chance to look things over yet but if you want an example of a command in the
form of:

command =

See @teleport. This should be very similar to page, since page is

page =

You'll be able to rip off the argument and '=' sign checking code.

I'll get aliases in shortly, I'm a bit railed with work right now but know how I'm
going to approach it.

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From tsalar...@gmail.com on April 28, 2007 08:07:36

Labels: -Priority-High Priority-Medium

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From tsalar...@gmail.com on April 29, 2007 06:57:57

Well, I've implemented it using that protocol, however I'm not able to successfully
page names with spaces for some reason. I'm still tracking down the issue, I suspect
it might be in the db functions or in the way I'm using the data.

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From squishywaffle@gmail.com on April 29, 2007 07:05:13

I'm not sure, but you're definitely going to want to change
message = eq_args[1]
To
message = eq_args[1:]
And then join() the message variable into a string (instead of the list). Someone may
put equal signs in their message, currently they'd lose any subsequent equal signs.

@Griatch
Copy link
Member Author

Griatch commented Jan 21, 2014

From tsalar...@gmail.com on May 04, 2007 10:04:48

After the changes you've made, we've got a working page. I'm going to close this out.

Status: Fixed

@Griatch Griatch closed this as completed Jan 21, 2014
Griatch pushed a commit that referenced this issue Feb 2, 2014
Fixed some unicode encoding mixups.
Griatch pushed a commit that referenced this issue Jun 12, 2018
Griatch pushed a commit that referenced this issue Dec 2, 2022
…attribute

Add unit test for seaching for objects by attribute
Griatch pushed a commit that referenced this issue Oct 19, 2023
…attribute

Add unit test for seaching for objects by attribute
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A suggestion for new functionality.
Projects
None yet
Development

No branches or pull requests

1 participant