Comments

lolandese’s picture

Issue tags: +Novice

In flickr.inc we could start by replacing
'tags' => $tags_combined, (two instances)
with
'text' => $tags_combined,
or just adding this line. No idea if it would perform AND or OR in that case. Also unsure how it deals with the comma separated list of tags.

To try and give it a test spin with Flickr Filter or Flickr Block. Also the Flickr API Explorer would turn handy (https://www.flickr.com/services/api/explore/?method=flickr.photos.search).

lolandese’s picture

Status: Active » Needs work
Issue tags: -Novice
StatusFileSize
new17.95 KB

Attached patch:

lolandese’s picture

Status: Needs work » Needs review
StatusFileSize
new41.05 KB
lolandese’s picture

StatusFileSize
new45.82 KB
lolandese’s picture

StatusFileSize
new48.22 KB
lolandese’s picture

StatusFileSize
new72.34 KB

Probably the last patch. Corrects also the counter besides numerous improvements of Flickr Style (unrelated to this issue).

  • lolandese committed 384feaf on 7.x-1.x
    Issue #2403623 by lolandese: Match taxonomy terms also with Flickr photo...
lolandese’s picture

Status: Needs review » Fixed
lolandese’s picture

Status: Fixed » Needs work
StatusFileSize
new3.49 KB

Extended the patch to all album types, Not just 'user'.

TODO:
Still seems to act with an AND operator instead of OR while in the album title 'or' is used.

EDIT: Is correct, but group albums with more than one tag are limited to query the last 500 photos. To solve with #2286373: Use flickr.photos.search for groups instead of flickr.groups.pools.getPhotos.

To correct instead the description of the block checkbox (extend searches on 'title' only if the album is not of the 'user' type).

lolandese’s picture

Status: Needs work » Fixed
StatusFileSize
new7.27 KB

  • lolandese committed 1f3e15d on 7.x-1.x
    Issue #2403623 by lolandese: Match taxonomy terms also with Flickr photo...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.