Yeah, space is not a valid URL character -- didn't show up in the adult terms and I didn't think of it. 

----- Original Message -----

> In util/sync_query_terms.py: > +        raise Exception("Unexpected
> results", result_string)
> > +      count += 1
> > +      ext = tldextract.extract(i["link"])
> > +      urls.append(".".join([ext.domain, ext.tld]))
> > +    return urls
> > +
> > +  def process_search_terms(self, input_file, output_file):
> > +    """Reads file with search terms, writes output_file with
> > search results."""
> > +    f_out = open(output_file, "w")
> > +    # Each line of the input contains an addon id
> > +    try:
> > +      f_in = open(input_file)
> > +    except IOError as error:
> > +      sys.exit("Can't find file: %s" % error)
> > +    for line in f_in.readlines():
> > +      search_term = line.strip()
> I had to change this line to 'search_term = line.strip().replace(" ",
> "+")' to deal with any spaces in search terms (e.g. "Jennifer
> Lopez") - otherwise google would reject it as an invalid request
> (b/c it was an invalid url?).
> —
> Reply to this email directly or view it on GitHub .