From: Moritz Bunkus <m.bunkus@linet-services.de>
Date: Mon, 29 Jun 2015 11:22:31 +0000 (+0200)
Subject: ClientJS: Ziel-Argument bestimmter Funktionen vorne trimmen
X-Git-Tag: release-3.3.0beta~34^2~9
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=35cf7c7029b4aa69eb9d02c7533d1a15aafb118b;p=kivitendo-erp.git

ClientJS: Ziel-Argument bestimmter Funktionen vorne trimmen

jQuery kann aus HTML-Strings DOM-Objekte bauen:
$("<p>stuff</p>"). Beginnt der HTML-String mit Leerzeichen, so croakt
jQuery daran. Daher bei den betroffenen Funktionen, die immer auf einem
so gebauten DOM-Objekt hantieren, das Ziel-Argument um führende
Leerzeichen bereinigen.
---

diff --git a/SL/ClientJS.pm b/SL/ClientJS.pm
index f1c949e5f..335bf8163 100644
--- a/SL/ClientJS.pm
+++ b/SL/ClientJS.pm
@@ -122,6 +122,8 @@ my %supported_methods = (
   scroll_into_view       => 1,  # $(<TARGET>)[0].scrollIntoView()
 );
 
+my %trim_target_for = map { ($_ => 1) } qw(insertAfter insertBefore appendTo prependTo);
+
 sub AUTOLOAD {
   our $AUTOLOAD;
 
@@ -154,6 +156,11 @@ sub action {
     $args[$idx] = "" . $args[$idx] if ref($args[$idx]) eq 'SL::Presenter::EscapedText';
   }
 
+  # Trim leading whitespaces for certain jQuery functions that operate
+  # on HTML code: $("<p>test</p>").appendTo('#some-id'). jQuery croaks
+  # on leading whitespaces, e.g. on $(" <p>test</p>").
+  $args[0] =~ s{^\s+}{} if $trim_target_for{$method};
+
   push @{ $self->_actions }, [ $method, @args ];
 
   return $self;