[Erp5-report] r46029 arnaud.fontaine - /erp5/trunk/utils/erp5.utils.test_browser/src/erp5/u...

[nobody at svn.erp5.org]

Author: arnaud.fontaine
Date: Wed Aug 31 16:51:53 2011
New Revision: 46029

URL: http://svn.erp5.org?rev=46029&view=rev
Log:
No  need to  define our  own  counter before  expiring Form  instance as  it's
already available in zope.testbrowser


Modified:
    erp5/trunk/utils/erp5.utils.test_browser/src/erp5/utils/test_browser/browser.py

Modified: erp5/trunk/utils/erp5.utils.test_browser/src/erp5/utils/test_browser/browser.py
URL: http://svn.erp5.org/erp5/trunk/utils/erp5.utils.test_browser/src/erp5/utils/test_browser/browser.py?rev=46029&r1=46028&r2=46029&view=diff
==============================================================================
--- erp5/trunk/utils/erp5.utils.test_browser/src/erp5/utils/test_browser/browser.py [utf8] (original)
+++ erp5/trunk/utils/erp5.utils.test_browser/src/erp5/utils/test_browser/browser.py [utf8] Wed Aug 31 16:51:53 2011
@@ -158,7 +158,6 @@ class Browser(ExtendedTestBrowser):
     """
     # Meaningful to re-create the MainForm class every time the page
     # has been changed
-    self._main_form_counter = -1
     self._main_form = None
 
     assert base_url[-1] == '/'
@@ -259,11 +258,9 @@ class Browser(ExtendedTestBrowser):
     """
     # If the page has not changed, no need to re-create a class, so
     # just return the main_form instance
-    if self._main_form_counter == self._counter and self._main_form:
+    if self._main_form and self._counter == self._main_form._browser_counter:
       return self._main_form
 
-    self._main_form_counter = self._counter
-
     main_form = None
     for form in self.mech_browser.forms():
       if form.attrs.get('id') == 'main_form':