From dc0c8b4a91ac443404c0a397cd53d2a78f23f97e Mon Sep 17 00:00:00 2001 From: David Baird Date: Tue, 8 Nov 2005 18:03:58 +0000 Subject: [PATCH] A few micro-edits, esp. wishlist.txt git-svn-id: http://svn.maypole.perl.org/Maypole/trunk@422 48953598-375a-da11-a14b-00016c27c3ee --- Changes | 2 +- lib/Maypole.pm | 3 ++- lib/Maypole/Model/CDBI/Plain.pm | 2 -- wishlist.txt | 20 +++++++++++++++++--- 4 files changed, 20 insertions(+), 7 deletions(-) diff --git a/Changes b/Changes index e7b6d37..8a93719 100644 --- a/Changes +++ b/Changes @@ -7,7 +7,7 @@ For information about current developments and future releases, see: Deprecated: Directly accessing the attributes of the request object, or the parameters in $r->params, or anything else, is DEPRECATED and likely to break in future - releases. + releases. Be good, and use proper method calls. Incompatible API changes: Maypole: diff --git a/lib/Maypole.pm b/lib/Maypole.pm index 8fead4d..9f1af35 100644 --- a/lib/Maypole.pm +++ b/lib/Maypole.pm @@ -431,7 +431,8 @@ sub handler : method $self->get_request($req); $self->parse_location; - # hook useful for declining static requests e.g. images + # hook useful for declining static requests e.g. images, or perhaps for + # sanitizing request parameters my $status = $self->start_request_hook; return $status unless $status == Maypole::Constants::OK(); diff --git a/lib/Maypole/Model/CDBI/Plain.pm b/lib/Maypole/Model/CDBI/Plain.pm index 7833c7b..37abc67 100644 --- a/lib/Maypole/Model/CDBI/Plain.pm +++ b/lib/Maypole/Model/CDBI/Plain.pm @@ -27,8 +27,6 @@ Maypole::Model::CDBI::Plain - Class::DBI model without ::Loader package Foo; use 'Maypole::Application'; - use Foo::SomeTable; - use Foo::Other::Table; Foo->config->model("Maypole::Model::CDBI::Plain"); Foo->setup([qw/ Foo::SomeTable Foo::Other::Table /]); diff --git a/wishlist.txt b/wishlist.txt index 8a73797..63e0ba3 100644 --- a/wishlist.txt +++ b/wishlist.txt @@ -10,6 +10,7 @@ send_output() should return a status code Move template test out of process() and into handler_guts() - maybe Fix bug 14570 - returning error codes breaks CGI::Maypole Write Maypole::Manual::Exceptions +Test and refactor external_redirect() Fix Mp::P::USC @@ -18,15 +19,28 @@ Fix Mp::P::USC Maypole::instance() Better plugin architecture, for models and bits of models. Investigate problems reported with adopt() - rt 15598 +Re-implement Maypole::Cache as Maypole::Plugin::Cache, probably using + start_request_hook, and not overriding handler_guts() +Handle repeat form submissions. +Implement internal_redirect(). +Build a more sophisticated app for testing. 3.0 ==== -encapsulate all request data in HTTP::Request object, and all response data +Encapsulate all request data in HTTP::Request object, and all response data in HTTP::Response object -add email handling - like Rails - via model plugins +Add email handling - like Rails - via model plugins. -add validation layer(s), or just an API +An e-commerce model plugin would be nice - or proof of concept. + +Add validation layer(s), or just an API killer apps: SVN model; mitiki; Pet Shop; +Multiple views - HTML, text-only, PDF, SOAP, XML - use request data to switch +to an alternate view_object - switch via a factory method. + +Maybe rename the model to PModel (Presentation Model)? + + -- 2.39.5