From afb246307af632fe5090bfa6aca74871379de361 Mon Sep 17 00:00:00 2001
From: ota42y <ota42y@gmail.com>
Date: Mon, 18 May 2020 21:50:19 +0900
Subject: [PATCH] bugfix for id_list parameter

`id_list` shoud be comma-delimited string.
https://arxiv.org/help/api/user-manual#_query_interface

But Query.ArticleIDs is array and url.Values#Encode() doesn't convert comma-delimited string.
So we need fix.

I think this convert is arxiv's spectific rules, so we shouldn't fix otils.ToURLValues.
---
 v1/arxiv.go | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/v1/arxiv.go b/v1/arxiv.go
index 815b4a2..d3b870d 100644
--- a/v1/arxiv.go
+++ b/v1/arxiv.go
@@ -21,6 +21,7 @@ import (
 	"fmt"
 	"io/ioutil"
 	"net/http"
+	"strings"
 	"sync"
 	"time"
 
@@ -110,7 +111,6 @@ type reqPage struct {
 	Terms      string   `json:"search_query"`
 	Start      int64    `json:"start"`
 	MaxResults int64    `json:"max_results"`
-	ArticleIDs []string `json:"id_list"`
 
 	SortBy    SortBy    `json:"sortBy"`
 	SortOrder SortOrder `json:"sortOrder"`
@@ -122,7 +122,6 @@ func (q *Query) reqPage() *reqPage {
 		Start:      q.PageNumber,
 		SortBy:     q.SortBy,
 		MaxResults: q.MaxResultsPerPage,
-		ArticleIDs: q.ArticleIDs[:],
 		SortOrder:  q.SortOrder,
 	}
 }
@@ -220,6 +219,9 @@ func (c *Client) Search(ctx context.Context, q *Query) (chan *ResultsPage, cance
 			if len(filters) > 0 {
 				qv.Set("search_query", filtersStr)
 			}
+			if len(q.ArticleIDs) > 0 {
+				qv.Set("id_list", strings.Join(q.ArticleIDs, ","))
+			}
 
 			cctx, span := trace.StartSpan(ctx, "paging")
 			span.Annotate([]trace.Attribute{