Jump to content
Sign in to follow this  
xtrme

jigoshop fatal error when switching themes

Recommended Posts

xtrme

Installed jigoshop plugin, then installed the Pagelines jigoshop plugin. Was having a little trouble with some of the variations within jigoshop so I switched to the Twenty Eleven theme to check things out and got a fatal php error related to the Pagelines jigoshop plugin. Had to remove the Pagelines jigoshop plugin via ftp in order to get the site back up. Good thing it's only my testing site.

Share this post


Link to post
Share on other sites
xtrme

Yes I know the Pagelines Jigoshop plugin only works with the Pagelines theme but when it (Pagelines Jigoshop Plugin) is installed, if you switch themes to something other than Pagelines theme, the site will go down and the only way to get it back up is to remove the Pagelines Jigoshop plugin but since the admin panel goes down with a fatal php error, the only way to fix it is to remove the plugin via an ftp client. It's a bug. Shouldn't happen.

Share this post


Link to post
Share on other sites
beardedavenger

Why did you reject the answer? PageLines Integration plugin, is for PageLines, and we have checks in place to make sure this doesn't happen.

	function check() {
	if ( ! function_exists( 'is_jigoshop' ) || ! function_exists( 'ploption' ) )
		return false;
	
	if ( ! is_jigoshop() )
		return false;
	
	return true;
	}

What other answer do you expect?


Something big is coming for DMS + photographers.

http://fotostheme.com

 

Nick Haskins & CO - New home for all of my PageLines Store products! 

http://nickhaskins.co

 

Better DMS - News, Tutorials, and Tips

http://betterdms.com

 

Share this post


Link to post
Share on other sites
xtrme

I rejected the answer because switching to another theme (Twenty Eleven) disabled the site. The only way to get it back up was to remove the Pagelines Jigoshop plugin. Is this something you'd consider normal?

Share this post


Link to post
Share on other sites
beardedavenger

No it's definitely not normal, which is the reason the check is in place. Is your integration plugin current? Should be 1.4.6. Also, is the framework up to date as well? The function above checks for checks for ploptions, which are specific to PageLines, and if they aren't present, then don't run.


Something big is coming for DMS + photographers.

http://fotostheme.com

 

Nick Haskins & CO - New home for all of my PageLines Store products! 

http://nickhaskins.co

 

Better DMS - News, Tutorials, and Tips

http://betterdms.com

 

Share this post


Link to post
Share on other sites
Rob

You have to forgive Nick. He's a coder and thinks in code. The reason this happened was that the plugin is intended to be used exclusively with the PageLines Framework. When you have any theme-specific plugin it's critical to deactivate it before switching themes. If you switch themes and a theme-specific plugin is still active, it continues doing its job, implementing hooks and other functions involved with the integrated, associated theme. If that theme isn't active, it will break WordPress. If you had deactivated the PageLines plugins prior to switching, Twenty-Eleven would not break and Jigo would probably work to your satisfaction. I apologize for the confusion here.


Former PageLines Moderator, Food Expert and Raconteur

Share this post


Link to post
Share on other sites
xtrme

I understand why it happened but there must be a way the plugin could be fixed so it doesn't break Wordpress (I have other Pagelines specific plugins that are ok). I bet Nick could write a piece of code that would work. Thanks for the clarification. If I want to troubleshoot using a different theme, from now on I'll disable that plugin.

Share this post


Link to post
Share on other sites
Simon

Update to the plugin version 1.4.7, i added some extra checking.

Share this post


Link to post
Share on other sites
Jenny

@xtrme As I mentioned by email, a fix has been pushed for the Jigoshop integration for PageLines so you should receive no more errors when switching themes. :) We apologize for any frustration this issue has caused.


Jenny :: Web designer at Simple Mama (follow me at @simplemamacom)

Check out Share Me, a social sharing add-on for DMS that is super simple to set up.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

Sign in to follow this  

×