Fix multi-action form.
authordrewc <drewc@tech.coop>
Sat, 23 Feb 2008 01:49:47 +0000 (17:49 -0800)
committerdrewc <drewc@tech.coop>
Sat, 23 Feb 2008 01:49:47 +0000 (17:49 -0800)
The 'standard' ucw forms have always required javascript if multiple actions are to be
submitted. I've never liked that, and this is my, IMO, superior solution.

darcs-hash:20080223014947-39164-8b219f9d816266e965fe4a8b08c2cb296ddb03b6.gz

src/ucw/standard-components.lisp

index dd39293..361e61e 100644 (file)
 
 (defmethod ucw::find-action-id :around ((context standard-request-context))
   (or 
-   (let (id)
-     (ucw::find-parameter 
-      (context.request context) ucw::+action-parameter-name+
-      :test (lambda (name parameter)
-             (declare (ignore name))
-             (destructuring-bind (param-name &optional action-id)
-                 (split-sequence:split-sequence 
-                  +action-compound-name-delimiter+ parameter)
-               (when (and action-id 
-                          (string= ucw::+action-parameter-name+ param-name))
-                 (setf id action-id)))))
-     id)
+   (loop
+
+      :for (k . v) in (ucw::parameters 
+                     (context.request context))
+      :do(destructuring-bind (param-name &optional action-id)
+             (split-sequence:split-sequence 
+              +action-compound-name-delimiter+ k)
+           (when (and action-id 
+                      (string= 
+                       ucw::+action-parameter-name+ param-name))
+             (return action-id))))
    (call-next-method)))
 
 (defcomponent standard-window-component