On Tue, 2007-11-20 at 18:32 -0600, commits mugshot org wrote:
> Modified: dumbhippo/trunk/.classpath
> ===================================================================
> --- dumbhippo/trunk/.classpath 2007-11-20 23:05:42 UTC (rev 6935)
> +++ dumbhippo/trunk/.classpath 2007-11-21 00:32:51 UTC (rev 6936)
> @@ -112,6 +112,6 @@
> <classpathentry kind="lib" path="openfire/src/plugins/userImportExport/lib/msv.jar"/>
> <classpathentry kind="lib" path="openfire/src/plugins/userImportExport/lib/isorelax.jar"/>
> <classpathentry kind="var" path="HIPPO_TRUNK_LIB/server/dom4j-1.6.1.jar"/>
> - <classpathentry kind="lib" path="lib/server/facebook.jar"/>
> + <classpathentry kind="lib" path="HIPPO_TRUNK_LIB/server/facebook.jar"/>
^^^
This looks wrong to me, though it does seem to work, somehow.
> Modified: dumbhippo/trunk/server/src/com/dumbhippo/server/impl/IdentitySpiderBean.java
> ===================================================================
> --- dumbhippo/trunk/server/src/com/dumbhippo/server/impl/IdentitySpiderBean.java 2007-11-20 23:05:42 UTC (rev 6935)
> +++ dumbhippo/trunk/server/src/com/dumbhippo/server/impl/IdentitySpiderBean.java 2007-11-21 00:32:51 UTC (rev 6936)
> @@ -39,6 +39,7 @@
> import com.dumbhippo.persistence.EmailResource;
> import com.dumbhippo.persistence.ExternalAccount;
> import com.dumbhippo.persistence.ExternalAccountType;
> +import com.dumbhippo.persistence.FacebookResource;
> import com.dumbhippo.persistence.Group;
> import com.dumbhippo.persistence.GuidPersistable;
> import com.dumbhippo.persistence.LinkResource;
> @@ -407,6 +408,9 @@
> DataService.currentSessionRW().changed(UserDMO.class, claimedOwner.getGuid(), "aim");
> else if (res instanceof XmppResource)
> DataService.currentSessionRW().changed(UserDMO.class, claimedOwner.getGuid(), "xmpp");
> + // TODO: is there a list of property names anywhere that needs to be updated?
> + else if (res instanceof FacebookResource)
> + DataService.currentSessionRW().changed(UserDMO.class, claimedOwner.getGuid(), "facebook");
The names here are names of properties on UserDMO. Since you didn't add
a property on UserDMO, there should be no call to changed() for that
property.
- Owen
Attachment:
signature.asc
Description: This is a digitally signed message part