patchFreeciv - Patches: patch #2121, [delegation02] add delegation...

 
 
Show feedback again

patch #2121: [delegation02] add delegation (especially for longturn)

Submitted by:  Matthias Pfafferodt <syntron>
Submitted on:  Fri Oct 29 20:15:41 2010  
 
Category: generalPriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: Matthias Pfafferodt <syntron>Open/Closed: Closed
Planned Release: 2.4.0Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

(Jump to the original submission Jump to the original submission)

Sun Jul 1 11:54:40 2012, comment #6:

Can someone point me at the original discussion / implementation of the delegate feature and its purpose (presumably for Longturn)? I think I understand the basics, but I'd like to work out what it should do in some corner cases that our version doesn't currently handle.

(If no documentation, I'll post my queries in a new ticket.)

Jacob Nevins <jtn>
Project Administrator
Fri Jan 14 22:29:20 2011, SVN revision 18942:

add delegation

  • delegate your control to another username
  • new command: /delegate
  • if a connection is detached restore the player state

(new wrapper function connection_close_server())

see gna patch #2121

(Browse SVN revision 18942)

Matthias Pfafferodt <syntron>
Project MemberIn charge of this item.
Wed Jan 5 18:36:55 2011, comment #4:

changes 20101230:

  • rewrite much of the code
  • move take/restore part to their own functions in connecthand.c
  • if a connection is detached restore player state (new function connection_close_server())

changes 20110105:

  • rebased directly to trunk (does not need fcdb patches)

(file #11704)

Matthias Pfafferodt <syntron>
Project MemberIn charge of this item.
Thu Dec 23 12:43:53 2010, comment #3:

new version of the patch:

  • no dependency on any database
  • save delegation
  • bugs:

- taking a player from an observer works but going back is not possible
- more?

please test!

(file #11661)

Matthias Pfafferodt <syntron>
Project MemberIn charge of this item.
Sat Oct 30 10:01:04 2010, comment #2:

> Why does it requires auth support enabled?


Because I did use the longturn usage as example and guideline?

Yea, it could be done entirely in the server without an external database ... I will have to think about this. Thanks for the comment.

Matthias Pfafferodt <syntron>
Project MemberIn charge of this item.
Fri Oct 29 23:06:00 2010, comment #1:

Why does it requires auth support enabled?

pepeto <pepeto>
Project Member
Fri Oct 29 20:15:41 2010, original submission:
  • new command 'delegate'
  • needs auth/luasql

limit commands in delegation mode?

Matthias Pfafferodt <syntron>
Project MemberIn charge of this item.

 

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

Attach File(s):
   
   
Comment:
   

 

Digest:
   patch dependencies.

Digest:
   patch dependencies.

 

Carbon-Copy List
  • -unavailable- added by jtn (Posted a comment)
  • -unavailable- added by pepeto (Posted a comment)
  • -unavailable- added by syntron (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.

     

    Error: not logged in

     

     

    Follow 12 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri Jan 14 22:30:45 2011syntronStatusIn Progress=>Done
      Open/ClosedOpen=>Closed
    Fri Jan 14 12:11:28 2011syntronDependencies-=>patch #1494 is dependent
    Wed Jan 5 18:43:46 2011syntronDependencies-=>Depends on patch #2276
      Dependencies-=>Depends on patch #2275
      Dependencies-=>Depends on patch #2273
      Dependencies-=>Depends on patch #2272
    Wed Jan 5 18:36:55 2011syntronAttached File-=>Added 20110105-delegation02-add-delegation.patch, #11704
      Summaryadd delegation (especially for longturn)=>[delegation02] add delegation (especially for longturn)
    Thu Dec 23 12:44:07 2010syntronSummary[fcdb] add delegation (especially for longturn)=>add delegation (especially for longturn)
    Thu Dec 23 12:43:53 2010syntronAttached File-=>Added 20101223-add-delegation.patch, #11661
    Fri Oct 29 20:15:41 2010syntronAttached File-=>Added 20101029-06-trunk-add-delegation.patch, #10989
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup