Bug #571

AP+ PS->IRC bridge doesn't work with timestamp update

Added by Fuco about 10 years ago. Updated about 10 years ago.

Status:ClosedStart date:2009-09-11
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:-
Target version:-

Description

Fix:
in autopilot.tcl (http://trac.openttdcoop.org/browser/tools/autopilot/branches/ap%2B/autopilot.tcl), add this code after line 225
---
regexp nocase {^(\[\d\d\d\d\d\d-\d\d \d\d:\d\d:\d\d\])\s+(.*)} $linestr line timestamp linestr
---

if the timestamp was provided by server, variable $timestamp will contain it. To check if it exist, you can use:
if [info exists timestamp] { do something with it }

The fix was tested on ActiveState ActiveTcl 8.5.5.0.287690/win64. It's just a one line fix so it shouldn't be hard to revert if it doesn't work.

Email if anything goes wrong...

timestamp.diff Magnifier (590 Bytes) Ammler, 2009-09-11 08:59

Associated revisions

Revision 69:a5cf176f1bf6
Added by Ammler about 10 years ago

(svn r740) Feature: Support for timestamps in console. (Fuco, closes #571)

History

#1 Updated by Fuco about 10 years ago

Update1: the website engine somehow removed dash (-) from the nocase switch. You have to add that as it obviously wont work. I don't know how to edit the original report :(

#2 Updated by Ammler about 10 years ago

I prefer patches, so I made the following one, will try it on ps now...

#3 Updated by Ammler about 10 years ago

  • Assignee deleted (Ammler)
  • % Done changed from 100 to 70

The patch works so far, very thanks. But there are some other things which need fixing, either. Like rcon and joining callback...

#4 Updated by Ammler about 10 years ago

  • Status changed from New to Closed
  • % Done changed from 70 to 100

Applied in changeset r740.

Also available in: Atom PDF