bugBattle for Wesnoth - Bugs: bug #5917, Allow commands to be performed on...

 
 
Show feedback again

bug #5917: Allow commands to be performed on undo in [allow_undo]

Submitted by:  Lari Nieminen <zookeeper>
Submitted on:  Sun 07 May 2006 08:07:57 PM UTC  
 
Category: Feature RequestSeverity: 1 - Wish
Priority: 3 - LowItem Group: WML
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Release: trunkOperating System: all

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sat 13 Jan 2007 11:27:34 PM UTC, comment #1:

Further notes: currently, [allow_undo] doesn't make the move undoable if there was an event triggered that either mutated something (I suppose the engine marks the state as mutated when changing variables and such) or if the move uncovered shroud/fog. A move that uncovers shroud/fog should IMO never be allowed to be undone, so this FR shouldn't affect that. However, an [allow_undo] should allow undoing of moves that merely caused some event that mutates something (and it's the WML writer's responsibility to make sure he then reverts all mutations inside [allow_undo]).

Lari Nieminen <zookeeper>
Project Member
Sun 07 May 2006 08:07:57 PM UTC, original submission:

Currently, for example a name=moveto [event] can be made undoable by putting [allow_undo][/allow_undo] inside it. However, events that actually make something happen (altering variables, units, whatever) can't really be made undoable, even though gameplay-wise undoing wouldn't be a problem. This could be fixed by executing the WML written inside the [allow_undo] tag when the move that triggered that event is undone. This way, even events that alter things can be made undoable by defining the commands that revert the involved variables, units or whatever to the previous state, so nothing gets messed up.

Lari Nieminen <zookeeper>
Project Member

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by zookeeper (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup