html menü rewrite v1

[mfinanz.git] / SL / Menu.pm

diff --git a/SL/Menu.pm b/SL/Menu.pm
index 9fcf15374eb3babffd34ac5d408db0b7d28b4269..bc1b4e753182e07db5eaff4bbb2798a75af91e4d 100644 (file)
--- a/SL/Menu.pm
+++ b/SL/Menu.pm
@@ -63,46 +63,30 @@ sub menuitem {
 
   my ($self, $myconfig, $form, $item) = @_;
 
 
   my ($self, $myconfig, $form, $item) = @_;
 

-  my $module = $form->{script};
-  my $action = "section_menu";
-  my $target = "";
-
-  if ($self->{$item}{module}) {
-    $module = $self->{$item}{module};
-  }
-  if ($self->{$item}{action}) {
-    $action = $self->{$item}{action};
-  }
-  if ($self->{$item}{target}) {
-    $target = $self->{$item}{target};
-  }
-
-  my $level = $form->escape($item);
-
-  my $str = qq|<a style="vertical-align:top" href=$module?action=$action&level=$level|;
-
-  my @vars = qw(module action target href);
-
-  if ($self->{$item}{href}) {
-    $str  = qq|<a href=$self->{$item}{href}|;
-    @vars = qw(module target href);
-  }
-
-  map { delete $self->{$item}{$_} } @vars;
-
+  my $module = $self->{$item}{module} || $form->{script};
+  my $action = $self->{$item}{action} || "section_menu";
+  my $target = $self->{$item}{target} || "";
+  my $level  = $form->escape($item);
+  my $target_token = ($target) ? "target='$target'" : '';
+
+  my $href = ($self->{$item}{href})
+           ? $form->escape($self->{$item}{href})
+           : "$module?action=$action";
+
+#  my @vars = ($self->{$item}{href})
+#           ? qw(module        target href)
+#           : qw(module action target href);
+#

   # add other params
   foreach my $key (keys %{ $self->{$item} }) {
   # add other params
   foreach my $key (keys %{ $self->{$item} }) {

-    $str .= "&" . $form->escape($key, 1) . "=";
+    next if $key =~ /target|module|action/;
+    $href .= "&amp;" . $form->escape($key, 1) . "=";

     my ($value, $conf) = split(/=/, $self->{$item}{$key}, 2);
     $value = $myconfig->{$value} . "/$conf" if ($conf);
     my ($value, $conf) = split(/=/, $self->{$item}{$key}, 2);
     $value = $myconfig->{$value} . "/$conf" if ($conf);

-    $str .= $form->escape($value, 1);
+    $href .= $form->escape($value, 1);

   }
 
   }
 

-  if ($target) {
-    $str .= qq| target=$target|;
-  }
-
-  $str .= ">";
+  my $str = "<a href='$href' $target_token>";

 
   $main::lxdebug->leave_sub();
 
 
   $main::lxdebug->leave_sub();