--- deliantra/server/ext/jeweler.ext	2007/07/31 09:40:15	1.12
+++ deliantra/server/ext/jeweler.ext	2010/04/09 18:29:19	1.20
@@ -1,9 +1,9 @@
 #! perl
 
-use Data::Dumper;
+use strict;
+
 use Jeweler;
 use List::Util qw/max min sum/;
-use strict;
 
 sub ingred_alias {
    my ($ing) = @_;
@@ -50,7 +50,7 @@
    my @rings = map { Jeweler::Object->new (object => $_) } @ring;
 
    @rings >= 2
-      or return $pl->reply (undef, "You slap yourself, you forgot to put at least 2 jewels in!");
+      or return $pl->message ("You slap yourself, you forgot to put at least 2 jewels in!");
 
    my $input_level = 0;
    my $value;
@@ -63,7 +63,7 @@
    $ring->improve_by_ring (@rings);
 
    if ($do_analyze) {
-      $pl->reply (undef, "You want to make a " . $ring->to_string . ": " . $ring->analyze ($sk, $pl, $input_level));
+      $pl->message ("You want to make a " . $ring->to_string . ": " . $ring->analyze ($sk, $pl, $input_level));
       $ring->wiz_analyze ($pl)
          if $pl->flag (cf::FLAG_WIZ);
       return;
@@ -82,7 +82,7 @@
 
    my $ch = $ring->get_chance_perc ($sk);
    my $succ = 0;
-   my $r = cf::random_roll (0, 100, $pl, cf::PREFER_HIGH);
+   my $r = cf::random_roll (0, 100, $pl, cf::PREFER_LOW);
 
    my $make_status;
    my $exp;
@@ -91,7 +91,8 @@
       $exp = $ring->projected_exp ($input_level);
 
       $pl->change_exp ($exp, "jeweler", cf::SK_EXP_SKILL_ONLY);
-      $pl->message ("You succeed and get $exp experience points.");
+      $pl->message (
+         "You succeed and get " . int ($exp) . " experience points.");
       $make_status = "succeeded";
 
       $ring->set_value ($value * 0.8); # 20% of the input values will vanish
@@ -114,9 +115,6 @@
             $ring->power_to_level, $exp, $make_status;
 
       warn "$make_info\n" if $make_status eq 'succeeded';
-
-      $ring_ob->set_ob_key_value (ext_jeweler_made_by   => $pl->name);
-      $ring_ob->set_ob_key_value (ext_jeweler_make_info => $make_info);
    }
 
    $chdl->put ($ring_ob);
@@ -154,6 +152,7 @@
                      $cfg->{connection},
                      $cfg->{state}
                   );
+                  $obj->decrease (1);
                   cf::override;
                }
             }
@@ -198,7 +197,9 @@
                return
             }
 
-            Jeweler::simple_converter ($player, $ingred, $chdl, $1);
+            Jeweler::simple_converter (
+               $player, $ingred, $chdl, $1,
+               cf::exp_to_level ($sk->stats->exp));
 
          } elsif ($msg =~ m/^\s*merge\s*analy[sz]e\s*$/i) {
             merge ($chdl, $sk, $pl, 1);
@@ -214,16 +215,15 @@
             if ($plan) {
                my @ring = $ingred->get_ring;
 
-               if ((@ring > 1) || ($ring[0]->nrof > 1)) {
-                  # actually the algorithm cant handle more than one improvement at a time
-                  $pl->message ("You can't manage to improve more than one thing at a time!");
-                  return;
-
-               } elsif (@ring < 1) {
+               if (!@ring) {
                   # actually the algorithm cant
                   $pl->message ("You slap yourself, you forgot the jewelery!");
                   return;
 
+               } elsif ((@ring > 1) || (grep { $_->nrof > 1 } @ring)) {
+                  # actually the algorithm cant handle more than one improvement at a time
+                  $pl->message ("You can't manage to improve more than one thing at a time!");
+                  return;
                } else {
                   my $ringo = Jeweler::Object->new (object => $ring[0]);
                   my $iring = $ingred->improve_ring_by_plan ($plan, $ringo);
@@ -275,5 +275,5 @@
    }
 );
 
-Jeweler::read_config "$DATADIR/jeweler.yaml";
+Jeweler::read_config "res/jeweler.yaml";