 |
ReedN
Wizard
Joined: 04 Jan 2006
Posts: 1279
Location: Portland, Oregon
|
 Posted: Thu Jul 30, 2009 4:46 am
[Fixed] Wrong direction sent for doors when a long direction value exists
|
Summary: Map sends the wrong movement command for doors when there exists compound long direction values for the direction.
1) Create a new session, enter it off-line
2) Import the modified 'n' direction:
| Code: |
<class name="directions" id="4">
<dir name="n" reverse="s" dir="n" id="7">n|jump north</dir>
</class>
|
3) Open the mapper with the map button
4) Change to edit mode and create two rooms.
5) Enter this into the command line to create a north south link between the two rooms.
| Code: |
| %roomlink(1, "n", 2) |
6) Add a door between the two rooms
7) Right click on the first room and select "Set Current Position"
8) Double click on the second room.
9) It erroneously sends:
When it should have sent:
|
|
Last edited by ReedN on Sat Oct 24, 2009 1:43 am; edited 2 times in total |
 |
|
|
Tech
GURU
Joined: 18 Oct 2000
Posts: 2733
Location: Atlanta, USA
|
Posted: Thu Jul 30, 2009 5:26 am
|
Note this only happens if the exit is has a door, but confirmed.
|
|
_________________
Asati di tempari! |
|
|
|
ReedN
Wizard
Joined: 04 Jan 2006
Posts: 1279
Location: Portland, Oregon
|
Posted: Thu Jul 30, 2009 5:29 am
|
I meant to have a door in there. I'll edit the original post.
|
|
|
|
|
Zugg
MASTER
Joined: 25 Sep 2000
Posts: 23379
Location: Colorado, USA
|
Posted: Thu Oct 22, 2009 11:07 pm
|
I'm going to go ahead and fix this, but it might have a bad side effect for other people.
In 3.10 (and previous versions), CMUD uses the *longest* string in the direction command string list as the primary direction name. That is why it was sending "jump north" instead of just "n".
In v3.11, I am changing this behavior to use the *first* string in the direction command string list. I think I remember us talking about this in the past and deciding that it was the right thing to do. Hopefully it won't cause problems for other mapper users. |
|
|
|
|
ReedN
Wizard
Joined: 04 Jan 2006
Posts: 1279
Location: Portland, Oregon
|
Posted: Thu Oct 22, 2009 11:56 pm
|
I agree, I believe that is the best way to do it. Anyone can change the order of their directions to put the primary one first.
|
|
|
|
|
ReedN
Wizard
Joined: 04 Jan 2006
Posts: 1279
Location: Portland, Oregon
|
Posted: Sat Oct 24, 2009 1:43 am
|
Confirmed fixed, hurrah!
|
|
|
|
|
|
|
|
