--- deliantra/Deliantra-Client/DC/Protocol.pm 2006/06/19 06:50:45 1.39 +++ deliantra/Deliantra-Client/DC/Protocol.pm 2007/07/12 17:56:51 1.113 @@ -1,124 +1,514 @@ -package CFClient::Protocol; +package CFPlus::Protocol; use utf8; use strict; use Crossfire::Protocol::Constants; -use CFClient; -use CFClient::UI; +use CFPlus; +use CFPlus::DB; +use CFPlus::UI; +use CFPlus::Pod; +use CFPlus::Macro; +use CFPlus::Item; -use base 'Crossfire::Protocol::Base'; +use Crossfire::Protocol::Base 0.95; -our %open_logs; +use base 'Crossfire::Protocol::Base'; sub new { - my $class = shift; + my ($class, %arg) = @_; - my $self = $class->SUPER::new (@_); + my $self = $class->SUPER::new (%arg, + setup_req => { + extmap => 1, + excmd => 1, + ywidget => 1, + %{$arg{setup_req} || {}}, + }, + ); $self->{map_widget}->clr_commands; - my $cmd_help = CFClient::load_pod CFClient::find_rcfile "pod/command_help.pod", command_help => 1, sub { - my ($pom) = @_; + my @cmd_help = map { + $_->{kw}[0] =~ /^(\S+) (?:\s+ \( ([^\)]*) \) )?/x + or die "unparseable command help: $_->{kw}[0]"; + + my $cmd = $1; + my @args = split /\|/, $2; + @args = (".*") unless @args; + + my (undef, @par) = CFPlus::Pod::section_of $_; + my $text = CFPlus::Pod::as_label @par; + + $_ = $_ eq ".*" ? "" : " $_" + for @args; + + map ["$cmd$_", $text], + sort { (length $a) <=> (length $b) } + @args + } 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 + my $pri = -100; + for my $face (@$faces) { + if (defined (my $chksum = $ev->{chksum}{$face})) { + utf8::downgrade $chksum; + + $chksum = unpack "H*", $chksum; + $self->{music_map}{$face} = $chksum; - my @cmd_help; + ::message ({ markup => "starting to download song #$face, check your output-rate setting if your connection gets laggy." }); + $self->ask_face ($face, $pri, undef, sub { + my $num = $_[0]; + my $len = length $_[1]; + my ($meta, $data) = unpack "(w/a*)*", $_[1]; - for my $head2 ($pom->head1->[-2]->head2) { - $head2->title =~ /^(\S+) (?:\s+ \( ([^\)]*) \) )?/x - or next; + CFPlus::DB::write_file $chksum, $data, sub { }; + CFPlus::DB::put resmap => $chksum => $meta, sub { }; - my $cmd = $1; - my @args = split /\|/, $2; - @args = (".*") unless @args; + $self->{music_meta}{$chksum} = $self->{json_coder}->decode ($meta); + ::message ({ markup => "downloaded song #$face, size $len octets" }); - $_ = $_ eq ".*" ? "" : " $_" - for @args; + &::audio_music_set ($self->{songs}); + }); + } + + push @songs, $self->{music_map}{$face}; + --$pri; + } + + &::audio_music_set ($self->{songs} = \@songs); + }); + + $self->connect_ext (event_capabilities => sub { + my ($cap) = @_; + + if (my $ts = $cap->{tileset}) { + if (my ($default) = grep $_->[2] & 1, @$ts) { + $self->{tileset} = $default; + $self->{tilesize} = $default->[3]; + $self->setup_req (tileset => $default->[0]); - my $text = CFClient::pod_to_pango $head2->content; + my $w = int $self->{mapw} * 32 / $self->{tilesize}; + my $h = int $self->{maph} * 32 / $self->{tilesize}; - push @cmd_help, ["$cmd$_", $text] - for sort { (length $a) <=> (length $b) } - @args; + $self->setup_req (mapsize => "${w}x${h}"); + } } + }); - \@cmd_help - }; + $self->{json_coder} + ->convert_blessed + ->filter_json_single_key_object (__widget_ref__ => sub { + $self->{widget}{$_[0]} + }); + + $self->connect_ext (ws_n => sub { + my ($arg) = @_; + + $self->{widgetset}{$arg{id}} = { + w => {}, + }; + }); + + $self->connect_ext (ws_d => sub { + my ($arg) = @_; + + my $ws = delete $self->{widgetset}{$arg{id}} + or return; + + $_->destroy + for values %{$ws->{w}}; + }); + + $self->connect_ext (ws_c => sub { + my ($arg) = @_; + + my $args = $arg->{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 => \@_); + + 1 + }; + } + + if (my $widget = eval { + local $SIG{__DIE__}; + "CFPlus::UI::$arg->{class}"->new ( + %$args, + s_ws => $arg->{ws}, + s_id => $arg->{id}, + ) + } + ) { + $self->{widget}{$arg->{id}} + = $self->{widgetset}{$arg->{ws}}{w}{$arg->{id}} + = $widget; + + $widget->connect (on_destroy => sub { + my ($widget) = @_; + + delete $self->{widget}{$widget->{s_id}}; + 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"); + } + }); + + $self->connect_ext (w_c => sub { + my ($arg) = @_; + + my $w = $self->{widget}{$arg->{id}} + or return; + my $m = $arg->{name}; + + my $a = $arg->{args} || []; + + if (exists $arg->{rid}) { + $self->send_exti_msg (w_r => rid => $arg->{rid}, res => [$w->$m (@$a)]); + } else { + $w->$m (@$a); + } + }); + + $self->connect_ext (w_s => sub { + my ($arg) = @_; + + my $w = $self->{widget}{$arg->{id}} + or return; + + $w->{$arg->{name}} = $arg->{value}; + }); + + $self->connect_ext (w_g => sub { + my ($arg) = @_; + + my $w = $self->{widget}{$arg->{id}} + or return; + + $self->send_exti_msg (w_r => rid => $arg->{rid}, res => [$w->{$arg->{name}}]); + }); + + $self->{on_stop_game_guard} = $self->{map_widget}{root}->connect (stop_game => sub { + for my $ws (values %{delete $self->{widgetset} || {}}) { + $_->destroy + for values %{delete $ws->{w} || {}}; + } + }); $self->{map_widget}->add_command (@$_) - for @$cmd_help; + for @cmd_help; - $self->{noface} = new_from_file CFClient::Texture - CFClient::find_rcfile "noface.png", minify => 1, mipmap => 1; + { + $self->{dialogue} = my $tex = new_from_file CFPlus::Texture + CFPlus::find_rcfile "dialogue.png", minify => 1, mipmap => 1; + $self->{map}->set_texture (1, @$tex{qw(name w h s t)}, @{$tex->{minified}}); + } - $self->{open_container} = 0; + { + $self->{noface} = my $tex = new_from_file CFPlus::Texture + CFPlus::find_rcfile "noface.png", minify => 1, mipmap => 1; + $self->{map}->set_texture (2, @$tex{qw(name w h s t)}, @{$tex->{minified}}); + } - # "global" - $self->{tilecache} = CFClient::db_table "tilecache"; - $self->{facemap} = CFClient::db_table "facemap"; + $self->{open_container} = 0; # per server - $self->{mapcache} = CFClient::db_table "mapcache_$self->{host}_$self->{port}"; + $self->{mapcache} = "mapcache_$self->{host}_$self->{port}"; $self } sub logprint { my ($self, @a) = @_; - my $filename = "$Crossfire::VARDIR/log.$self->{host}"; - my $fh = $open_logs{$filename}; - unless ($fh) { - # FIXME: handle this more gracefully? - open $fh, ">>", $filename - or die "Couldn't open logfile: log.$self->{host}: $!"; + $self->{log_fh} ||= do { + my $path = "$Crossfire::VARDIR/log.$self->{host}"; - $open_logs{$filename} = $fh; - } + open my $fh, ">>:utf8", $path + or die "Couldn't open logfile $path: $!"; - my ($sec, $min, $hour, $mday, $mon, $year) = localtime (time); + $fh->autoflush (1); + + $fh; + }; + + my ($sec, $min, $hour, $mday, $mon, $year) = localtime time; my $ts = sprintf "%04d-%02d-%02d %02d:%02d:%02d", $year + 1900, $mon + 1, $mday, $hour, $min, $sec; - print $fh "$ts ", @a, "\n"; - $fh->flush; + print {$self->{log_fh}} "$ts ", @a, "\n"; +} + +sub _stat_numdiff { + my ($self, $name, $old, $new) = @_; + + my $diff = $new - $old; + + $diff = 0.01 * int $diff * 100; + + 0.1 >= abs $diff ? () + : $diff < 0 ? "$name$diff" : "$name+$diff" } +sub _stat_skillmaskdiff { + my ($self, $name, $old, $new) = @_; + + my $diff = $old ^ $new + or return; + + my @diff = map + { + $diff & $_ + ? (($new & $_ ? "+" : "-") . $self->{spell_paths}{$_}) + : () + } + sort { $a <=> $b } keys %{$self->{spell_paths}}; + + join "", @diff +} + +# all stats that are chacked against changes +my @statchange = ( + [&CS_STAT_STR => \&_stat_numdiff, "Str"], + [&CS_STAT_INT => \&_stat_numdiff, "Int"], + [&CS_STAT_WIS => \&_stat_numdiff, "Wis"], + [&CS_STAT_DEX => \&_stat_numdiff, "Dex"], + [&CS_STAT_CON => \&_stat_numdiff, "Con"], + [&CS_STAT_CHA => \&_stat_numdiff, "Cha"], + [&CS_STAT_POW => \&_stat_numdiff, "Pow"], + [&CS_STAT_WC => \&_stat_numdiff, "Wc"], + [&CS_STAT_AC => \&_stat_numdiff, "Ac"], + [&CS_STAT_DAM => \&_stat_numdiff, "Dam"], + [&CS_STAT_SPEED => \&_stat_numdiff, "Speed"], + [&CS_STAT_WEAP_SP => \&_stat_numdiff, "WSp"], + [&CS_STAT_MAXHP => \&_stat_numdiff, "HP"], + [&CS_STAT_MAXSP => \&_stat_numdiff, "Mana"], + [&CS_STAT_MAXGRACE => \&_stat_numdiff, "Grace"], + [&CS_STAT_WEIGHT_LIM => \&_stat_numdiff, "Weight"], + [&CS_STAT_SPELL_ATTUNE => \&_stat_skillmaskdiff, "attuned"], + [&CS_STAT_SPELL_REPEL => \&_stat_skillmaskdiff, "repelled"], + [&CS_STAT_SPELL_DENY => \&_stat_skillmaskdiff, "denied"], + [&CS_STAT_RES_PHYS => \&_stat_numdiff, "phys"], + [&CS_STAT_RES_MAG => \&_stat_numdiff, "magic"], + [&CS_STAT_RES_FIRE => \&_stat_numdiff, "fire"], + [&CS_STAT_RES_ELEC => \&_stat_numdiff, "electricity"], + [&CS_STAT_RES_COLD => \&_stat_numdiff, "cold"], + [&CS_STAT_RES_CONF => \&_stat_numdiff, "confusion"], + [&CS_STAT_RES_ACID => \&_stat_numdiff, "acid"], + [&CS_STAT_RES_DRAIN => \&_stat_numdiff, "drain"], + [&CS_STAT_RES_GHOSTHIT => \&_stat_numdiff, "ghosthit"], + [&CS_STAT_RES_POISON => \&_stat_numdiff, "poison"], + [&CS_STAT_RES_SLOW => \&_stat_numdiff, "slow"], + [&CS_STAT_RES_PARA => \&_stat_numdiff, "paralyse"], + [&CS_STAT_TURN_UNDEAD => \&_stat_numdiff, "turnundead"], + [&CS_STAT_RES_FEAR => \&_stat_numdiff, "fear"], + [&CS_STAT_RES_DEPLETE => \&_stat_numdiff, "depletion"], + [&CS_STAT_RES_DEATH => \&_stat_numdiff, "death"], + [&CS_STAT_RES_HOLYWORD => \&_stat_numdiff, "godpower"], + [&CS_STAT_RES_BLIND => \&_stat_numdiff, "blind"], +); + sub stats_update { my ($self, $stats) = @_; - if (my $exp = $stats->{+CS_STAT_EXP64}) { - my $diff = $exp - $self->{prev_exp}; - $self->{statusbox}->add ("$diff experience gained", group => "experience $diff", fg => [0.5, 1, 0.5, 0.8], timeout => 5) - if exists $self->{prev_exp} && $diff; - $self->{prev_exp} = $exp; + my $prev = $self->{prev_stats} || { }; + + if (my @diffs = + ( + ($stats->{+CS_STAT_EXP64} > $prev->{+CS_STAT_EXP64} ? ($stats->{+CS_STAT_EXP64} - $prev->{+CS_STAT_EXP64}) . " experience gained" : ()), + map { + $stats->{$_} && $prev->{$_} + && $stats->{$_}[1] > $prev->{$_}[1] ? "($self->{skill_info}{$_}+" . ($stats->{$_}[1] - $prev->{$_}[1]) . ")" : () + } sort { $a <=> $b } keys %{$self->{skill_info}} + ) + ) { + my $msg = join " ", @diffs; + $self->{statusbox}->add ($msg, group => "experience $msg", fg => [0.5, 1, 0.5, 0.8], timeout => 5); } - ::update_stats_window ($stats); + if ( + my @diffs = map $_->[1]->($self, $_->[2], $prev->{$_->[0]}, $stats->{$_->[0]}), @statchange + ) { + my $msg = "stat change: " . (join " ", @diffs); + $self->{statusbox}->add ($msg, group => "stat $msg", fg => [0.8, 1, 0.2, 1], timeout => 10); + } + + $self->update_stats_window ($stats, $prev); + + $self->{prev_stats} = { %$stats }; +} + +my %RES_TBL = ( + phys => CS_STAT_RES_PHYS, + magic => CS_STAT_RES_MAG, + fire => CS_STAT_RES_FIRE, + elec => CS_STAT_RES_ELEC, + cold => CS_STAT_RES_COLD, + conf => CS_STAT_RES_CONF, + acid => CS_STAT_RES_ACID, + drain => CS_STAT_RES_DRAIN, + ghit => CS_STAT_RES_GHOSTHIT, + pois => CS_STAT_RES_POISON, + slow => CS_STAT_RES_SLOW, + para => CS_STAT_RES_PARA, + tund => CS_STAT_TURN_UNDEAD, + fear => CS_STAT_RES_FEAR, + depl => CS_STAT_RES_DEPLETE, + deat => CS_STAT_RES_DEATH, + holyw => CS_STAT_RES_HOLYWORD, + blind => CS_STAT_RES_BLIND, +); + +sub update_stats_window { + my ($self, $stats, $prev) = @_; + + # I love text protocols... + + my $hp = $stats->{+CS_STAT_HP} * 1; + my $hp_m = $stats->{+CS_STAT_MAXHP} * 1; + my $sp = $stats->{+CS_STAT_SP} * 1; + my $sp_m = $stats->{+CS_STAT_MAXSP} * 1; + my $fo = $stats->{+CS_STAT_FOOD} * 1; + my $fo_m = 999; + my $gr = $stats->{+CS_STAT_GRACE} * 1; + my $gr_m = $stats->{+CS_STAT_MAXGRACE} * 1; + + $::GAUGES->{hp} ->set_value ($hp, $hp_m); + $::GAUGES->{mana} ->set_value ($sp, $sp_m); + $::GAUGES->{food} ->set_value ($fo, $fo_m); + $::GAUGES->{grace} ->set_value ($gr, $gr_m); + $::GAUGES->{exp} ->set_text ("Exp: " . (::formsep ($stats->{+CS_STAT_EXP64})) + . " (lvl " . ($stats->{+CS_STAT_LEVEL} * 1) . ")"); + $::GAUGES->{range} ->set_text ($stats->{+CS_STAT_RANGE}); + my $title = $stats->{+CS_STAT_TITLE}; + $title =~ s/^Player: //; + $::STATWIDS->{title} ->set_text ("Title: " . $title); + + $::STATWIDS->{st_str} ->set_text (sprintf "%d" , $stats->{+CS_STAT_STR}); + $::STATWIDS->{st_dex} ->set_text (sprintf "%d" , $stats->{+CS_STAT_DEX}); + $::STATWIDS->{st_con} ->set_text (sprintf "%d" , $stats->{+CS_STAT_CON}); + $::STATWIDS->{st_int} ->set_text (sprintf "%d" , $stats->{+CS_STAT_INT}); + $::STATWIDS->{st_wis} ->set_text (sprintf "%d" , $stats->{+CS_STAT_WIS}); + $::STATWIDS->{st_pow} ->set_text (sprintf "%d" , $stats->{+CS_STAT_POW}); + $::STATWIDS->{st_cha} ->set_text (sprintf "%d" , $stats->{+CS_STAT_CHA}); + $::STATWIDS->{st_wc} ->set_text (sprintf "%d" , $stats->{+CS_STAT_WC}); + $::STATWIDS->{st_ac} ->set_text (sprintf "%d" , $stats->{+CS_STAT_AC}); + $::STATWIDS->{st_dam} ->set_text (sprintf "%d" , $stats->{+CS_STAT_DAM}); + $::STATWIDS->{st_arm} ->set_text (sprintf "%d" , $stats->{+CS_STAT_RES_PHYS}); + $::STATWIDS->{st_spd} ->set_text (sprintf "%.1f", $stats->{+CS_STAT_SPEED}); + $::STATWIDS->{st_wspd}->set_text (sprintf "%.1f", $stats->{+CS_STAT_WEAP_SP}); + + $self->update_weight; + + $::STATWIDS->{"res_$_"}->set_text (sprintf "%d%", $stats->{$RES_TBL{$_}}) + for keys %RES_TBL; + + my $sktbl = $::STATWIDS->{skill_tbl}; + my @skills = keys %{ $self->{skill_info} }; + + 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 $TOOLTIP_ALL = "\n\nLeft click - ready skill\nMiddle click - use spell\nRight click - further options"; + + my @TOOLTIP_LVL = (tooltip => "Level. The level of the skill.$TOOLTIP_ALL", can_events => 1, can_hover => 1); + my @TOOLTIP_EXP = (tooltip => "Experience. The experience points you have in this skill.$TOOLTIP_ALL", can_events => 1, can_hover => 1); + + my ($x, $y) = (0, 1); + for ( + sort { $stats->{$b->[0]}[1] <=> $stats->{$a->[0]}[1] or $a->[1] cmp $b->[1] } + map [$_, $self->{skill_info}{$_}], + grep exists $stats->{$_}, + @skills + ) { + my ($idx, $name) = @$_; + + my $spell_cb = sub { + my ($widget, $ev) = @_; + + if ($ev->{button} == 1) { + $::CONN->user_send ("ready_skill $name"); + } elsif ($ev->{button} == 2) { + $::CONN->user_send ("use_skill $name"); + } elsif ($ev->{button} == 3) { + my $shortname = CFPlus::shorten $name, 14; + (new CFPlus::UI::Menu + items => [ + ["bind ready_skill $shortname to a key" => sub { CFPlus::Macro::quick_macro ["ready_skill $name"] }], + ["bind use_skill $shortname to a key" => sub { CFPlus::Macro::quick_macro ["use_skill $name"] }], + ], + )->popup ($ev); + } else { + return 0; + } + + 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); + + $x++ and ($x, $y) = (0, $y + 1); + } + } + + for (grep exists $stats->{$_}, @skills) { + $self->{stat_widget_exp}{$_}->set_text (::formsep ($stats->{$_}[1])); + $self->{stat_widget_lvl}{$_}->set_text ($stats->{$_}[0] * 1); + } +} + +sub macro_send { + my ($self, $macro) = @_; + + for my $cmd (@{ $macro->{action} }) { + $self->send_command ($cmd); + } } sub user_send { my ($self, $command) = @_; - if ($self->{record}) { - push @{$self->{record}}, $command; - } + $self->{record}->($command) + if $self->{record}; $self->logprint ("send: ", $command); $self->send_command ($command); - ::status $command; + ::status ($command); } -sub start_record { - my ($self) = @_; - - $self->{record} = []; -} +sub record { + my ($self, $cb) = @_; -sub stop_record { - my ($self) = @_; - return delete $self->{record}; + $self->{record} = $cb; } sub map_scroll { @@ -130,7 +520,7 @@ sub feed_map1a { my ($self, $data) = @_; - $self->{map}->map1a_update ($data); + $self->{map}->map1a_update ($data, $self->{setup}{extmap}); $self->{map_widget}->update; } @@ -149,7 +539,7 @@ my ($hash, $x, $y, $w, $h) = @$map_info; my $data = $self->{map}->get_rect ($x, $y, $w, $h); - $self->{mapcache}->put ($hash => Compress::LZF::compress $data); + CFPlus::DB::put $self->{mapcache} => $hash => Compress::LZF::compress $data, sub { }; #warn sprintf "SAVEmap[%s] length %d\n", $hash, length $data;#d# } @@ -163,20 +553,47 @@ delete $self->{map_widget}{magicmap}; } +sub bg_fetch { + my ($self) = @_; + + my $id; + + do { + $id = pop @{$self->{bg_fetch}} + or return; + } while $self->{texture}[$id]; + + CFPlus::DB::get tilecache => $id, sub { + my ($data) = @_; + + return unless $self->{map}; # stop when destroyed + + $self->set_texture ($id => $data) + if defined $data; + + $self->bg_fetch; + }; +} sub load_map($$$) { my ($self, $hash, $x, $y) = @_; - if (defined (my $data = $self->{mapcache}->get ($hash))) { - $data = Compress::LZF::decompress $data; - #warn sprintf "LOADmap[%s,%d,%d] length %d\n", $hash, $x, $y, length $data;#d# - for my $id ($self->{map}->set_rect ($x, $y, $data)) { - my $data = $self->{tilecache}->get ($id) - or next; + my $gen = $self->{map_change_gen}; + + CFPlus::DB::get $self->{mapcache} => $hash, sub { + return unless $gen == $self->{map_change_gen}; + + my ($data) = @_; - $self->set_texture ($id => $data); + if (defined $data) { + $data = Compress::LZF::decompress $data; + #warn sprintf "LOADmap[%s,%d,%d] length %d\n", $hash, $x, $y, length $data;#d# + + my $inprogress = @{ $self->{bg_fetch} || [] }; + unshift @{ $self->{bg_fetch} }, $self->{map}->set_rect ($x, $y, $data); + $self->bg_fetch unless $inprogress; } - } + }; } # hardcode /world/world_xxx_xxx map names, the savings are enourmous, @@ -250,7 +667,10 @@ if $x >= $x0 && $x + $w < $x1 && $y >= $y0 && $y + $h < $y1; } else { + my $gen = $self->{map_change_gen}; $self->send_mapinfo ("spatial $path$tile", sub { + return unless $gen == $self->{map_change_gen}; + my ($mode, $flags, $x, $y, $w, $h, $hash) = @_; return if $mode ne "spatial"; @@ -275,6 +695,8 @@ $self->flush_map; + ++$self->{map_change_gen}; + my ($ox, $oy) = ($::MAP->ox, $::MAP->oy); my $mapmapw = $self->{mapmap}->{w}; @@ -300,71 +722,44 @@ } sub face_find { - my ($self, $facenum, $face) = @_; + my ($self, $facenum, $face, $cb) = @_; my $hash = "$face->{chksum},$face->{name}"; - my $id = $self->{facemap}->get ($hash); - - unless ($id) { - # create new id for face - # I love transactions - for (1..100) { - my $txn = $CFClient::DB_ENV->txn_begin; - my $status = $self->{facemap}->db_get (id => $id); - if ($status == 0 || $status == BerkeleyDB::DB_NOTFOUND) { - $id = ($id || 64) + 1; - if ($self->{facemap}->put (id => $id) == 0 - && $self->{facemap}->put ($hash => $id) == 0) { - $txn->txn_commit; - - goto gotid; - } - } - $txn->txn_abort; - } + my $id = CFPlus::DB::get_tile_id_sync $hash; - CFClient::fatal "maximum number of transaction retries reached - database problems?"; - } + $face->{id} = $id; + $self->{faceid}[$facenum] = $id; -gotid: - $face->{id} = $id; - $self->{map}->set_face ($facenum => $id); - $self->{faceid}[$facenum] = $id;#d# + $self->{map}->set_tileid ($facenum => $id); - my $face = $self->{tilecache}->get ($id); - - if ($face) { - #$self->face_prefetch; - $face - } else { - my $tex = $self->{noface}; - $self->{map}->set_texture ($id, @$tex{qw(name w h s t)}, @{$tex->{minified}}); - undef - }; + CFPlus::DB::get tilecache => $id, $cb; } sub face_update { - my ($self, $facenum, $face) = @_; + my ($self, $facenum, $face, $changed) = @_; - $self->{tilecache}->put ($face->{id} => $face->{image}); #TODO: try to avoid duplicate writes + CFPlus::DB::put tilecache => $face->{id} => $face->{image}, sub { } + if $changed; $self->set_texture ($face->{id} => delete $face->{image}); } +sub smooth_update { + my ($self, $facenum, $face) = @_; + + $self->{map}->set_smooth ($facenum, $face->{smoothface}, $face->{smoothlevel}); +} + sub set_texture { my ($self, $id, $data) = @_; - $self->{texture}[$id] ||= do { - my $tex = - new_from_image CFClient::Texture - $data, minify => 1, mipmap => 1; - - $self->{map}->set_texture ($id, @$tex{qw(name w h s t)}, @{$tex->{minified}}); - $self->{map_widget}->update; + $self->{texture}[$id] = my $tex = + new_from_image CFPlus::Texture + $data, minify => 1, mipmap => 1; - $tex - }; + $self->{map}->set_texture ($id, @$tex{qw(name w h s t)}, @{$tex->{minified}}); + $self->{map_widget}->update; } sub sound_play { @@ -384,94 +779,129 @@ $self->{query}-> ($self, $flags, $prompt); } -sub drawinfo { - my ($self, $color, $text) = @_; - - my @color = ( - [1.00, 1.00, 1.00], #[0.00, 0.00, 0.00], - [1.00, 1.00, 1.00], - [0.50, 0.50, 1.00], #[0.00, 0.00, 0.55] - [1.00, 0.00, 0.00], - [1.00, 0.54, 0.00], - [0.11, 0.56, 1.00], - [0.93, 0.46, 0.00], - [0.18, 0.54, 0.34], - [0.56, 0.73, 0.56], - [0.80, 0.80, 0.80], - [0.55, 0.41, 0.13], - [0.99, 0.77, 0.26], - [0.74, 0.65, 0.41], - ); - - $self->logprint ("info: ", $text); - - my $time = sprintf "%02d:%02d:%02d", (localtime time)[2,1,0]; - - # try to create single paragraphs of multiple lines sent by the server - $text =~ s/(?<=\S)\n(?=\w)/ /g; +sub sanitise_xml($) { + local $_ = shift; - $text = CFClient::UI::Label::escape $text; - $text =~ s/\[b\](.*?)\[\/b\]/\1<\/b>/g; - $text =~ s/\[color=(.*?)\](.*?)\[\/color\]/\2<\/span>/g; - - $self->{logview}->add_paragraph ($color[$color], - join "\n", map "$time $_", split /\n/, $text); - - $self->{statusbox}->add ($text, - group => $text, - fg => $color[$color], - timeout => $color >= 2 ? 60 : 10, - tooltip_font => $::FONT_FIXED, - ); -} + # we now weed out all tags we do not support + s%<(?!/?i>|/?u>|/?b>|fg |/fg>)%<%g; + # now all entities + s/&(?!amp;|lt;|gt;|apos;|quot;|#[0-9]+;|#x[0-9a-fA-F]+;)/&/g; + + # handle some elements + s/(.*?)<\/fg>/$2<\/span>/gs; + s/(.*?)<\/fg>/$2<\/span>/gs; + + $_ +} + +our %NAME_TO_COLOR = ( + black => 0, + white => 1, + darkblue => 2, + red => 3, + orange => 4, + lightblue => 5, + darkorange => 6, + green => 7, + darkgreen => 8, + grey => 9, + brown => 10, + yellow => 11, + tan => 12, +); + +our @CF_COLOR = ( + [1.00, 1.00, 1.00], #[0.00, 0.00, 0.00], + [1.00, 1.00, 1.00], + [0.50, 0.50, 1.00], #[0.00, 0.00, 0.55] + [1.00, 0.00, 0.00], + [1.00, 0.54, 0.00], + [0.11, 0.56, 1.00], + [0.93, 0.46, 0.00], + [0.18, 0.54, 0.34], + [0.56, 0.73, 0.56], + [0.80, 0.80, 0.80], + [0.75, 0.61, 0.20], + [0.99, 0.77, 0.26], + [0.74, 0.65, 0.41], +); + +sub msg { + my ($self, $color, $type, $text, @extra) = @_; + + $text = sanitise_xml $text; + + if (my $cb = $self->{cb_msg}{$type}) { + $_->($self, $color, $type, $text, @extra) for values %$cb; + } elsif ($type =~ /^(?:chargen-race-title|chargen-race-description)$/) { + $type =~ s/-/_/g; + $self->{$type} = $text; + } else { + $self->logprint ("msg: ", $text); + return if $color < 0; # negative color == ignore if not understood -sub drawextinfo { - my ($self, $color, $type, $subtype, $message) = @_; + my $fg = $CF_COLOR[$color % @CF_COLOR]; - $self->drawinfo ($color, $message); + ## 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 => $_ }) + for split /\n/, $text; + + $self->{statusbox}->add ($text, + group => $text, + fg => $fg, + timeout => $color >= 2 ? 180 : 10, + tooltip_font => $::FONT_FIXED, + ); + } } sub spell_add { my ($self, $spell) = @_; - # try to create single paragraphs of multiple lines sent by the server + # try to create single paragraphs out of the multiple lines sent by the server $spell->{message} =~ s/(?<=\S)\n(?=\w)/ /g; $spell->{message} =~ s/\n+$//; $spell->{message} ||= "Server did not provide a description for this spell."; - $::SETUP_SPELLS->add_spell ($spell); + $::SPELL_LIST->add_spell ($spell); - $self->{map_widget}->add_command ("invoke $spell->{name}", CFClient::UI::Label::escape $spell->{message}); - $self->{map_widget}->add_command ("cast $spell->{name}", CFClient::UI::Label::escape $spell->{message}); + $self->{map_widget}->add_command ("invoke $spell->{name}", CFPlus::asxml $spell->{message}); + $self->{map_widget}->add_command ("cast $spell->{name}", CFPlus::asxml $spell->{message}); } sub spell_delete { my ($self, $spell) = @_; - $::SETUP_SPELLS->remove_spell ($spell); + + $::SPELL_LIST->remove_spell ($spell); } -sub addme_success { - my ($self) = @_; +sub setup { + my ($self, $setup) = @_; - my $skill_help = CFClient::load_pod CFClient::find_rcfile "pod/skill_help.pod", skill_help => 1, sub { - my ($pom) = @_; + $self->{map_widget}->set_tilesize ($self->{tilesize}); + $::MAP->resize ($self->{mapw}, $self->{maph}); +} - my %skill_help; +sub addme_success { + my ($self) = @_; - for my $head2 ($pom->head1->[3]->head2) { - $skill_help{$head2->title} = CFClient::pod_to_pango $head2->content; - } + my %skill_help; - \%skill_help + for my $node (CFPlus::Pod::find skill_description => "*") { + my (undef, @par) = CFPlus::Pod::section_of $node; + $skill_help{$node->{kw}[0]} = CFPlus::Pod::as_label @par; }; - + for my $skill (values %{$self->{skill_info}}) { $self->{map_widget}->add_command ("ready_skill $skill", - (CFClient::UI::Label::escape "Ready the skill '$skill'\n\n") - . $skill_help->{$skill}); + (CFPlus::asxml "Ready the skill '$skill'\n\n") + . $skill_help{$skill}); $self->{map_widget}->add_command ("use_skill $skill", - (CFClient::UI::Label::escape "Immediately use the skill '$skill'\n\n") - . $skill_help->{$skill}); + (CFPlus::asxml "Immediately use the skill '$skill'\n\n") + . $skill_help{$skill}); } } @@ -505,57 +935,64 @@ $self->send ("requestinfo image_sums $face $face"); - $self->{statusbox}->add (CFClient::UI::Label::escape "prefetching $todo", + $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 (CFClient::UI::Label::escape "starting to prefetch", + $self->{statusbox}->add (CFPlus::asxml "starting to prefetch", group => "prefetch", timeout => 3, fg => [1, 1, 0, 0.5]); } } sub update_floorbox { - $CFClient::UI::ROOT->on_refresh ($::FLOORBOX => sub { + $CFPlus::UI::ROOT->on_refresh ($::FLOORBOX => sub { return unless $::CONN; - + $::FLOORBOX->clear; my $row; - for (@{ $::CONN->{container}{0} }) { + for (sort { $a->{count} <=> $b->{count} } values %{ $::CONN->{container}{$::CONN->{open_container} || 0} }) { if ($row < 6) { local $_->{face_widget}; # hack to force recreation of widget local $_->{desc_widget}; # hack to force recreation of widget - CFClient::Item::update_widgets $_; + CFPlus::Item::update_widgets $_; $::FLOORBOX->add (0, $row, $_->{face_widget}); $::FLOORBOX->add (1, $row, $_->{desc_widget}); $row++; } else { - $::FLOORBOX->add (1, $row, new CFClient::UI::Button + $::FLOORBOX->add (1, $row, new CFPlus::UI::Button text => "More...", - on_activate => sub { $::INV_WINDOW->toggle_visibility; 0 }, + on_activate => sub { ::toggle_player_page ($::INVENTORY_PAGE); 0 }, ); last; } } + if ($::CONN->{open_container}) { + $::FLOORBOX->add (1, $row++, new CFPlus::UI::Button + text => "Close container", + on_activate => sub { $::CONN->send ("apply $::CONN->{open_container}") } + ); + } }); - + $::WANT_REFRESH++; } sub set_opencont { my ($conn, $tag, $name) = @_; $conn->{open_container} = $tag; + update_floorbox; $::INV_RIGHT_HB->clear (); - $::INV_RIGHT_HB->add (new CFClient::UI::Label align => 0, expand => 1, text => $name); + $::INV_RIGHT_HB->add (new CFPlus::UI::Label align => 0, expand => 1, text => $name); if ($tag != 0) { # Floor isn't closable, is it? - $::INV_RIGHT_HB->add (new CFClient::UI::Button + $::INV_RIGHT_HB->add (new CFPlus::UI::Button text => "Close container", tooltip => "Close the currently open container (if one is open)", on_activate => sub { @@ -570,63 +1007,49 @@ $::INVR->set_items ($conn->{container}{$tag}); } -sub update_container { - my ($tag) = @_; +sub update_containers { + my ($self) = @_; + + $CFPlus::UI::ROOT->on_refresh ("update_containers_$self" => sub { + my $todo = delete $self->{update_container} + or return; - $::INVR->set_items ($::CONN->{container}{$::CONN->{open_container}}) - if $tag == $::CONN->{open_container}; + for my $tag (keys %$todo) { + update_floorbox if $tag == 0 or $tag == $self->{open_container}; + if ($tag == 0) { + $::INVR->set_items ($self->{container}{0}) + if $tag == $self->{open_container}; + } elsif ($tag == $self->{player}{tag}) { + $::INV->set_items ($self->{container}{$tag}) + } else { + $::INVR->set_items ($self->{container}{$tag}) + if $tag == $self->{open_container}; + } + } + }); } sub container_add { my ($self, $tag, $items) = @_; - #d# print "container_add: container $tag ($self->{player}{tag})\n"; - - if ($tag == 0) { - update_floorbox; - update_container (0); - } elsif ($tag == $self->{player}{tag}) { - $::INV->set_items ($self->{container}{$self->{player}{tag}}) - } else { - update_container ($tag); - } - - # $self-<{player}{tag} => player inv - #use PApp::Util; warn PApp::Util::dumpval $self->{container}{$self->{player}{tag}}; + $self->{update_container}{$tag}++; + $self->update_containers; } sub container_clear { my ($self, $tag) = @_; - #d# print "container_clear: container $tag ($self->{player}{tag})\n"; - - if ($tag == 0) { - update_floorbox; - update_container (0); - } elsif ($tag == $self->{player}{tag}) { - $::INV->set_items ($self->{container}{$tag}) - } else { - update_container ($tag); - } - -# use PApp::Util; warn PApp::Util::dumpval $self->{container}{0}; + $self->{update_container}{$tag}++; + $self->update_containers; } sub item_delete { my ($self, @items) = @_; - for (@items) { - #d# print "item_delete: $_->{tag} from $_->{container} ($self->{player}{tag})\n"; - - if ($_->{container} == 0) { - update_floorbox; - update_container ($_->{tag}); - } elsif ($_->{container} == $self->{player}{tag}) { - $::INV->set_items ($self->{container}{$self->{player}{tag}}) - } else { - update_container ($_->{container}); - } - } + $self->{update_container}{$_->{container}}++ + for @items; + + $self->update_containers; } sub item_update { @@ -634,32 +1057,35 @@ #d# print "item_update: $item->{tag} in $item->{container} ($self->{player}{tag}) ($::CONN->{open_container})\n"; - if ($item->{tag} == $self->{player}{tag}) { - $::STATWIDS->{weight}->set_text (sprintf "Weight: %.1fkg", $item->{weight} / 1000); - return; - } - - CFClient::Item::update_widgets $item; + CFPlus::Item::update_widgets $item; if ($item->{tag} == $::CONN->{open_container} && not ($item->{flags} & F_OPEN)) { set_opencont ($::CONN, 0, "Floor"); } elsif ($item->{flags} & F_OPEN) { - set_opencont ($::CONN, $item->{tag}, CFClient::Item::desc_string $item); + set_opencont ($::CONN, $item->{tag}, CFPlus::Item::desc_string $item); + } else { - if ($item->{container} == 0) { - update_floorbox; - update_container (0); - } elsif ($item->{container} == $self->{player}{tag}) { - $::INV->set_items ($self->{container}{$item->{container}}) - } + $self->{update_container}{$item->{container}}++; + $self->update_containers; } } sub player_update { my ($self, $player) = @_; - $::STATWIDS->{weight}->set_text (sprintf "Weight: %.1fkg", $player->{weight} / 1000); + $self->update_weight; +} + +sub update_weight { + my ($self) = @_; + + my $weight = .001 * $self->{player}{weight}; + my $limit = .001 * $self->{stat}{+CS_STAT_WEIGHT_LIM}; + + $::STATWIDS->{weight}->set_text (sprintf "Weight: %.1fkg", $weight); + $::STATWIDS->{m_weight}->set_text (sprintf "%.1fkg", $limit); + $::STATWIDS->{i_weight}->set_text (sprintf "%.1f/%.1fkg", $weight, $limit); } sub update_server_info { @@ -672,40 +1098,66 @@ . "protocol version $self->{version}\n" . "minimap support $yesno[$self->{setup}{mapinfocmd} > 0]\n" . "extended command support $yesno[$self->{setup}{extcmd} > 0]\n" + . "examine command support $yesno[$self->{setup}{excmd} > 0]\n" + . "editing support $yesno[!!$self->{editor_support}]\n" + . "map attributes $yesno[$self->{setup}{extmap} > 0]\n" + . "big image protocol support $yesno[$self->{setup}{fxix} > 0]\n" . "cfplus support $yesno[$self->{cfplus_ext} > 0]" . ($self->{cfplus_ext} > 0 ? ", version $self->{cfplus_ext}" : "") ."\n" . "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 => "1", sub { - $self->{cfplus_ext} = $_[0]; + $self->send_ext_req (cfplus_support => version => 1, sub { + $self->{cfplus_ext} = $_[0]{version}; $self->update_server_info; + + if ($self->{cfplus_ext} >= 2) { + $self->send_ext_req ("editor_support", sub { + $self->{editor_support} = $_[0]; + $self->update_server_info; + + 0 + }); + } + + 0 }); $self->update_server_info; $self->send_command ("output-sync $::CFG->{output_sync}"); $self->send_command ("output-count $::CFG->{output_count}"); + $self->send_command ("output-rate $::CFG->{output_rate}") if $::CFG->{output_rate} > 0; $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 => "$x $y", sub { - my %res = split /\x00/, $_[0]; + $self->send_ext_req (lookat => dx => $x, dy => $y, sub { + my ($msg) = @_; - if (exists $res{npc_dialog}) { + if (exists $msg->{npc_dialog}) { # start npc chat dialog - $self->{npc_dialog} = new CFClient::NPCDialog:: + $self->{npc_dialog} = new CFPlus::NPCDialog:: dx => $x, dy => $y, - title => "$res{npc_dialog} (NPC)", + title => "$msg->{npc_dialog} (NPC)", conn => $self, ; } @@ -718,15 +1170,17 @@ sub destroy { my ($self) = @_; - $self->{npc_dialog}->destroy + (delete $self->{npc_dialog})->destroy if $self->{npc_dialog}; $self->SUPER::destroy; + + %$self = (); } -package CFClient::NPCDialog; +package CFPlus::NPCDialog; -our @ISA = 'CFClient::UI::FancyFrame'; +our @ISA = 'CFPlus::UI::Toplevel'; sub new { my $class = shift; @@ -739,24 +1193,23 @@ force_h => $::HEIGHT * 0.7, title => "NPC Dialog", kw => { hi => 0, yes => 0, no => 0 }, + has_close_button => 1, @_, ); - Scalar::Util::weaken (my $this = $self); + CFPlus::weaken (my $this = $self); + + $self->connect (delete => sub { $this->destroy; 1 }); # better use a pane... - $self->add (my $hbox = new CFClient::UI::HBox); - $hbox->add ($self->{textview} = new CFClient::UI::TextScroller expand => 1); + $self->add (my $hbox = new CFPlus::UI::HBox); + $hbox->add ($self->{textview} = new CFPlus::UI::TextScroller expand => 1); - $hbox->add (my $vbox = new CFClient::UI::VBox); + $hbox->add (my $vbox = new CFPlus::UI::VBox); - $vbox->add (new CFClient::UI::Label text => "Message Entry:"); - $vbox->add ($self->{entry} = new CFClient::UI::Entry - tooltip => "Enter a message you want to tell the NPC and press return.\n\n" - . "Sometimes you have to tell an NPC something you cannot find out during " - . "a normal conversation (such as a password). In those cases you have to use " - . "this text entry. You can also enter responses manually instead of using the response " - . "buttons below.", + $vbox->add (new CFPlus::UI::Label text => "Message Entry:"); + $vbox->add ($self->{entry} = new CFPlus::UI::Entry + tooltip => "#npc_message_entry", on_activate => sub { my ($entry, $text) = @_; @@ -769,23 +1222,27 @@ }, ); - $vbox->add ($self->{options} = new CFClient::UI::VBox); + $vbox->add ($self->{options} = new CFPlus::UI::VBox); - $self->{bye_button} = new CFClient::UI::Button + $self->{bye_button} = new CFPlus::UI::Button text => "Bye (close)", tooltip => "Use this button to end talking to the NPC. This also closes the dialog window.", - on_activate => sub { $this->destroy; 0 }, + on_activate => sub { $this->destroy; 1 }, ; $self->update_options; - $self->{token} = $self->{conn}->ext_token; - $self->{conn}->connect_ext ($self->{token} => sub { $this->feed (@_) }); - $self->{conn}->send ("ext npc_dialog_begin $self->{token} $self->{dx} $self->{dy}"); + $self->{id} = $self->{conn}->send_ext_req ( + npc_dialog_begin => dx => $self->{dx}, dy => $self->{dy}, + sub { $this && $this->feed (@_) } + ); $self->{entry}->grab_focus; - $self->{textview}->add_paragraph ([1, 1, 0, 1], "[starting conversation with $self->{title}]\n\n"); + $self->{textview}->add_paragraph ({ + fg => [1, 1, 0, 1], + markup => "[starting conversation with $self->{title}]\n\n", + }); $self->show; $self @@ -794,13 +1251,13 @@ sub update_options { my ($self) = @_; - Scalar::Util::weaken $self; + CFPlus::weaken $self; $self->{options}->clear; $self->{options}->add ($self->{bye_button}); for my $kw (sort keys %{ $self->{kw} }) { - $self->{options}->add (new CFClient::UI::Button + $self->{options}->add (new CFPlus::UI::Button text => $kw, on_activate => sub { $self->send ($kw); @@ -811,19 +1268,37 @@ } sub feed { - my ($self, $data) = @_; + my ($self, $msg) = @_; - my ($type, $msg) = split / /, $data, 2; + CFPlus::weaken $self; - if ($type eq "msg") { - my ($msg, @kw) = split /\x00/, $msg; - $self->{kw}{$_} = 1 for @kw; + if ($msg->{msgtype} eq "reply") { + $self->{kw}{$_} = 1 for @{$msg->{add_topics} || []}; + $self->{kw}{$_} = 0 for @{$msg->{del_topics} || []}; - $msg = CFClient::UI::Label::escape $msg; + 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} }; - $msg =~ s/($match)/$1<\/span>/gi; # underline when http-ready, huh. + 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 ([1, 1, 1, 1], "\n$msg"); + $self->{textview}->add_paragraph ({ markup => $text, widget => \@link }); + $self->{textview}->scroll_to_bottom; $self->update_options; } else { $self->destroy; @@ -835,8 +1310,10 @@ sub send { my ($self, $msg) = @_; - $self->{conn}->send ("ext npc_dialog_tell $self->{token} $msg"); - $self->{textview}->add_paragraph ([1, 1, 0, 1], "\n" . CFClient::UI::Label::escape $msg); + $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); } sub destroy { @@ -844,10 +1321,14 @@ #Carp::cluck "debug\n";#d# #todo# enable: destroy gets called twice because scalar keys {} is 1 - delete $self->{conn}{npc_dialog}; - $self->{conn}->disconnect_ext ($self->{token}); + if ($self->{conn}) { + $self->{conn}->send_ext_msg (npc_dialog_end => msgid => $self->{id}) if $self->{id}; + delete $self->{conn}{npc_dialog}; + $self->{conn}->disconnect_ext ($self->{id}); + } $self->SUPER::destroy; } -1; +1 +