Fix miscellaneous prod bugs.

service/datastore/multiarg.go

Index: service/datastore/multiarg.go
diff --git a/service/datastore/multiarg.go b/service/datastore/multiarg.go
index 3efa1493df86692ed1947c2985ba17e753f54b46..091596b4884e451b02f58b22e35b999e24215af5 100644
--- a/service/datastore/multiarg.go
+++ b/service/datastore/multiarg.go
@@ -267,7 +267,8 @@ func mkPLSName(o interface{}) (PropertyLoadSaver, string) {
 		return pls, pls.o.Type().Name()
 	}
 	if pls, ok := o.(PropertyLoadSaver); ok {
-		return pls, ""
+		v, _ := getPLSValue(o)
+		return pls, v.Type().Name()

[iannucci, 2015/08/15 02:23:53]

This looks really wrong to me. We already see if it's a *structPLS above. This
is for implementors of just the PropertyLoadSaver interface (e.g. PropertyMap),
which won't have .Type()

[dnj (Google), 2015/08/15 02:32:32]

The problem occurs when I implement a custom PropertyLoadSaver. That
PropertyLoadSaver doesn't set the $kind meta, so newKeyObjErr gets back "" for
the name. It then runs GetMetaDefault("kind", ""), and since my PLS doesn't
declare/export $kind, the name is "" and I hit the ensuing error.

 	}
 	pls := GetPLS(o)
 	name := pls.(*structPLS).o.Type().Name()