--- deliantra/Deliantra-Client/DC/Protocol.pm 2006/06/12 13:26:14 1.32 +++ deliantra/Deliantra-Client/DC/Protocol.pm 2006/07/02 21:07:26 1.51 @@ -10,6 +10,8 @@ use base 'Crossfire::Protocol::Base'; +our %open_logs; + sub new { my $class = shift; @@ -61,6 +63,28 @@ $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}: $!"; + + $open_logs{$filename} = $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; +} + sub stats_update { my ($self, $stats) = @_; @@ -81,8 +105,9 @@ push @{$self->{record}}, $command; } + $self->logprint ("send: ", $command); $self->send_command ($command); - ::status $command; + ::status ($command); } sub start_record { @@ -109,6 +134,12 @@ $self->{map_widget}->update; } +sub magicmap { + my ($self, $w, $h, $x, $y, $data) = @_; + + $self->{map_widget}->set_magicmap ($w, $h, $x, $y, $data); +} + sub flush_map { my ($self) = @_; @@ -129,6 +160,7 @@ delete $self->{neigh_map}; $self->{map}->clear; + delete $self->{map_widget}{magicmap}; } @@ -281,7 +313,7 @@ my $txn = $CFClient::DB_ENV->txn_begin; my $status = $self->{facemap}->db_get (id => $id); if ($status == 0 || $status == BerkeleyDB::DB_NOTFOUND) { - $id = ($id || 16) + 1; + $id = ($id || 64) + 1; if ($self->{facemap}->put (id => $id) == 0 && $self->{facemap}->put ($hash => $id) == 0) { $txn->txn_commit; @@ -371,6 +403,8 @@ [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 @@ -380,8 +414,9 @@ $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->{logview}->add_paragraph ($color[$color], $_) + for map "$time $_", split /\n/, $text; + $self->{logview}->scroll_to_bottom; $self->{statusbox}->add ($text, group => $text, @@ -405,7 +440,7 @@ $spell->{message} =~ s/\n+$//; $spell->{message} ||= "Server did not provide a description for this spell."; - $::SETUP_SPELLS->add_spell ($spell); + $::SPELL_PAGE->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}); @@ -413,7 +448,8 @@ sub spell_delete { my ($self, $spell) = @_; - $::SETUP_SPELLS->remove_spell ($spell); + + $::SPELL_PAGE->remove_spell ($spell); } sub addme_success { @@ -490,7 +526,7 @@ $::FLOORBOX->clear; my $row; - for (@{ $::CONN->{container}{0} }) { + for (sort { $a->{count} <=> $b->{count} } values %{ $::CONN->{container}{0} }) { if ($row < 6) { local $_->{face_widget}; # hack to force recreation of widget local $_->{desc_widget}; # hack to force recreation of widget @@ -503,7 +539,7 @@ } else { $::FLOORBOX->add (1, $row, new CFClient::UI::Button text => "More...", - on_activate => sub { $::INV_WINDOW->toggle_visibility }, + on_activate => sub { ::toggle_player_page ($::INVENTORY_PAGE); 0 }, ); last; } @@ -528,6 +564,7 @@ $::CONN->send ("apply $tag") # $::CONN->{open_container}") if $tag != 0; #if $CONN->{open_container} != 0; + 0 }, ); } @@ -535,63 +572,46 @@ $::INVR->set_items ($conn->{container}{$tag}); } -sub update_container { - my ($tag) = @_; +sub update_containers { + my ($self) = @_; - $::INVR->set_items ($::CONN->{container}{$::CONN->{open_container}}) - if $tag == $::CONN->{open_container}; + $CFClient::UI::ROOT->on_refresh ("update_containers_$self" => sub { + for my $tag (keys %{ delete $self->{update_container} }) { + if ($tag == 0) { + update_floorbox; + $::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 { @@ -599,11 +619,6 @@ #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; if ($item->{tag} == $::CONN->{open_container} && not ($item->{flags} & F_OPEN)) { @@ -611,19 +626,21 @@ } elsif ($item->{flags} & F_OPEN) { set_opencont ($::CONN, $item->{tag}, CFClient::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; +# if ($item->{container} == 0) { +# update_floorbox; +# update_container (0); +# } elsif ($item->{container} == $self->{player}{tag}) { +# $::INV->set_items ($self->{container}{$item->{container}}) +# } } } sub player_update { my ($self, $player) = @_; - $::STATWIDS->{weight}->set_text (sprintf "Weight: %.1fkg", $player->{weight} / 1000); } @@ -729,15 +746,17 @@ $entry->set_text (""); $this->send ($text); + + 0 }, ); $vbox->add ($self->{options} = new CFClient::UI::VBox); - $self->{close_button} = new CFClient::UI::Button + $self->{bye_button} = new CFClient::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 }, + on_activate => sub { $this->destroy; 0 }, ; $self->update_options; @@ -746,7 +765,7 @@ $self->{conn}->connect_ext ($self->{token} => sub { $this->feed (@_) }); $self->{conn}->send ("ext npc_dialog_begin $self->{token} $self->{dx} $self->{dy}"); - $self->{entry}->focus_in; + $self->{entry}->grab_focus; $self->{textview}->add_paragraph ([1, 1, 0, 1], "[starting conversation with $self->{title}]\n\n"); @@ -760,13 +779,14 @@ Scalar::Util::weaken $self; $self->{options}->clear; - $self->{options}->add ($self->{close_button}); + $self->{options}->add ($self->{bye_button}); for my $kw (sort keys %{ $self->{kw} }) { $self->{options}->add (new CFClient::UI::Button text => $kw, on_activate => sub { $self->send ($kw); + 0 }, ); } @@ -775,19 +795,37 @@ sub feed { my ($self, $data) = @_; + Scalar::Util::weaken $self; + my ($type, $msg) = split / /, $data, 2; if ($type eq "msg") { my ($msg, @kw) = split /\x00/, $msg; $self->{kw}{$_} = 1 for @kw; - warn "<<<<@kw>>>>\n";#d# - - $msg = CFClient::UI::Label::escape $msg; + $msg = "\n" . CFClient::UI::Label::escape $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; + $msg =~ s{ + ($match) + }{ + my $kw = $1; + + push @link, new CFClient::UI::Label + markup => "$kw", + can_hover => 1, + can_events => 1, + padding_x => 0, + padding_y => 0, + on_button_up => sub { + $self->send ($kw); + }; + + chr 0xfffc + }giex; - $self->{textview}->add_paragraph ([1, 1, 1, 1], "\n$msg"); + $self->{textview}->add_paragraph ([1, 1, 1, 1], [$msg, @link]); + $self->{textview}->scroll_to_bottom; $self->update_options; } else { $self->destroy; @@ -801,12 +839,13 @@ $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}->scroll_to_bottom; } sub destroy { my ($self) = @_; - #Carp::cluck "debug\n";#d# #todo# enable: destroyx gets called twice because scalar keys {} is 1 + #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});