[Zope3-checkins] CVS: Zope3/src/zope/app/browser/content - sql.py:1.4

Viktorija Zaksiene ryzaja@codeworks.lt
Fri, 24 Jan 2003 09:54:11 -0500


Update of /cvs-repository/Zope3/src/zope/app/browser/content
In directory cvs.zope.org:/tmp/cvs-serv6371/src/zope/app/browser/content

Modified Files:
	sql.py 
Log Message:
Marius and Steve found out why an edit form stops working when the view mixin
class specifies its own __implements__.  Added an explaination of that.
Perhaps someone who has the time could look into it?



=== Zope3/src/zope/app/browser/content/sql.py 1.3 => 1.4 ===
--- Zope3/src/zope/app/browser/content/sql.py:1.3	Thu Jan 23 04:46:28 2003
+++ Zope3/src/zope/app/browser/content/sql.py	Fri Jan 24 09:54:09 2003
@@ -24,6 +24,14 @@
 
     # XXX: if the following line is uncommented, @@test.html stops working
     # __implements__ = BrowserView.__implements__
+    #
+    # Just found the reason: if you specify __implements__ here, it overrides
+    # the one defined in zope.app.pagetemplate.simpeviewclass.simple,
+    # and IBrowserPublisher disappears from the interface list.  Instead,
+    # __implements__ of the newly created class (see SimpleViewClass in the
+    # same module) ought to be a union of __implements__ of all the base
+    # classes.  Or perhaps it should be done by zope.app.browser.form.editview?
+    
     __used_for__ = ISQLScript
 
     error = None