Closed (fixed)
Project:
Album Photos
Version:
8.x-4.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
26 Sep 2018 at 20:57 UTC
Updated:
17 Oct 2018 at 15:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
nathaniel commentedHi, thanks for posting in the issue queue. Do you have access to the error log? I tested adding an album on the demo site (just upgraded to 8.6.1) and also on a local copy of the site running PHP 7.2. I wasn't able to reproduce the error. I can test a fresh install though if needed.
What theme are you using?
Comment #3
chikeI am using TopPlus Lite theme
Here is the error log
Thanks so much.
Comment #4
nathaniel commentedOkay, looks like the error is coming from the theme.
Quick fix topplus_lite.theme (line 139):
I'll update the photos.routing.yml to load the node object to prevent this from happening in the future. Might be a day or two though. Needs a few edits and thorough testing.
Comment #5
chikesame error screen but the error log changes to
Comment #6
nathaniel commentedI think it is the same issue there:
Line 315 adding " && is_object($variables['node'])" should fix it for now:
} elseif ((!$sidebar_first && !$sidebar_second) && (isset($variables['node']) && !empty($variables['node']) && is_object($variables['node'])) && ($variables['node']->getType() == 'page')) {The problem is the photos module is passing just the node id instead of the full node object.
Comment #7
nathaniel commentedOkay, here's a patch to fix the photos module so it loads the node object instead of only the node id. If you are available to test it that would help.
Comment #8
chikeMy site is working, thank you sir.
Comment #10
nathaniel commented