The module presently only parses attachments made with the standard upload module. I would like it to also parse those made with the filemanager module (since I'm using that module on my site...).

CommentFileSizeAuthor
#1 search_attachments.module.patch13.77 KBJürgen Depicker

Comments

Jürgen Depicker’s picture

Jürgen Depicker commented
Status: Active » Needs review
StatusFileSize
new search_attachments.module.patch13.77 KB

I've patched the code in this way:

  • added two helper functions to prevent code duplication
  • added support for attachments created by the filemanager and attachment module.
  • cleaned up some whitespace (two spaces instead of tabs)

The code works without a hitch on my site. I hope the code gets inserted into the cvs version, so others may benefit from it as well.
The patch is in unidiff format. I hope this is fine?

markj’s picture

markj commented

Thanks for the filemanager patch. I've added a driver system that allows you to add support for any file management api or module. I'll try to create a driver for filemanager based on your code.

markj’s picture

markj commented
Status: Needs review » Closed (fixed)

There is now a driver for attachments/filemanager.

Jürgen Depicker’s picture

Jürgen Depicker commented

Thanks!!! I would like to see the new version; if only you would commit ;-).

markj’s picture

markj commented

The latest version is available at http://interoperating.info/mark/search_attachments