Added image fields for brand, season and episode. Also simplified the repetetive...

diff --git a/src/epg.c b/src/epg.c
index 0a3576e4b59d4478a91ba7d00fcd1451b2229ec2..2c07d5d9bb438cabb6e8b32f05a2ba5a525c3718 100644 (file)
--- a/src/epg.c
+++ b/src/epg.c
@@ -399,6 +399,21 @@ static epg_object_t *_epg_object_deserialize ( htsmsg_t *m, epg_object_t *eo )
   return eo;
 }
 
+static int _epg_object_set_str
+  ( void *o, char **old, const char *new )
+{
+  epg_object_t *eo = (epg_object_t*)o;
+  int save = 0;
+  if ( !eo || !new ) return 0;
+  if ( !*old || strcmp(*old, new) ) {
+    if ( *old ) free(*old);
+    *old = strdup(new);
+    _epg_object_set_updated(eo);
+    save = 1;
+  }
+  return save;
+}
+
 /* **************************************************************************
  * Brand
  * *************************************************************************/
@@ -453,28 +468,17 @@ epg_brand_t *epg_brand_find_by_id ( uint64_t id )
 
 int epg_brand_set_title ( epg_brand_t *brand, const char *title )
 {
-  int save = 0;
-  if ( !brand || !title ) return 0;
-  if ( !brand->title || strcmp(brand->title, title) ) {
-    if ( brand->title ) free(brand->title);
-    brand->title     = strdup(title);
-    _epg_object_set_updated((epg_object_t*)brand);
-    save = 1;
-  }
-  return save;
+  return _epg_object_set_str(brand, &brand->title, title);
 }
 
 int epg_brand_set_summary ( epg_brand_t *brand, const char *summary )
 {
-  int save = 0;
-  if ( !brand || !summary ) return 0;
-  if ( !brand->summary || strcmp(brand->summary, summary) ) {
-    if ( brand->summary ) free(brand->summary);
-    brand->summary   = strdup(summary);
-    _epg_object_set_updated((epg_object_t*)brand);
-    save = 1;
-  }
-  return save;
+  return _epg_object_set_str(brand, &brand->summary, summary);
+}
+
+int epg_brand_set_image ( epg_brand_t *brand, const char *image )
+{
+  return _epg_object_set_str(brand, &brand->image, image);
 }
 
 int epg_brand_set_season_count ( epg_brand_t *brand, uint16_t count )
@@ -616,15 +620,12 @@ epg_season_t *epg_season_find_by_id ( uint64_t id )
 
 int epg_season_set_summary ( epg_season_t *season, const char *summary )
 {
-  int save = 0;
-  if ( !season || !summary ) return 0;
-  if ( !season->summary || strcmp(season->summary, summary) ) {
-    if ( season->summary ) free(season->summary);
-    season->summary = strdup(summary);
-    _epg_object_set_updated((epg_object_t*)season);
-    save = 1;
-  }
-  return save;
+  return _epg_object_set_str(season, &season->summary, summary);
+}
+
+int epg_season_set_image ( epg_season_t *season, const char *image )
+{
+  return _epg_object_set_str(season, &season->image, image);
 }
 
 int epg_season_set_episode_count ( epg_season_t *season, uint16_t count )
@@ -783,53 +784,27 @@ epg_episode_t *epg_episode_find_by_id ( uint64_t id )
 
 int epg_episode_set_title ( epg_episode_t *episode, const char *title )
 {
-  int save = 0;
-  if ( !episode || !title ) return 0;
-  if ( !episode->title || strcmp(episode->title, title) ) {
-    if ( episode->title ) free(episode->title);
-    episode->title = strdup(title);
-    save = 1;
-  }
-  return save;
+  return _epg_object_set_str(episode, &episode->title, title);
 }
 
 int epg_episode_set_subtitle ( epg_episode_t *episode, const char *subtitle )
 {
-  int save = 0;
-  if ( !episode || !subtitle ) return 0;
-  if ( !episode->subtitle || strcmp(episode->subtitle, subtitle) ) {
-    if ( episode->subtitle ) free(episode->subtitle);
-    episode->subtitle = strdup(subtitle);
-    _epg_object_set_updated((epg_object_t*)episode);
-    save = 1;
-  }
-  return save;
+  return _epg_object_set_str(episode, &episode->subtitle, subtitle);
 }
 
 int epg_episode_set_summary ( epg_episode_t *episode, const char *summary )
 {
-  int save = 0;
-  if ( !episode || !summary ) return 0;
-  if ( !episode->summary || strcmp(episode->summary, summary) ) {
-    if ( episode->summary ) free(episode->summary);
-    episode->summary = strdup(summary);
-    _epg_object_set_updated((epg_object_t*)episode);
-    save = 1;
-  }
-  return save;
+  return _epg_object_set_str(episode, &episode->summary, summary);
 }
 
 int epg_episode_set_description ( epg_episode_t *episode, const char *desc )
 {
-  int save = 0;
-  if ( !episode || !desc ) return 0;
-  if ( !episode->description || strcmp(episode->description, desc) ) {
-    if ( episode->description ) free(episode->description);
-    episode->description = strdup(desc);
-    _epg_object_set_updated((epg_object_t*)episode);
-    save = 1;
-  }
-  return save;
+  return _epg_object_set_str(episode, &episode->description, desc);
+}
+
+int epg_episode_set_image ( epg_episode_t *episode, const char *image )
+{
+  return _epg_object_set_str(episode, &episode->image, image);
 }
 
 int epg_episode_set_number ( epg_episode_t *episode, uint16_t number )

diff --git a/src/epg.h b/src/epg.h
index dff7f2688a5897a42a5771a65ee288b0f2bf82b9..77c30dfaf174f0065fb1fa644a636227ae04ca98 100644 (file)
--- a/src/epg.h
+++ b/src/epg.h
@@ -16,13 +16,6 @@
  *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/*
- * TODO LIST:
- *
- *   URI in the objects limits us to single grabber, might want something
- *   more flexible to try and merge grabbers? Is that feasible?
- */
-
 #ifndef EPG_H
 #define EPG_H
 
@@ -107,6 +100,7 @@ struct epg_brand
   char                      *title;        ///< Brand name
   char                      *summary;      ///< Brand summary
   uint16_t                   season_count; ///< Total number of seasons
+  char                      *image;        ///< Brand image
 
   epg_season_tree_t          seasons;      ///< Season list
   epg_episode_tree_t         episodes;     ///< Episode list
@@ -124,6 +118,8 @@ int epg_brand_set_summary      ( epg_brand_t *b, const char *summary )
   __attribute__((warn_unused_result));
 int epg_brand_set_season_count ( epg_brand_t *b, uint16_t season_count )
   __attribute__((warn_unused_result));
+int epg_brand_set_image        ( epg_brand_t *b, const char *i )
+  __attribute__((warn_unused_result));
 
 /* Serialization */
 htsmsg_t    *epg_brand_serialize   ( epg_brand_t *b );
@@ -144,6 +140,7 @@ struct epg_season
   char                      *summary;       ///< Season summary
   uint16_t                   number;        ///< The season number
   uint16_t                   episode_count; ///< Total number of episodes
+  char                      *image;         ///< Season image
 
   RB_ENTRY(epg_season)       blink;         ///< Brand list link
   epg_brand_t               *brand;         ///< Parent brand
@@ -165,6 +162,8 @@ int epg_season_set_episode_count ( epg_season_t *s, uint16_t episode_count )
   __attribute__((warn_unused_result));
 int epg_season_set_brand         ( epg_season_t *s, epg_brand_t *b, int u )
   __attribute__((warn_unused_result));
+int epg_season_set_image         ( epg_season_t *s, const char *image )
+  __attribute__((warn_unused_result));
 
 /* Serialization */
 htsmsg_t    *epg_season_serialize    ( epg_season_t *b );
@@ -188,6 +187,7 @@ struct epg_episode
   uint16_t                   number;        ///< The episode number
   uint16_t                   part_number;   ///< For multipart episodes
   uint16_t                   part_count;    ///< For multipart episodes
+  char                      *image;         ///< Episode image
 
   RB_ENTRY(epg_episode)      blink;         ///< Brand link
   RB_ENTRY(epg_episode)      slink;         ///< Season link
@@ -226,6 +226,8 @@ int epg_episode_set_genre        ( epg_episode_t *e, const uint8_t *g, int c )
   __attribute__((warn_unused_result));
 int epg_episode_set_genre_str    ( epg_episode_t *e, const char **s )
   __attribute__((warn_unused_result));
+int epg_episode_set_image        ( epg_episode_t *e, const char *i )
+  __attribute__((warn_unused_result));
 
 /* EpNum format helper */
 // output string will be: