ThomBedford

Hi Steve,

Thanks for the advice, currently taking a backup of my blog, and will then try the health check followed by disabling everything and enabling piece-by-piece.

Everything else on the blog works fine, and I can install plugins etc fine, it’s just these few portrait images that are rotating. Will run the health checks and see if anything comes of it and will update accordingly.

Resolved my latest issue, I needed to change the code in the lightbox php file as well as the search mechanisms were duplicated. So in the BWGModelGalleryBox.php file in the frontend/models directory there’s a similar search string to my previous comment (which is for some reason held for moderation).

Before:

    if ($filter_search_name != '') {
     $where = ' AND (image.alt LIKE "%%' . $filter_search_name . '%%" OR image.description LIKE "%%' . $filter_search_name . '%%")';
    }

After

    if ($filter_search_name != '') {
      $where = ' AND ((';
      $filter_search_name_array = explode(' ', $filter_search_name);
      foreach($filter_search_name_array as $filter_search_name_1){
        $where .='image.alt LIKE "%%' . $filter_search_name_1 . '%%" AND ';
      }
      $where = substr($where, 0, -4) . ') OR (';
	  foreach($filter_search_name_array as $filter_search_name_2){
        $where .='image.description LIKE "%%' . $filter_search_name_2 . '%%" AND ';
      }
      $where = substr($where, 0, -4) . '))';
    }

I’ve made pretty much the exact same change here that I made in the other file and it fixed the issue, just changed the variable names slightly.

Hi,

I’ve coded the change to the search, however when clicking an image I get the error “The image has been deleted.” in the lightbox even though the links are all correct. Here’s the change I made:

In the framework/WDGLibrary.php file, the function get_image_rows_data, there is a WHERE clause to add search items if they exist:

    if ( $bwg_search ) {
      $where = 'AND (image.alt LIKE "%%' . $bwg_search . '%%" OR image.description LIKE "%%' . $bwg_search . '%%")';
    }

Change the contents of this if-statement to the following:

    if ( $bwg_search ) {
      $where = 'AND ((';
      $bwg_search_array = explode(' ', $bwg_search);
      foreach($bwg_search_array as $bwg_search){
        $where .='image.alt LIKE "%%' . $bwg_search . '%%" AND ';
      }
      $where = substr($where, 0, -4) . ') OR (';
	  foreach($bwg_search_array as $bwg_search){
        $where .='image.description LIKE "%%' . $bwg_search . '%%" AND ';
      }
      $where = substr($where, 0, -4) . '))';
    }

This will create an array based on a space-separated search string, then loop through that array and add on an extra where clause on to the end of the where string.

Now I need to figure out how to resolve the “The image has been deleted.” issue…

• This reply was modified 8 years, 3 months ago by ThomBedford.
• This reply was modified 8 years, 3 months ago by ThomBedford.

Hi,

I think by changing the default.js code from:

  jQuery("#search_by_name .search_by_name").on("input keyup", function() {
    var search_by_name = jQuery(this).val().toLowerCase();
    if (search_by_name) {
      jQuery("#explorer_body .explorer_item").hide();
      jQuery("#explorer_body .explorer_item").each(function () {
        var filename = jQuery(this).attr("filename").toLowerCase();
        if (filename.indexOf(search_by_name) != -1) {
          jQuery(this).show();
        }
      });
    }
    else {
      jQuery("#explorer_body .explorer_item").show();
    }
  });

To something like this:

  jQuery("#search_by_name .search_by_name").on("input keyup", function() {
    var search_by_name = jQuery(this).val().toLowerCase();
    if (search_by_name) {
      jQuery("#explorer_body .explorer_item").hide();
      jQuery("#explorer_body .explorer_item").each(function () {
        var filename = jQuery(this).attr("filename").toLowerCase();
        if (filename.match("/("+search_by_name.replace(" ","|")+")/")) {
          jQuery(this).show();
        }
      });
    }
    else {
      jQuery("#explorer_body .explorer_item").show();
    }
  });

This may fix the problem, it simply changes the indexOf call to a match call and alters the string slightly to fit the method.

Can you confirm if this would work?

Thanks,
Thom