SurfSD
Forum Replies Created
-
Forum: Plugins
In reply to: [WooCommerce Stripe Payment Gateway] Stripe Settings Show Disabledas above resolved.
Forum: Plugins
In reply to: [WooCommerce Stripe Payment Gateway] Stripe Settings Show DisabledThank you for looking at this, came home from work today and all the errors are gone, it appears to be resolved. Maybe there was some sort of action W.C. needed to take and finally got around to it, dno’t know, but it would be great if that information was somewhere. Anyway resolved.
Forum: Plugins
In reply to: [WooCommerce] Error in wc-template-functions.php V 2.67Thank you, yes. Look at the second post. It’s not the strong tags, those were errantly added here.
What’s interesting is the repo commit says
claudiosanches Removed extra quote from variations attribute dropdown, closes #12251
“Updated three days ago” but I updated WC **yesterday** and the extra quote was still there.
Forum: Plugins
In reply to: [WooCommerce] Error in wc-template-functions.php V 2.67Apologies, been a long time since I’ve shared code here and didn’t realize the bold’s would not parse. Ignore the STRONG tags, here is a clear code chunk of the error:
$html = '<select id="' . esc_attr( $id ) . '" class="' . esc_attr( $class ) . '" name="' . esc_attr( $name ) . '" data-attribute_name="attribute_' . esc_attr( sanitize_title( $attribute ) ) . '"' . // <----- error '" data-show_option_none="' . ( $show_option_none ? 'yes' : 'no' ) . '">';Forum: Plugins
In reply to: [Cyclone Slider] 404's on included JSIt just got more strange. I checked the settings and the only script I’ve checked to load is “Cycle 2. This is the core script needed by the plugin.” All others unchecked, and the only template is standard.
Forum: Reviews
In reply to: [WooCommerce] Overall awesome piece of softwareSorry, I forgot the documentation. There is nothing that says “we believe in what we do” more than deep and accurate documentation that is actually helpful.
Forum: Plugins
In reply to: [Product Grid for WooCommerce] plugin asks for licence???After a short look at the code, this plugin is not yet ready. It is a **very** good start though and look forward to re-visiting it in the future.
The main reason I say so is it doesn’t appear to easily allow you to create themes, there is only 1, “flat.” I would prefer to be able to copy the flat directory, create a new one, and modify that, leaving “flat” alone (never hack the core. 🙂
Forum: Plugins
In reply to: [Product Grid for WooCommerce] plugin asks for licence???It appears this is an incomplete function added before it was ready, clearly evident if you hit the license link.
Until they sort it out locate this line at the bottom (-ish) of product-grid-functions.php and comment it out as shown.
//add_action('admin_notices', 'product_grid_admin_notices');Doesn’t appear to affect functionality, yet.
Forum: Plugins
In reply to: [Cyclone Slider] Fatal Error After Update<facepalm> Been working too long of hours. 🙂
Forum: Plugins
In reply to: [Cyclone Slider] Fatal Error After UpdateJust updated to 2.10.3, I still see
function cycloneslider_service_zip_archive( $plugin ){ return 'ZipArchive'; }Which should check if the class exists . . . but previous users who don’t have Zip compiled have reported 2.10.2 fixed it, so you’re probably addressing it in another way . . . all good. 😛
Forum: Plugins
In reply to: [Cyclone Slider] Fatal Error After UpdateAuthor released version 2.10.2 recently, claims this is fixed, but I’ve already re-compiled PHP with Zip so I can’t verify. Anyone without Zip compiled in PHP willing to install the new version and let us know if it’s fixed? Looks to me like it might cause the same problem, lines 402 – 404 of cyclone-slider.php:
function cycloneslider_service_zip_archive( $plugin ){ return new ZipArchive; }If it’s still broken for you, see code above which first checks if the class exists before instantiating it.
Forum: Plugins
In reply to: [Cyclone Slider] Fatal Error After UpdateI think you misunderstand the logic . . . look . . . .
The original: something somewhere (we don’t care what) calls the function cycloneslider_service_zip_archive and it originally just assumes the class exists and returns an instance of the object.
function cycloneslider_service_zip_archive( $plugin ){ return new ZipArchive; }That is the source of the error if the class isn’t compiled with PHP. what the below code does says “only return the class if it DOES exist.” The advantage is that if the Zip class does get installed it will work normally. (Note I’ve added the extra “return” line I mentioned.)
function cycloneslider_service_zip_archive( $plugin ){ if (class_exists('ZipArchive')) { return new ZipArchive; } return null; }Line 1: Call the function to **attempt** to include ZipArchive
Line 2: Ask PHP to see if this class exists
Line 3: If it does, return instance of the object.
Line 4: end of logic block; if it has returned it will go no further.
Line 5, If the Zip class is not installed, as in your case, lines 2-4 are skipped and NOW we return null; there is no ZipArchive object.What you’ve done there is return null in any case, which is the same as commenting it out:
function cycloneslider_service_zip_archive( $plugin ){ // return new ZipArchive; }Forum: Plugins
In reply to: [Cyclone Slider] Fatal Error After UpdateUpdated:
you should probably add
return null;
… after the if block to be explicit, even though if not found the function returns void. Which is almost the same thing.
I recompiled Apache. There is no ZipArchive extension, it is compiled with the Zip extension. I am on WHM so EasyApache makes it pretty painless, just go to Exhaustive List and include Zip in the PHP compile . . .
Command line add ‘–enable-zip’ to the params. Cyclone Slider 2 works fine without hacking the plugin after that.
Forum: Plugins
In reply to: [Cyclone Slider] Fatal Error After UpdateFor a quick fix, comment out line 403 In cyclone-slider.php as below
I know this is an old post, I encountered this slider with a theme (that I’m on my third week of cleaning up) and got to this issue. Surprised it’s still going on.
Just an FYI, the half correct way to remove the code that includes the ZipArchive class is to change line 403 to this:
function cycloneslider_service_zip_archive( $plugin ){ if (class_exists('ZipArchive')) { return new ZipArchive; } }That way if the class exists it will still function.
The fully correct way would be to throw an exception and inform the user what is actually wrong so they can install the extension. I don’t think I’ll put that much work into it, I’ll just recompile PHP and be done with it. 🙂
Forum: Themes and Templates
In reply to: Rewrites with Custom Post TypeGot it. Sheesh.
I had previously tried escaping matches but apparently something ELSE was wrong at that point. With the latest iteration of changes, it turns out the most basic coding knowledge was overlooked.
Double quoted strings are interpolated. I double quoted because I needed to interpolate the $type variable, and also did so in the second params for consistency.
In the context of add_my_portfolio_rules, $matches is undefined. I overlooked it because that **should** have issued a PHP warning, and somewhere in this mess of code warnings are disabled. That’s my story and I’m sticking to it. 😛
By escaping $matches,
"index.php?post_type=portfolio&name=\$matches[1]",or single-quoting it,
'index.php?post_type=portfolio&name=$matches[1]',The rewrites now work and I can move on to finishing the cleanup.
Thanks for all the responses, you have all been very helpful. 😀
At least, this may help someone else . . .