]> git.decadent.org.uk Git - maypole.git/commitdiff
Merged Apache2::MVC into Apache::MVC, deprecated $r->{query}
authorSebastian Riedel <sri@labs.kraih.com>
Thu, 9 Sep 2004 16:59:57 +0000 (16:59 +0000)
committerSebastian Riedel <sri@labs.kraih.com>
Thu, 9 Sep 2004 16:59:57 +0000 (16:59 +0000)
git-svn-id: http://svn.maypole.perl.org/Maypole/trunk@170 48953598-375a-da11-a14b-00016c27c3ee

Changes
lib/Apache/MVC.pm
lib/Apache/MVC/Base.pm [deleted file]
lib/Apache2/MVC.pm [deleted file]
lib/Maypole/Application.pm

diff --git a/Changes b/Changes
index 2d0b1a33812b36a20736b75114a28eeb31bb8b0e..1b2bc7626e78eb5248c8d46714b1782eb1a8c6af 100644 (file)
--- a/Changes
+++ b/Changes
@@ -3,9 +3,10 @@ Revision history for Perl extension Maypole
 1.8   XXX XXX XX XX:XX:XX XXX XXXX
     - Added parse_args() (Simon Flack)
     - call additional_data() and authenticate() for plain templates
-    - added Apache2::MVC mod_perl2 front-end
+    - merged Apache2::MVC (mod_perl2 support) into Apache::MVC
     - added Maypole::Application universal loader
-    - added Apache::MVC::Base base class for Apache front-ends
+    - config parameter handling for Maypole::Model::CDBI
+    - $r->{query} is now deprecated, use $r->{params} for GET and POST
 
 1.7   Sat Jul 17 20:15:26 BST 2004
     - Emergency release - we lost the "use Maypole::Constants" from
index 5005effb96f244fa0eaeae8526fc64d146160b80..320b6b78252227139477c9c6e1ffc15e6f40c9c7 100644 (file)
@@ -1,24 +1,64 @@
 package Apache::MVC;
 
-use base qw(Apache::MVC::Base Maypole);
-use Apache;
-use Apache::Request;
 use strict;
 use warnings;
 
-our $VERSION = "0.3";
+use base 'Maypole';
+use mod_perl;
+
+if ( $mod_perl::VERSION >= 1.99 ) {
+    require Apache2;
+    require Apache::RequestRec;
+    require Apache::RequestUtil;
+    require APR::URI;
+}
+else { require Apache }
+require Apache::Request;
+
+our $VERSION = "0.4";
 
 sub get_request {
-    shift->{ar} = Apache::Request->new( Apache->request );
+    my ( $self, $r ) = @_;
+    $self->{ar} = Apache::Request->new($r);
+}
+
+sub parse_location {
+    my $self = shift;
+    $self->{path} = $self->{ar}->uri;
+    my $loc = $self->{ar}->location;
+    no warnings 'uninitialized';
+    $self->{path} =~ s/^($loc)?\///;
+    $self->parse_path;
+    $self->parse_args;
 }
 
 sub parse_args {
     my $self = shift;
-    $self->{params} = { $self->{ar}->content };
-    while ( my ( $key, $value ) = each %{ $self->{params} } ) {
-        $self->{params}{$key} = '' unless defined $value;
+    $self->{params} = { $self->_mod_perl_args( $self->{ar} ) };
+    $self->{query}  = { $self->_mod_perl_args( $self->{ar} ) };
+}
+
+sub send_output {
+    my $r = shift;
+    $r->{ar}->content_type( $r->{content_type} );
+    $r->{ar}->headers_out->set( "Content-Length" => length $r->{output} );
+    $r->{ar}->send_http_header;
+    $r->{ar}->print( $r->{output} );
+}
+
+sub get_template_root {
+    my $r = shift;
+    $r->{ar}->document_root . "/" . $r->{ar}->location;
+}
+
+sub _mod_perl_args {
+    my ( $self, $apr ) = @_;
+    my %args;
+    foreach my $key ( $apr->param ) {
+        my @values = $apr->param($key);
+        $args{$key} = @values == 1 ? $values[0] : \@values;
     }
-    $self->{query} = { $self->{ar}->args };
+    return %args;
 }
 
 1;
@@ -113,6 +153,7 @@ see L<Maypole>.
 =head1 AUTHOR
 
 Simon Cozens, C<simon@cpan.org>
+Marcus Ramberg, C<marcus@thefeed.no>
 Screwed up by Sebastian Riedel, C<sri@oook.de>
 
 =head1 LICENSE
diff --git a/lib/Apache/MVC/Base.pm b/lib/Apache/MVC/Base.pm
deleted file mode 100644 (file)
index 996240d..0000000
+++ /dev/null
@@ -1,50 +0,0 @@
-package Apache::MVC::Base;
-
-use strict;
-use warnings;
-
-sub parse_location {
-    my $self = shift;
-    $self->{path} = $self->{ar}->uri;
-    my $loc = $self->{ar}->location;
-    no warnings 'uninitialized';
-    $self->{path} =~ s/^($loc)?\///;
-    $self->parse_path;
-    $self->parse_args;
-}
-
-sub send_output {
-    my $r = shift;
-    $r->{ar}->content_type( $r->{content_type} );
-    $r->{ar}->headers_out->set( "Content-Length" => length $r->{output} );
-    $r->{ar}->send_http_header;
-    $r->{ar}->print( $r->{output} );
-}
-
-sub get_template_root {
-    my $r = shift;
-    $r->{ar}->document_root . "/" . $r->{ar}->location;
-}
-
-1;
-
-=head1 NAME
-
-Apache::MVC::Base - Apache front-end base class
-
-=head1 SYNOPSIS
-
-    use base 'Apache::Maypole::Base';
-
-=head1 DESCRIPTION
-
-The base class for the Apache and Apache2 front-ends.
-
-=head1 AUTHOR
-
-Simon Cozens, C<simon@cpan.org>
-Screwed up by Sebastian Riedel, C<sri@oook.de>
-
-=head1 LICENSE
-
-You may distribute this code under the same terms as Perl itself.
diff --git a/lib/Apache2/MVC.pm b/lib/Apache2/MVC.pm
deleted file mode 100644 (file)
index 2c6ceb7..0000000
+++ /dev/null
@@ -1,132 +0,0 @@
-package Apache2::MVC;
-
-use base qw(Apache::MVC::Base Maypole);
-use Apache2;
-use Apache::RequestRec;
-use Apache::RequestUtil;
-use Apache::Request;
-use APR::URI;
-use strict;
-use warnings;
-
-our $VERSION = "0.1";
-
-sub get_request {
-    my ( $self, $r ) = @_;
-    $self->{ar} = Apache::Request->new($r);
-}
-
-sub parse_args {
-    my $self = shift;
-    $self->{params} = { $self->_mod_perl_args( $self->{ar} ) };
-    $self->{query}  = { $self->_mod_perl_args( $self->{ar} ) };
-}
-
-sub _mod_perl_args {
-    my ( $self, $apr ) = @_;
-    my %args;
-    foreach my $key ( $apr->param ) {
-        my @values = $apr->param($key);
-        $args{$key} = @values == 1 ? $values[0] : \@values;
-    }
-    return %args;
-}
-
-1;
-
-=head1 NAME
-
-Apache2::MVC - Apache2 front-end to Maypole
-
-=head1 SYNOPSIS
-
-    package BeerDB;
-    use base 'Apache::MVC';
-    BeerDB->setup("dbi:mysql:beerdb");
-    BeerDB->config->{uri_base} = "http://your.site/";
-    BeerDB->config->{display_tables} = [qw[beer brewery pub style]];
-    # Now set up your database:
-    # has-a relationships
-    # untaint columns
-
-    1;
-
-=head1 DESCRIPTION
-
-Maypole is a Perl web application framework to Java's struts. It is 
-essentially completely abstracted, and so doesn't know anything about
-how to talk to the outside world. C<Apache2::MVC> is a mod_perl2 based
-subclass of Maypole.
-
-To use it, you need to create a package which represents your entire
-application. In our example above, this is the C<BeerDB> package.
-
-This needs to first inherit from C<Apache2::MVC>, and then call setup.
-This will give your package an Apache-compatible C<handler> subroutine,
-and then pass any parameters onto the C<setup_database> method of the
-model class. The default model class for Maypole uses L<Class::DBI> to 
-map a database to classes, but this can be changed by messing with the
-configuration. (B<Before> calling setup.)
-
-Next, you should configure your application through the C<config>
-method. Configuration parameters at present are:
-
-=over
-
-=item uri_base
-
-You B<must> specify this; it is the base URI of the application, which
-will be used to construct links.
-
-=item display_tables
-
-If you do not want all of the tables in the database to be accessible,
-then set this to a list of only the ones you want to display
-
-=item rows_per_page
-
-List output is paged if you set this to a positive number of rows.
-
-=back
-
-You should also set up relationships between your classes, such that,
-for instance, calling C<brewery> on a C<BeerDB::Beer> object returns an
-object representing its associated brewery.
-
-For a full example, see the included "beer database" application.
-
-=head1 INSTALLATION
-
-Create a driver module like the one above.
-
-Put the following in your Apache config:
-
-    <Location /beer>
-        SetHandler perl-script
-        PerlHandler BeerDB
-    </Location>
-
-Copy the templates found in F<templates/factory> into the
-F<beer/factory> directory off the web root. When the designers get
-back to you with custom templates, they are to go in
-F<beer/custom>. If you need to do override templates on a
-database-table-by-table basis, put the new template in
-F<beer/I<table>>. 
-
-This will automatically give you C<add>, C<edit>, C<list>, C<view> and
-C<delete> commands; for instance, a list of breweries, go to 
-
-    http://your.site/beer/brewery/list
-
-For more information about how the system works and how to extend it,
-see L<Maypole>.
-
-=head1 AUTHOR
-
-Simon Cozens, C<simon@cpan.org>
-Marcus Ramberg, C<marcus@thefeed.no>
-Screwed up by Sebastian Riedel, C<sri@oook.de>
-
-=head1 LICENSE
-
-You may distribute this code under the same terms as Perl itself.
index 2727c9a062a7f5d5cc74319c7fa925feadf087fc..c2d58dc8bae21fc74a9f81cda564ec850251cfb9 100644 (file)
@@ -4,15 +4,8 @@ use strict;
 use warnings;
 
 if ( $ENV{MOD_PERL} ) {
-    require mod_perl;
-    if ( $mod_perl::VERSION >= 1.99 ) {
-        require Apache2::MVC;
-        our @ISA = qw(Apache2::MVC);
-    }
-    else {
-        require Apache::MVC;
-        our @ISA = qw(Apache::MVC);
-    }
+    require Apache::MVC;
+    our @ISA = qw(Apache::MVC);
 }
 else {
     require CGI::Maypole;