ViewVC Help
View File | Revision Log | Show Annotations | Download File
/cvs/deliantra/Deliantra-Client/DC/Protocol.pm
(Generate patch)

Comparing deliantra/Deliantra-Client/DC/Protocol.pm (file contents):
Revision 1.109 by root, Mon Jun 25 05:43:37 2007 UTC vs.
Revision 1.162 by root, Wed Aug 22 21:14:06 2007 UTC

… …
21 21
22 my $self = $class->SUPER::new (%arg, 22 my $self = $class->SUPER::new (%arg,
23 setup_req => { 23 setup_req => {
24 extmap => 1, 24 extmap => 1,
25 excmd => 1, 25 excmd => 1,
26 xwidget => 1, 26 xwidget1 => 1,#d#
27 %{$arg{setup_req} || {}}, 27 %{$arg{setup_req} || {}},
28 }, 28 },
29 ); 29 );
30 30
31 $self->{map_widget}->clr_commands; 31 $self->{map_widget}->clr_commands;
… …
48 sort { (length $a) <=> (length $b) } 48 sort { (length $a) <=> (length $b) }
49 @args 49 @args
50 } sort { $a->{par} <=> $b->{par} } 50 } sort { $a->{par} <=> $b->{par} }
51 CFPlus::Pod::find command => "*"; 51 CFPlus::Pod::find command => "*";
52 52
53 $self->connect_ext (event_capabilities => sub { 53 $self->{json_coder}
54 my ($cap) = @_; 54 ->convert_blessed
55 55 ->filter_json_single_key_object (__w_ => sub {
56 if (my $ts = $cap->{tileset}) { 56 $self->{widget}{$_[0]}
57 if (my ($default) = grep $_->[2] & 1, @$ts) {
58 $self->{tileset} = $default;
59 $self->{tilesize} = $default->[3];
60 $self->setup_req (tileset => $default->[0]);
61
62 my $w = int $self->{mapw} * 32 / $self->{tilesize};
63 my $h = int $self->{maph} * 32 / $self->{tilesize};
64
65 $self->setup_req (mapsize => "${w}x${h}");
66 }
67 }
68 });
69
70 $self->connect_ext (ws_n => sub {
71 my ($arg) = @_;
72
73 $self->{widgetset}{$arg{id}} = {
74 w => {},
75 }; 57 });
76 });
77 58
78 $self->connect_ext (ws_d => sub { 59 # destroy widgets on logout
79 my ($arg) = @_;
80
81 my $ws = delete $self->{widgetset}{$arg{id}}
82 or return;
83
84 $_->destroy
85 for values %{$ws->{w}};
86 });
87
88 $self->connect_ext (ws_c => sub {
89 my ($arg) = @_;
90
91 my $args = $arg->{args} || {};
92
93 for my $ev (grep /^on_/, keys %$args) {
94 $args->{$ev} = sub {
95 my $id = shift->{s_id};
96 $self->send_exti_msg (w_e => id => $id, name => $ev, args => \@_);
97
98 1
99 };
100 }
101
102 if (my $widget = eval {
103 "CFPlus::UI::$arg->{class}"->new (
104 %$args,
105 s_ws => $arg->{ws},
106 s_id => $arg->{id},
107 )
108 }
109 ) {
110 $self->{widget}{$arg->{id}}
111 = $self->{widgetset}{$arg->{ws}}{w}{$arg->{id}}
112 = $widget;
113
114 $widget->connect (on_destroy => sub {
115 my ($widget) = @_;
116
117 delete $self->{widget}{$widget->{s_id}};
118 delete $self->{widgetset}{$widget->{s_ws}}{$widget->{s_id}};
119 });
120 } else {
121 warn "server failed creating client-side widget " . (CFPlus::to_json $arg) . ": $@\n";
122 $self->send_exti_msg (w_e => id => $arg->{id}, name => "destroy");
123 }
124 });
125
126 $self->connect_ext (w_c => sub {
127 my ($arg) = @_;
128
129 my $w = $self->{widget}{$arg->{id}}
130 or return;
131 my $m = $arg->{name};
132
133 my @a = map {
134 "HASH" eq ref && 1 == (scalar keys %$_) && exists $_->{__widget_ref__}
135 ? $self->{widget}{$_->{__widget_ref__}}
136 : $_
137 } @{ $arg->{args} || [] };
138
139 if (exists $arg->{rid}) {
140 $self->send_exti_msg (w_r => rid => $arg->{rid}, res => [$w->$m (@a)]);
141 } else {
142 $w->$m (@a);
143 }
144 });
145
146 $self->connect_ext (w_s => sub {
147 my ($arg) = @_;
148
149 my $w = $self->{widget}{$arg->{id}}
150 or return;
151
152 $w->{$arg->{name}} = $arg->{value};
153 });
154
155 $self->connect_ext (w_g => sub {
156 my ($arg) = @_;
157
158 my $w = $self->{widget}{$arg->{id}}
159 or return;
160
161 $self->send_exti_msg (w_r => rid => $arg->{rid}, res => [$w->{$arg->{name}}]);
162 });
163
164 $self->{on_stop_game_guard} = $self->{map_widget}{root}->connect (stop_game => sub { 60 $self->{on_stop_game_guard} = $self->{map_widget}{root}->connect (stop_game => sub {
165 for my $ws (values %{delete $self->{widgetset} || {}}) { 61 for my $ws (values %{delete $self->{widgetset} || {}}) {
166 $_->destroy 62 $_->destroy
167 for values %{delete $ws->{w} || {}}; 63 for values %{delete $ws->{w} || {}};
168 } 64 }
… …
189 $self->{mapcache} = "mapcache_$self->{host}_$self->{port}"; 85 $self->{mapcache} = "mapcache_$self->{host}_$self->{port}";
190 86
191 $self 87 $self
192} 88}
193 89
90sub update_fx_want {
91 my ($self) = @_;
92
93 $self->send_exti_msg (fx_want => {
94 3 => !!$::CFG->{bgm_enable}, # FT_MUSIC
95 5 => !!$::CFG->{audio_enable}, # FT_SOUND
96 6 => 1, # FT_RSRC
97 });
98}
99
100sub ext_capabilities {
101 my ($self, %cap) = @_;
102
103 #$self->send ("setup sound 0"); # we use a different protocol
104 $self->update_fx_want;
105
106 $self->send_exti_req (resource => "exp_table", sub {
107 my ($exp_table) = @_;
108
109 $self->register_face_handler ($exp_table, sub {
110 my ($face) = @_;
111
112 $self->{exp_table} = $self->{json_coder}->decode (delete $face->{data});
113 $_->() for values %{ $self->{on_exp_update} || {} };
114 });
115
116 ()
117 });
118
119 if (my $ts = $cap{tileset}) {
120 if (my ($default) = grep $_->[2] & 1, @$ts) {
121 $self->{tileset} = $default;
122 $self->{tilesize} = $default->[3];
123 $self->setup_req (tileset => $default->[0]);
124
125 my $w = int $self->{mapw} * 32 / $self->{tilesize};
126 my $h = int $self->{maph} * 32 / $self->{tilesize};
127
128 $self->setup_req (mapsize => "${w}x${h}");
129 }
130 }
131}
132
133sub ext_ambient_music {
134 my ($self, $songs) = @_;
135 &::audio_music_set_ambient ($songs);
136}
137
138#############################################################################
139
140sub widget_associate {
141 my ($self, $ws, $id, $widget) = @_;
142
143 if ($widget) {
144 $widget->{s_id} = $id;
145 $self->{widget}{$id} = $widget;
146
147 if ($ws) {
148 $widget->{s_ws} = $ws;
149 $self->{widgetset}{$ws}{w}{$id} = $widget;
150 }
151
152 $widget->connect (on_destroy => sub {
153 my ($widget) = @_;
154
155 delete $self->{widget}{$widget->{s_id}};
156 delete $self->{widgetset}{$widget->{s_ws}}{$widget->{s_id}}
157 if exists $widget->{s_ws};
158 });
159
160 1
161 } else {
162 $self->send_exti_msg (w_e => $id, undef);
163
164 0
165 }
166}
167
168# widgetset new
169sub ext_ws_n {
170 my ($self, $id) = @_;
171
172 $self->{widgetset}{$id} = {
173 w => {},
174 };
175}
176
177# widgetset destroy
178sub ext_ws_d {
179 my ($self, $id) = @_;
180
181 my $ws = delete $self->{widgetset}{$id}
182 or return;
183
184 $_->destroy
185 for values %{$ws->{w}};
186}
187
188# widgetset create
189sub ext_ws_c {
190 my ($self, $ws, $id, $class, $args) = @_;
191
192 for my $ev (grep /^on_/, keys %$args) {
193 my $rid = $args->{$ev};
194 $args->{$ev} = sub {
195 my $id = shift->{s_id};
196 $self->send_exti_msg (w_e => $id, $rid, @_);
197
198 1
199 };
200 }
201
202 $self->widget_associate (
203 $ws, $id => scalar eval {
204 local $SIG{__DIE__};
205 "CFPlus::UI::$class"->new (%$args)
206 }
207 ) or warn "server failed creating client-side widget " . (CFPlus::to_json $class) . ": $@\n";
208}
209
210# widgetset associate
211sub ext_ws_a {
212 my ($self, %ass) = @_;
213
214 # everything that has a name, wether conceivably useful or not
215 my %wkw = (
216 root => $CFPlus::UI::ROOT,
217 tooltip => $CFPlus::UI::TOOLTIP,
218
219 mapwidget => $::MAPWIDGET,
220 buttonbar => $::BUTTONBAR,
221 metaserver => $::METASERVER,
222 buttonbar => $::BUTTONBAR,
223 login_button => $::LOGIN_BUTTON,
224 quit_dialog => $::QUIT_DIALOG,
225 host_entry => $::HOST_ENTRY,
226 metaserver => $::METASERVER,
227 server_info => $::SERVER_INFO,
228
229 setup_dialog => $::SETUP_DIALOG,
230 setup_notebook => $::SETUP_NOTEBOOK,
231 setup_server => $::SETUP_SERVER,
232 setup_keyboard => $::SETUP_KEYBOARD,
233
234 pl_notebook => $::PL_NOTEBOOK,
235 pl_window => $::PL_WINDOW,
236 inventory_page => $::INVENTORY_PAGE,
237 stats_page => $::STATS_PAGE,
238 skill_page => $::SKILL_PAGE,
239 spell_page => $::SPELL_PAGE,
240 spell_list => $::SPELL_LIST,
241
242 floorbox => $::FLOORBOX,
243 help_window => $::HELP_WINDOW,
244 message_window => $::MESSAGE_WINDOW,
245 statusbox => $::SDTATUSBOX,
246
247 inv => $::INV,
248 invr => $::INVR,
249 invr_hb => $::INVR_HB,
250 );
251
252 while (my ($id, $name) = each %ass) {
253 $self->widget_associate (undef, $id => $wkw{$name})
254 or warn "server failed to associate non-existent well-known widget $name\n";
255 }
256}
257
258# widget call
259sub ext_w_c {
260 my ($self, $id, $rid, $method, @args) = @_;
261
262 my $w = $self->{widget}{$id}
263 or return;
264
265 if ($rid) {
266 $self->send_exti_msg (w_r => $rid, $w->$method (@args));
267 } else {
268 $w->$method (@args);
269 }
270}
271
272# widget set
273sub ext_w_s {
274 my ($self, $id, $attr) = @_;
275
276 my $w = $self->{widget}{$id}
277 or return;
278
279 for (my $i = 0; $i < $#$attr; $i += 2) {
280 my ($member, $value) = @$attr[$i, $i+1];
281 if (defined $value) {
282 $w->{$member} = $value;
283 } else {
284 delete $w->{$member};
285 }
286 $w->{parent}->realloc if $member =~ /^c_/ && $w->{visible};
287 }
288}
289
290# widget get
291sub ext_w_g {
292 my ($self, $id, $rid, $attr) = @_;
293
294 my $w = $self->{widget}{$id}
295 or return;
296
297 $self->send_exti_msg (w_r => $rid, [map $w->{$_}, @$attr]);
298}
299
300# message window
301sub ext_channel_info {
302 my ($self, $info) = @_;
303 $self->{channels}->{$info->{id}} = $info;
304 $::MESSAGE_WINDOW->add_channel ($info);
305}
306
307#############################################################################
308
194sub logprint { 309sub logprint {
195 my ($self, @a) = @_; 310 my ($self, @a) = @_;
196 311
197 $self->{log_fh} ||= do { 312 CFPlus::DB::logprint "$Crossfire::VARDIR/log.$self->{host}" => (join "", @a), sub { };
198 my $path = "$Crossfire::VARDIR/log.$self->{host}";
199
200 open my $fh, ">>:utf8", $path
201 or die "Couldn't open logfile $path: $!";
202
203 $fh->autoflush (1);
204
205 $fh;
206 };
207
208 my ($sec, $min, $hour, $mday, $mon, $year) = localtime time;
209
210 my $ts = sprintf "%04d-%02d-%02d %02d:%02d:%02d",
211 $year + 1900, $mon + 1, $mday, $hour, $min, $sec;
212
213 print {$self->{log_fh}} "$ts ", @a, "\n";
214} 313}
215 314
216sub _stat_numdiff { 315sub _stat_numdiff {
217 my ($self, $name, $old, $new) = @_; 316 my ($self, $name, $old, $new) = @_;
218 317
… …
351 $::GAUGES->{mana} ->set_value ($sp, $sp_m); 450 $::GAUGES->{mana} ->set_value ($sp, $sp_m);
352 $::GAUGES->{food} ->set_value ($fo, $fo_m); 451 $::GAUGES->{food} ->set_value ($fo, $fo_m);
353 $::GAUGES->{grace} ->set_value ($gr, $gr_m); 452 $::GAUGES->{grace} ->set_value ($gr, $gr_m);
354 $::GAUGES->{exp} ->set_text ("Exp: " . (::formsep ($stats->{+CS_STAT_EXP64})) 453 $::GAUGES->{exp} ->set_text ("Exp: " . (::formsep ($stats->{+CS_STAT_EXP64}))
355 . " (lvl " . ($stats->{+CS_STAT_LEVEL} * 1) . ")"); 454 . " (lvl " . ($stats->{+CS_STAT_LEVEL} * 1) . ")");
455 $::GAUGES->{prg} ->set_value ($stats->{+CS_STAT_LEVEL}, $stats->{+CS_STAT_EXP64});
356 $::GAUGES->{range} ->set_text ($stats->{+CS_STAT_RANGE}); 456 $::GAUGES->{range} ->set_text ($stats->{+CS_STAT_RANGE});
357 my $title = $stats->{+CS_STAT_TITLE}; 457 my $title = $stats->{+CS_STAT_TITLE};
358 $title =~ s/^Player: //; 458 $title =~ s/^Player: //;
359 $::STATWIDS->{title} ->set_text ("Title: " . $title); 459 $::STATWIDS->{title} ->set_text ("Title: " . $title);
360 460
… …
378 for keys %RES_TBL; 478 for keys %RES_TBL;
379 479
380 my $sktbl = $::STATWIDS->{skill_tbl}; 480 my $sktbl = $::STATWIDS->{skill_tbl};
381 my @skills = keys %{ $self->{skill_info} }; 481 my @skills = keys %{ $self->{skill_info} };
382 482
383 if (grep +(exists $stats->{$_}) != (exists $prev->{$_}), @skills) { 483 my @order = sort { $stats->{$b->[0]}[1] <=> $stats->{$a->[0]}[1] or $a->[1] cmp $b->[1] }
484 map [$_, $self->{skill_info}{$_}],
485 grep exists $stats->{$_},
486 @skills;
487
488 if ($self->{stat_order} ne join ",", map $_->[0], @order) {
489 $self->{stat_order} = join ",", map $_->[0], @order;
490
384 $sktbl->clear; 491 $sktbl->clear;
385 492
493 my $sw = $self->{skillwid}{""} ||= [
386 $sktbl->add (0, 0, new CFPlus::UI::Label text => "Experience", align => 1); 494 0, 0, (new CFPlus::UI::Label text => "Experience", align => 1),
387 $sktbl->add (1, 0, new CFPlus::UI::Label text => "Lvl.", align => 1); 495 1, 0, (new CFPlus::UI::Label text => "Lvl.", align => 1),
496 2, 0, (new CFPlus::UI::Label text => "Progress", align => 0),
388 $sktbl->add (2, 0, new CFPlus::UI::Label text => "Skill", expand => 1); 497 3, 0, (new CFPlus::UI::Label text => "Skill", expand => 1),
389 $sktbl->add (3, 0, new CFPlus::UI::Label text => "Experience", align => 1); 498 4, 0, (new CFPlus::UI::Label text => "Experience", align => 1),
390 $sktbl->add (4, 0, new CFPlus::UI::Label text => "Lvl.", align => 1); 499 5, 0, (new CFPlus::UI::Label text => "Lvl.", align => 1),
500 6, 0, (new CFPlus::UI::Label text => "Progress", align => 0),
391 $sktbl->add (5, 0, new CFPlus::UI::Label text => "Skill", expand => 1); 501 7, 0, (new CFPlus::UI::Label text => "Skill", expand => 1),
502 ];
503
504 my @add = @$sw;
392 505
393 my $TOOLTIP_ALL = "\n\n<small>Left click - ready skill\nMiddle click - use spell\nRight click - further options</small>"; 506 my $TOOLTIP_ALL = "\n\n<small>Left click - ready skill\nMiddle click - use spell\nRight click - further options</small>";
394 507
395 my @TOOLTIP_LVL = (tooltip => "<b>Level</b>. The level of the skill.$TOOLTIP_ALL", can_events => 1, can_hover => 1); 508 my @TOOLTIP_LVL = (tooltip => "<b>Level</b>. The level of the skill.$TOOLTIP_ALL", can_events => 1, can_hover => 1);
396 my @TOOLTIP_EXP = (tooltip => "<b>Experience</b>. The experience points you have in this skill.$TOOLTIP_ALL", can_events => 1, can_hover => 1); 509 my @TOOLTIP_EXP = (tooltip => "<b>Experience</b>. The experience points you have in this skill.$TOOLTIP_ALL", can_events => 1, can_hover => 1);
397 510
398 my ($x, $y) = (0, 1); 511 my ($x, $y) = (0, 1);
399 for ( 512 for (@order) {
400 sort { $stats->{$b->[0]}[1] <=> $stats->{$a->[0]}[1] or $a->[1] cmp $b->[1] }
401 map [$_, $self->{skill_info}{$_}],
402 grep exists $stats->{$_},
403 @skills
404 ) {
405 my ($idx, $name) = @$_; 513 my ($idx, $name) = @$_;
406 514
407 my $spell_cb = sub { 515 my $spell_cb = sub {
408 my ($widget, $ev) = @_; 516 my ($widget, $ev) = @_;
409 517
… …
424 } 532 }
425 533
426 1 534 1
427 }; 535 };
428 536
429 $sktbl->add ($x * 3 + 0, $y, $self->{stat_widget_exp}{$idx} = new CFPlus::UI::Label 537 my $sw = $self->{skillwid}{$idx} ||= [
538 # exp
539 (new CFPlus::UI::Label
430 text => "0", align => 1, font => $::FONT_FIXED, fg => [1, 1, 0], on_button_down => $spell_cb, @TOOLTIP_EXP); 540 align => 1, font => $::FONT_FIXED, fg => [1, 1, 0], on_button_down => $spell_cb, @TOOLTIP_EXP),
431 $sktbl->add ($x * 3 + 1, $y, $self->{stat_widget_lvl}{$idx} = new CFPlus::UI::Label 541
542 # level
543 (new CFPlus::UI::Label
432 text => "0", align => 1, font => $::FONT_FIXED, fg => [0, 1, 0], padding_x => 4, on_button_down => $spell_cb, @TOOLTIP_LVL); 544 text => "0", align => 1, font => $::FONT_FIXED, fg => [0, 1, 0], padding_x => 4, on_button_down => $spell_cb, @TOOLTIP_LVL),
545
546 # progress
547 (new CFPlus::UI::ExperienceProgress),
548
549 # label
433 $sktbl->add ($x * 3 + 2, $y, new CFPlus::UI::Label text => $name, on_button_down => $spell_cb, 550 (new CFPlus::UI::Label text => $name, on_button_down => $spell_cb,
434 can_events => 1, can_hover => 1, tooltip => (CFPlus::Pod::section_label skill_description => $name) . $TOOLTIP_ALL); 551 can_events => 1, can_hover => 1, tooltip => (CFPlus::Pod::section_label skill_description => $name) . $TOOLTIP_ALL),
552 ];
553
554 push @add,
555 $x * 4 + 0, $y, $sw->[0],
556 $x * 4 + 1, $y, $sw->[1],
557 $x * 4 + 2, $y, $sw->[2],
558 $x * 4 + 3, $y, $sw->[3],
559 ;
435 560
436 $x++ and ($x, $y) = (0, $y + 1); 561 $x++ and ($x, $y) = (0, $y + 1);
437 } 562 }
563
564 $sktbl->add_at (@add);
438 } 565 }
439 566
440 for (grep exists $stats->{$_}, @skills) { 567 for (grep exists $stats->{$_}, @skills) {
568 next if $prev->{$_} == $stats->{$_};
569
570 my $sw = $self->{skillwid}{$_};
441 $self->{stat_widget_exp}{$_}->set_text (::formsep ($stats->{$_}[1])); 571 $sw->[0]->set_text (::formsep ($stats->{$_}[1]));
442 $self->{stat_widget_lvl}{$_}->set_text ($stats->{$_}[0] * 1); 572 $sw->[1]->set_text ($stats->{$_}[0] * 1);
573 $sw->[2]->set_value (@{$stats->{$_}});
443 } 574 }
444} 575}
445 576
446sub macro_send { 577sub macro_send {
447 my ($self, $macro) = @_; 578 my ($self, $macro) = @_;
… …
475} 606}
476 607
477sub feed_map1a { 608sub feed_map1a {
478 my ($self, $data) = @_; 609 my ($self, $data) = @_;
479 610
480 $self->{map}->map1a_update ($data, $self->{setup}{extmap}); 611 my $missing = $self->{map}->map1a_update ($data, $self->{setup}{extmap});
612 my $delay;
613
614 for my $tile (@$missing) {
615 next if $self->{delay}{$tile};
616
617 $delay = 1;
618
619 if (my $tex = $::CONN->{texture}[$tile]) {
620 $tex->upload;
621 } else {
622 $self->{delay}{$tile} = 1;
623
624 # we assume the face is in-flight and will eventually come
625 push @{$self->{tile_cb}{$tile}}, sub {
626 delete $self->{delay}{$tile};
627 $_[0]->upload;
628 };
629 }
630 }
631
632 if ($delay) {
633 # delay the map drawing a tiny bit in the hope of getting the missing fetched
634 Event->timer (after => 0.03, cb => sub {
635 $_[0]->w->cancel;
636 $self->{map_widget}->update
637 if $self->{map_widget};
638 });
639 } else {
481 $self->{map_widget}->update; 640 $self->{map_widget}->update;
641 }
482} 642}
483 643
484sub magicmap { 644sub magicmap {
485 my ($self, $w, $h, $x, $y, $data) = @_; 645 my ($self, $w, $h, $x, $y, $data) = @_;
486 646
… …
493 my $map_info = delete $self->{map_info} 653 my $map_info = delete $self->{map_info}
494 or return; 654 or return;
495 655
496 my ($hash, $x, $y, $w, $h) = @$map_info; 656 my ($hash, $x, $y, $w, $h) = @$map_info;
497 657
498 my $data = $self->{map}->get_rect ($x, $y, $w, $h); 658 my $data = Compress::LZF::compress $self->{map}->get_rect ($x, $y, $w, $h);
659 $self->{map_cache_new}{$hash} = \$data;
499 CFPlus::DB::put $self->{mapcache} => $hash => Compress::LZF::compress $data, sub { }; 660 CFPlus::DB::put $self->{mapcache} => $hash => $data, sub { };
500 #warn sprintf "SAVEmap[%s] length %d\n", $hash, length $data;#d#
501} 661}
502 662
503sub map_clear { 663sub map_clear {
504 my ($self) = @_; 664 my ($self) = @_;
505 665
… …
511} 671}
512 672
513sub bg_fetch { 673sub bg_fetch {
514 my ($self) = @_; 674 my ($self) = @_;
515 675
516 my $id; 676 my $tile;
517 677
518 do { 678 do {
519 $id = pop @{$self->{bg_fetch}} 679 $tile = pop @{$self->{bg_fetch}}
520 or return; 680 or return;
521 } while $self->{texture}[$id]; 681 } while $self->{texture}[$tile];
522 682
523 CFPlus::DB::get tilecache => $id, sub { 683 CFPlus::DB::get tilecache => $tile, sub {
524 my ($data) = @_; 684 my ($data) = @_;
525 685
526 return unless $self->{map}; # stop when destroyed 686 return unless $self->{map}; # stop when destroyed
527 687
528 $self->set_texture ($id => $data)
529 if defined $data; 688 if (defined $data) {
689 $self->have_tile ($tile, $data);
690 $self->{texture}[$tile]->upload;
691 }
530 692
531 $self->bg_fetch; 693 $self->bg_fetch;
532 }; 694 };
533} 695}
534 696
535sub load_map($$$) { 697sub load_map($$$) {
536 my ($self, $hash, $x, $y) = @_; 698 my ($self, $hash, $x, $y) = @_;
537 699
538 my $gen = $self->{map_change_gen}; 700 my $gen = $self->{map_change_gen};
539 701
540 CFPlus::DB::get $self->{mapcache} => $hash, sub { 702 my $cb = sub {
541 return unless $gen == $self->{map_change_gen}; 703 return unless $gen == $self->{map_change_gen};
542 704
543 my ($data) = @_; 705 my ($data) = @_;
544 706
545 if (defined $data) { 707 if (defined $data) {
708 $self->{map_cache_new}{$hash} = \$data;
709
546 $data = Compress::LZF::decompress $data; 710 my $data = Compress::LZF::decompress $data;
547 #warn sprintf "LOADmap[%s,%d,%d] length %d\n", $hash, $x, $y, length $data;#d#
548 711
549 my $inprogress = @{ $self->{bg_fetch} || [] }; 712 my $inprogress = @{ $self->{bg_fetch} || [] };
550 unshift @{ $self->{bg_fetch} }, $self->{map}->set_rect ($x, $y, $data); 713 unshift @{ $self->{bg_fetch} }, $self->{map}->set_rect ($x, $y, $data);
551 $self->bg_fetch unless $inprogress; 714 $self->bg_fetch unless $inprogress;
552 } 715 }
553 }; 716 };
717
718 if (my $rdata = $self->{map_cache_old}{$hash}) {
719 $cb->($$rdata);
720 } else {
721 CFPlus::DB::get $self->{mapcache} => $hash, $cb;
722 }
554} 723}
555 724
556# hardcode /world/world_xxx_xxx map names, the savings are enourmous, 725# hardcode /world/world_xxx_xxx map names, the savings are enourmous,
557# (server resource,s latency, bandwidth), so this hack is warranted. 726# (server resource,s latency, bandwidth), so this hack is warranted.
558# the right fix is to make real tiled maps with an overview file 727# the right fix is to make real tiled maps with an overview file
… …
651 my ($self, $mode, $flags, $x, $y, $w, $h, $hash) = @_; 820 my ($self, $mode, $flags, $x, $y, $w, $h, $hash) = @_;
652 821
653 $self->flush_map; 822 $self->flush_map;
654 823
655 ++$self->{map_change_gen}; 824 ++$self->{map_change_gen};
825 $self->{map_cache_old} = delete $self->{map_cache_new};
656 826
657 my ($ox, $oy) = ($::MAP->ox, $::MAP->oy); 827 my ($ox, $oy) = ($::MAP->ox, $::MAP->oy);
658 828
659 my $mapmapw = $self->{mapmap}->{w}; 829 my $mapmapw = $self->{mapmap}->{w};
660 my $mapmaph = $self->{mapmap}->{h}; 830 my $mapmaph = $self->{mapmap}->{h};
… …
679} 849}
680 850
681sub face_find { 851sub face_find {
682 my ($self, $facenum, $face, $cb) = @_; 852 my ($self, $facenum, $face, $cb) = @_;
683 853
684 my $hash = "$face->{chksum},$face->{name}"; 854 if ($face->{type} == 0) { # FT_FACE
685
686 my $id = CFPlus::DB::get_tile_id_sync $hash; 855 my $id = CFPlus::DB::get_tile_id_sync $face->{name};
687 856
688 $face->{id} = $id; 857 $face->{id} = $id;
689 $self->{faceid}[$facenum] = $id;
690
691 $self->{map}->set_tileid ($facenum => $id); 858 $self->{map}->set_tileid ($facenum => $id);
692 859
693 CFPlus::DB::get tilecache => $id, $cb; 860 CFPlus::DB::get tilecache => $id, $cb;
861
862 } elsif ($face->{type} & 1) { # with metadata
863 CFPlus::DB::get res_meta => $face->{name}, $cb;
864
865 } else { # no metadata
866 CFPlus::DB::get res_data => $face->{name}, $cb;
867 }
694} 868}
695 869
696sub face_update { 870sub face_update {
697 my ($self, $facenum, $face, $changed) = @_; 871 my ($self, $facenum, $face, $changed) = @_;
698 872
873 if ($face->{type} == 0) {
874 # image, FT_FACE
699 CFPlus::DB::put tilecache => $face->{id} => $face->{image}, sub { } 875 CFPlus::DB::put tilecache => $face->{id} => $face->{data}, sub { }
700 if $changed; 876 if $changed;
701 877
702 $self->set_texture ($face->{id} => delete $face->{image}); 878 $self->have_tile ($face->{id}, delete $face->{data});
879
880 } elsif ($face->{type} & 1) {
881 # split metadata case, FT_MUSIC, FT_SOUND
882 if ($changed) { # new data
883 my ($meta, $data) = unpack "(w/a*)*", $face->{data};
884 $face->{data} = $meta;
885
886 CFPlus::DB::put res_data => $face->{name} => $data, sub { };
887 CFPlus::DB::put res_meta => $face->{name} => $meta, sub { };
888 }
889
890 $face->{data} = $self->{json_coder}->decode ($face->{data});
891
892 ::add_license ($face);
893
894 if ($face->{type} == 3) { # FT_MUSIC
895 ::message ({ markup => "downloaded song #$facenum" })
896 if $changed;
897
898 &::audio_music_push ($facenum);
899 } elsif ($face->{type} == 5) { # FT_SOUND
900 ::message ({ markup => "downloaded sound #$facenum" })
901 if $changed;
902
903 &::audio_sound_push ($facenum);
904 }
905
906 } else {
907 # flat resource case, FT_RSRC
908 CFPlus::DB::put res_data => $face->{name} => $face->{data}, sub { }
909 if $changed;
910 }
911
912 if (my $cbs = $self->{face_cb}{$facenum}) {
913 $_->($face, $changed) for @$cbs;
914 }
703} 915}
704 916
705sub smooth_update { 917sub smooth_update {
706 my ($self, $facenum, $face) = @_; 918 my ($self, $facenum, $face) = @_;
707 919
708 $self->{map}->set_smooth ($facenum, $face->{smoothface}, $face->{smoothlevel}); 920 $self->{map}->set_smooth ($facenum, $face->{smoothface}, $face->{smoothlevel});
709} 921}
710 922
711sub set_texture { 923sub have_tile {
712 my ($self, $id, $data) = @_; 924 my ($self, $tile, $data) = @_;
713 925
714 $self->{texture}[$id] = my $tex = 926 return unless $self->{map};
927
928 my $tex = $self->{texture}[$tile] ||=
715 new_from_image CFPlus::Texture 929 new CFPlus::Texture
930 tile => $tile,
931 image => $data, delete_image => 1,
716 $data, minify => 1, mipmap => 1; 932 minify => 1, mipmap => 1;
717 933
718 $self->{map}->set_texture ($id, @$tex{qw(name w h s t)}, @{$tex->{minified}}); 934 if (my $cbs = delete $self->{tile_cb}{$tile}) {
719 $self->{map_widget}->update; 935 $_->($tex) for @$cbs;
936 }
937}
938
939# call in non-void context registers a temporary
940# hook with handle, otherwise its permanent
941sub on_face_change {
942 my ($self, $num, $cb) = @_;
943
944 push @{$self->{face_cb}{$num}}, $cb;
945
946 defined wantarray
947 ? CFPlus::guard {
948 @{$self->{face_cb}{$num}}
949 = grep $_ != $cb,
950 @{$self->{face_cb}{$num}};
951 }
952 : ()
953}
954
955# call in non-void context registers a temporary
956# hook with handle, otherwise its permanent
957sub register_face_handler {
958 my ($self, $num, $cb) = @_;
959
960 return unless $num;
961
962 # invoke if available right now
963 $cb->($self->{face}[$num], 0)
964 unless exists $self->{face}[$num]{loading};
965
966 # future changes
967 $self->on_face_change ($num => $cb)
720} 968}
721 969
722sub sound_play { 970sub sound_play {
723 my ($self, $x, $y, $soundnum, $type) = @_; 971 my ($self, $type, $face, $dx, $dy, $vol) = @_;
724 972
725 $self->{sound_play}->($x, $y, $soundnum, $type); 973 &::audio_sound_play ($face, $dx, $dy, $vol)
974 unless $type & 1; # odd types are silent for future expansion
726} 975}
727 976
728my $LAST_QUERY; # server is stupid, stupid, stupid 977my $LAST_QUERY; # server is stupid, stupid, stupid
729 978
730sub query { 979sub query {
… …
795 $self->{$type} = $text; 1044 $self->{$type} = $text;
796 } else { 1045 } else {
797 $self->logprint ("msg: ", $text); 1046 $self->logprint ("msg: ", $text);
798 return if $color < 0; # negative color == ignore if not understood 1047 return if $color < 0; # negative color == ignore if not understood
799 1048
800 my $fg = $CF_COLOR[$color % @CF_COLOR]; 1049 my $fg = $CF_COLOR[$color & NDI_COLOR_MASK] || [1, 0, 0];
801 1050
802 ## try to create single paragraphs of multiple lines sent by the server 1051 ## try to create single paragraphs of multiple lines sent by the server
803 # no longer neecssary with TRT servers 1052 # no longer neecssary with TRT servers
804 #$text =~ s/(?<=\S)\n(?=\w)/ /g; 1053 #$text =~ s/(?<=\S)\n(?=\w)/ /g;
805 1054
806 ::message ({ fg => $fg, markup => $_ }) 1055 ::message ({
1056 fg => $fg,
1057 markup => $_,
1058 type => $type,
1059 extra => [@extra],
1060 color_flags => $color
807 for split /\n/, $text; 1061 }) for split /\n/, $text;
808 1062
809 $self->{statusbox}->add ($text, 1063 $self->{statusbox}->add ($text,
810 group => $text, 1064 group => $text,
811 fg => $fg, 1065 fg => $fg,
812 timeout => $color >= 2 ? 180 : 10, 1066 timeout => $color >= 2 ? 180 : 10,
… …
868 $self->{map_widget}->clr_commands; 1122 $self->{map_widget}->clr_commands;
869 1123
870 ::stop_game (); 1124 ::stop_game ();
871} 1125}
872 1126
873sub image_info {
874 my ($self, $numfaces) = @_;
875
876 $self->{num_faces} = $numfaces;
877 $self->{face_prefetch} = [1 .. $numfaces];
878 $self->face_prefetch;
879}
880
881sub face_prefetch {
882 my ($self) = @_;
883
884 return unless $::CFG->{face_prefetch};
885
886 if ($self->{num_faces}) {
887 return if @{ $self->{send_queue} || [] };
888 my $todo = @{ $self->{face_prefetch} }
889 or return;
890
891 my ($face) = splice @{ $self->{face_prefetch} }, + rand @{ $self->{face_prefetch} }, 1, ();
892
893 $self->send ("requestinfo image_sums $face $face");
894
895 $self->{statusbox}->add (CFPlus::asxml "prefetching $todo",
896 group => "prefetch", timeout => 3, fg => [1, 1, 0, 0.5]);
897 } elsif (!exists $self->{num_faces}) {
898 $self->send ("requestinfo image_info");
899
900 $self->{num_faces} = 0;
901
902 $self->{statusbox}->add (CFPlus::asxml "starting to prefetch",
903 group => "prefetch", timeout => 3, fg => [1, 1, 0, 0.5]);
904 }
905}
906
907sub update_floorbox { 1127sub update_floorbox {
908 $CFPlus::UI::ROOT->on_refresh ($::FLOORBOX => sub { 1128 $CFPlus::UI::ROOT->on_refresh ($::FLOORBOX => sub {
909 return unless $::CONN; 1129 return unless $::CONN;
910 1130
911 $::FLOORBOX->clear; 1131 $::FLOORBOX->clear;
1132
1133 my @add;
912 1134
913 my $row; 1135 my $row;
914 for (sort { $a->{count} <=> $b->{count} } values %{ $::CONN->{container}{$::CONN->{open_container} || 0} }) { 1136 for (sort { $a->{count} <=> $b->{count} } values %{ $::CONN->{container}{$::CONN->{open_container} || 0} }) {
915 if ($row < 6) { 1137 if ($row < 6) {
916 local $_->{face_widget}; # hack to force recreation of widget 1138 local $_->{face_widget}; # hack to force recreation of widget
917 local $_->{desc_widget}; # hack to force recreation of widget 1139 local $_->{desc_widget}; # hack to force recreation of widget
918 CFPlus::Item::update_widgets $_; 1140 CFPlus::Item::update_widgets $_;
919 1141
1142 push @add,
920 $::FLOORBOX->add (0, $row, $_->{face_widget}); 1143 0, $row, $_->{face_widget},
921 $::FLOORBOX->add (1, $row, $_->{desc_widget}); 1144 1, $row, $_->{desc_widget};
922 1145
923 $row++; 1146 $row++;
924 } else { 1147 } else {
925 $::FLOORBOX->add (1, $row, new CFPlus::UI::Button 1148 push @add, 1, $row, new CFPlus::UI::Button
926 text => "More...", 1149 text => "More...",
927 on_activate => sub { ::toggle_player_page ($::INVENTORY_PAGE); 0 }, 1150 on_activate => sub { ::toggle_player_page ($::INVENTORY_PAGE); 0 },
928 ); 1151 ;
929 last; 1152 last;
930 } 1153 }
931 } 1154 }
932 if ($::CONN->{open_container}) { 1155 if ($::CONN->{open_container}) {
933 $::FLOORBOX->add (1, $row++, new CFPlus::UI::Button 1156 push @add, 1, $row++, new CFPlus::UI::Button
934 text => "Close container", 1157 text => "Close container",
935 on_activate => sub { $::CONN->send ("apply $::CONN->{open_container}") } 1158 on_activate => sub { $::CONN->send ("apply $::CONN->{open_container}") }
936 ); 1159 ;
937 } 1160 }
1161
1162 $::FLOORBOX->add_at (@add);
938 }); 1163 });
939 1164
940 $::WANT_REFRESH++; 1165 $::WANT_REFRESH->start;
941} 1166}
942 1167
943sub set_opencont { 1168sub set_opencont {
944 my ($conn, $tag, $name) = @_; 1169 my ($conn, $tag, $name) = @_;
945 $conn->{open_container} = $tag; 1170 $conn->{open_container} = $tag;
946 update_floorbox; 1171 update_floorbox;
947 1172
948 $::INV_RIGHT_HB->clear (); 1173 $::INVR_HB->clear ();
949 $::INV_RIGHT_HB->add (new CFPlus::UI::Label align => 0, expand => 1, text => $name); 1174 $::INVR_HB->add (new CFPlus::UI::Label align => 0, expand => 1, text => $name);
950 1175
951 if ($tag != 0) { # Floor isn't closable, is it? 1176 if ($tag != 0) { # Floor isn't closable, is it?
952 $::INV_RIGHT_HB->add (new CFPlus::UI::Button 1177 $::INVR_HB->add (new CFPlus::UI::Button
953 text => "Close container", 1178 text => "Close container",
954 tooltip => "Close the currently open container (if one is open)", 1179 tooltip => "Close the currently open container (if one is open)",
955 on_activate => sub { 1180 on_activate => sub {
956 $::CONN->send ("apply $tag") # $::CONN->{open_container}") 1181 $::CONN->send ("apply $tag") # $::CONN->{open_container}")
957 if $tag != 0; 1182 if $tag != 0;
958 #if $CONN->{open_container} != 0;
959 0 1183 0
960 }, 1184 },
961 ); 1185 );
962 } 1186 }
963 1187
… …
1062 . "cfplus support $yesno[$self->{cfplus_ext} > 0]" 1286 . "cfplus support $yesno[$self->{cfplus_ext} > 0]"
1063 . ($self->{cfplus_ext} > 0 ? ", version $self->{cfplus_ext}" : "") ."\n" 1287 . ($self->{cfplus_ext} > 0 ? ", version $self->{cfplus_ext}" : "") ."\n"
1064 . "map size $self->{mapw}×$self->{maph}\n" 1288 . "map size $self->{mapw}×$self->{maph}\n"
1065 ); 1289 );
1066 1290
1067 ::setup_build_button ($self->{editor_support}->{builder_ui});
1068} 1291}
1069 1292
1070sub logged_in { 1293sub logged_in {
1071 my ($self) = @_; 1294 my ($self) = @_;
1072 1295
1073 $self->send_ext_req (cfplus_support => version => 1, sub { 1296 $self->send_ext_req (cfplus_support => version => 2, sub {
1297 my (%msg) = @_;
1298
1074 $self->{cfplus_ext} = $_[0]{version}; 1299 $self->{cfplus_ext} = $msg{version};
1075 $self->update_server_info; 1300 $self->update_server_info;
1076 1301
1077 if ($self->{cfplus_ext} >= 2) { 1302 if ($self->{cfplus_ext} >= 2) {
1078 $self->send_ext_req ("editor_support", sub { 1303 $self->send_ext_req ("editor_support", sub {
1079 $self->{editor_support} = $_[0]; 1304 $self->{editor_support} = { @_ };
1080 $self->update_server_info; 1305 $self->update_server_info;
1081 1306
1082 0 1307 0
1083 }); 1308 });
1084 } 1309 }
… …
1092 $self->send_command ("output-count $::CFG->{output_count}"); 1317 $self->send_command ("output-count $::CFG->{output_count}");
1093 $self->send_command ("output-rate $::CFG->{output_rate}") if $::CFG->{output_rate} > 0; 1318 $self->send_command ("output-rate $::CFG->{output_rate}") if $::CFG->{output_rate} > 0;
1094 $self->send_command ("pickup $::CFG->{pickup}"); 1319 $self->send_command ("pickup $::CFG->{pickup}");
1095} 1320}
1096 1321
1097sub buildat {
1098 my ($self, $builditem, $x, $y) = @_;
1099
1100 if ($self->{cfplus_ext}) {
1101 $self->send_ext_msg (builder_build => dx => $x, dy => $y, (ref ($builditem) eq 'HASH') ? %$builditem : (item => $builditem));
1102 }
1103}
1104
1105sub lookat { 1322sub lookat {
1106 my ($self, $x, $y) = @_; 1323 my ($self, $x, $y) = @_;
1107 1324
1108 if ($self->{cfplus_ext}) { 1325 if ($self->{cfplus_ext}) {
1109 $self->send_ext_req (lookat => dx => $x, dy => $y, sub { 1326 $self->send_ext_req (lookat => $x, $y, sub {
1110 my ($msg) = @_; 1327 my (%msg) = @_;
1111 1328
1112 if (exists $msg->{npc_dialog}) { 1329 if (exists $msg{npc_dialog}) {
1113 # start npc chat dialog 1330 # start npc chat dialog
1114 $self->{npc_dialog} = new CFPlus::NPCDialog:: 1331 $self->{npc_dialog} = new CFPlus::NPCDialog::
1115 dx => $x, 1332 token => $msg{npc_dialog},
1116 dy => $y,
1117 title => "$msg->{npc_dialog} (NPC)", 1333 title => "$msg{npc_dialog}[0] (NPC)",
1118 conn => $self, 1334 conn => $self,
1119 ; 1335 ;
1120 } 1336 }
1121 }); 1337 });
1122 } 1338 }
… …
1129 1345
1130 (delete $self->{npc_dialog})->destroy 1346 (delete $self->{npc_dialog})->destroy
1131 if $self->{npc_dialog}; 1347 if $self->{npc_dialog};
1132 1348
1133 $self->SUPER::destroy; 1349 $self->SUPER::destroy;
1350
1351 %$self = ();
1134} 1352}
1135 1353
1136package CFPlus::NPCDialog; 1354package CFPlus::NPCDialog;
1137 1355
1138our @ISA = 'CFPlus::UI::Toplevel'; 1356our @ISA = 'CFPlus::UI::Toplevel';
… …
1185 on_activate => sub { $this->destroy; 1 }, 1403 on_activate => sub { $this->destroy; 1 },
1186 ; 1404 ;
1187 1405
1188 $self->update_options; 1406 $self->update_options;
1189 1407
1190 $self->{id} = $self->{conn}->send_ext_req ( 1408 $self->{id} = "npc-channel-" . $self->{conn}->token;
1191 npc_dialog_begin => dx => $self->{dx}, dy => $self->{dy}, 1409 $self->{conn}->connect_ext ($self->{id} => sub {
1192 sub { $this && $this->feed (@_) } 1410 $this->feed (@_) if $this;
1193 ); 1411 });
1412
1413 $self->{conn}->send_ext_msg (npc_dialog_begin => $self->{id}, $self->{token});
1194 1414
1195 $self->{entry}->grab_focus; 1415 $self->{entry}->grab_focus;
1196 1416
1197 $self->{textview}->add_paragraph ({ 1417 $self->{textview}->add_paragraph ({
1198 fg => [1, 1, 0, 1], 1418 fg => [1, 1, 0, 1],
… …
1221 ); 1441 );
1222 } 1442 }
1223} 1443}
1224 1444
1225sub feed { 1445sub feed {
1226 my ($self, $msg) = @_; 1446 my ($self, $type, @arg) = @_;
1227 1447
1228 CFPlus::weaken $self; 1448 CFPlus::weaken $self;
1229 1449
1230 if ($msg->{msgtype} eq "reply") { 1450 if ($type eq "update") {
1451 my (%info) = @arg;
1452
1231 $self->{kw}{$_} = 1 for @{$msg->{add_topics} || []}; 1453 $self->{kw}{$_} = 1 for @{$info{add_topics} || []};
1232 $self->{kw}{$_} = 0 for @{$msg->{del_topics} || []}; 1454 $self->{kw}{$_} = 0 for @{$info{del_topics} || []};
1233
1234 my $text = "\n" . CFPlus::Protocol::sanitise_xml $msg->{msg};
1235 my $match = join "|", map "\\b\Q$_\E\\b", sort { (length $b) <=> (length $a) } keys %{ $self->{kw} };
1236 my @link;
1237 $text =~ s{
1238 ($match)
1239 }{
1240 my $kw = $1;
1241
1242 push @link, new CFPlus::UI::Label
1243 markup => "<span foreground='#c0c0ff' underline='single'>$kw</span>",
1244 can_hover => 1,
1245 can_events => 1,
1246 padding_x => 0,
1247 padding_y => 0,
1248 on_button_up => sub {
1249 $self->send ($kw);
1250 };
1251
1252 "\x{fffc}"
1253 }giex;
1254 1455
1456 if (exists $info{msg}) {
1457 my $text = "\n" . CFPlus::Protocol::sanitise_xml $info{msg};
1458 my $match = join "|", map "\\b\Q$_\E\\b", sort { (length $b) <=> (length $a) } keys %{ $self->{kw} };
1459 my @link;
1460 $text =~ s{
1461 ($match)
1462 }{
1463 my $kw = $1;
1464
1465 push @link, new CFPlus::UI::Label
1466 markup => "<span foreground='#c0c0ff' underline='single'>$kw</span>",
1467 can_hover => 1,
1468 can_events => 1,
1469 padding_x => 0,
1470 padding_y => 0,
1471 on_button_up => sub {
1472 $self->send ($kw);
1473 };
1474
1475 "\x{fffc}"
1476 }giex;
1477
1255 $self->{textview}->add_paragraph ({ markup => $text, widget => \@link }); 1478 $self->{textview}->add_paragraph ({ markup => $text, widget => \@link });
1256 $self->{textview}->scroll_to_bottom; 1479 $self->{textview}->scroll_to_bottom;
1480 }
1481
1257 $self->update_options; 1482 $self->update_options;
1258 } else { 1483 } else {
1259 $self->destroy; 1484 $self->destroy;
1260 } 1485 }
1261 1486
… …
1266 my ($self, $msg) = @_; 1491 my ($self, $msg) = @_;
1267 1492
1268 $self->{textview}->add_paragraph ({ markup => "\n" . CFPlus::asxml $msg }); 1493 $self->{textview}->add_paragraph ({ markup => "\n" . CFPlus::asxml $msg });
1269 $self->{textview}->scroll_to_bottom; 1494 $self->{textview}->scroll_to_bottom;
1270 1495
1271 $self->{conn}->send_ext_msg (npc_dialog_tell => msgid => $self->{id}, msg => $msg); 1496 $self->{conn}->send_ext_msg (npc_dialog_tell => $self->{id}, $msg);
1272} 1497}
1273 1498
1274sub destroy { 1499sub destroy {
1275 my ($self) = @_; 1500 my ($self) = @_;
1276 1501
1277 #Carp::cluck "debug\n";#d# #todo# enable: destroy gets called twice because scalar keys {} is 1 1502 #Carp::cluck "debug\n";#d# #todo# enable: destroy gets called twice because scalar keys {} is 1
1278 1503
1279 if ($self->{conn}) { 1504 if ($self->{conn}) {
1280 $self->{conn}->send_ext_msg (npc_dialog_end => msgid => $self->{id}) if $self->{id}; 1505 $self->{conn}->send_ext_msg (npc_dialog_end => $self->{id}) if $self->{id};
1281 delete $self->{conn}{npc_dialog}; 1506 delete $self->{conn}{npc_dialog};
1282 $self->{conn}->disconnect_ext ($self->{id}); 1507 $self->{conn}->disconnect_ext ($self->{id});
1283 } 1508 }
1284 1509
1285 $self->SUPER::destroy; 1510 $self->SUPER::destroy;

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines