Project

General

Profile

Actions

Feature #1132

closed

warden-client: součástí ID souboru by měl být i požadovaný typ

Added by Pavel Kácha over 11 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
07/31/2013
Due date:
% Done:

0%

Estimated time:

Description

Bylo by vhodné ukládat ID pro každý tahaný typ zpráv, což by se mohlo hodit klientům, kteří jsou zaregistrování pro tahání více typů nebo všech typů. V současné době stáhnu-li jeden typ a začnu-li tahat jiný typ, už tahám od nového ID - což není právě intuitivní.

Actions #1

Updated by Pavel Kácha over 11 years ago

Vzniklo ze:

Bodík <> ve :

myslite ze by bylo mozne prijmout tento patch ?
https://gist.github.com/bodik/5b0b47b45466851ecfbb#file-gistfile1-txt-L96

Bodík <> ve :

nevim jak vy ale v soucasne chvili kdyz chci tahat 4 typy zprav musim mit 4 separatni skripty. na zaklade jmena skriptu totiz warden-client generuje jmeno souboru s ID ktery se pouziva pro naslede dotazy getNewEvents

ja navrhuju aby soucasti jmena byl i typ zpravy, coz by umoznilo v jednom skriptu tahat vsechny, nebo jenom jejich vycet pokud by mi nevyhovovalo all ...

Actions #2

Updated by Pavel Kácha over 11 years ago

Bodíkův patch

warden client 2.1 id file patch
----------------------------------------
diff --git a/WardenClientReceive.pm b/WardenClientReceive.pm.orig
index 7e1d24f..dc52f62 100755
--- a/WardenClientReceive.pm
+++ b/WardenClientReceive.pm.orig
@@ -42,7 +42,7 @@ sub getNewEvents

     # set name of ID file for each client aplication 
     my $caller_name = $FindBin::Script;
-    my $id_file = $vardir . $caller_name . "-". $requested_type .".id";
+    my $id_file = $vardir . $caller_name . ".id";

     #-----------------------------------------------------------------------------
     # get last ID from ID file (if exist) or

Actions #3

Updated by Pavel Kácha over 11 years ago

  • Status changed from New to Resolved

Patch ošetřuje i undef v $requested_type (který serveru říká, že klient chce vše) a doplňuje text do dokumentace: 339fbb23

Bude vhodné zdůraznit v informačním mailu o nové verzi.

Actions #4

Updated by Pavel Kácha over 11 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF