A few micro-edits, esp. wishlist.txt

git-svn-id: http://svn.maypole.perl.org/Maypole/trunk@422 48953598-375a-da11-a14b-00016c27c3ee

diff --git a/Changes b/Changes
index e7b6d378c873e579bd447b30bdd68ab33ad94e47..8a93719a1504a5ae1264dcb84f6ae818ec72436b 100644 (file)
--- 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 8fead4d2a16ffad298370e5c63130eb05ec6753f..9f1af35366433899221fa6cf1fe191eb42dd8cfa 100644 (file)
--- 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 7833c7b25e0bfe11c49e25cece5f2f6338293df3..37abc6734be882746d77d6b3475ed2bc64d56f72 100644 (file)
--- 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 8a73797503edb1a40084486400f38075b91d8058..63e0ba3fa9732b6c6fa55620a86345e30c4ab529 100644 (file)
--- 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\r
 Fix bug 14570 - returning error codes breaks CGI::Maypole\r
 Write Maypole::Manual::Exceptions\r
+Test and refactor external_redirect()\r
 \r
 Fix Mp::P::USC\r
 \r
@@ -18,15 +19,28 @@ Fix Mp::P::USC
 Maypole::instance()\r
 Better plugin architecture, for models and bits of models. \r
 Investigate problems reported with adopt() - rt 15598\r
+Re-implement Maypole::Cache as Maypole::Plugin::Cache, probably using \r
+    start_request_hook, and not overriding handler_guts()\r
+Handle repeat form submissions.\r
+Implement internal_redirect().\r
+Build a more sophisticated app for testing. \r
 \r
 3.0\r
 ====\r
-encapsulate all request data in HTTP::Request object, and all response data \r
+Encapsulate all request data in HTTP::Request object, and all response data \r
 in HTTP::Response object\r
 \r
-add email handling - like Rails - via model plugins\r
+Add email handling - like Rails - via model plugins. \r
 \r
-add validation layer(s), or just an API\r
+An e-commerce model plugin would be nice - or proof of concept.  \r
+\r
+Add validation layer(s), or just an API\r
 \r
 killer apps: SVN model; mitiki; Pet Shop; \r
 \r
+Multiple views - HTML, text-only, PDF, SOAP, XML  - use request data to switch\r
+to an alternate view_object - switch via a factory method.\r
+\r
+Maybe rename the model to PModel (Presentation Model)?\r
+\r
+\r