IMasterObject adaptation doesn't work for native storm model objects

[Julian Edwards]

I have this code:

{{{
    IMasterObject(subscriber).status = ArchiveSubscriberStatus.EXPIRED
}}}

where 'subscriber' is an instance of ArchiveSubscriber, a native Storm class. This code falls over with an error:

*** TypeError: ('could not adapt', <lp.soyuz.model.archivesubscriber.ArchiveSubscriber object at 0x74b1690, <InterfaceClass canonical.launchpad.interfaces.lpstorm.IMasterObject>)

Looking in the zcml it appears that IDBObject is only implemented by SQLBase:

    <class class="canonical.database.sqlbase.SQLBase">
        <implements interface="canonical.launchpad.interfaces.IDBObject" />
    </class>

so I can't adapt native Storm objects in this way.

[Stuart Bishop (stub)]

=== modified file 'lib/canonical/launchpad/webapp/configure.zcml'
--- lib/canonical/launchpad/webapp/configure.zcml 2010-06-08 15:57:09 +0000
+++ lib/canonical/launchpad/webapp/configure.zcml 2010-06-10 11:33:24 +0000
@@ -633,7 +633,7 @@
         />
     <adapter
         provides="canonical.launchpad.interfaces.IMasterObject"
- for="canonical.launchpad.interfaces.IDBObject"
+ for="zope.interface.Interface"
         trusted="yes"
         factory="canonical.launchpad.webapp.adapter.get_object_from_master_store"
         />

[Gary Poster (gary)]

Stub's suggestion is pragmatic if not precise. I'm OK with it.

Is there a root Storm class (not interface) we can use instead? IIRC, that should work as well. If it doesn't, we can use zcml to slam an interface on the root Storm class, and make the adapter work off of that interface.

[Julian Edwards (julian-edwards)]

My Storm classes always inherit from storm.locals.Storm, but I don't know if that's strictly necessary.

[Julian Edwards (julian-edwards)]

I'll make this change later today, where's a good place to unit test this? I don't want to clutter that lovely db-policy.txt doctest :)

[Stuart Bishop (stub)]

On Thu, Jun 10, 2010 at 8:10 PM, Gary Poster <email address hidden> wrote:
> Stub's suggestion is pragmatic if not precise.  I'm OK with it.
>
> Is there a root Storm class (not interface) we can use instead?  IIRC,
> that should work as well.  If it doesn't, we can use zcml to slam an
> interface on the root Storm class, and make the adapter work off of that
> interface.

Last time I tried I couldn't do it. There appears to be no way to say
that the Storm and all its subclasses implements an Interface.

--
Stuart Bishop <email address hidden>
http://www.stuartbishop.net/

[Ursula Junque (ursinha)]

Fixed in stable r11098 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/11098>.

[Launchpad QA Bot (lpqabot)]

Fixed in stable r11143 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/11143>.

[Launchpad QA Bot (lpqabot)]

Fixed in stable r11227 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/11227>.