about summary refs log tree commit diff stats
path: root/cpanfile
diff options
context:
space:
mode:
authorAndinus <andinus@nand.sh>2020-06-30 12:32:26 +0530
committerAndinus <andinus@nand.sh>2020-06-30 12:32:26 +0530
commit8dd9f42bfae6bb62b93b91f9c47ff410b12fa58b (patch)
treec4aa5f98164b15e10cc33a6ce5d4846bd2e0fbb5 /cpanfile
parent420a084a2d6d28b4ece2f369c881965fe5ea89b8 (diff)
downloadcrux-8dd9f42bfae6bb62b93b91f9c47ff410b12fa58b.tar.gz
Fix logical error in $url generation
This issue started with f3d1bc5f9d86300c42d6a57a9f60bd508bdf9bb4:

  Use URI to encode the url for random_search

  I'll use this for everything eventually. Before this search terms
  like "rocky mountains" would break, also segments, query_keywords
  makes it easier.

Now the problem was that query_keywords forms urls like
`?nature+water' which totally makes sense & this is why it's logical
error. I want to form it in `?nature,water' format so first we use
`join' to form `nature,water' & then use URI::Encode to encode it
correctly & then use query_keywords to form $url.

We have to use URI::Encode because otherwise we would end up with same
problem that I tried fixing in
f3d1bc5f9d86300c42d6a57a9f60bd508bdf9bb4. "rocky mountains" would've
been seperated by space.

Now it's all good, every space will be encoded & keywords will be
seperated by `,' which is what Unsplash Source wants.
Diffstat (limited to 'cpanfile')
0 files changed, 0 insertions, 0 deletions