PeanutBase / PeanutbaseWebsite

Repo to document and track issues pertaining to PeanutBase website.
0 stars 0 forks source link

Images of germplasm collections fail to appear (Brilliant Gallery) #1

Closed sdash-github closed 4 years ago

sdash-github commented 4 years ago

Sudhansu Roshan noticed that the germplasm images have disappeared. They come up as broken links, e.g.: https://peanutbase.org/germplasm/images/Citra_2011_regen_seeds I believe you set these up with Brilliant Gallery. It's been disabled as a block: https://peanutbase.org/admin/structure/block You set this up so long ago you probably don't remember the details, but I suspect you can solve this one as quickly as I can. Ethy

sdash-github commented 4 years ago

Checking in -stage. Dependency (admin/modules):

  1. Brilliant gallery enabled.
  2. But Lightbox2 disabled. Likely because, Requires: Entity (missing)

Error messages (from /admin/reports/dblog):

Type php Location https://localhost:6802/sites/default/files/brilliant_gallery_temp/bg_cached_resized_bd868817d17d8ce89aab3c7d043c7762.jpg Referrer https://localhost:6802/admin/reports/event/3154433 Message PDOException: SQLSTATE[42601]: Syntax error: 7 ERROR: syntax error at or near "FROM" LINE 1: SELECT array FROM brilliant_gallery_image_arrays WHERE hash ... ^: SELECT array FROM {brilliant_gallery_image_arrays} WHERE hash = :hash LIMIT 1; Array ( [:hash] => bd868817d17d8ce89aab3c7d043c7762 ) in brilliant_gallery_image_properties_get() (line 158 of /usr/local/www/drupal7/sites/all/modules/brilliant_gallery/brilliant_gallery_functions.inc). Severity error

sdash-github commented 4 years ago

Nathan's patch: Sudhansu. FWIW, regarding the first issue (broken Brilliant Gallery), on peanutbase-stage I installed Entity, enabled Lightbox2, and updated Brilliant Gallery to version 7.x-1.11, but the error persisted. Digging through my notes, I think we're hitting this issue (likely resurfaced after a prior Brilliant Gallery update overwrote the listed patch):

https://www.drupal.org/project/brilliant_gallery/issues/357376#comment-9678217

Unfortunately, the patch didn't apply cleanly to Brilliant Gallery 7.x-1.11; I'll look into modernizing it.

sdash-github commented 4 years ago

Nathan fixed it.

-------- Forwarded Message -------- Subject: Re: Github repo for Peanubase website issues Date: Fri, 11 Oct 2019 07:50:39 -0400 From: Nathan Weeks weeks@iastate.edu To: Sudhansu Dash sdash@ncgr.org CC: pb-developers@googlegroups.com pb-developers@googlegroups.com

I updated the brilliant_gallery PostgreSQL patch for version 7.x-11 (https://www.drupal.org/project/brilliant_gallery/issues/357376#comment-13305117),

I'm not sure if there's additional functionality that needs to be tested, but at least the images appear to display: https://peanutbase-stage.usda.iastate.edu/germplasm/images/Citra_2011_regen_seeds

I've pushed a new branch (postgres) & tag (7.x-1.11-postgres) to GitHub: https://github.com/legumeinfo/brilliant_gallery/tree/postgresql (and now I see there's already an old branch issue/postgresql-support...)

-- Nathan