[PATCH] Pagination problem in jamendo plugin

[Xabier Rodriguez Calvar <xrcalvar igalia com>]

I am debugging a problem in mafw-grilo-source and realized that there is
a problem with the pagination. According to
http://developer.jamendo.com/fr/wiki/MusiclistApi_draft , pn parameter
is the page number, and in grl_jamendo we where just passing the
skip_count instead of the page number. I corrected it by dividing by the
count.

Br.

-- 
Xabier Rodríguez Calvar
Enxeñeiro en Informática
IGALIA http://www.igalia.com

From 245a3128e218430d76715d63617097281e410c93 Mon Sep 17 00:00:00 2001
From: Xabier Rodriguez Calvar <xrcalvar igalia com>
Date: Tue, 15 Jun 2010 11:54:59 +0200
Subject: [PATCH] jamendo: Fixed the page number when building the url request

---
 src/jamendo/grl-jamendo.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/jamendo/grl-jamendo.c b/src/jamendo/grl-jamendo.c
index 66a3267..6230985 100644
--- a/src/jamendo/grl-jamendo.c
+++ b/src/jamendo/grl-jamendo.c
@@ -986,7 +986,7 @@ grl_jamendo_source_browse (GrlMediaSource *source,
           g_strdup_printf (JAMENDO_GET_ALBUMS_FROM_ARTIST,
                            jamendo_keys,
                            bs->count,
-                           bs->skip + 1,
+                           bs->skip / bs->count + 1,
                            container_split[1]);
       } else {
         /* Browsing through artists */
@@ -994,7 +994,7 @@ grl_jamendo_source_browse (GrlMediaSource *source,
         url = g_strdup_printf (JAMENDO_GET_ARTISTS,
                                jamendo_keys,
                                bs->count,
-                               bs->skip + 1);
+                               bs->skip / bs->count + 1);
       }
       g_free (jamendo_keys);
 
@@ -1006,7 +1006,7 @@ grl_jamendo_source_browse (GrlMediaSource *source,
           g_strdup_printf (JAMENDO_GET_TRACKS_FROM_ALBUM,
                            jamendo_keys,
                            bs->count,
-                           bs->skip + 1,
+                           bs->skip / bs->count + 1,
                            container_split[1]);
       } else {
         /* Browsing through albums */
@@ -1014,7 +1014,7 @@ grl_jamendo_source_browse (GrlMediaSource *source,
         url = g_strdup_printf (JAMENDO_GET_ALBUMS,
                                jamendo_keys,
                                bs->count,
-                               bs->skip + 1);
+                               bs->skip / bs->count + 1);
       }
       g_free (jamendo_keys);
 
-- 
1.7.1

Attachment: signature.asc
Description: Esta =?ISO-8859-1?Q?=E9?= unha parte de mensaxe asinada dixitalmente