--- deliantra/Deliantra-Client/DC/Protocol.pm 2007/07/13 15:44:48 1.118
+++ deliantra/Deliantra-Client/DC/Protocol.pm 2007/08/13 14:15:09 1.152
@@ -23,7 +23,7 @@
setup_req => {
extmap => 1,
excmd => 1,
- ywidget => 1,
+ ywidget => 1,#d#
%{$arg{setup_req} || {}},
},
);
@@ -50,57 +50,18 @@
} sort { $a->{par} <=> $b->{par} }
CFPlus::Pod::find command => "*";
- $self->connect_ext (event_music => sub {
- my ($ev) = @_;
-
- return unless $::CFG->{bgm_enable};
-
- my $faces = $ev->{faces};
- my @songs;
-
- # request music from server if appropriate
- if (my $faces = $ev->{faces}) {
- for (@$faces) {
- my ($face, $pri, $chksum) = @$_;
- utf8::downgrade $chksum;
-
- $chksum = unpack "H*", $chksum;
- $self->{music_map}{$face} = $chksum;
-
- # check if already cached
- CFPlus::DB::get "res-metadata" => $chksum, sub {
- my ($meta) = @_;
-
- if (defined $meta) {
- $self->{music_meta}{$chksum} = $self->{json_coder}->decode ($meta);
- } else {
- ::message ({ markup => "starting to download song #$face/$pri, check your output-rate setting if your connection gets laggy." });
- $self->ask_face ($face, -100 - $pri, undef, sub {
- my $len = length $_[1];
- my ($meta, $data) = unpack "(w/a*)*", $_[1];
-
- CFPlus::DB::write_file $chksum, $data, sub { };
- CFPlus::DB::put "res-metadata" => $chksum => $meta, sub { };
-
- $self->{music_meta}{$chksum} = $self->{json_coder}->decode ($meta);
- ::message ({ markup => "downloaded song #$face, size $len octets" });
-
- &::audio_music_set ($self->{songs});
- });
- }
- }
- }
- }
-
- if (my $play = $ev->{play}) {
- &::audio_music_set ($self->{songs} = [map $self->{music_map}{$_}, @$play]);
- }
+ $self->connect_ext (ambient_music => sub {
+ my ($songs) = @_;
+ &::audio_music_set_ambient ($songs);
});
- $self->connect_ext (event_capabilities => sub {
- my ($cap) = @_;
+ $self->connect_ext (capabilities => sub {
+ my (%cap) = @_;
+
+ #$self->send ("setup sound 0"); # we use a different protocol
+ $self->update_fx_want;
- if (my $ts = $cap->{tileset}) {
+ if (my $ts = $cap{tileset}) {
if (my ($default) = grep $_->[2] & 1, @$ts) {
$self->{tileset} = $default;
$self->{tilesize} = $default->[3];
@@ -116,37 +77,38 @@
$self->{json_coder}
->convert_blessed
- ->filter_json_single_key_object (__widget_ref__ => sub {
+ ->filter_json_single_key_object (__w_ => sub {
$self->{widget}{$_[0]}
});
+ # widgetset new
$self->connect_ext (ws_n => sub {
- my ($arg) = @_;
+ my ($id) = @_;
- $self->{widgetset}{$arg{id}} = {
+ $self->{widgetset}{$id} = {
w => {},
};
});
+ # widgetset destroy
$self->connect_ext (ws_d => sub {
- my ($arg) = @_;
+ my ($id) = @_;
- my $ws = delete $self->{widgetset}{$arg{id}}
+ my $ws = delete $self->{widgetset}{$id}
or return;
$_->destroy
for values %{$ws->{w}};
});
+ # widgetset create
$self->connect_ext (ws_c => sub {
- my ($arg) = @_;
-
- my $args = $arg->{args} || {};
+ my ($ws, $id, $class, $args) = @_;
for my $ev (grep /^on_/, keys %$args) {
$args->{$ev} = sub {
my $id = shift->{s_id};
- $self->send_exti_msg (w_e => id => $id, name => $ev, args => \@_);
+ $self->send_exti_msg (w_e => $id, $ev, \@_);
1
};
@@ -154,15 +116,15 @@
if (my $widget = eval {
local $SIG{__DIE__};
- "CFPlus::UI::$arg->{class}"->new (
+ "CFPlus::UI::$class"->new (
%$args,
- s_ws => $arg->{ws},
- s_id => $arg->{id},
+ s_ws => $ws,
+ s_id => $id,
)
}
) {
- $self->{widget}{$arg->{id}}
- = $self->{widgetset}{$arg->{ws}}{w}{$arg->{id}}
+ $self->{widget}{$id}
+ = $self->{widgetset}{$ws}{w}{$id}
= $widget;
$widget->connect (on_destroy => sub {
@@ -172,45 +134,63 @@
delete $self->{widgetset}{$widget->{s_ws}}{$widget->{s_id}};
});
} else {
- warn "server failed creating client-side widget " . (CFPlus::to_json $arg) . ": $@\n";
- $self->send_exti_msg (w_e => id => $arg->{id}, name => "destroy");
+ warn "server failed creating client-side widget " . (CFPlus::to_json $class) . ": $@\n";
+ $self->send_exti_msg (w_e => $id, "destroy");
}
});
+ # widget call
$self->connect_ext (w_c => sub {
- my ($arg) = @_;
+ my ($id, $rid, $method, $args) = @_;
- my $w = $self->{widget}{$arg->{id}}
+ my $w = $self->{widget}{$id}
or return;
- my $m = $arg->{name};
- my $a = $arg->{args} || [];
+ $args ||= [];
- if (exists $arg->{rid}) {
- $self->send_exti_msg (w_r => rid => $arg->{rid}, res => [$w->$m (@$a)]);
+ if ($rid) {
+ $self->send_exti_msg (w_r => $rid, [$w->$method (@$args)]);
} else {
- $w->$m (@$a);
+ $w->$method (@$args);
}
});
+ # widget set
$self->connect_ext (w_s => sub {
- my ($arg) = @_;
+ my ($id, $attr) = @_;
- my $w = $self->{widget}{$arg->{id}}
+ my $w = $self->{widget}{$id}
or return;
- $w->{$arg->{name}} = $arg->{value};
+ for (my $i = 0; $i < $#$attr; $i += 2) {
+ my ($member, $value) = @$attr[$i, $i+1];
+ if (defined $value) {
+ $w->{$member} = $value;
+ } else {
+ delete $w->{$member};
+ }
+ $w->{parent}->realloc if $member =~ /^c_/ && $w->{visible};
+ }
});
+ # widget get
$self->connect_ext (w_g => sub {
- my ($arg) = @_;
+ my ($id, $rid, $attr) = @_;
- my $w = $self->{widget}{$arg->{id}}
+ my $w = $self->{widget}{$id}
or return;
- $self->send_exti_msg (w_r => rid => $arg->{rid}, res => [$w->{$arg->{name}}]);
+ $self->send_exti_msg (w_r => $rid, [map $w->{$_}, @$attr]);
+ });
+
+ # message window
+ $self->connect_ext (channel_info => sub {
+ my ($info) = @_;
+ $self->{channels}->{$info->{id}} = $info;
+ $::MESSAGE_WINDOW->add_channel ($info);
});
+ # destroy widgets on logout
$self->{on_stop_game_guard} = $self->{map_widget}{root}->connect (stop_game => sub {
for my $ws (values %{delete $self->{widgetset} || {}}) {
$_->destroy
@@ -241,26 +221,19 @@
$self
}
-sub logprint {
- my ($self, @a) = @_;
-
- $self->{log_fh} ||= do {
- my $path = "$Crossfire::VARDIR/log.$self->{host}";
-
- open my $fh, ">>:utf8", $path
- or die "Couldn't open logfile $path: $!";
-
- $fh->autoflush (1);
-
- $fh;
- };
+sub update_fx_want {
+ my ($self) = @_;
- my ($sec, $min, $hour, $mday, $mon, $year) = localtime time;
+ $self->send_exti_msg (fx_want => {
+ 3 => !!$::CFG->{bgm_enable}, # FT_MUSIC
+ 5 => !!$::CFG->{audio_enable}, # FT_SOUND
+ });
+}
- my $ts = sprintf "%04d-%02d-%02d %02d:%02d:%02d",
- $year + 1900, $mon + 1, $mday, $hour, $min, $sec;
+sub logprint {
+ my ($self, @a) = @_;
- print {$self->{log_fh}} "$ts ", @a, "\n";
+ CFPlus::DB::logprint "$Crossfire::VARDIR/log.$self->{host}" => (join "", @a), sub { };
}
sub _stat_numdiff {
@@ -433,12 +406,16 @@
if (grep +(exists $stats->{$_}) != (exists $prev->{$_}), @skills) {
$sktbl->clear;
- $sktbl->add (0, 0, new CFPlus::UI::Label text => "Experience", align => 1);
- $sktbl->add (1, 0, new CFPlus::UI::Label text => "Lvl.", align => 1);
- $sktbl->add (2, 0, new CFPlus::UI::Label text => "Skill", expand => 1);
- $sktbl->add (3, 0, new CFPlus::UI::Label text => "Experience", align => 1);
- $sktbl->add (4, 0, new CFPlus::UI::Label text => "Lvl.", align => 1);
- $sktbl->add (5, 0, new CFPlus::UI::Label text => "Skill", expand => 1);
+ my @add;
+
+ push @add,
+ 0, 0, (new CFPlus::UI::Label text => "Experience", align => 1),
+ 1, 0, (new CFPlus::UI::Label text => "Lvl.", align => 1),
+ 2, 0, (new CFPlus::UI::Label text => "Skill", expand => 1),
+ 3, 0, (new CFPlus::UI::Label text => "Experience", align => 1),
+ 4, 0, (new CFPlus::UI::Label text => "Lvl.", align => 1),
+ 5, 0, (new CFPlus::UI::Label text => "Skill", expand => 1),
+ ;
my $TOOLTIP_ALL = "\n\nLeft click - ready skill\nMiddle click - use spell\nRight click - further options";
@@ -476,15 +453,19 @@
1
};
- $sktbl->add ($x * 3 + 0, $y, $self->{stat_widget_exp}{$idx} = new CFPlus::UI::Label
- text => "0", align => 1, font => $::FONT_FIXED, fg => [1, 1, 0], on_button_down => $spell_cb, @TOOLTIP_EXP);
- $sktbl->add ($x * 3 + 1, $y, $self->{stat_widget_lvl}{$idx} = new CFPlus::UI::Label
- text => "0", align => 1, font => $::FONT_FIXED, fg => [0, 1, 0], padding_x => 4, on_button_down => $spell_cb, @TOOLTIP_LVL);
- $sktbl->add ($x * 3 + 2, $y, new CFPlus::UI::Label text => $name, on_button_down => $spell_cb,
- can_events => 1, can_hover => 1, tooltip => (CFPlus::Pod::section_label skill_description => $name) . $TOOLTIP_ALL);
+ push @add,
+ $x * 3 + 0, $y, ($self->{stat_widget_exp}{$idx} = new CFPlus::UI::Label
+ align => 1, font => $::FONT_FIXED, fg => [1, 1, 0], on_button_down => $spell_cb, @TOOLTIP_EXP),
+ $x * 3 + 1, $y, ($self->{stat_widget_lvl}{$idx} = new CFPlus::UI::Label
+ text => "0", align => 1, font => $::FONT_FIXED, fg => [0, 1, 0], padding_x => 4, on_button_down => $spell_cb, @TOOLTIP_LVL),
+ $x * 3 + 2, $y, (new CFPlus::UI::Label text => $name, on_button_down => $spell_cb,
+ can_events => 1, can_hover => 1, tooltip => (CFPlus::Pod::section_label skill_description => $name) . $TOOLTIP_ALL),
+ ;
$x++ and ($x, $y) = (0, $y + 1);
}
+
+ $sktbl->add_at (@add);
}
for (grep exists $stats->{$_}, @skills) {
@@ -527,8 +508,36 @@
sub feed_map1a {
my ($self, $data) = @_;
- $self->{map}->map1a_update ($data, $self->{setup}{extmap});
- $self->{map_widget}->update;
+ my $missing = $self->{map}->map1a_update ($data, $self->{setup}{extmap});
+ my $delay;
+
+ for my $tile (@$missing) {
+ next if $self->{delay}{$tile};
+
+ $delay = 1;
+
+ if (my $tex = $::CONN->{texture}[$tile]) {
+ $tex->upload;
+ } else {
+ $self->{delay}{$tile} = 1;
+
+ # we assume the face is in-flight and will eventually come
+ push @{$self->{face_cb}{$tile}}, sub {
+ delete $self->{delay}{$tile};
+ $_[0]->upload;
+ };
+ }
+ }
+
+ if ($delay) {
+ # delay the map drawing a tiny bit in the hope of getting the missing fetched
+ Event->timer (after => 0.03, cb => sub {
+ $_[0]->w->cancel;
+ $self->{map_widget}->update;
+ });
+ } else {
+ $self->{map_widget}->update;
+ }
}
sub magicmap {
@@ -570,13 +579,15 @@
or return;
} while $self->{texture}[$id];
- CFPlus::DB::get tilecache => $id, sub {
- my ($data) = @_;
+ CFPlus::DB::exists tilecache => $id, sub {
+ my ($exists) = @_;
return unless $self->{map}; # stop when destroyed
- $self->set_texture ($id => $data)
- if defined $data;
+ if ($exists) {
+ $self->have_tile ($id);
+ $self->{texture}[$id]->upload;
+ }
$self->bg_fetch;
};
@@ -587,20 +598,27 @@
my $gen = $self->{map_change_gen};
- CFPlus::DB::get $self->{mapcache} => $hash, sub {
+ my $cb = sub {
return unless $gen == $self->{map_change_gen};
my ($data) = @_;
if (defined $data) {
- $data = Compress::LZF::decompress $data;
- #warn sprintf "LOADmap[%s,%d,%d] length %d\n", $hash, $x, $y, length $data;#d#
+ $self->{map_cache_new}{$hash} = \$data;
+
+ my $data = Compress::LZF::decompress $data;
my $inprogress = @{ $self->{bg_fetch} || [] };
unshift @{ $self->{bg_fetch} }, $self->{map}->set_rect ($x, $y, $data);
$self->bg_fetch unless $inprogress;
}
};
+
+ if (my $rdata = $self->{map_cache_old}{$hash}) {
+ $cb->($$rdata);
+ } else {
+ CFPlus::DB::get $self->{mapcache} => $hash, $cb;
+ }
}
# hardcode /world/world_xxx_xxx map names, the savings are enourmous,
@@ -703,6 +721,7 @@
$self->flush_map;
++$self->{map_change_gen};
+ $self->{map_cache_old} = delete $self->{map_cache_new};
my ($ox, $oy) = ($::MAP->ox, $::MAP->oy);
@@ -731,25 +750,60 @@
sub face_find {
my ($self, $facenum, $face, $cb) = @_;
- my $hash = "$face->{chksum},$face->{name}";
+ if ($face->{type} == 0) { # FT_FACE
+ my $id = CFPlus::DB::get_tile_id_sync $face->{name};
- my $id = CFPlus::DB::get_tile_id_sync $hash;
+ $face->{id} = $id;
+ $self->{faceid}[$facenum] = $id;
- $face->{id} = $id;
- $self->{faceid}[$facenum] = $id;
+ $self->{map}->set_tileid ($facenum => $id);
- $self->{map}->set_tileid ($facenum => $id);
+ CFPlus::DB::get tilecache => $id, $cb;
- CFPlus::DB::get tilecache => $id, $cb;
+ } elsif ($face->{type} & 1) { # with metadata
+ CFPlus::DB::get res_meta => $face->{name}, $cb;
+ }
}
sub face_update {
my ($self, $facenum, $face, $changed) = @_;
- CFPlus::DB::put tilecache => $face->{id} => $face->{image}, sub { }
- if $changed;
+ if ($face->{type} == 0) { # FT_FACE
+ CFPlus::DB::put tilecache => $face->{id} => $face->{data}, sub { }
+ if $changed;
+
+ $self->have_tile ($face->{id});
+
+ } elsif ($face->{type} & 1) { # split metadata
+ if ($changed) { # new data
+ my ($meta_json, $data) = unpack "(w/a*)*", delete $face->{data};
+ $face->{meta} = $self->{json_coder}->decode ($meta_json);
- $self->set_texture ($face->{id} => delete $face->{image});
+ CFPlus::DB::put res_data => $face->{name} => $data, sub { };
+ CFPlus::DB::put res_meta => $face->{name} => $meta_json, sub { };
+ } else {
+ $face->{meta} = $self->{json_coder}->decode (delete $face->{data});
+ }
+
+ ::add_license ($face);
+
+ if ($face->{type} == 3) { # FT_MUSIC
+ $self->{music_meta}{$facenum} = $face;
+
+ ::message ({ markup => "downloaded song #$facenum" })
+ if $changed;
+
+ &::audio_music_push ($facenum);
+ } elsif ($face->{type} == 5) { # FT_SOUND
+ $self->{sound_meta}{$facenum} = $face;
+
+ ::message ({ markup => "downloaded sound #$facenum" })
+ if $changed;
+
+ &::audio_sound_push ($facenum);
+ }
+
+ }
}
sub smooth_update {
@@ -758,23 +812,35 @@
$self->{map}->set_smooth ($facenum, $face->{smoothface}, $face->{smoothlevel});
}
-sub set_texture {
- my ($self, $id, $data) = @_;
+sub have_tile {
+ my ($self, $tile) = @_;
return unless $self->{map};
- $self->{texture}[$id] = my $tex =
- new_from_image CFPlus::Texture
- $data, minify => 1, mipmap => 1;
+ my $tex = $self->{texture}[$tile] ||=
+ new CFPlus::Texture
+ tile => $tile, minify => 1, mipmap => 1, delay => 1;
+
+ $_->($tex) for @{(delete $self->{face_cb}{$tile}) || []};
+}
+
+sub connect_face_update {
+ my ($self, $id, $cb) = @_;
+
+ push @{$self->{face_cb}{$id}}, $cb;
- $self->{map}->set_texture ($id, @$tex{qw(name w h s t)}, @{$tex->{minified}});
- $self->{map_widget}->update;
+ CFPlus::guard {
+ @{$self->{face_cb}{$id}}
+ = grep $_ != $cb,
+ @{$self->{face_cb}{$id}};
+ }
}
sub sound_play {
- my ($self, $x, $y, $soundnum, $type) = @_;
+ my ($self, $type, $face, $dx, $dy, $vol) = @_;
- $self->{sound_play}->($x, $y, $soundnum, $type);
+ &::audio_sound_play ($face, $dx, $dy, $vol)
+ unless $type & 1; # odd types are silent for future expansion
}
my $LAST_QUERY; # server is stupid, stupid, stupid
@@ -849,13 +915,13 @@
$self->logprint ("msg: ", $text);
return if $color < 0; # negative color == ignore if not understood
- my $fg = $CF_COLOR[$color % @CF_COLOR];
+ my $fg = $CF_COLOR[$color & 0x1f] || [1, 0, 0];
## try to create single paragraphs of multiple lines sent by the server
# no longer neecssary with TRT servers
#$text =~ s/(?<=\S)\n(?=\w)/ /g;
- ::message ({ fg => $fg, markup => $_ })
+ ::message ({ fg => $fg, markup => $_, type => $type, extra => [@extra] })
for split /\n/, $text;
$self->{statusbox}->add ($text,
@@ -922,46 +988,14 @@
::stop_game ();
}
-sub image_info {
- my ($self, $numfaces) = @_;
-
- $self->{num_faces} = $numfaces;
- $self->{face_prefetch} = [1 .. $numfaces];
- $self->face_prefetch;
-}
-
-sub face_prefetch {
- my ($self) = @_;
-
- return unless $::CFG->{face_prefetch};
-
- if ($self->{num_faces}) {
- return if @{ $self->{send_queue} || [] };
- my $todo = @{ $self->{face_prefetch} }
- or return;
-
- my ($face) = splice @{ $self->{face_prefetch} }, + rand @{ $self->{face_prefetch} }, 1, ();
-
- $self->send ("requestinfo image_sums $face $face");
-
- $self->{statusbox}->add (CFPlus::asxml "prefetching $todo",
- group => "prefetch", timeout => 3, fg => [1, 1, 0, 0.5]);
- } elsif (!exists $self->{num_faces}) {
- $self->send ("requestinfo image_info");
-
- $self->{num_faces} = 0;
-
- $self->{statusbox}->add (CFPlus::asxml "starting to prefetch",
- group => "prefetch", timeout => 3, fg => [1, 1, 0, 0.5]);
- }
-}
-
sub update_floorbox {
$CFPlus::UI::ROOT->on_refresh ($::FLOORBOX => sub {
return unless $::CONN;
$::FLOORBOX->clear;
+ my @add;
+
my $row;
for (sort { $a->{count} <=> $b->{count} } values %{ $::CONN->{container}{$::CONN->{open_container} || 0} }) {
if ($row < 6) {
@@ -969,27 +1003,30 @@
local $_->{desc_widget}; # hack to force recreation of widget
CFPlus::Item::update_widgets $_;
- $::FLOORBOX->add (0, $row, $_->{face_widget});
- $::FLOORBOX->add (1, $row, $_->{desc_widget});
+ push @add,
+ 0, $row, $_->{face_widget},
+ 1, $row, $_->{desc_widget};
$row++;
} else {
- $::FLOORBOX->add (1, $row, new CFPlus::UI::Button
+ push @add, 1, $row, new CFPlus::UI::Button
text => "More...",
on_activate => sub { ::toggle_player_page ($::INVENTORY_PAGE); 0 },
- );
+ ;
last;
}
}
if ($::CONN->{open_container}) {
- $::FLOORBOX->add (1, $row++, new CFPlus::UI::Button
+ push @add, 1, $row++, new CFPlus::UI::Button
text => "Close container",
on_activate => sub { $::CONN->send ("apply $::CONN->{open_container}") }
- );
+ ;
}
+
+ $::FLOORBOX->add_at (@add);
});
- $::WANT_REFRESH++;
+ $::WANT_REFRESH->start;
}
sub set_opencont {
@@ -1116,19 +1153,20 @@
. "map size $self->{mapw}×$self->{maph}\n"
);
- ::setup_build_button ($self->{editor_support}->{builder_ui});
}
sub logged_in {
my ($self) = @_;
- $self->send_ext_req (cfplus_support => version => 1, sub {
- $self->{cfplus_ext} = $_[0]{version};
+ $self->send_ext_req (cfplus_support => version => 2, sub {
+ my (%msg) = @_;
+
+ $self->{cfplus_ext} = $msg{version};
$self->update_server_info;
if ($self->{cfplus_ext} >= 2) {
$self->send_ext_req ("editor_support", sub {
- $self->{editor_support} = $_[0];
+ $self->{editor_support} = { @_ };
$self->update_server_info;
0
@@ -1146,27 +1184,18 @@
$self->send_command ("pickup $::CFG->{pickup}");
}
-sub buildat {
- my ($self, $builditem, $x, $y) = @_;
-
- if ($self->{cfplus_ext}) {
- $self->send_ext_msg (builder_build => dx => $x, dy => $y, (ref ($builditem) eq 'HASH') ? %$builditem : (item => $builditem));
- }
-}
-
sub lookat {
my ($self, $x, $y) = @_;
if ($self->{cfplus_ext}) {
- $self->send_ext_req (lookat => dx => $x, dy => $y, sub {
- my ($msg) = @_;
+ $self->send_ext_req (lookat => $x, $y, sub {
+ my (%msg) = @_;
- if (exists $msg->{npc_dialog}) {
+ if (exists $msg{npc_dialog}) {
# start npc chat dialog
$self->{npc_dialog} = new CFPlus::NPCDialog::
- dx => $x,
- dy => $y,
- title => "$msg->{npc_dialog} (NPC)",
+ token => $msg{npc_dialog},
+ title => "$msg{npc_dialog}[0] (NPC)",
conn => $self,
;
}
@@ -1241,10 +1270,12 @@
$self->update_options;
- $self->{id} = $self->{conn}->send_ext_req (
- npc_dialog_begin => dx => $self->{dx}, dy => $self->{dy},
- sub { $this && $this->feed (@_) }
- );
+ $self->{id} = "npc-channel-" . $self->{conn}->token;
+ $self->{conn}->connect_ext ($self->{id} => sub {
+ $this->feed (@_) if $this;
+ });
+
+ $self->{conn}->send_ext_msg (npc_dialog_begin => $self->{id}, $self->{token});
$self->{entry}->grab_focus;
@@ -1277,37 +1308,42 @@
}
sub feed {
- my ($self, $msg) = @_;
+ my ($self, $type, @arg) = @_;
CFPlus::weaken $self;
- if ($msg->{msgtype} eq "reply") {
- $self->{kw}{$_} = 1 for @{$msg->{add_topics} || []};
- $self->{kw}{$_} = 0 for @{$msg->{del_topics} || []};
-
- my $text = "\n" . CFPlus::Protocol::sanitise_xml $msg->{msg};
- my $match = join "|", map "\\b\Q$_\E\\b", sort { (length $b) <=> (length $a) } keys %{ $self->{kw} };
- my @link;
- $text =~ s{
- ($match)
- }{
- my $kw = $1;
-
- push @link, new CFPlus::UI::Label
- markup => "$kw",
- can_hover => 1,
- can_events => 1,
- padding_x => 0,
- padding_y => 0,
- on_button_up => sub {
- $self->send ($kw);
- };
+ if ($type eq "update") {
+ my (%info) = @arg;
- "\x{fffc}"
- }giex;
+ $self->{kw}{$_} = 1 for @{$info{add_topics} || []};
+ $self->{kw}{$_} = 0 for @{$info{del_topics} || []};
- $self->{textview}->add_paragraph ({ markup => $text, widget => \@link });
- $self->{textview}->scroll_to_bottom;
+ if (exists $info{msg}) {
+ my $text = "\n" . CFPlus::Protocol::sanitise_xml $info{msg};
+ my $match = join "|", map "\\b\Q$_\E\\b", sort { (length $b) <=> (length $a) } keys %{ $self->{kw} };
+ my @link;
+ $text =~ s{
+ ($match)
+ }{
+ my $kw = $1;
+
+ push @link, new CFPlus::UI::Label
+ markup => "$kw",
+ can_hover => 1,
+ can_events => 1,
+ padding_x => 0,
+ padding_y => 0,
+ on_button_up => sub {
+ $self->send ($kw);
+ };
+
+ "\x{fffc}"
+ }giex;
+
+ $self->{textview}->add_paragraph ({ markup => $text, widget => \@link });
+ $self->{textview}->scroll_to_bottom;
+ }
+
$self->update_options;
} else {
$self->destroy;
@@ -1322,7 +1358,7 @@
$self->{textview}->add_paragraph ({ markup => "\n" . CFPlus::asxml $msg });
$self->{textview}->scroll_to_bottom;
- $self->{conn}->send_ext_msg (npc_dialog_tell => msgid => $self->{id}, msg => $msg);
+ $self->{conn}->send_ext_msg (npc_dialog_tell => $self->{id}, $msg);
}
sub destroy {
@@ -1331,7 +1367,7 @@
#Carp::cluck "debug\n";#d# #todo# enable: destroy gets called twice because scalar keys {} is 1
if ($self->{conn}) {
- $self->{conn}->send_ext_msg (npc_dialog_end => msgid => $self->{id}) if $self->{id};
+ $self->{conn}->send_ext_msg (npc_dialog_end => $self->{id}) if $self->{id};
delete $self->{conn}{npc_dialog};
$self->{conn}->disconnect_ext ($self->{id});
}