From 63e4355fba1ec90bec9cf751b8b6fc3483062377 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Gra=C3=B1a?= <dangra@gmail.com>
Date: Thu, 19 Apr 2012 14:10:55 -0300
Subject: [PATCH] find form input elements using one xpath #111 #121

---
 scrapy/http/request/form.py | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/scrapy/http/request/form.py b/scrapy/http/request/form.py
index 50533ea42..3b3fb4fca 100644
--- a/scrapy/http/request/form.py
+++ b/scrapy/http/request/form.py
@@ -32,10 +32,8 @@ class FormRequest(Request):
     @classmethod
     def from_response(cls, response, formname=None, formnumber=0, formdata=None,
                       clickdata=None, dont_click=False, **kwargs):
-        from scrapy.selector.lxmldocument import LxmlDocument
         kwargs.setdefault('encoding', response.encoding)
-        root = LxmlDocument(response, lxml.html.HTMLParser)
-        form = _get_form(root, formname, formnumber, response)
+        form = _get_form(response, formname, formnumber)
         formdata = _get_inputs(form, formdata, dont_click, clickdata, response)
         url = form.action or form.base_url
         return cls(url, method=form.method, formdata=formdata, **kwargs)
@@ -47,11 +45,10 @@ def _urlencode(seq, enc):
               for v in (vs if hasattr(vs, '__iter__') else [vs])]
     return urllib.urlencode(values, doseq=1)
 
-def _get_form(root, formname, formnumber, response):
-    """
-    Uses all the passed arguments to get the required form
-    element
-    """
+def _get_form(response, formname, formnumber):
+    """Find the form element """
+    from scrapy.selector.lxmldocument import LxmlDocument
+    root = LxmlDocument(response, lxml.html.HTMLParser)
     if not root.forms:
         raise ValueError("No <form> element found in %s" % response)
 
@@ -77,15 +74,18 @@ def _get_inputs(form, formdata, dont_click, clickdata, response):
     except (ValueError, TypeError):
         raise ValueError('formdata should be a dict or iterable of tuples')
 
-    inputs = [(n, u'' if v is None else v) for n, v in form.fields.items() if n not in formdata]
+    inputs = form.xpath('descendant::input[@type!="submit"]|descendant::textarea|descendant::select')
+    values = [(k, u'' if v is None else v) \
+              for k, v in ((e.name, e.value) for e in inputs) \
+              if k not in formdata]
 
     if not dont_click:
         clickable = _get_clickable(clickdata, form)
         if clickable and clickable[0] not in formdata and not clickable[0] is None:
-            inputs.append(clickable)
+            values.append(clickable)
 
-    inputs.extend(formdata.iteritems())
-    return inputs
+    values.extend(formdata.iteritems())
+    return values
 
 def _get_clickable(clickdata, form):
     """