Micro-modularization.

[pr0n] / perl / Sesse / pr0n / Common.pm

diff --git a/perl/Sesse/pr0n/Common.pm b/perl/Sesse/pr0n/Common.pm
index b9a651cb5261e6b0981c015772387121d8907300..098c9d36d22ea766ce6f828dc6f0633f15fa00a6 100644 (file)
--- a/perl/Sesse/pr0n/Common.pm
+++ b/perl/Sesse/pr0n/Common.pm
@@ -131,11 +131,7 @@ sub get_query_string {
                next unless defined($value);
                next if (defined($defparam->{$key}) && $value == $defparam->{$key});
 
-               $value = URI::Escape::uri_escape($value);
-
-               # Unescape a few for prettiness (we'll need something for a real _, though)
-               $value =~ s/%20/_/g;
-               $value =~ s/%2F/\//g;
+               $value = pretty_escape($value);
        
                $str .= ($first) ? "?" : ';';
                $str .= "$key=$value";
@@ -143,6 +139,27 @@ sub get_query_string {
        }
        return $str;
 }
+               
+sub pretty_escape {
+       my $value = shift;
+
+       $value = URI::Escape::uri_escape($value);
+
+       # Unescape a few for prettiness (we'll need something for a real _, though)
+       $value =~ s/%20/_/g;
+       $value =~ s/%2F/\//g;
+
+       return $value;
+}
+
+sub pretty_unescape {
+       my $value = shift;
+
+       # URI unescaping is already done for us
+       $value =~ s/_/ /g;
+
+       return $value;
+}
 
 sub print_link {
        my ($r, $title, $baseurl, $param, $defparam, $accesskey) = @_;