Author: lresende
Date: Thu May 28 23:21:15 2009
New Revision: 779795

URL: http://svn.apache.org/viewvc?rev=779795&view=rev
Log:
Merge branch 'work'

Modified:
    
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java
    
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingUtil.java
    
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/binding/atom/aggregator/AggregatorTestCase.java

Modified: 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java
URL: 
http://svn.apache.org/viewvc/tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java?rev=779795&r1=779794&r2=779795&view=diff
==============================================================================
--- 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java
 (original)
+++ 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingInvoker.java
 Thu May 28 23:21:15 2009
@@ -383,10 +383,11 @@
                     }
 
                 } else if (status == 404) {
-                       if ( provider.supportsFeedEntries())
+                       if ( provider.supportsFeedEntries()) {
                                msg.setFaultBody(new NotFoundException());
-                       else
-                               msg.setFaultBody(new 
org.apache.tuscany.sca.data.collection.NotFoundException());                   
+                       } else {
+                               msg.setFaultBody(new 
org.apache.tuscany.sca.data.collection.NotFoundException());
+                       }
                 } else {
                     msg.setFaultBody(new ServiceRuntimeException("HTTP status 
code: " + status));
                 }

Modified: 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingUtil.java
URL: 
http://svn.apache.org/viewvc/tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingUtil.java?rev=779795&r1=779794&r2=779795&view=diff
==============================================================================
--- 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingUtil.java
 (original)
+++ 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/main/java/org/apache/tuscany/sca/binding/atom/provider/AtomBindingUtil.java
 Thu May 28 23:21:15 2009
@@ -24,6 +24,7 @@
 import org.apache.abdera.factory.Factory;
 import org.apache.abdera.model.Content;
 import org.apache.abdera.model.Link;
+import org.apache.abdera.model.Content.Type;
 import org.apache.tuscany.sca.data.collection.Entry;
 import org.apache.tuscany.sca.data.collection.Item;
 import org.apache.tuscany.sca.databinding.Mediator;
@@ -47,13 +48,13 @@
     static Entry<Object, Object> entry(org.apache.abdera.model.Entry feedEntry,
                                        DataType<?> itemClassType, DataType<?> 
itemXMLType, Mediator mediator) {
         if (feedEntry != null) {
-            if (itemClassType.getPhysical() == Item.class) {
+            if (itemClassType.getPhysical() == Item.class || 
feedEntry.getContentType() == Type.HTML ) {
                 String key = feedEntry.getId().toString();
-                
+
                 Item item = new Item();
                 item.setTitle(feedEntry.getTitle());
                 item.setContents(feedEntry.getContent());
-                
+
                 for (Link link : feedEntry.getLinks()) {
                     if (link.getRel() == null || "self".equals(link.getRel())) 
{
                         if (item.getLink() == null) {
@@ -65,25 +66,25 @@
                         item.setAlternate(link.getHref().toString());
                     }
                 }
-                
+
                 item.setDate(feedEntry.getUpdated());
-                
+
                 return new Entry<Object, Object>(key, item);
-                
+
             } else {
                 String key = null; 
                 if ( feedEntry.getId() != null) {
-                       key = feedEntry.getId().toString();
+                    key = feedEntry.getId().toString();
                 }
-                
+
                 // Create the item from XML
                 if (feedEntry.getContentElement().getElements().size() == 0) {
-                        return null;
+                    return null;
                 }
-                
+
                 String value = feedEntry.getContent();
                 Object data = mediator.mediate(value, itemXMLType, 
itemClassType, null);
-    
+
                 return new Entry<Object, Object>(key, data);
             }
         } else {
@@ -107,19 +108,19 @@
         Object data = entry.getData();
         if (data instanceof Item) {
             Item item = (Item)data;
-            
+
             org.apache.abdera.model.Entry feedEntry = factory.newEntry();
             if (key != null) {
                 feedEntry.setId(key.toString());
             }
             feedEntry.setTitle(item.getTitle());
             feedEntry.setContentAsHtml(item.getContents());
-    
+
             String href = item.getLink();
             if (href == null && key != null) {
                 href = key.toString();
             }
-    
+
             if (href != null) {
                 feedEntry.addLink(href);
             }
@@ -131,31 +132,31 @@
             if (alternate != null) {
                 feedEntry.addLink(alternate, "alternate");
             }
-                
+
             Date date = item.getDate();
             if (date != null) {
                 feedEntry.setUpdated(date);
             }
             return feedEntry;
-            
+
         } else if (data != null) {
             org.apache.abdera.model.Entry feedEntry = factory.newEntry();
-             feedEntry.setId(key.toString());
-             feedEntry.setTitle("item");
-             
-             
-             // Convert the item to XML
-             String value = mediator.mediate(data, itemClassType, itemXMLType, 
null).toString();
-             
-             Content content = factory.newContent();
-             content.setContentType(Content.Type.XML);
-             content.setValue(value);
-             
-             feedEntry.setContentElement(content);
-    
-             feedEntry.addLink(key.toString());
-                  
-             return feedEntry;
+            feedEntry.setId(key.toString());
+            feedEntry.setTitle("item");
+
+
+            // Convert the item to XML
+            String value = mediator.mediate(data, itemClassType, itemXMLType, 
null).toString();
+
+            Content content = factory.newContent();
+            content.setContentType(Content.Type.XML);
+            content.setValue(value);
+
+            feedEntry.setContentElement(content);
+
+            feedEntry.addLink(key.toString());
+
+            return feedEntry;
         } else {
             return null;
         }

Modified: 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/binding/atom/aggregator/AggregatorTestCase.java
URL: 
http://svn.apache.org/viewvc/tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/binding/atom/aggregator/AggregatorTestCase.java?rev=779795&r1=779794&r2=779795&view=diff
==============================================================================
--- 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/binding/atom/aggregator/AggregatorTestCase.java
 (original)
+++ 
tuscany/branches/sca-java-1.x/modules/binding-atom-abdera/src/test/java/org/apache/tuscany/sca/binding/atom/aggregator/AggregatorTestCase.java
 Thu May 28 23:21:15 2009
@@ -55,8 +55,8 @@
         new Socket("127.0.0.1", 8085);
     }
     
-    @Ignore("TUSCANY-2939")
-    //@Test
+    //@Ignore("TUSCANY-2939")
+    @Test
     public void testAggregator() throws Exception {
         Entry<String, Item>[] entries = aggregatorService.getAll();
         


Reply via email to