[Jifty-commit] r5421 - in jifty/trunk: lib/Jifty

[Jifty commits]

Author: alexmv
Date: Fri May  9 11:56:11 2008
New Revision: 5421

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Collection.pm
   jifty/trunk/lib/Jifty/CurrentUser.pm
   jifty/trunk/lib/Jifty/Request.pm

Log:
 r31451 at kohr-ah:  chmrr | 2008-05-09 11:55:27 -0400
  * Collection inherits a pager already
  * POD nitpick (must have blank line before pod directives)
  * URI unescaping is now consistent between FastCGI and standalone


Modified: jifty/trunk/lib/Jifty/Collection.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Collection.pm	(original)
+++ jifty/trunk/lib/Jifty/Collection.pm	Fri May  9 11:56:11 2008
@@ -52,7 +52,7 @@
 
 =cut
 
-__PACKAGE__->mk_accessors(qw(pager results_are_readable));
+__PACKAGE__->mk_accessors(qw(results_are_readable));
 
 =head2 as_search_action PARAMHASH
 

Modified: jifty/trunk/lib/Jifty/CurrentUser.pm
==============================================================================
--- jifty/trunk/lib/Jifty/CurrentUser.pm	(original)
+++ jifty/trunk/lib/Jifty/CurrentUser.pm	Fri May  9 11:56:11 2008
@@ -69,7 +69,7 @@
 	}
 
 That is, it will attempt to load the columns given in the model named C<App::Model::User> (where I<App> is the name of your application class). If your notion of a user object isn't a typical Jifty model or named something else, you will definitely need to override this method. If you need to perform any additional initialization for user objects, you may want to override this as well.
-	
+
 =cut
 
 sub _init {

Modified: jifty/trunk/lib/Jifty/Request.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Request.pm	(original)
+++ jifty/trunk/lib/Jifty/Request.pm	Fri May  9 11:56:11 2008
@@ -153,7 +153,7 @@
     my $path = $data->{'path'};
     
     if ($cgi && ! $path) {
-        $path = URI::Escape::uri_unescape($cgi->path_info);
+        $path = $cgi->path_info;
         $path =~ s/\?.*//;
     };
 
@@ -249,9 +249,7 @@
     my $self = shift;
     my ($cgi) = @_;
 
-    # always get the unescaped path for dispatcher, which is already
-    # the case for fastcgi, but not standalone.
-    my $path = URI::Escape::uri_unescape($cgi->path_info);
+    my $path = $cgi->path_info;
     $path =~ s/\?.*//;
     $self->path( $path );