Jump to content

Screenshots on Downloads Search Results


Adriano Faria

Recommended Posts

This is an excellent hook, I've been meaning to add this functionality myself, but it breaks if you're community is using sphinx.

Fatal error: Call to undefined method screenshotSearchResuls_search::_buildWhereStatement() in hooks/screenshotSearchResuls_search_01d905c3b1d3a9dd0ba4e7a5e8585d6e.php on line 53
Any chance you'll be able to patch in support for forums using sphinx?
Link to comment
Share on other sites

  • 1 year later...

Found a bug with files that have a plus sign in their titles.

Latest SQL Error Today (Today, 10:50 PM)
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 Date: Mon, 05 May 2014 02:50:11 +0000
 Error: 1064 - syntax error, unexpected '+'
 IP Address: - /files/file/12-foo-bar-cake/
 ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 mySQL query error: SELECT COUNT(*) as total_results FROM downloads_files f  LEFT JOIN downloads_ccontent cc ON ( cc.file_id=f.file_id )   WHERE (MATCH( f.file_name ) AGAINST( 'foo bar cake++' IN BOOLEAN MODE ) OR MATCH( f.file_desc ) AGAINST( 'foo bar cake++' IN BOOLEAN MODE )) AND f.file_cat IN(1,2,3,4,5) AND f.file_id<>12
 .--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------.
 | File                                                                       | Function                                                                      | Line No.          |
 |----------------------------------------------------------------------------+-------------------------------------------------------------------------------+-------------------|
 | admin/applications_addon/ips/downloads/extensions/search/engines/sql.php    | [screenshotSearchResuls_search]._filesSearch                                  | 91                |
 '----------------------------------------------------------------------------+-------------------------------------------------------------------------------+-------------------'
 | admin/sources/classes/search/controller.php                                 | [search_engine_downloads].search                                              | 544               |
 '----------------------------------------------------------------------------+-------------------------------------------------------------------------------+-------------------'
 | admin/applications_addon/ips/downloads/modules_public/display/file.php      | [IPSSearch].search                                                            | 590               |
 '----------------------------------------------------------------------------+-------------------------------------------------------------------------------+-------------------'
 | admin/applications_addon/ips/downloads/modules_public/display/file.php      | [public_downloads_display_file].getSimilarFiles                               | 462               |
 '----------------------------------------------------------------------------+-------------------------------------------------------------------------------+-------------------'
 | admin/sources/base/ipsController.php                                        | [public_downloads_display_file].doExecute                                     | 306               |
 '----------------------------------------------------------------------------+-------------------------------------------------------------------------------+-------------------'


Attempting to view any of these files on the front-end will result in a database error.

Edit: This doesn't seem to be related to your hook actually..? I was just assuming from "screenshotSearchResuls_search", my second guess is the most recent security patch may have caused it?

Sorry to waste your time if this isn't relevant.

Link to comment
Share on other sites

Yeah, I did that immediately after posting to confirm, and it still gave an error.

I figured out the issue though. It was a bug with fulltext index searching in InnoDB 5.6. (So it wasn't specifically related to this hook or IP.Board at all)

Sorry for the trouble.

Link to comment
Share on other sites

  • 9 months later...

Archived

This topic is now archived and is closed to further replies.

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...