[Yanel-commits] rev 30429 - public/yanel/trunk/src/core/java/org/wyona/yanel/core/source

[josias at wyona.com]

Author: josias
Date: 2008-01-16 10:33:50 +0100 (Wed, 16 Jan 2008)
New Revision: 30429

Modified:
   public/yanel/trunk/src/core/java/org/wyona/yanel/core/source/ResourceResolver.java
Log:
added some TODOs about parameter handling

Modified: public/yanel/trunk/src/core/java/org/wyona/yanel/core/source/ResourceResolver.java
===================================================================
--- public/yanel/trunk/src/core/java/org/wyona/yanel/core/source/ResourceResolver.java	2008-01-16 09:22:56 UTC (rev 30428)
+++ public/yanel/trunk/src/core/java/org/wyona/yanel/core/source/ResourceResolver.java	2008-01-16 09:33:50 UTC (rev 30429)
@@ -57,6 +57,7 @@
             ResourceManager manager = Yanel.getInstance().getResourceManager();
             Resource targetResource = manager.getResource(resource.getEnvironment(), 
                     resource.getRealm(), uri);
+            // TODO: don't overwrite existing parameters, but add the new ones
             targetResource.setParameters(parameters);
             if (ResourceAttributeHelper.hasAttributeImplemented(targetResource, "Viewable", "1")) {
                 String viewV1path = resource.getRealm().getMountPoint() + uri.substring(1);
@@ -92,6 +93,7 @@
             for (int i=0; i<paramPairs.length; i++) {
                 String[] tokens = paramPairs[i].split("=");
                 String name = tokens[0];
+                // TODO: accept empty value
                 String value = tokens[1];
                 if (log.isDebugEnabled()) {
                     log.debug("adding parameter: name: " + name + " value: " + value);