squarestar
Forum Replies Created
-
Forum: Plugins
In reply to: [Simple Membership] Unified renewal dateThanks – What I mean is: Can I set an expiry date of 31st of December without specifying the year, so that renewals are required from the 1st of January each year? I see that it is not possible in the access duration section, so I’m thinking the answer is no…
Would I need to create new memberships for each year, ending on the 31st?
Forum: Plugins
In reply to: [Favorites] Custom markup Breaking entire site please fixThe cause is un-escaped inputs in the code. I have submitted a bug report here: https://wordpress.org/support/topic/bug-settings-inputs-not-escaped-causes-malformed-html-on-the-frontend/
Forum: Plugins
In reply to: [WP Remote Users Sync] Login failsNo worries 🙂 It’s a great plugin – thanks.
Forum: Plugins
In reply to: [WP Remote Users Sync] Login failsYep, my bad.
Forum: Plugins
In reply to: [WP Remote Users Sync] Login failsAfter adding some debugging code to the plugin I finally discovered that the reason login was failing was that I had the “Force Login Redirects & Logout Everywhere” setting checked on only the destination site.
As I wasn’t using Safari or iOS I didn’t realise this would matter, but maybe it’s actually because the 2 sites weren’t set to the same value?
Anyway, problem solved.
Forum: Plugins
In reply to: [WP Remote Users Sync] Allow requests through custom processActually I answered my own question by logging the $_POST array and finding an element I can check for.
Thanks for the plugin 🙂
Thanks Robert – the email templates work under normal operation but the test emails that get sent when you’re setting the plugin up don’t get run through the
replace_email_strings()method. It gives the impression that the feature is broken.I made the changes to the plugin that I indicated in my second message above and it fixed the issue. It’s a much better experience when testing to see the correct values in the test emails.
Having looked at the code, the solution is to change lines 811 and 812 of
/includes/classes/class-wp2fa.phpto the following:$subject = wp_strip_all_tags( self::replace_email_strings(self::get_wp2fa_email_templates( $email_id . '_email_subject' ) ) );$message = wpautop( self::replace_email_strings(self::get_wp2fa_email_templates( $email_id . '_email_body' ), $user_id ) );Additions in italics
Forum: Plugins
In reply to: [WooCommerce PayPal Payments] Error connecting to PayPalLol, I tried again after writing my initial message and it worked! 7th time lucky it seems 😉
It would be helpful to have a more detailed error message; if it can take that many attempts I would think many people, like me, would assume something more serious was wrong after 3 or 4 failures.
Forum: Plugins
In reply to: [CAPTCHA 4WP - Antispam CAPTCHA solution for WordPress] Bug causes PHP errorThanks @robert681 – that seems to address that problem, however I notice the same issue in one of the other class methods. The
reset_password_verify()takes$errorsas the first argument, but passes$userback out under one condition, and nothing under another condition.With filters, you always have to return the first argument no matter what you do or don’t do with it in your own code. Otherwise other plugins that hook into the same filter afterwards will get nothing (or the wrong thing) and may not function.
Thanks for your reply. I have added the filters that fix the problem for me.
The only thing I would say is this: My solution is below, and you can see that I am not providing any information that is specific to my installation. I’m not really clear why the plugin can’t use this information by default and still allow other users to supersede it.
The priority feature of filters should mean that the plugin itself could use a priority of say 5, and any customisation would just need to use a later priority.
function filterWebpcDirPath($path, $directory) { if ($directory === 'uploads') { return WP_CONTENT_DIR . '/uploads'; } if ($directory === 'webp') { return WP_CONTENT_DIR . '/uploads-webpc'; } return $path; }After reverting to v1.22.5 I can see that the problem is no longer present
I am having this problem as well. Any information on an upcoming fix?
Forum: Plugins
In reply to: [Relevanssi Light] Possible to use partial matches?Thanks Mikko
Forum: Plugins
In reply to: [Relevanssi Light] Javascript TypeError when clicking Process All PostsThanks! Much appreciated 🙂