OSSP CVS Repository

ossp - Check-in [2220]
Not logged in
[Honeypot]  [Browse]  [Home]  [Login]  [Reports
[Search]  [Ticket]  [Timeline
  [Patchset]  [Tagging/Branching

Check-in Number: 2220
Date: 2002-Jul-04 08:54:48 (local)
2002-Jul-04 06:54:48 (UTC)
User:rse
Branch:
Comment: add my issues
Tickets:
Inspections:
Files:
ossp-pkg/tabea/tabea-brainstorming      1.6 -> 1.7     13 inserted, 0 deleted

ossp-pkg/tabea/tabea-brainstorming 1.6 -> 1.7

--- tabea-brainstorming  2002/07/04 06:40:45     1.6
+++ tabea-brainstorming  2002/07/04 06:54:48     1.7
@@ -376,6 +376,19 @@
 
     $rc = $handle->destroy();
 
+    Issues:
+    - [rse] $type should be not on constructor, but on render() IMHO
+    - [rse] $prefix should be not on constructor, but on render() IMHO
+    - [rse] if thinking about programming a CGI, it is not clear to me
+            how one has to process the POST data and inject it again
+            into the $handle object for re-rendering. I would expect
+            the reverse of render() for loading in the POST results.
+            I recommend: $rc = $handle->parse($type, $post_data);
+                         $rc = $handle->render($type, $html_data);
+            This way we get - load/save for internal/backend I/O
+                            - parse/render for input-device-specific I/O
+                            - import/export for output-device-specific I/O
+
     Example: Generic RAID configuration
     -----------------------------------
 

CVSTrac 2.0.1