[ 
https://issues.apache.org/jira/browse/SHINDIG-124?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Papp updated SHINDIG-124:
---------------------------------

    Attachment: fix-124-3-bug.patch.txt

This should do it.  Attached is a patch that adds the Enum functionality 
without any pre-conditions -except- that attempts to set the displayValue null 
will throw a NullPointerException (as required by the OS 0.7 spec which states 
that this field must never be null).  The Message Java model class is also 
added in this patch, as well as the attendant changes to the Person class for 
Enum.  Hope this fits the bill.

> Implementation POJO for opensocial.Enum.xx and opensocial.Message
> -----------------------------------------------------------------
>
>                 Key: SHINDIG-124
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-124
>             Project: Shindig
>          Issue Type: New Feature
>          Components: Gadgets Server - Java
>            Reporter: Michael Papp
>         Attachments: fix-124-1-bug.patch.txt, fix-124-2-bug.patch.txt, 
> fix-124-3-bug.patch.txt, fix-124-bug.patch.txt
>
>
> Submitting proposed implementation for opensocial.Enum and 
> opensocial.Enum.Drinker, opensocial.Enum.Gender, and opensocial.Enum.Smoker.  
> This is a best guess pass as there are a number of ways of implementing this, 
> and opensocial 0.7 spec for the getKey() is rather nebulous.  Anyway, hope 
> this helps.  If approach is right, then I can amend patch to include tests.
> M. Papp

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to