Require atom entries to have ID, title and authors

[cascardo/atompub.git] / atom / entry.c

diff --git a/atom/entry.c b/atom/entry.c
index 99ab439..31994c3 100644 (file)
--- a/atom/entry.c
+++ b/atom/entry.c
@@ -26,24 +26,29 @@
 struct _atom_entry
 {
   xmlDocPtr doc;
-  char *id;
+  AtomID *id;
   char *title;
   GPtrArray *authors;
+  GPtrArray *categories;
   char *summary;
 };
 
 void atom_entry_author_add (AtomEntry *, AtomPerson *);
 static void atom_entry_authors_delete (AtomEntry *);
 
+void atom_entry_category_add (AtomEntry *, AtomCategory *);
+static void atom_entry_categories_delete (AtomEntry *);
+
 AtomEntry *
-atom_entry_new (char *title, AtomPerson *author)
+atom_entry_new (char *id, char *title, AtomPerson *author)
 {
   AtomEntry *entry;
   entry = g_slice_new (AtomEntry);
   entry->doc = NULL;
-  entry->id = NULL;
+  entry->id = atom_id_new (id);
   entry->title = g_strdup (title);
   entry->authors = NULL;
+  entry->categories = NULL;
   atom_entry_author_add (entry, author);
   entry->summary = NULL;
   return entry;
@@ -75,15 +80,22 @@ atom_entry_new_data_len (char *data, size_t len)
       char * content;
       content = xmlNodeGetContent (child->xmlChildrenNode);
       if (!xmlStrcmp (child->name, "id"))
-       entry->id = content;
+        entry->id = atom_id_new (content);
       else if (!xmlStrcmp (child->name, "title"))
-       entry->title = content;
+        entry->title = g_strdup (content);
       else if (!xmlStrcmp (child->name, "summary"))
-       entry->summary = content;
+        entry->summary = g_strdup (content);
       else if (!xmlStrcmp (child->name, "author"))
-       atom_entry_author_add (entry, atom_person_new_from_xmlnode (child));
+        atom_entry_author_add (entry, atom_person_new_from_xmlnode (child));
+      else if (!xmlStrcmp (child->name, "category"))
+        atom_entry_category_add (entry, atom_category_new_from_xmlnode (child));
       else
-       xmlFree (content);
+        xmlFree (content);
+    }
+  if (entry->id == NULL || entry->title == NULL || entry->authors == NULL)
+    {
+      atom_entry_delete (entry);
+      return NULL;
     }
   return entry;
 }
@@ -94,28 +106,32 @@ atom_entry_delete (AtomEntry *entry)
   if (entry->doc)
     xmlFreeDoc (entry->doc);
   if (entry->id)
-    g_free (entry->id);
+    atom_id_delete (entry->id);
   if (entry->title)
     g_free (entry->title);
   if (entry->authors)
     atom_entry_authors_delete (entry);
+  if (entry->categories)
+    atom_entry_categories_delete (entry);
   if (entry->summary)
     g_free (entry->summary);
   g_slice_free (AtomEntry, entry);
 }
 
-char *
+AtomID *
 atom_entry_id (AtomEntry *entry)
 {
   return entry->id;
 }
 
 void
-atom_entry_id_set (AtomEntry *entry, char *id)
+atom_entry_id_set (AtomEntry *entry, AtomID *id)
 {
+  if (id == NULL)
+    return;
   if (entry->id)
-    g_free (entry->id);
-  entry->id = g_strdup (id);
+    atom_id_delete (entry->id);
+  entry->id = id;
 }
 
 char *
@@ -127,6 +143,8 @@ atom_entry_title (AtomEntry *entry)
 void
 atom_entry_title_set (AtomEntry *entry, char *title)
 {
+  if (title == NULL)
+    return;
   if (entry->title)
     g_free (title);
   entry->title = g_strdup (title);
@@ -161,6 +179,37 @@ atom_entry_authors_delete (AtomEntry *entry)
   g_ptr_array_free (entry->authors, TRUE);
 }
 
+void
+atom_entry_categories (AtomEntry *entry, AtomCategory *** categories,
+                       size_t *len)
+{
+  if (len)
+    *len = entry->categories->len;
+  if (categories)
+    *categories = entry->categories->pdata;
+}
+
+void
+atom_entry_category_add (AtomEntry *entry, AtomCategory *category)
+{
+  g_ptr_array_add (entry->categories, category);
+  if (entry->categories == NULL)
+    {
+      entry->categories = g_ptr_array_new ();
+    }
+  g_ptr_array_add (entry->categories, category);
+}
+
+static void
+atom_entry_categories_delete (AtomEntry *entry)
+{
+  size_t len = entry->categories->len;
+  int i;
+  for (i = 0; i < len; i++)
+    atom_category_delete (g_ptr_array_index (entry->categories, i));
+  g_ptr_array_free (entry->categories, TRUE);
+}
+
 char *
 atom_entry_summary (AtomEntry *entry)
 {
@@ -183,6 +232,7 @@ atom_entry_update_xmlnode (AtomEntry *entry)
   xmlNodePtr title;
   xmlNodePtr summary;
   xmlNodePtr author;
+  xmlNodePtr cat;
   int i;
   if (entry->doc == NULL)
     {
@@ -203,7 +253,8 @@ atom_entry_update_xmlnode (AtomEntry *entry)
           if (!xmlStrcmp (child->name, "id") ||
               !xmlStrcmp (child->name, "title") ||
               !xmlStrcmp (child->name, "summary") ||
-              !xmlStrcmp (child->name, "author"))
+              !xmlStrcmp (child->name, "author") ||
+              !xmlStrcmp (child->name, "category"))
             {
               xmlUnlinkNode (child);
               xmlFreeNode (child);
@@ -211,7 +262,7 @@ atom_entry_update_xmlnode (AtomEntry *entry)
           child = next;
         }
     }
-  id = xmlNewTextChild (root, NULL, "id", entry->id);
+  id = xmlNewTextChild (root, NULL, "id", atom_id_string (entry->id));
   title = xmlNewTextChild (root, NULL, "title", entry->title);
   if (entry->summary)
     summary = xmlNewTextChild (root, NULL, "summary", entry->summary);
@@ -222,6 +273,13 @@ atom_entry_update_xmlnode (AtomEntry *entry)
       author = atom_person_to_xmlnode (person, "author");
       xmlAddChild (root, author);
     }
+  for (i = 0; entry->categories && i < entry->categories->len; i++)
+    {
+      AtomCategory *category;
+      category = g_ptr_array_index (entry->categories, i);
+      cat = atom_category_to_xmlnode (category, "category");
+      xmlAddChild (root, category);
+    }
 }
 
 void