]> git.decadent.org.uk Git - maypole.git/blobdiff - lib/CGI/Maypole.pm
applied parse_path fix to CGI::Maypole
[maypole.git] / lib / CGI / Maypole.pm
index b7e5ca26c7966570f32afb669052d4dd08ed51ad..df244326a4c6b497161c324678b27b6376aea0cc 100644 (file)
@@ -5,73 +5,11 @@ use strict;
 use warnings;
 use CGI::Simple;
 use Maypole::Headers;
+use Maypole::Constants;
 
-our $VERSION = '2.10';
+our $VERSION = '2.11';
 
-sub run {
-    my $self = shift;
-    return $self->handler();
-}
-
-sub get_request {
-    shift->{cgi} = CGI::Simple->new();
-}
-
-
-sub parse_location {
-    my $self = shift;
-    my $cgi = $self->{cgi};
-
-    # Reconstruct the request headers (as far as this is possible)
-    $self->headers_in(Maypole::Headers->new);
-    for my $http_header ($cgi->http) {
-        (my $field_name = $http_header) =~ s/^HTTPS?_//;
-        $self->headers_in->set($field_name => $cgi->http($http_header));
-    }
-
-    $self->{path} = $cgi->url( -absolute => 1, -path_info => 1 );
-    my $loc = $cgi->url( -absolute => 1 );
-    no warnings 'uninitialized';
-    $self->{path} .= '/' if $self->{path} eq $loc;
-    $self->{path} =~ s/^($loc)?\///;
-    $self->parse_path;
-    $self->parse_args;
-}
-
-sub parse_args {
-    my $self = shift;
-    my (%vars) = $self->{cgi}->Vars;
-    while ( my ( $key, $value ) = each %vars ) {
-        my @values = split "\0", $value;
-        $vars{$key} = @values <= 1 ? $values[0] : \@values;
-    }
-    $self->{params} = {%vars};
-    $self->{query}  = {%vars};
-}
-
-sub send_output {
-    my $r = shift;
-
-    # Collect HTTP headers
-    my %headers = (
-        -type            => $r->{content_type},
-        -charset         => $r->{document_encoding},
-        -content_length  => do { use bytes; length $r->{output} },
-    );
-    foreach ($r->headers_out->field_names) {
-        next if /^Content-(Type|Length)/;
-        $headers{"-$_"} = $r->headers_out->get($_);
-    }
-
-    print $r->{cgi}->header(%headers), $r->{output};
-}
-
-sub get_template_root {
-    my $r = shift;
-    $r->{cgi}->document_root . "/" . $r->{cgi}->url( -relative => 1 );
-}
-
-1;
+__PACKAGE__->mk_accessors( qw/cgi/ );
 
 =head1 NAME
 
@@ -80,16 +18,7 @@ CGI::Maypole - CGI-based front-end to Maypole
 =head1 SYNOPSIS
 
      package BeerDB;
-     use base 'CGI::Maypole';
-     BeerDB->setup("dbi:mysql:beerdb");
-     BeerDB->config->uri_base("http://your.site/cgi-bin/beer.cgi/");
-     BeerDB->config->display_tables([qw[beer brewery pub style]]);
-     BeerDB->config->template_root("/var/www/beerdb/");
-     # Now set up your database:
-     # has-a relationships
-     # untaint columns
-
-     1;
+     use Maypole::Application;
 
      ## example beer.cgi:
 
@@ -101,7 +30,8 @@ CGI::Maypole - CGI-based front-end to Maypole
 Now to access the beer database, type this URL into your browser:
 http://your.site/cgi-bin/beer.cgi/frontpage
 
-NOTE: this Maypole frontend requires additional modules that won't be installed or included with Maypole. Please see below.
+NOTE: this Maypole frontend requires additional modules that won't be installed
+or included with Maypole. Please see below.
 
 =head1 DESCRIPTION
 
@@ -109,7 +39,8 @@ This is a CGI platform driver for Maypole. Your application can inherit from
 CGI::Maypole directly, but it is recommended that you use
 L<Maypole::Application>.
 
-This module requires CGI::Simple which you will have to install yourself via CPAN or manually.
+This module requires CGI::Simple which you will have to install yourself via
+CPAN or manually.
 
 =head1 METHODS
 
@@ -121,6 +52,14 @@ Call this from your CGI script to start the Maypole application.
 
 =back
 
+=cut
+
+sub run 
+{
+    my $self = shift;
+    return $self->handler;
+}
+
 =head1 Implementation
 
 This class overrides a set of methods in the base Maypole class to provide it's
@@ -130,14 +69,164 @@ functionality. See L<Maypole> for these:
 
 =item get_request
 
-=item get_template_root
+=cut
 
-=item parse_args
+sub get_request 
+{
+    shift->cgi( CGI::Simple->new );
+}
 
 =item parse_location
 
+=cut
+
+sub parse_location 
+{
+    my $r = shift;
+    my $cgi = $r->cgi;
+
+    # Reconstruct the request headers (as far as this is possible)
+    $r->headers_in(Maypole::Headers->new);
+    for my $http_header ($cgi->http) {
+        (my $field_name = $http_header) =~ s/^HTTPS?_//;
+        $r->headers_in->set($field_name => $cgi->http($http_header));
+    }
+
+    my $path = $cgi->url( -absolute => 1, -path_info => 1 );
+    my $loc = $cgi->url( -absolute => 1 );
+    {
+        no warnings 'uninitialized';
+        $path .= '/' if $path eq $loc;
+       if ($loc =~ /\/$/) {
+         $path =~ s/^($loc)?//;
+       } else {
+         $path =~ s/^($loc)?\///;
+       }
+    }
+    $r->path($path);
+    
+    $r->parse_path;
+    $r->parse_args;
+}
+
+=item warn
+
+=cut
+
+sub warn {
+    my ($self,@args) = @_;
+    my ($package, $line) = (caller)[0,2];
+    warn "[$package line $line] ", @args ;
+    return;
+}
+
+=item parse_args
+
+=cut
+
+sub parse_args 
+{
+    my $r = shift;
+    my (%vars) = $r->cgi->Vars;
+    while ( my ( $key, $value ) = each %vars ) {
+        my @values = split "\0", $value;
+        $vars{$key} = @values <= 1 ? $values[0] : \@values;
+    }
+    $r->params( {%vars} );
+    $r->query( $r->params );
+}
+
+=item redirect_request
+
+=cut
+
+# FIXME: use headers_in to gather host and other information?
+sub redirect_request 
+{
+  my $r = shift;
+  my $redirect_url = $_[0];
+  my $status = "302";
+  if ($_[1]) {
+    my %args = @_;
+    if ($args{url}) {
+      $redirect_url = $args{url};
+    } else {
+      my $path = $args{path} || $r->cgi->url(-absolute => 1, -query=>1);
+      my $host = $args{domain};
+      ($host = $r->cgi->url(-base => 1)) =~ s/^https?:\/\///i unless ($host);
+      my $protocol = $args{protocol} || $r->get_protocol;
+      $redirect_url = "${protocol}://${host}/${path}";
+    }
+    $status = $args{status} if ($args{status});
+  }
+
+  $r->headers_out->set('Status' => $status);
+  $r->headers_out->set('Location' => $redirect_url);
+
+  return;
+}
+
+=item get_protocol
+
+=cut
+
+sub get_protocol 
+{
+  my $self = shift;
+  my $protocol = ($self->cgi->https) ? 'https' : 'http';
+  return $protocol;
+}
+
 =item send_output
 
+Generates output (using C<collect_output>) and prints it. 
+
+=cut
+
+sub send_output 
+{
+    my $r = shift;
+    print $r->collect_output;
+}
+
+=item collect_output
+
+Gathers headers and output together into a string and returns it.
+
+Splitting this code out of C<send_output> supports L<Maypole::HTTPD::Frontend>.
+
+=cut
+
+sub collect_output
+{
+    my $r = shift;
+    
+    # Collect HTTP headers
+    my %headers = (
+        -type            => $r->content_type,
+        -charset         => $r->document_encoding,
+        -content_length  => do { use bytes; length $r->output },
+    );
+    foreach ($r->headers_out->field_names) {
+        next if /^Content-(Type|Length)/;
+        $headers{"-$_"} = $r->headers_out->get($_);
+    }
+
+    return $r->cgi->header(%headers) . $r->output;
+}
+
+=item get_template_root
+
+=cut
+
+sub get_template_root {
+    my $r = shift;
+    $r->cgi->document_root . "/" . $r->cgi->url( -relative => 1 );
+}
+
+1;
+
+
 =back
 
 =head1 DEPENDANCIES