Now the module provides autocomplete search just using file names. It would be good to search also by the file description if using CCK filefield and if the file name is overriden by user using the description option.

CommentFileSizeAuthor
#5 filefield_sources.patch3.7 KBNor4a
#3 filefield_sources.patch3.66 KBNor4a
#2 filefield_sources.patch3.46 KBNor4a
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented

Unfortunately this is nearly impossible because the description for a file is not stored in the "files" database table. The description is stored by CCK, and in any number of different tables depending on which content type the file was originally uploaded into.

Nor4a’s picture

Nor4a CreditAttribution: Nor4a commented
FileSize
filefield_sources.patch3.46 KB

This is not so complicated.
A bit of coding and the patch is made to do the job for "Reference existing" option.

Hope to see this in the next release :)

Nor4a’s picture

Nor4a CreditAttribution: Nor4a commented
FileSize
filefield_sources.patch3.66 KB

In the new path fixed small search issue when searching by description.

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Status: Active » Needs work

Thanks for the patch. I hadn't realized that we're already querying every in individual field already. This patch looks good but it could use some cleanup to following coding standards (http://drupal.org/coding-standards).

Nor4a’s picture

Nor4a CreditAttribution: Nor4a commented
Status: Needs work » Needs review
FileSize
filefield_sources.patch3.7 KB

Yes - there were some Coder issues - fixed in the attached patch.

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Component: Code » General

This issue already handles most of #837894: Reuse existing metadata, it'd probably be wise of us to go ahead and make this support other properties besides description (such as alt and title).

asb’s picture

asb CreditAttribution: asb commented

sub

temelkoff’s picture

temelkoff CreditAttribution: temelkoff commented

sub

asb’s picture

asb CreditAttribution: asb commented

Any news on this issue?

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch commented
Status: Needs review » Needs work

At this point I'm not sure about this patch, we'll need to port it to Drupal 7 and see if the same functionality is reasonable there.

quicksketch’s picture

quicksketch
he/him
CreditAttribution: quicksketch at Lullabot commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Issue summary: View changes

D6 is no longer supported, so moving this forward.