Command: @cpattr about evennia HOT 13 CLOSED

From [email protected] on August 07, 2007 10:40:13

Owner: tsalaroth

from evennia.

From [email protected] on September 14, 2007 10:08:41

Owner: ---

from evennia.

From [email protected] on January 13, 2009 05:14:01

Owner: jamesvclemence

from evennia.

From [email protected] on January 13, 2009 05:14:55

I'm very happy to take on this one, I do have a few questions before I start work,
but will ask them when I have a moment!

from evennia.

From [email protected] on January 13, 2009 09:27:33

Status: Started

from evennia.

From [email protected] on January 17, 2009 19:02:56

Labels: -Priority-Medium Priority-High

from evennia.

From [email protected] on January 23, 2009 09:57:09

First draft implemented and issues found already fixed. Any further issues please
contact me.

Status: Implemented

from evennia.

From [email protected] on January 24, 2009 11:01:41

It looks like your editor is using tabs instead of spaces. Please make sure you're
indenting with 4 spaces, and not tabs. It really messes up things for the rest of us.
Take a peek at the Code Style document on the wiki at code.evennia.com.

Thanks!

from evennia.

From [email protected] on January 24, 2009 11:31:33

Corrected, my apologies, it should all be fixed now, please let me know if not - I'd
intended to do spaces as per the code style but a config error got in the way.

from evennia.

From [email protected] on January 24, 2009 12:52:35

I've got to re-open this, it doesn't work as per MUX2's syntax. Also, take a look and
note that you don't have to figure out if attributes already exist for the sake of
either updating or creating them, you can just use object.set_attribute("someattrib",
"someval") (which does that for you).

This is a bit of a complex one due to the four different styles of the command. What
I'd suggest is breaking it out into another module under the commands directory and
compartmentalize it out so it's easy to follow and doesn't pollute objmanip.py with
extra functions.

I've updated the comment docstring to reflect the full MUX2 syntax. You've got a good
bit of the work done already, you just need to write it in such a way that you're not
duplicating code for each of the syntaxes. See @search for some ideas, that has a lot
of different arrangements.

Status: Started

from evennia.

From [email protected] on January 25, 2009 10:15:46

Thats fine, my apologies I didn't realise the multiplicity of cpattr syntax! I'm
pretty busy the next few days but I'll run with hacking on it over the next week or
so. Cheers for the pointers, I had realised the set_attribute thing I'd taken a look
a couple of days ago, but hadn't updated the code accordingly.

Thanks once again I'll get you a more functioning commit in about a week.

from evennia.

From [email protected] on April 27, 2009 10:35:07

This hasn't seen action for a long time, opening it back up to whoever wants to
implement it as per MUX/MUSH.

Status: Accepted
Owner: ---

from evennia.

From griatch on August 29, 2009 12:16:43

Implemented as per MUX specifications.

Status: Implemented
Owner: griatch

from evennia.