--- deliantra/Deliantra-Client/DC/Protocol.pm	2006/07/30 13:16:44	1.69
+++ deliantra/Deliantra-Client/DC/Protocol.pm	2006/08/13 15:14:17	1.73
@@ -18,34 +18,27 @@
 
    $self->{map_widget}->clr_commands;
 
-   my $cmd_help = CFPlus::Pod::load CFPlus::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
+  } CFPlus::Pod::find command_help => "*";
 
-      my @cmd_help;
-
-      for my $head2 ($pom->head1->[-2]->head2) {
-         $head2->title =~ /^(\S+) (?:\s+ \( ([^\)]*) \) )?/x
-            or next;
-
-         my $cmd = $1;
-         my @args = split /\|/, $2;
-         @args = (".*") unless @args;
-
-         $_ = $_ eq ".*" ? "" : " $_"
-            for @args;
-
-         my $text = CFPlus::Pod::as_markup $head2->content;
-
-         push @cmd_help, ["$cmd$_", $text]
-            for sort { (length $a) <=> (length $b) }
-                     @args;
-      }
-
-      \@cmd_help
-   };
-
-   $self->{map_widget}->add_command (@$_)
-      for @$cmd_help;
+  $self->{map_widget}->add_command (@$_)
+     for @cmd_help;
 
    $self->{noface} = new_from_file CFPlus::Texture
       CFPlus::find_rcfile "noface.png", minify => 1, mipmap => 1;
@@ -638,7 +631,7 @@
    $self->{statusbox}->add ($text,
       group        => $text,
       fg           => $color[$color],
-      timeout      => $color >= 2 ? 60 : 10,
+      timeout      => $color >= 2 ? 180 : 10,
       tooltip_font => $::FONT_FIXED,
    );
 }
@@ -672,25 +665,20 @@
 sub addme_success {
    my ($self) = @_;
 
-   my $skill_help = CFPlus::Pod::load CFPlus::find_rcfile "pod/skill_help.pod", skill_help => 1, sub {
-      my ($pom) = @_;
-
-      my %skill_help;
-
-      for my $head2 ($pom->head1->[3]->head2) {
-         $skill_help{$head2->title} = CFPlus::Pod::as_markup $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",
                                         (CFPlus::asxml "Ready the skill '$skill'\n\n")
-                                        . $skill_help->{$skill});
+                                        . $skill_help{$skill});
       $self->{map_widget}->add_command ("use_skill $skill",
                                         (CFPlus::asxml "Immediately use the skill '$skill'\n\n")
-                                        . $skill_help->{$skill});
+                                        . $skill_help{$skill});
    }
 }
 
@@ -896,9 +884,11 @@
 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;
+
+      0
    });
 
    $self->update_server_info;
@@ -912,15 +902,15 @@
    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 CFPlus::NPCDialog::
                dx    => $x,
                dy    => $y,
-               title => "$res{npc_dialog} (NPC)",
+               title => "$msg->{npc_dialog} (NPC)",
                conn  => $self,
             ;
          }
@@ -970,11 +960,7 @@
 
    $vbox->add (new CFPlus::UI::Label text => "Message Entry:");
    $vbox->add ($self->{entry} = new CFPlus::UI::Entry
-      tooltip     => "Enter a message you want to tell the NPC and press <b>return</b>.\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.",
+      tooltip     => "#npc_message_entry",
       on_activate => sub {
          my ($entry, $text) = @_;
 
@@ -997,9 +983,10 @@
 
    $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;
 
@@ -1032,23 +1019,18 @@
 }
 
 sub feed {
-   my ($self, $data) = @_;
+   my ($self, $msg) = @_;
 
    Scalar::Util::weaken $self;
 
-   my ($type, $msg) = split / /, $data, 2;
-
-   if ($type eq "msg") {
-      my ($msg, @kw) = split /\x00/, $msg;
+   if ($msg->{msgtype} eq "reply") {
+      $self->{kw}{$_} = 1 for @{$msg->{add_topics} || []};
+      $self->{kw}{$_} = 0 for @{$msg->{del_topics} || []};
 
-      utf8::decode $_ for ($msg, @kw);
-      
-      $self->{kw}{$_} = 1 for @kw;
-
-      $msg = "\n" . CFPlus::UI::asxml $msg;
+      my $text = "\n" . CFPlus::asxml $msg->{msg};
       my $match = join "|", map "\\b\Q$_\E\\b", sort { (length $b) <=> (length $a) } keys %{ $self->{kw} };
       my @link;
-      $msg =~ s{
+      $text =~ s{
          ($match)
       }{
          my $kw = $1;
@@ -1066,7 +1048,7 @@
          chr 0xfffc
       }giex;
       
-      $self->{textview}->add_paragraph ({ markup => $msg, widget => \@link });
+      $self->{textview}->add_paragraph ({ markup => $text, widget => \@link });
       $self->{textview}->scroll_to_bottom;
       $self->update_options;
    } else {
@@ -1082,8 +1064,7 @@
    $self->{textview}->add_paragraph ({ markup => "\n" . CFPlus::asxml $msg });
    $self->{textview}->scroll_to_bottom;
 
-   utf8::encode $msg;
-   $self->{conn}->send ("ext npc_dialog_tell $self->{token} $msg");
+   $self->{conn}->send_ext_msg (npc_dialog_tell => msgid => $self->{id}, msg => $msg);
 }
 
 sub destroy {
@@ -1091,9 +1072,11 @@
 
    #Carp::cluck "debug\n";#d# #todo# enable: destroy gets called twice because scalar keys {} is 1
 
-   $self->{conn}->send ("ext npc_dialog_end $self->{token}") if $self->{token};
-   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;
 }