Update to Enfold Caused Critical Error

Hello,
I set up a staging site and finally got the issue down to some PHP, which was given to me by Yigit in this thread.

add_filter('avia_load_shortcodes', 'avia_include_shortcode_template', 15, 1);
function avia_include_shortcode_template($paths)
{
	$template_url = get_stylesheet_directory();
    	array_unshift($paths, $template_url.'/shortcodes/');

	return $paths;
}

Going even further back Josue helped me here, and the fix Yigit gave me basically allowed me to copy shortcodes/av-helper-slideshow.php and have my custom version living in the child theme. Has the name or location of av-helper-slideshow.php in the main theme changed?

This is a long story, but the client didn’t want the slider to be too big on a wide page, so Josue gave me code to switch out to a widescreen version at a certain pixel width.

Just wondering if there’s a simple fix,

Cheers
J

Hi Yigit,
I don’t have FTP any more, but I’ve created a login below which I hope helps.
The php on staging current has a /* */ wrapped around it
Thank you!

Hi Nikko,

Thanks for getting back to me. I have no clue when it comes to PHP and just followed Yigit and Josue’s instructions.
Have you amended the file using the admin access? If not, could you tell me where to add please?

Thanks

It seems the lines of code Josue gave me here don’t have the same attributes so I can’t apply the modifications he gave me to the latest version of av-helper-slideshow.php

Hi Nikko,

My av-helper-slideshow.php doesn’t have those functions in it, it begins:

class avia_slideshow
	{
		static  $slider = 0; 				//slider count for the current page
		protected $config;	 				//base config set on initialization
		protected $slides;	 				//attachment posts for the current slider
		protected $slide_count = 0;			//number of slides
		protected $id_array = array();
		function __construct($config)

It’s the same code Yigit helped me placing into the child theme with five years ago. The code in the latest version of the theme doesn’t match up with what Josue gave me initially, which is what I’ve been using ever since (part of it looks like this):

$html .= "<img class='img_default' src='".$img[0]."' width='".$img[1]."' height='".$img[2]."' title='".$linktitle."' alt='".$linkalt."' $markup_url />";

The closest thing I can find in the new theme version is

$img_tag = "<img src='{$img[0]}' width='{$img[1]}' height='{$img[2]}' title='{$linktitle}' alt='{$linkalt}' {$markup_url} {$img_style} />";

I’ve tried to amend the new theme using the original instructions here and it broke the slider!

Really not sure what to do now. Originally the client was unhappy that on large screens the slider was virtually 100% vh. Essentially, the purpose was to change the slider height when viewing the site on larger screens. Josue helped me so that when the screen went over 1500px wide, the height of the slider changes to a scaled version of the original image, which are 3264 x 765 px. It’s been working fine for five years. You can see it working on the live site – linked below.

Hi Nikko,
That’s a great service, I’ve just done that – link in private content. Thanks for all your help with this.

Hi Nikko,

Thanks so much for this. I added it to the child theme within the staging environment. Sadly, it is not quite working. If you check the link in the private area, it does not resize when reducing down to 1500px or less. It just uses the wide image and scales it down. This is the CSS I was provided with, do you see anything in this which needs to change?

img.img_full{
    display: none !important;
}
@media only screen and (min-width: 1500px) {
    img.img_full {
        display: block !important;
    }
    img.img_default{
        display: none !important;
    }
}

Thanks again for you patience and perseverance!

• This reply was modified 3 years, 9 months ago by jonrouse. Reason: Enfold versions in private

Thank you Nikko.
I’ve just replaced that in the staging child theme (and emptied cache) and it’s the same I’m afraid. Sorry!

For some reason some of the links on the staged version are linking to the live site, easier for me to see as the WP tool bar changes from orange to black.
The staging pages seem to be those with plain permalinks, not sure if that’s any help, but I thought it was worth mentioning.

Thanks again.

Hi Nikko,
Sure, I did set this up a few days ago

Hi Nikko,

Reply below

Thanks Nikko, give it a try. Fingers crossed!

Hi Nikko,
Maybe I need to grant access to the main site and you can access staging through there.
Please check private content.
Thanks

Thanks Nikko,
I can see that something has definitely changed, so thank you. However, when I visit the pages on staging I am seeing both versions of the image stacked on top of one another. I’m guessing you’re not seeing this? Link below for reference.

Sorry this has turned into such a pain of a job!

That’s nearly there! It should be the other way around; so the taller image is displayed on sizes below 1500px, the wider image on sizes above 1500px. Hope this makes sense!

Perfect, that’s working great on the staging site!

Just to check, essentially, all I need to do now is
• update Enfold on the live site
• replace the av-helper-slideshow.php in the child theme
• replace those few lines of css with what you added above?

Thank you, I owe you a beer or a cake or something!